Merge branch 'MDL-53778' of https://github.com/stronk7/moodle
authorAndrew Nicols <andrew@nicols.co.uk>
Thu, 27 Jun 2019 00:35:56 +0000 (08:35 +0800)
committerAndrew Nicols <andrew@nicols.co.uk>
Thu, 27 Jun 2019 00:35:56 +0000 (08:35 +0800)
media/classes/manager.php
media/upgrade.txt
search/classes/manager.php
search/upgrade.txt

index 39bca4c..2b24d86 100644 (file)
@@ -138,19 +138,10 @@ final class core_media_manager {
     }
 
     /**
-     * Setup page requirements.
-     *
-     * This should must only be called once per page request.
-     *
-     * @deprecated Moodle 3.3, The setup is now done in ::instance() so there is no need to call this
-     * @param moodle_page $page The page we are going to add requirements to.
-     * @see core_media_manager::instance()
-     * @todo MDL-57632 final deprecation
+     * @deprecated since Moodle 3.3. The setup is now done in ::instance() so there is no need to call this.
      */
-    public function setup($page) {
-        debugging('core_media_manager::setup() is deprecated.' .
-                  'You only need to call core_media_manager::instance() now', DEBUG_DEVELOPER);
-        // No need to call ::instance from here, because the instance has already be set up.
+    public function setup() {
+        throw new coding_exception('core_media_manager::setup() can not be used any more because it is done in ::instance()');
     }
 
     /**
index 8e1594c..b49be62 100644 (file)
@@ -1,8 +1,11 @@
 This files describes API changes in /media/ plugins,
 information provided here is intended especially for developers.
 
-=== 3.6 ===
+=== 3.8 ===
+* The final deprecation of core_media_manager::setup() means that this function will no longer be called.
+The setup is now done in ::instance() so there is no need to call this.
 
+=== 3.6 ===
 * The following functions have been finally deprecated and can not be used anymore:
   * core_media_player::is_enabled()
   * core_media_player::compare_by_rank()
index f462480..fea5402 100644 (file)
@@ -1120,11 +1120,7 @@ class manager {
             if (count($result) === 5) {
                 list($numrecords, $numdocs, $numdocsignored, $lastindexeddoc, $partial) = $result;
             } else {
-                // Backward compatibility for engines that don't support partial adding.
-                list($numrecords, $numdocs, $numdocsignored, $lastindexeddoc) = $result;
-                debugging('engine::add_documents() should return $partial (4-value return is deprecated)',
-                        DEBUG_DEVELOPER);
-                $partial = false;
+                throw new coding_exception('engine::add_documents() should return $partial (4-value return is deprecated)');
             }
 
             if ($numdocs > 0) {
index 7453c2c..6603406 100644 (file)
@@ -1,6 +1,13 @@
 This files describes API changes in /search/*,
 information provided here is intended especially for developers.
 
+=== 3.8 ===
+
+* Search indexing supports time limits to make the scheduled task run more neatly since 3.4. In order for
+  this to work, search engine plugins will need to implement the 'stopat' parameter if they
+  override the add_documents() function, and return an extra parameter from this function (see base
+  class in engine.php). Unmodified plugins will not work anymore.
+
 === 3.7 ===
 
 * Search areas now have categories and can optionally implement get_category_names method to