Changeset 4641 for trunk/bp-members/bp-members-adminbar.php

Timestamp:

07/10/2011 03:23:12 PM (14 years ago)

Author:

boonebgorges

Message:

Changes main Notifications WP Admin Bar menu id, to avoid clashing with Settings > Notifications. Fixes #3294. Props johnjamesjacoby

File:

• trunk/bp-members/bp-members-adminbar.php (modified) (3 diffs)

trunk/bp-members/bp-members-adminbar.php

@@ -168 +168 @@
     // Add the top-level Notifications button
     $wp_admin_bar->add_menu( array(
-        'id'    => 'notifications',
+        'id'    => 'bp-notifications',
         'title' => $menu_title,
         'href'  => bp_loggedin_user_domain()
@@ -176 +176 @@
         foreach ( (array)$notifications as $notification ) {
             $wp_admin_bar->add_menu( array(
-                'parent' => 'notifications',
+                'parent' => 'bp-notifications',
                 'id'     => 'notification-' . $notification->id,
                 'title'  => $notification->content,
@@ -184 +184 @@
     } else {
         $wp_admin_bar->add_menu( array(
-            'parent' => 'notifications',
+            'parent' => 'bp-notifications',
             'id'     => 'no-notifications',
             'title'  => __( 'No new notifications', 'buddypress' ),