From 48dcaf5892330b60324e30751515b0899ec5e0ea Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Wed, 17 Dec 2008 18:10:58 +0000
Subject: [PATCH] MDL-14876 fixed regressions in user subscriptions - $cm used
 incorrectly instead of $cm->id; merged from MOODLE_19_STABLE

---
 mod/forum/lib.php | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/mod/forum/lib.php b/mod/forum/lib.php
index caae6c3456..284d1d93f0 100644
--- a/mod/forum/lib.php
+++ b/mod/forum/lib.php
@@ -390,8 +390,8 @@ function forum_cron() {
 
             // caching subscribed users of each forum
             if (!isset($subscribedusers[$forumid])) {
-                if ($subusers = forum_subscribed_users($courses[$courseid], $forums[$forumid], 0,
-                        get_context_instance(CONTEXT_MODULE, $coursemodules[$forumid]))) {
+                $modcontext = get_context_instance(CONTEXT_MODULE, $coursemodules[$forumid]->id);
+                if ($subusers = forum_subscribed_users($courses[$courseid], $forums[$forumid], 0, $modcontext)) {
                     foreach ($subusers as $postuser) {
                         // do not try to mail users with stopped email
                         if ($postuser->emailstop) {
@@ -2722,8 +2722,9 @@ function forum_subscribed_users($course, $forum, $groupid=0, $context = NULL) {
               u.mnethostid";
 
     if (forum_is_forcesubscribed($forum)) {
-        if (is_null($context)) {
-            $context = get_context_instance(CONTEXT_MODULE, get_coursemodule_from_instance('forum', $forum->id, $course->id));
+        if (empty($context)) {
+            $cm = get_coursemodule_from_instance('forum', $forum->id, $course->id);
+            $context = get_context_instance(CONTEXT_MODULE, $cm->id);
         }
         $sort = "u.email ASC";
         $results = forum_get_potential_subscribers($context, $groupid, $fields, $sort);
-- 
2.39.5