Changeset 5087

Timestamp:

08/24/2013 02:12:45 PM (11 years ago)

Author:

johnjamesjacoby

Message:

In bbp_user_can_view_forum(), pass the $user_id into bbp_is_user_keymaster().

Fixes issue where bbp_user_can_view_forum() may return incorrect results for users with the Keymaster role.

Props alex-ye. Fixes #2393.

File:

• trunk/includes/users/template.php (modified) (1 diff)

trunk/includes/users/template.php

@@ -1506 +1506 @@
 
     // Validate parsed values
-    $user_id  = bbp_get_user_id ( $r['user_id'], false, false );
+    $user_id  = bbp_get_user_id( $r['user_id'], false, false );
     $forum_id = bbp_get_forum_id( $r['forum_id'] );
     $retval   = false;
 
     // User is a keymaster
-    if ( bbp_is_user_keymaster() ) {
+    if ( bbp_is_user_keymaster( $user_id ) ) {
         $retval = true;
 
     // Forum is public, and user can read forums or is not logged in
-    } elseif ( bbp_is_forum_public ( $forum_id, $r['check_ancestors'] ) ) {
+    } elseif ( bbp_is_forum_public( $forum_id, $r['check_ancestors'] ) ) {
         $retval = true;