Merge lp:~smajevic-irfan/ubuntu-qa-website/gci into lp:ubuntu-qa-website

[Irfan Smajevic]

Changed "Top 20 (of all time)" page to include the top 100 testers and changed the name "Top 100 (of all time)"

[Nicholas Skaggs (nskaggs)]

LGTM, thanks!

review: Approve

=== modified file 'modules/qatracker/qatracker.pages.php'
--- modules/qatracker/qatracker.pages.php	2012-11-19 19:45:41 +0000
+++ modules/qatracker/qatracker.pages.php	2015-12-13 21:49:57 +0000
@@ -207,9 +207,9 @@
         'type' => MENU_NORMAL_ITEM,
         'weight' => -8);
 
-    $items['qatracker/reports/testers/top20'] = array(
-        'title' => t("Top 20 (of all time)"),
-        'description' => t("Top 20 (of all time)"),
+    $items['qatracker/reports/testers/top100'] = array(
+        'title' => t("Top 100 (of all time)"),
+        'description' => t("Top 100 (of all time)"),
         'file' => 'report/qatracker.report.testers.php',
         'page arguments' => array("qatracker_report_testers"),
         'access arguments' => array("access content"),

=== modified file 'modules/qatracker/report/qatracker.report.testers.php'
--- modules/qatracker/report/qatracker.report.testers.php	2015-05-31 14:53:33 +0000
+++ modules/qatracker/report/qatracker.report.testers.php	2015-12-13 21:49:57 +0000
@@ -39,7 +39,7 @@
     $query->leftjoin('qatracker_build', 'qatracker_build', 'qatracker_build.id = qatracker_build_milestone.buildid');
     $query->leftjoin('qatracker_result', 'qatracker_result', 'qatracker_result.buildid = qatracker_build.id');
     $query->leftjoin('users', 'users', 'users.uid = qatracker_result.reporterid');
-    if (arg(3) != "top20") {
+    if (arg(3) != "top100") {
         $query->fields('qatracker_milestone', array('title', 'status'));
         $query->condition('qatracker_milestone.status', array(0,1), 'IN');
         $query->orderBy('qatracker_milestone.status', 'ASC');
@@ -47,7 +47,7 @@
         $query->groupBy('qatracker_milestone.id, qatracker_milestone.title, qatracker_milestone.status, users.name');
     }
     else {
-        $query->range(0, 20);
+        $query->range(0, 100);
         $query->groupBy('users.name');
     }
     if(array_key_exists("date_from", $_POST) && $_POST['date_from']) {
@@ -70,11 +70,11 @@
 
     if (!function_exists("new_table")) {
         function new_table($rows, $milestone, $status) {
-            if (arg(3) != "top20") {
+            if (arg(3) != "top100") {
                 $title = $milestone." (".$status.")";
             }
             else {
-                $title = t("Top 20 (of all time)");
+                $title = t("Top 100 (of all time)");
             }
             return array(
                 '#theme' => 'table',
@@ -89,7 +89,7 @@
     }
 
     foreach ($result as $record) {
-        if (arg(3) != "top20" && $milestone && $milestone != $record->title) {
+        if (arg(3) != "top100" && $milestone && $milestone != $record->title) {
             $items[] = new_table($rows, $milestone, $status);
             $rows = array();
             $position = 1;
@@ -102,7 +102,7 @@
             l($record->name, "http://launchpad.net/~".$record->name),
             $record->count,
         );
-        if (arg(3) != "top20") {
+        if (arg(3) != "top100") {
             $milestone = $record->title;
             $status = $qatracker_milestone_status[$record->status];
         }