Merge branch 'MDL-54542-master-fixup' of https://github.com/FMCorz/moodle
authorAndrew Nicols <andrew@nicols.co.uk>
Thu, 19 May 2016 03:52:29 +0000 (11:52 +0800)
committerAndrew Nicols <andrew@nicols.co.uk>
Thu, 19 May 2016 03:52:29 +0000 (11:52 +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;