Changeset 4553

Timestamp:

12/07/2012 09:09:10 AM (11 years ago)

Author:

johnjamesjacoby

Message:

BuddyPress Groups:

• If group does not have forums active, do not show forum tab.
• See #2083 (trunk)

File:

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

trunk/includes/extend/buddypress/group.php

@@ -32 +32 @@
      */
     public function __construct() {
+        $this->setup_variables();
+        $this->setup_actions();
+        $this->setup_filters();
+        $this->maybe_unset_forum_menu();
+    }
+
+    /**
+     * Setup the group forums class variables
+     *
+     * @since bbPress ()
+     */
+    private function setup_variables() {
 
         // Name and slug
@@ -55 +67 @@
         $this->template_file        = 'groups/single/plugins';
         $this->display_hook         = 'bp_template_content';
+    }
+
+    /**
+     * Setup the group forums class actions
+     *
+     * @since bbPress (r4552)
+     */
+    private function setup_actions() {
 
         // Possibly redirect
-        add_action( 'bbp_template_redirect',     array( $this, 'redirect_canonical' ) );
+        add_action( 'bbp_template_redirect',         array( $this, 'redirect_canonical'        ) );
+
+        // Remove topic cap map when view is done
+        add_action( 'bbp_after_group_forum_display', array( $this, 'remove_topic_meta_cap_map' ) );
+    }
+
+    /**
+     * Setup the group forums class filters
+     *
+     * @since bbPress (r4552)
+     */
+    private function setup_filters() {
 
         // Group forum pagination
@@ -84 +115 @@
         add_filter( 'bbp_map_reply_meta_caps',   array( $this, 'map_topic_meta_caps'          ), 10, 4 );
 
-        // Remove topic cap map when view is done
-        add_action( 'bbp_after_group_forum_display', array( $this, 'remove_topic_meta_cap_map' ) );
-
         // Map group forum activity items to groups
         add_filter( 'bbp_before_record_activity_parse_args', array( $this, 'map_activity_to_group' ) );
@@ -113 +141 @@
 
         $this->display_forums( 0 );
+    }
+
+    /**
+     * Maybe unset the group forum nav item if group does not have a forum
+     *
+     * @since bbPress (r4552)
+     *
+     * @return If not viewing a single group
+     */
+    public function maybe_unset_forum_menu() {
+
+        // Bail if not viewing a single group
+        if ( ! bp_is_group() )
+            return;
+
+        // Are forums enabled for this group?
+        $checked = (bool) ( bp_get_new_group_enable_forum() );
+
+        // Tweak the nav item variable based on if group has forum or not
+        $this->enable_nav_item = $checked;
     }