MDL-49651 mod_forum: Respect course visibility in cron

This checks the courses' current visibility and the users permissions.
If they have the permission to view the course when it is hidden they'll get messages.

We already have checks for the users enrolled & subscription status.
This commit is contained in:
Mathew
2019-01-03 10:50:39 +08:00
committed by Jake Dallimore
parent b63c41d762
commit aaf4b2b489
+6
View File
@@ -658,6 +658,12 @@ function forum_cron() {
}
}
$coursecontext = context_course::instance($course->id);
if (!$course->visible and !has_capability('moodle/course:viewhiddencourses', $coursecontext, $userto->id)) {
// The course is hidden and the user does not have access to it.
continue;
}
// Don't send email if the forum is Q&A and the user has not posted.
// Initial topics are still mailed.
if ($forum->type == 'qanda' && !forum_get_user_posted_time($discussion->id, $userto->id) && $pid != $discussion->firstpost) {