From: danmarsden <danmarsden>
Date: Sun, 20 Jan 2008 20:05:45 +0000 (+0000)
Subject: Fix for MDL-12979 forum sends out notification to users not in course - due to call... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=c37fdcb550e1c66ecac7feb4413255493152723f;p=moodle.git

Fix for MDL-12979 forum sends out notification to users not in course - due to call to deprec function get_course_users - This also fixes MDL-10163
---

diff --git a/mod/forum/lib.php b/mod/forum/lib.php
index ef155954b3..6f5df3caab 100644
--- a/mod/forum/lib.php
+++ b/mod/forum/lib.php
@@ -1918,7 +1918,11 @@ function forum_subscribed_users($course, $forum, $groupid=0, $cache=false) {
     }
 
     if (forum_is_forcesubscribed($forum)) {
-        $results = get_course_users($course->id);     // Otherwise get everyone in the course
+        $context = get_context_instance(CONTEXT_COURSE, $course->id);
+        $sort = "u.email ASC";
+        $fields ="u.id, u.username, u.firstname, u.lastname, u.maildisplay, u.mailformat, u.maildigest, u.emailstop, u.imagealt,
+                  u.email, u.city, u.country, u.lastaccess, u.lastlogin, u.picture, u.timezone, u.theme, u.lang, u.trackforums, u.mnethostid";
+        $results = get_users_by_capability($context, 'mod/forum:initialsubscriptions', $fields, $sort, '','','','', false, true);
     } else {
         $results = get_records_sql("SELECT u.id, u.username, u.firstname, u.lastname, u.maildisplay, u.mailformat, u.maildigest, u.emailstop, u.imagealt,
                                    u.email, u.city, u.country, u.lastaccess, u.lastlogin, u.picture, u.timezone, u.theme, u.lang, u.trackforums, u.mnethostid