gradebook MDL-25713 switched grade_object fetch_all_helper() over to using a recordset
authorandyjdavis <andrew@moodle.com>
Mon, 20 Dec 2010 08:39:46 +0000 (16:39 +0800)
committerandyjdavis <andrew@moodle.com>
Mon, 20 Dec 2010 08:39:46 +0000 (16:39 +0800)
lib/grade/grade_object.php

index 6f4ba06..2978ccd 100644 (file)
@@ -184,20 +184,21 @@ abstract class grade_object {
         }
 
         global $DB;
-        if ($datas = $DB->get_records_select($table, $wheresql, $newparams)) {
-
-            $result = array();
-            foreach($datas as $data) {
-                $instance = new $classname();
-                grade_object::set_properties($instance, $data);
-                $result[$instance->id] = $instance;
-            }
-            return $result;
-
-        } else {
+        $rs = $DB->get_recordset_select($table, $wheresql, $newparams);
+        //returning false rather than empty array if nothing found
+        if (!$rs->valid()) {
+             return false;
+        }
 
-            return false;
+        $result = array();
+        foreach($rs as $data) {
+            $instance = new $classname();
+            grade_object::set_properties($instance, $data);
+            $result[$instance->id] = $instance;        
         }
+        $rs->close();
+        
+        return $result;
     }
 
     /**