Changeset 4721

Timestamp:

01/27/2013 12:38:01 AM (12 years ago)

Author:

johnjamesjacoby

Message:

Rearrange the way map_group_forum_meta_caps() is used. See #2089.

File:

• trunk/includes/extend/buddypress/group.php (modified) (3 diffs)

trunk/includes/extend/buddypress/group.php

@@ -111 +111 @@
         add_filter( 'post_type_link',            array( $this, 'post_type_link'               ), 10, 2 );
 
-        // Allow group member to view private/hidden forums
-        add_filter( 'bbp_map_meta_caps',         array( $this, 'map_group_forum_meta_caps'    ), 99, 4 );
-
         // Map group forum activity items to groups
         add_filter( 'bbp_before_record_activity_parse_args', array( $this, 'map_activity_to_group' ) );
@@ -120 +117 @@
         add_filter( 'bbp_get_forum_subforum_count_int', array( $this, 'no_subforums_yet' ) );
 
-        // Group member permissions to view the topic and reply forms
-        add_filter( 'bbp_current_user_can_access_create_topic_form', array( $this, 'form_permissions' ) );
-        add_filter( 'bbp_current_user_can_access_create_reply_form', array( $this, 'form_permissions' ) );
+        /** Caps **************************************************************/
+
+        // Only add these filters if inside a group forum
+        if ( bp_is_single_item() && bp_is_groups_component() && bp_is_current_action( 'forum' ) ) {
+
+            // Allow group member to view private/hidden forums
+            add_filter( 'bbp_map_meta_caps', array( $this, 'map_group_forum_meta_caps' ), 10, 4 );
+
+            // Group member permissions to view the topic and reply forms
+            add_filter( 'bbp_current_user_can_access_create_topic_form', array( $this, 'form_permissions' ) );
+            add_filter( 'bbp_current_user_can_access_create_reply_form', array( $this, 'form_permissions' ) );
+        }
     }
 
@@ -178 +184 @@
      */
     public function map_group_forum_meta_caps( $caps = array(), $cap = '', $user_id = 0, $args = array() ) {
-
-        // Bail if not viewing a group forum
-        if ( ! bp_is_single_item() || ! bp_is_groups_component() || ! bp_is_current_action( 'forum' ) )
-            return $caps;
 
         switch ( $cap ) {