Move the update of digestmailtimelast to the end of the

loop to allow digests to be sent over multiple cron
executions if something crash in the middle. MDL-11657

Merged from MOODLE_19_STABLE

diff --git a/mod/forum/lib.php b/mod/forum/lib.php
index 7091a858e86fd313583d1d9fa5e3494684e969f0..43077f4e2e8311a632631a89edf591f32ad966f9 100644 (file)
--- a/mod/forum/lib.php
+++ b/mod/forum/lib.php
@@ -512,8 +512,6 @@ function forum_cron() {
 
     if ($CFG->digestmailtimelast < $digesttime and $timenow > $digesttime) {
 
-        set_config('digestmailtimelast', $timenow);
-
         mtrace('Sending forum digests: '.userdate($timenow, '', $sitetimezone));
 
         $digestposts_rs = get_recordset('forum_queue');
@@ -738,6 +736,8 @@ function forum_cron() {
                 }
             }
         }
+    /// We have finishied all digest emails, update $CFG->digestmailtimelast
+        set_config('digestmailtimelast', $timenow);
     }
 
     if (!empty($usermailcount)) {