Changeset 5126

Timestamp:

09/08/2011 11:34:17 AM (14 years ago)

Author:

boonebgorges

Message:

Makes BP respect show_admin_bar_front and _admin when using WP Admin Bar. Fixes #3560

File:

• trunk/bp-core/bp-core-buddybar.php (modified) (2 diffs)

trunk/bp-core/bp-core-buddybar.php

@@ -525 +525 @@
 
 /**
+ * Retrieve the admin bar display preference of a user based on context.
+ *
+ * This is a direct copy of WP's private _get_admin_bar_pref()
+ *
+ * @since 1.5.0
+ *
+ * @param string $context Context of this preference check, either 'admin' or 'front'.
+ * @param int $user Optional. ID of the user to check, defaults to 0 for current user.
+ *
+ * @uses get_user_option()
+ *
+ * @return bool Whether the admin bar should be showing for this user.
+ */
+function bp_get_admin_bar_pref( $context, $user = 0 ) {
+    $pref = get_user_option( "show_admin_bar_{$context}", $user );
+    if ( false === $pref )
+        return true;
+
+    return 'true' === $pref;
+}
+
+/**
  * Handle the Admin Bar/BuddyBar business
  *
@@ -551 +573 @@
     // Show the WordPress admin bar
     if ( bp_use_wp_admin_bar() && $wp_version >= 3.1 ) {
+        // Respect user's admin bar display preferences
+        if ( bp_get_admin_bar_pref( 'front', bp_loggedin_user_id() ) || bp_get_admin_bar_pref( 'admin', bp_loggedin_user_id() ) )
+            return;
+
         show_admin_bar( true );