dashboard: optimize DB access

Author: nunoplopes

db.php

@@ -282,6 +282,22 @@ function db_get_patch_stats() {
                        ->getArrayResult();
 }
 
+function db_get_pr_stats($year) {
+  global $entityManager;
+  return $entityManager->createQueryBuilder()
+                       ->from('Patch', 'p')
+                       ->select(['p.status',
+                                 'p.type',
+                                 'g.repository',
+                                 'COUNT(p.status) AS count'])
+                       ->join('p.group', 'g')
+                       ->where('g.year = :year')
+                       ->groupBy('p.status, p.type, g.repository')
+                       ->setParameter('year', $year)
+                       ->getQuery()
+                       ->getArrayResult();
+}
+
 function db_delete_cache() {
   global $connection;
   $connection->executeStatement('DROP TABLE IF EXISTS cache_items');

pages/dashboard.php

@@ -73,12 +73,16 @@
     @++$projs[$repo->name()];
 
     $prs_per_project[$repo->name()]['url'] = (string)$repo;
-    foreach ($group->patches as $patch) {
-      @++$prs_per_project[$repo->name()][$patch->type][$patch->wasMerged()];
-    }
   }
 }
 
+foreach (db_get_pr_stats($selected_year) as $data) {
+  $repo   = (new Repository($data['repository']))->name();
+  $merged = $data['status'] == PATCH_MERGED ||
+            $data['status'] == PATCH_MERGED_ILLEGAL;
+  @$prs_per_project[$repo][$data['type']][$merged] += $data['count'];
+}
+
 if (sizeof($projs) > 10) {
   foreach ($projs as $proj => $n) {
     if ($n == 1)