From 1ddfb914fdec9d01813e20a64d0d46c956789086 Mon Sep 17 00:00:00 2001
From: tjhunt <tjhunt>
Date: Wed, 22 Jul 2009 09:20:04 +0000
Subject: [PATCH] quiz reports: Fix ordering of reports broken by MDL-16438.

---
 mod/quiz/report/reportlib.php | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/mod/quiz/report/reportlib.php b/mod/quiz/report/reportlib.php
index fd8433ae41..424158788f 100644
--- a/mod/quiz/report/reportlib.php
+++ b/mod/quiz/report/reportlib.php
@@ -354,22 +354,20 @@ function quiz_report_list($context){
         return $reportlist;
     }
     $reports = $DB->get_records('quiz_report', null, 'displayorder DESC', 'name, capability');
-
     $reportdirs = get_plugin_list("quiz");
-    //order the reports tab in descending order of displayorder
+
+    // Order the reports tab in descending order of displayorder
     $reportcaps = array();
-    if ($reports){
-        foreach ($reports as $key => $obj) {
-            if (in_array($obj->name, $reportdirs)) {
-                $reportcaps[$obj->name]=$obj->capability;
-            }
+    foreach ($reports as $key => $obj) {
+        if (array_key_exists($obj->name, $reportdirs)) {
+            $reportcaps[$obj->name] = $obj->capability;
         }
     }
 
-    //add any other reports on the end
-    foreach ($reportdirs as $reportname => $reportdir) {
+    // Add any other reports on the end
+    foreach ($reportdirs as $reportname => $notused) {
         if (!isset($reportcaps[$reportname])) {
-            $reportcaps[$reportname]= null;
+            $reportcaps[$reportname] = null;
         }
     }
     $reportlist = array();
@@ -377,7 +375,7 @@ function quiz_report_list($context){
         if (empty($capability)){
             $capability = 'mod/quiz:viewreports';
         }
-        if ($has = has_capability($capability, $context)){
+        if (has_capability($capability, $context)){
             $reportlist[] = $name;
         }
     }
-- 
2.39.5