Permalink
Browse files

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

  • Loading branch information...
1 parent d4b3034 commit e163fbc943c9c96da89939ff97813d5f7e94cbb1 @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);
@@ -417,7 +422,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 e163fbc

Please sign in to comment.