MDL-55082 search: Update core areas to return results in time ASC order
authorEric Merrill <merrill@oakland.edu>
Thu, 30 Jun 2016 15:06:56 +0000 (11:06 -0400)
committerEric Merrill <merrill@oakland.edu>
Thu, 30 Jun 2016 15:11:35 +0000 (11:11 -0400)
course/classes/search/mycourse.php
mod/glossary/classes/search/entry.php
mod/survey/classes/search/activity.php
search/classes/area/base_activity.php
search/tests/fixtures/mock_search_area.php

index 7a5f30b..b75defd 100644 (file)
@@ -49,7 +49,7 @@ class mycourse extends \core_search\area\base {
      */
     public function get_recordset_by_timestamp($modifiedfrom = 0) {
         global $DB;
-        return $DB->get_recordset_select('course', 'timemodified >= ?', array($modifiedfrom));
+        return $DB->get_recordset_select('course', 'timemodified >= ?', array($modifiedfrom), 'timemodified ASC');
     }
 
     /**
index e030e4e..72f1671 100644 (file)
@@ -53,7 +53,7 @@ class entry extends \core_search\area\base_mod {
 
         $sql = "SELECT ge.*, g.course FROM {glossary_entries} ge
                   JOIN {glossary} g ON g.id = ge.glossaryid
-                WHERE ge.timemodified >= ?";
+                WHERE ge.timemodified >= ? ORDER BY ge.timemodified ASC";
         return $DB->get_recordset_sql($sql, array($modifiedfrom));
     }
 
index 3cb9ee7..f4033eb 100644 (file)
@@ -46,7 +46,8 @@ class activity extends \core_search\area\base_activity {
     public function get_recordset_by_timestamp($modifiedfrom = 0) {
         global $DB;
         $select = 'course != ? AND ' . static::MODIFIED_FIELD_NAME . ' >= ?';
-        return $DB->get_recordset_select($this->get_module_name(), $select, array(0, $modifiedfrom));
+        return $DB->get_recordset_select($this->get_module_name(), $select, array(0, $modifiedfrom),
+                static::MODIFIED_FIELD_NAME . ' ASC');
     }
 
 }
index 0970f6e..0f1b3a5 100644 (file)
@@ -62,7 +62,8 @@ abstract class base_activity extends base_mod {
      */
     public function get_recordset_by_timestamp($modifiedfrom = 0) {
         global $DB;
-        return $DB->get_recordset_select($this->get_module_name(), static::MODIFIED_FIELD_NAME . ' >= ?', array($modifiedfrom));
+        return $DB->get_recordset_select($this->get_module_name(), static::MODIFIED_FIELD_NAME . ' >= ?', array($modifiedfrom),
+                static::MODIFIED_FIELD_NAME . ' ASC');
     }
 
     /**
index 8a638d7..1adfeef 100644 (file)
@@ -41,8 +41,8 @@ class mock_search_area extends \core_search\area\base {
     public function get_recordset_by_timestamp($modifiedfrom = 0) {
         global $DB;
 
-        // Filter by capability as we want this quick.
-        return $DB->get_recordset_sql("SELECT * FROM {temp_mock_search_area} WHERE timemodified >= ?", array($modifiedfrom));
+        $sql = "SELECT * FROM {temp_mock_search_area} WHERE timemodified >= ? ORDER BY timemodified ASC";
+        return $DB->get_recordset_sql($sql, array($modifiedfrom));
     }
 
     public function get_document($record, $options = array()) {