MDL-30635 quiz cron: fixup - enable in version.php & fn name quiz_statistics_cron
authorTim Hunt <T.J.Hunt@open.ac.uk>
Wed, 11 Jan 2012 10:09:30 +0000 (10:09 +0000)
committerTim Hunt <T.J.Hunt@open.ac.uk>
Wed, 11 Jan 2012 14:02:57 +0000 (14:02 +0000)
mod/quiz/report/statistics/lib.php
mod/quiz/version.php

index efa3ed7..0f3e262 100644 (file)
@@ -49,14 +49,14 @@ function quiz_statistics_questiontext_preview_pluginfile($context, $questionid,
 }
 
 /**
-* Quiz statistics report cron code. Deletes cached data more than a certain age.
-*/
-function quiz_report_statistics_cron() {
+ * Quiz statistics report cron code. Deletes cached data more than a certain age.
+ */
+function quiz_statistics_cron() {
     global $DB;
 
     $expiretime = time() - 5*HOURSECS;
-    $todelete = $DB->get_records_select_menu('quiz_statistics', 'timemodified < ?',
-    array($expiretime), '', 'id, 1');
+    $todelete = $DB->get_records_select_menu('quiz_statistics',
+            'timemodified < ?', array($expiretime), '', 'id, 1');
 
     if (!$todelete) {
         return true;
@@ -64,20 +64,14 @@ function quiz_report_statistics_cron() {
 
     list($todeletesql, $todeleteparams) = $DB->get_in_or_equal(array_keys($todelete));
 
-    if (!$DB->delete_records_select('quiz_question_statistics',
-            'quizstatisticsid ' . $todeletesql, $todeleteparams)) {
-    mtrace('Error deleting out of date quiz_question_statistics records.');
-    }
+    $DB->delete_records_select('quiz_question_statistics',
+            'quizstatisticsid ' . $todeletesql, $todeleteparams);
 
-    if (!$DB->delete_records_select('quiz_question_response_stats',
-            'quizstatisticsid ' . $todeletesql, $todeleteparams)) {
-    mtrace('Error deleting out of date quiz_question_response_stats records.');
-    }
+    $DB->delete_records_select('quiz_question_response_stats',
+            'quizstatisticsid ' . $todeletesql, $todeleteparams);
 
-    if (!$DB->delete_records_select('quiz_statistics',
-            'id ' . $todeletesql, $todeleteparams)) {
-    mtrace('Error deleting out of date quiz_statistics records.');
-    }
+    $DB->delete_records_select('quiz_statistics',
+            'id ' . $todeletesql, $todeleteparams);
 
     return true;
 }
index 39f8bfd..33124f0 100644 (file)
@@ -25,7 +25,7 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$module->version   = 2011120701;       // The current module version (Date: YYYYMMDDXX)
+$module->version   = 2011120702;       // The current module version (Date: YYYYMMDDXX)
 $module->requires  = 2011112900;       // Requires this Moodle version
 $module->component = 'mod_quiz';       // Full name of the plugin (used for diagnostics)
-$module->cron      = 0;
+$module->cron      = 60;