Merge branch 'MDL-65051-master' of git://github.com/lameze/moodle
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Wed, 27 Mar 2019 21:27:25 +0000 (22:27 +0100)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Wed, 27 Mar 2019 21:27:25 +0000 (22:27 +0100)
lib/classes/task/plagiarism_cron_task.php
lib/plagiarismlib.php
plagiarism/lib.php
plagiarism/upgrade.txt

index b5010a8..9d600ff 100644 (file)
@@ -44,8 +44,22 @@ class plagiarism_cron_task extends scheduled_task {
     public function execute() {
         global $CFG;
 
-        require_once($CFG->libdir.'/plagiarismlib.php');
-        plagiarism_cron();
+        if (!empty($CFG->enableplagiarism)) {
+            require_once($CFG->libdir.'/plagiarismlib.php');
+            $plagiarismplugins = plagiarism_load_available_plugins();
+            foreach ($plagiarismplugins as $plugin => $dir) {
+                require_once($dir . '/lib.php');
+                $plagiarismclass = "plagiarism_plugin_$plugin";
+                $plagiarismplugin = new $plagiarismclass;
+                if (method_exists($plagiarismplugin, 'cron')) {
+                    mtrace('Processing cron function for plagiarism_plugin_' . $plugin . '...', '');
+                    cron_trace_time_and_memory();
+                    mtrace('It has been detected the class ' . $plagiarismclass . ' has a legacy cron method
+                            implemented. Plagiarism plugins should implement their own schedule tasks.', '');
+                    $plagiarismplugin->cron();
+                }
+            }
+        }
     }
 
 }
index 0492ad1..5f6207f 100644 (file)
@@ -160,27 +160,7 @@ function plagiarism_print_disclosure($cmid) {
     }
     return $output;
 }
-/**
- * used by admin/cron.php to get similarity scores from submitted files.
- *
- */
-function plagiarism_cron() {
-    global $CFG;
-    if (empty($CFG->enableplagiarism)) {
-        return '';
-    }
-    $plagiarismplugins = plagiarism_load_available_plugins();
-    foreach($plagiarismplugins as $plugin => $dir) {
-        require_once($dir.'/lib.php');
-        $plagiarismclass = "plagiarism_plugin_$plugin";
-        $plagiarismplugin = new $plagiarismclass;
-        if (method_exists($plagiarismplugin, 'cron')) {
-            mtrace('Processing cron function for plagiarism_plugin_' . $plugin . '...', '');
-            cron_trace_time_and_memory();
-            $plagiarismplugin->cron();
-        }
-    }
-}
+
 /**
  * helper function - also loads lib file of plagiarism plugin
  * @return array of available plugins
index 9cfd91d..c86c5da 100644 (file)
@@ -97,17 +97,4 @@ abstract class plagiarism_plugin {
      */
     public function update_status($course, $cm) {
     }
-
-    /**
-     * Deprecated cron method.
-     *
-     * This method was added by mistake in the previous versions of Moodle, do not override it since it is never called.
-     * To implement cron you need to register a scheduled task, see https://docs.moodle.org/dev/Task_API.
-     * For backward compatibility with the old cron API the method cron() from this class can also be used.
-     *
-     * @deprecated since Moodle 3.1 MDL-52702 - please use scheduled tasks instead.
-     */
-    public function plagiarism_cron() {
-        debugging('plagiarism_plugin::plagiarism_cron() is deprecated. Please use scheduled tasks instead', DEBUG_DEVELOPER);
-    }
 }
index 67d625c..e43b510 100644 (file)
@@ -1,4 +1,7 @@
 This files describes API changes for code that uses the plagiarism API.
+=== 3.7 ===
+
+* The plagiarism_cron() function has been deleted, plugins should implement their own scheduled tasks.
 
 === 3.4 ===