Changeset 4554

Timestamp:

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

Author:

johnjamesjacoby

Message:

BuddyPress Groups:

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

File:

• branches/2.2/includes/extend/buddypress/group.php (modified) (8 diffs)

branches/2.2/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';
-
-        // Add handlers to bp_actions
-        add_action( 'bp_actions', 'bbp_new_forum_handler'  );
-        add_action( 'bp_actions', 'bbp_new_topic_handler'  );
-        add_action( 'bp_actions', 'bbp_new_reply_handler'  );
-        add_action( 'bp_actions', 'bbp_edit_forum_handler' );
-        add_action( 'bp_actions', 'bbp_edit_topic_handler' );
-        add_action( 'bp_actions', 'bbp_edit_reply_handler' );
+    }
+
+    /**
+     * 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
@@ -92 +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' ) );
@@ -121 +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;
     }
 
@@ -578 +618 @@
 
                         bbp_set_query_name( 'bbp_forum_form' );
-
                         bbp_get_template_part( 'form', 'forum' );
 
@@ -632 +671 @@
                         if ( !empty( $_GET['action'] ) && 'merge' == $_GET['action'] ) :
                             bbp_set_query_name( 'bbp_topic_merge' );
-
                             bbp_get_template_part( 'form', 'topic-merge' );
 
@@ -638 +676 @@
                         elseif ( !empty( $_GET['action'] ) && 'split' == $_GET['action'] ) :
                             bbp_set_query_name( 'bbp_topic_split' );
-
                             bbp_get_template_part( 'form', 'topic-split' );
 
@@ -650 +687 @@
                     // Single Topic
                     else:
-
                         bbp_set_query_name( 'bbp_single_topic' ); ?>