Merge pull request #99 from Chloe070196/ils_usage_graphs

fix: group ils record usage by year and month
PTFS-Europe · Jul 30, 2024 · a18215a · a18215a
2 parents a35d2a8 + 85a68d4
commit a18215a
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/code/web/services/ILS/UsageGraphs.php b/code/web/services/ILS/UsageGraphs.php
@@ -75,6 +75,7 @@ function launch() {
 			$userILSUsage->selectAdd('year');
 			$userILSUsage->selectAdd('month');
 			$userILSUsage->orderBy('year, month');
+
 			if ($stat == 'userLogins') {
 				$dataSeries['User Logins'] = [
 					'borderColor' => 'rgba(255, 99, 132, 1)',
@@ -166,6 +167,7 @@ function launch() {
 			$stat == 'totalHolds'
 		) {
 			$recordILSUsage = new ILSRecordUsage();
+			$recordILSUsage->groupBy('year, month');
 			if (!empty($instanceName)) {
 				$recordILSUsage->instance = $instanceName;
 			}