Permalink
Browse files

MDL-31426 forum: don't show/allow any subscription to not enrolled users

  • Loading branch information...
1 parent c0ac40e commit 0e487bcfde11d146d032407d441fbf78504b3b46 @stronk7 stronk7 committed Jan 30, 2012
Showing with 9 additions and 3 deletions.
  1. +9 −3 mod/forum/index.php
View
@@ -149,8 +149,13 @@
}
}
-/// Do course wide subscribe/unsubscribe
-if (!is_null($subscribe) and !isguestuser()) {
+// Do course wide subscribe/unsubscribe if requested
+if (!is_null($subscribe)) {
+ if (isguestuser() or !$can_subscribe) {
+ // there should not be any links leading to this place, just redirect
+ redirect(new moodle_url('/mod/forum/index.php', array('id' => $id)), get_string('subscribeenrolledonly', 'forum'));
+ }
+ // Can proceed now, the user is not guest and is enrolled
foreach ($modinfo->instances['forum'] as $forumid=>$cm) {
$forum = $forums[$forumid];
$modcontext = get_context_instance(CONTEXT_MODULE, $cm->id);
@@ -411,7 +416,8 @@
$PAGE->set_button($searchform);
echo $OUTPUT->header();
-if (!isguestuser() && isloggedin()) {
+// Show the subscribe all options only to non-guest, enrolled users
+if (!isguestuser() && isloggedin() && $can_subscribe) {
echo $OUTPUT->box_start('subscription');
echo html_writer::tag('div',
html_writer::link(new moodle_url('/mod/forum/index.php', array('id'=>$course->id, 'subscribe'=>1, 'sesskey'=>sesskey())),

0 comments on commit 0e487bc

Please sign in to comment.