Changeset 6321

Timestamp:

02/26/2017 07:25:15 PM (8 years ago)

Author:

johnjamesjacoby

Message:

Engagements: Allow engagements to be toggled on/off, for forums that have not run the upgrade routine yet.

Also missed a spot in the BuddyPress integration.

See #3068.

Location:

trunk/src

Files:

• includes/admin/settings.php (modified) (2 diffs)
• includes/admin/tools/upgrades.php (modified) (1 diff)
• includes/core/options.php (modified) (1 diff)
• includes/extend/buddypress/loader.php (modified) (1 diff)
• includes/extend/buddypress/members.php (modified) (2 diffs)
• templates/default/bbpress/user-details.php (modified) (1 diff)

trunk/src/includes/admin/settings.php

@@ -156 +156 @@
             ),
 
+            // Allow engagements setting
+            '_bbp_enable_engagements' => array(
+                'title'             => __( 'Engagements', 'bbpress' ),
+                'callback'          => 'bbp_admin_setting_callback_engagements',
+                'sanitize_callback' => 'intval',
+                'args'              => array()
+            ),
+
             // Allow topic tags
             '_bbp_allow_topic_tags' => array(
@@ -602 +610 @@
     <input name="_bbp_enable_subscriptions" id="_bbp_enable_subscriptions" type="checkbox" value="1" <?php checked( bbp_is_subscriptions_active( true ) ); bbp_maybe_admin_setting_disabled( '_bbp_enable_subscriptions' ); ?> />
     <label for="_bbp_enable_subscriptions"><?php esc_html_e( 'Allow users to subscribe to forums and topics', 'bbpress' ); ?></label>
+
+<?php
+}
+
+/**
+ * Allow engagements setting field
+ *
+ * @since 2.0.0 bbPress (r2737)
+ *
+ * @uses checked() To display the checked attribute
+ */
+function bbp_admin_setting_callback_engagements() {
+?>
+
+    <input name="_bbp_enable_engagements" id="_bbp_enable_engagements" type="checkbox" value="1" <?php checked( bbp_is_engagements_active( true ) ); bbp_maybe_admin_setting_disabled( '_bbp_enable_engagements' ); ?> />
+    <label for="_bbp_enable_engagements"><?php esc_html_e( 'Track topics each user engages in', 'bbpress' ); ?></label>
 
 <?php

trunk/src/includes/admin/tools/upgrades.php

@@ -158 +158 @@
  * Upgrade user engagements for bbPress 2.6 and higher
  *
- * @since 2.6.0 bbPress (r63010)
+ * @since 2.6.0 bbPress (r6320)
  *
  * @return array An array of the status code and the message

trunk/src/includes/core/options.php

@@ -232 +232 @@
 
 /**
+ * Checks if engagements feature is enabled.
+ *
+ * @since 2.6.0 bbPress (r6320)
+ *
+ * @param $default bool Optional.Default value true
+ * @uses get_option() To get the engagements option
+ * @return bool Is engagements enabled or not
+ */
+function bbp_is_engagements_active( $default = 1 ) {
+    return (bool) apply_filters( 'bbp_is_engagements_active', (bool) get_option( '_bbp_enable_engagements', $default ) );
+}
+
+/**
  * Are topic tags allowed
  *

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

@@ -220 +220 @@
 
         // Engagements
-        $sub_nav[] = array(
-            'name'            => __( 'Engagements', 'bbpress' ),
-            'slug'            => bbp_get_user_engagements_slug(),
-            'parent_url'      => $forums_link,
-            'parent_slug'     => $this->slug,
-            'screen_function' => 'bbp_member_forums_screen_engagements',
-            'position'        => 60,
-            'item_css_id'     => 'engagements'
-        );
+        if ( bbp_is_engagements_active() ) {
+            $sub_nav[] = array(
+                'name'            => __( 'Engagements', 'bbpress' ),
+                'slug'            => bbp_get_user_engagements_slug(),
+                'parent_url'      => $forums_link,
+                'parent_slug'     => $this->slug,
+                'screen_function' => 'bbp_member_forums_screen_engagements',
+                'position'        => 60,
+                'item_css_id'     => 'engagements'
+            );
+        }
 
         // Favorite topics

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

@@ -115 +115 @@
             } elseif ( bbp_is_subscriptions_active() && bp_is_current_action( bbp_get_user_subscriptions_slug() ) ) {
                 $profile_url = $this->get_subscriptions_permalink( '', $user_id );
+
+            // 'engagements' action
+            } elseif ( bbp_is_engagements_active() && bp_is_current_action( bbp_get_user_engagements_slug() ) ) {
+                $profile_url = $this->get_subscriptions_permalink( '', $user_id );
             }
 
@@ -168 +172 @@
 
     /**
+     * Override bbPress engagements URL with BuddyPress profile URL
+     *
+     * @since 2.6.0 bbPress (r6320)
+     *
+     * @param string $url
+     * @param int $user_id
+     * @return string
+     */
+    public function get_engagements_permalink( $url, $user_id ) {
+
+        // Do not filter if not on BuddyPress root blog
+        if ( ! bp_is_root_blog() ) {
+            return false;
+        }
+
+        $component_slug = bbpress()->extend->buddypress->slug;
+        $url            = trailingslashit( bp_core_get_user_domain( $user_id ) . $component_slug . '/' . bbp_get_user_engagements_slug() );
+        return $url;
+    }
+
+    /**
      * Set favorites and subscriptions query variables if viewing member profile
      * pages.

trunk/src/templates/default/bbpress/user-details.php

@@ -44 +44 @@
             </li>
 
-            <li class="<?php if ( bbp_is_single_user_engagements() ) :?>current<?php endif; ?>">
-                <span class='bbp-user-engagements-created-link'>
-                    <a href="<?php bbp_user_engagements_url(); ?>" title="<?php printf( esc_attr__( "%s's Engagements", 'bbpress' ), bbp_get_displayed_user_field( 'display_name' ) ); ?>"><?php esc_html_e( 'Engagements', 'bbpress' ); ?></a>
-                </span>
-            </li>
+            <?php if ( bbp_is_engagements_active() ) : ?>
+                <li class="<?php if ( bbp_is_single_user_engagements() ) :?>current<?php endif; ?>">
+                    <span class='bbp-user-engagements-created-link'>
+                        <a href="<?php bbp_user_engagements_url(); ?>" title="<?php printf( esc_attr__( "%s's Engagements", 'bbpress' ), bbp_get_displayed_user_field( 'display_name' ) ); ?>"><?php esc_html_e( 'Engagements', 'bbpress' ); ?></a>
+                    </span>
+                </li>
+            <?php endif; ?>
 
             <?php if ( bbp_is_favorites_active() ) : ?>