MDL-54542 competency: Really handle false negatives in plans to review
authorFrederic Massart <fred@moodle.com>
Thu, 19 May 2016 03:42:15 +0000 (11:42 +0800)
committerFrederic Massart <fred@moodle.com>
Thu, 19 May 2016 03:42:15 +0000 (11:42 +0800)
competency/classes/api.php

index 004c8de..b11daa4 100644 (file)
@@ -2409,11 +2409,11 @@ class api {
         $sql .= " AND p.userid $insql";
         $params += $inparams;
 
-        // Order by last updated, seconded by ID to prevent random ordering.
-        $sql .= " ORDER BY p.timemodified DESC, p.id ASC";
+        // Order by ID just to have some ordering in place.
+        $ordersql = " ORDER BY p.id ASC";
 
         $plans = array();
-        $records = $DB->get_recordset_sql($select . $sql, $params, $skip, $limit);
+        $records = $DB->get_recordset_sql($select . $sql . $ordersql, $params, $skip, $limit);
         foreach ($records as $record) {
             $plan = new plan(0, plan::extract_record($record, 'plan_'));
             $template = null;