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

Timestamp:

11/07/2013 05:15:51 PM (13 years ago)

Author:

johnjamesjacoby

Message:

First pass at bp-notifications component. Includes:

• Backwards compatibility for old core functions.
• Screens, functions, classes, and actions for Notifications.
• Improved class methods for getting, updating, and deleting notifications.
• Template parts in bp-legacy for theme compatibility.
• A few basic unit tests.

@todo's:

• Improve template output with dedicated functions, markup, classes, et all.
• More unit tests.
• Pagination links.
• Auto-activate on update, so existing installations do not lose previously core functionality.

See #5148. Props johnjamesjacoby, boonebgorges, r-a-y.

File:

• trunk/bp-members/bp-members-adminbar.php (modified) (1 diff)

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

@@ -140 +140 @@
  */
 function bp_members_admin_bar_notifications_menu() {
-    global $wp_admin_bar;
 
-    if ( !is_user_logged_in() )
+    // Bail if notifications is not active
+    if ( ! bp_is_active( 'notifications' ) ) {
         return false;
-
-    $notifications = bp_core_get_notifications_for_user( bp_loggedin_user_id(), 'object' );
-    $count         = !empty( $notifications ) ? count( $notifications ) : 0;
-    $alert_class   = (int) $count > 0 ? 'pending-count alert' : 'count no-alert';
-    $menu_title    = '<span id="ab-pending-notifications" class="' . $alert_class . '">' . $count . '</span>';
-
-    // Add the top-level Notifications button
-    $wp_admin_bar->add_menu( array(
-        'parent'    => 'top-secondary',
-        'id'        => 'bp-notifications',
-        'title'     => $menu_title,
-        'href'      => bp_loggedin_user_domain(),
-    ) );
-
-    if ( !empty( $notifications ) ) {
-        foreach ( (array) $notifications as $notification ) {
-            $wp_admin_bar->add_menu( array(
-                'parent' => 'bp-notifications',
-                'id'     => 'notification-' . $notification->id,
-                'title'  => $notification->content,
-                'href'   => $notification->href
-            ) );
-        }
-    } else {
-        $wp_admin_bar->add_menu( array(
-            'parent' => 'bp-notifications',
-            'id'     => 'no-notifications',
-            'title'  => __( 'No new notifications', 'buddypress' ),
-            'href'   => bp_loggedin_user_domain()
-        ) );
     }
 
-    return;
+    bp_notifications_toolbar_menu();
 }
 add_action( 'admin_bar_menu', 'bp_members_admin_bar_notifications_menu', 90 );