diff --git a/mod/forum/db/access.php b/mod/forum/db/access.php index 0828dfc95e4..724a6945431 100644 --- a/mod/forum/db/access.php +++ b/mod/forum/db/access.php @@ -33,20 +33,6 @@ $mod_forum_capabilities = array( - 'mod/forum:viewforum' => array( - - 'captype' => 'read', - 'contextlevel' => CONTEXT_MODULE, - 'legacy' => array( - 'guest' => CAP_ALLOW, - 'student' => CAP_ALLOW, - 'teacher' => CAP_ALLOW, - 'editingteacher' => CAP_ALLOW, - 'coursecreator' => CAP_ALLOW, - 'admin' => CAP_ALLOW - ) - ), - 'mod/forum:viewdiscussion' => array( 'captype' => 'read', diff --git a/mod/forum/index.php b/mod/forum/index.php index 39c12e90e88..dbd892f2b37 100644 --- a/mod/forum/index.php +++ b/mod/forum/index.php @@ -93,7 +93,7 @@ $cm = get_coursemodule_from_instance("forum", $forum->id, $course->id); $context = get_context_instance(CONTEXT_MODULE, $cm->id); - if (!has_capability('mod/forum:viewforum', $context)) { + if (!has_capability('mod/forum:viewdiscussions', $context)) { if (isset($forum->keyreference)) { unset($learningforums[$forum->keyreference]); } diff --git a/mod/forum/lib.php b/mod/forum/lib.php index 8759e3001fb..95bd34ff03b 100644 --- a/mod/forum/lib.php +++ b/mod/forum/lib.php @@ -1183,8 +1183,7 @@ function forum_get_readable_forums($userid, $courseid=0) { $forum->accessallgroups = true; } - if (has_capability('mod/forum:viewforum', $forumcontext) - && has_capability('mod/forum:viewdiscussion', $forumcontext)) { + if (has_capability('mod/forum:viewdiscussion', $forumcontext)) { $forum->viewhiddentimedposts = has_capability('mod/forum:viewhiddentimedposts', $forumcontext); @@ -4513,7 +4512,6 @@ function forum_convert_to_roles($forum, $forummodid, $teacherroles=array(), // Create overrides for default student and guest roles (prevent). foreach ($studentroles as $studentrole) { - assign_capability('mod/forum:viewforum', CAP_PREVENT, $studentrole->id, $context->id); assign_capability('mod/forum:viewdiscussion', CAP_PREVENT, $studentrole->id, $context->id); assign_capability('mod/forum:viewhiddentimedposts', CAP_PREVENT, $studentrole->id, $context->id); assign_capability('mod/forum:startdiscussion', CAP_PREVENT, $studentrole->id, $context->id); @@ -4533,7 +4531,6 @@ function forum_convert_to_roles($forum, $forummodid, $teacherroles=array(), assign_capability('mod/forum:throttlingapplies', CAP_PREVENT, $studentrole->id, $context->id); } foreach ($guestroles as $guestrole) { - assign_capability('mod/forum:viewforum', CAP_PREVENT, $guestrole->id, $context->id); assign_capability('mod/forum:viewdiscussion', CAP_PREVENT, $guestrole->id, $context->id); assign_capability('mod/forum:viewhiddentimedposts', CAP_PREVENT, $guestrole->id, $context->id); assign_capability('mod/forum:startdiscussion', CAP_PREVENT, $guestrole->id, $context->id); diff --git a/mod/forum/view.php b/mod/forum/view.php index 5eac0fc9ac1..ad4c492e705 100644 --- a/mod/forum/view.php +++ b/mod/forum/view.php @@ -78,7 +78,7 @@ notice(get_string("activityiscurrentlyhidden")); } - if (!has_capability('mod/forum:viewforum', $context)) { + if (!has_capability('mod/forum:viewdiscussions', $context)) { notice(get_string('noviewdiscussionspermission', 'forum')); }