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

Timestamp:

10/19/2023 11:34:45 PM (3 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 );