Changeset 6485

Timestamp:

06/04/2017 09:49:50 PM (8 years ago)

Author:

johnjamesjacoby

Message:

BuddyPress: Root group forum updates

• Include ability to create a new group root forum if none exists
• Update group-forum setting verbiage to accommodate above
• When a forum is deleted, check if it's the group root, and delete the setting accordingly
• Introduce bbp_maybe_delete_group_forum_root() and bbp_maybe_create_group_forum_root() functions to carry out the above functionalities

Location:

trunk/src/includes

Files:

• admin/settings.php (modified) (8 diffs)
• extend/buddypress/functions.php (modified) (3 diffs)

trunk/src/includes/admin/settings.php

@@ -55 +55 @@
         ),
         'bbp_settings_single_slugs' => array(
-            'title'    => esc_html__( 'Single Forum Slugs', 'bbpress' ),
+            'title'    => esc_html__( 'Forum Single Slugs', 'bbpress' ),
             'callback' => 'bbp_admin_setting_callback_single_slug_section',
             'page'     => 'permalink',
@@ -65 +65 @@
         ),
         'bbp_settings_buddypress' => array(
-            'title'    => esc_html__( 'BuddyPress Integration', 'bbpress' ),
+            'title'    => esc_html__( 'Forum Integration for BuddyPress', 'bbpress' ),
             'callback' => 'bbp_admin_setting_callback_buddypress_section',
             'page'     => 'buddypress',
         ),
         'bbp_settings_akismet' => array(
-            'title'    => esc_html__( 'Akismet Integration', 'bbpress' ),
+            'title'    => esc_html__( 'Forum Integration for Akismet', 'bbpress' ),
             'callback' => 'bbp_admin_setting_callback_akismet_section',
             'page'     => 'discussion'
@@ -431 +431 @@
             // Are group forums enabled?
             '_bbp_enable_group_forums' => array(
-                'title'             => esc_html__( 'Enable Group Forums', 'bbpress' ),
+                'title'             => esc_html__( 'Group Forums', 'bbpress' ),
                 'callback'          => 'bbp_admin_setting_callback_group_forums',
                 'sanitize_callback' => 'intval',
@@ -439 +439 @@
             // Group forums parent forum ID
             '_bbp_group_forums_root_id' => array(
-                'title'             => esc_html__( 'Group Forums Parent', 'bbpress' ),
+                'title'             => esc_html__( 'Primary Forum', 'bbpress' ),
                 'callback'          => 'bbp_admin_setting_callback_group_forums_root_id',
                 'sanitize_callback' => 'intval',
@@ -1301 +1301 @@
 ?>
 
-    <p><?php esc_html_e( 'Forum settings for BuddyPress', 'bbpress' ); ?></p>
+    <p><?php esc_html_e( 'Configure how Forum settings for BuddyPress', 'bbpress' ); ?></p>
 
 <?php
@@ -1331 +1331 @@
 function bbp_admin_setting_callback_group_forums_root_id() {
 
+    // Group root ID
+    $group_root = bbp_get_group_forums_root_id();
+    if ( ! bbp_get_forum( $group_root ) ) {
+        delete_option( '_bbp_group_forums_root_id' );
+        $group_root = 0;
+    }
+
     // Output the dropdown for all forums
-    bbp_dropdown( array(
-        'selected'           => bbp_get_group_forums_root_id(),
-        'show_none'          => esc_attr__( '&mdash; Forum root &mdash;', 'bbpress' ),
+    $select = bbp_get_dropdown( array(
+        'selected'           => $group_root,
+        'show_none'          => esc_html__( '&mdash; No parent &mdash;', 'bbpress' ),
         'orderby'            => 'title',
         'order'              => 'ASC',
@@ -1340 +1347 @@
         'disable_categories' => false,
         'disabled'           => '_bbp_group_forums_root_id'
-    ) ); ?>
-
-    <?php esc_html_e( 'is the parent for all group forums', 'bbpress' ); ?>
-    <p class="description"><?php esc_html_e( 'Using the Forum Root is not recommended. Changing this does not move existing forums.', 'bbpress' ); ?></p>
+    ) );
+
+    // Check cap one time
+    $can_add_new = current_user_can( 'publish_forums' );
+    $button = '';
+
+    // Text variations based on configuration
+    if ( empty( $group_root ) && ( true === $can_add_new ) ) {
+
+        // New URL
+        $new_url = wp_nonce_url( add_query_arg( array(
+            'page'   => 'bbpress',
+            'create' => 'bbp-group-forum-root'
+        ), admin_url( 'options-general.php' ) ), '_bbp_group_forums_root_id' );
+
+        // Button & text
+        $button = '<a href="' . esc_url( $new_url ) . '">' . esc_html__( 'create a new one', 'bbpress' ) . '</a>';
+        $text   = esc_html__( 'Use %s to contain your group forums, or %s', 'bbpress' );
+    } else {
+        $text = esc_html__( 'Use %s to contain your group forums', 'bbpress' );
+    }
+
+    // Output
+    printf( $text, $select, $button ); ?>
+    <p class="description"><?php esc_html_e( 'Changing this will not move existing forums.', 'bbpress' ); ?></p>
 
 <?php
@@ -1444 +1472 @@
     } ?>
 
-    <select name="_bbp_converter_platform" id="_bbp_converter_platform" /><?php echo $options ?></select>
+    <select name="_bbp_converter_platform" id="_bbp_converter_platform"><?php echo $options ?></select>
     <label for="_bbp_converter_platform"><?php esc_html_e( 'is the previous forum software', 'bbpress' ); ?></label>
 

trunk/src/includes/extend/buddypress/functions.php

@@ -16 +16 @@
 add_filter( 'bbp_is_single_user',   'bbp_filter_is_single_user',    10, 1 );
 add_filter( 'bbp_is_user_home',     'bbp_filter_is_user_home',      10, 1 );
+
+// Group Forum Root
+add_action( 'load-settings_page_bbpress', 'bbp_maybe_create_group_forum_root' );
+add_action( 'bbp_delete_forum',           'bbp_maybe_delete_group_forum_root' );
 
 /** BuddyPress Helpers ********************************************************/
@@ -306 +310 @@
 }
 
+/** Forum Group Root **********************************************************/
+
+/**
+ * Clean up the group root setting if the forum is being deleted
+ *
+ * @since 2.6.0 bbPress (r6479)
+ *
+ * @param int $forum_id The forum ID being deleted
+ */
+function bbp_maybe_delete_group_forum_root( $forum_id = 0 ) {
+
+    // Bail if no forum ID
+    $forum_id = bbp_get_forum_id();
+    if ( empty( $forum_id ) ) {
+        return;
+    }
+
+    // Get the group root
+    $group_root = (int) get_option( '_bbp_group_forums_root_id', 0 );
+
+    // Delete the group root if the forum just got deleted
+    if ( $group_root === $forum_id ) {
+        delete_option( '_bbp_group_forums_root_id' );
+    }
+}
+
+/**
+ * Handle the new group forum root creation
+ *
+ * @since 2.6.0 bbPress (r6479)
+ *
+ * @return
+ */
+function bbp_maybe_create_group_forum_root() {
+
+    // Bail if no nonce
+    if ( empty( $_GET['_wpnonce'] ) || ( empty( $_GET['create'] ) || ( 'bbp-group-forum-root' !== $_GET['create'] ) ) ) {
+        return;
+    }
+
+    // Bail if user cannot publish forums
+    if ( ! current_user_can( 'publish_forums' ) ) {
+        return;
+    }
+
+    // Bail if nonce check fails
+    if ( ! wp_verify_nonce( $_GET['_wpnonce'], '_bbp_group_forums_root_id' ) ) {
+        return;
+    }
+
+    // Create new forum
+    $forum_id = bbp_insert_forum(
+
+        // Post
+        array( 'post_title' => esc_html__( 'Group Forums', 'bbpress' ) ),
+
+        // Meta
+        array( 'forum_type' => 'category' )
+    );
+
+    // Update & redirect
+    if ( ! empty( $forum_id ) ) {
+
+        // Create
+        update_option( '_bbp_group_forums_root_id', $forum_id );
+
+        // Redirect
+        wp_safe_redirect( add_query_arg( array(
+            'page'    => 'bbpress',
+            'updated' => true // Lame, but still supported
+        ), admin_url( 'options-general.php' ) ) );
+        die;
+    }
+}
+
 /** Forum/Group Sync **********************************************************/
 
 /**
  * These functions are used to keep the many-to-many relationships between
- * groups and forums synchronized. Each forum and group stores ponters to each
+ * groups and forums synchronized. Each forum and group stores pointers to each
  * other in their respective meta. This way if a group or forum is deleted
- * their associattions can be updated without much effort.
+ * their associations can be updated without much effort.
  */
 
@@ -512 +591 @@
 
 /**
- * Remove a group from aall forums
+ * Remove a group from all forums
  *
  * @param type $group_id