Changeset 13614 for trunk/src/bp-notifications/bp-notifications-adminbar.php

Timestamp:

10/19/2023 11:34:45 PM (2 years ago)

Author:

imath

Message:

Introduce Admin Notifications to warn Admin about important BP changes

• Add a new Administration screen to list Admin notifications.
• Put the logic in place to dismiss an Admin notification.
• Add a 11.4 update task to add unread Admin notifications.
• Create a 11.4 Admin Notification to inform Admin users about whether they need to install BP Classic to be ready for BuddyPress 12.0.0

Props dcavins

Closes ​https://github.com/buddypress/buddypress/pull/176
See #9007 (trunk)

File:

• trunk/src/bp-notifications/bp-notifications-adminbar.php (modified) (2 diffs)

trunk/src/bp-notifications/bp-notifications-adminbar.php

@@ -23 +23 @@
  */
 function bp_notifications_toolbar_menu() {
-    global $wp_admin_bar;
-
     if ( ! is_user_logged_in() ) {
         return false;
@@ -30 +28 @@
 
     $notifications = bp_notifications_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 . '">' . number_format_i18n( $count ) . '</span>';
-    $menu_link   = bp_loggedin_user_url( bp_members_get_path_chunks( array( bp_get_notifications_slug() ) ) );
+    $menu_link     = bp_loggedin_user_url( bp_members_get_path_chunks( array( bp_get_notifications_slug() ) ) );
 
-    // Add the top-level Notifications button.
-    $wp_admin_bar->add_node( array(
-        'parent' => 'top-secondary',
-        'id'     => 'bp-notifications',
-        'title'  => $menu_title,
-        'href'   => $menu_link,
-    ) );
-
-    if ( ! empty( $notifications ) ) {
-        foreach ( (array) $notifications as $notification ) {
-            $wp_admin_bar->add_node( array(
-                'parent' => 'bp-notifications',
-                'id'     => 'notification-' . $notification->id,
-                'title'  => $notification->content,
-                'href'   => $notification->href,
-            ) );
-        }
-    } else {
-        $wp_admin_bar->add_node( array(
-            'parent' => 'bp-notifications',
-            'id'     => 'no-notifications',
-            'title'  => __( 'No new notifications', 'buddypress' ),
-            'href'   => $menu_link,
-        ) );
-    }
-
-    return true;
+    return bp_members_admin_bar_notifications_dropdown( $notifications, $menu_link );
 }
-add_action( 'admin_bar_menu', 'bp_members_admin_bar_notifications_menu', 90 );