Changeset 13036 for trunk/src/bp-templates/bp-nouveau/includes/messages/loader.php

Timestamp:

08/10/2021 04:55:25 PM (4 years ago)

Author:

imath

Message:

BP Nouveau: include a notification to the active Site Wide Notice

When a Site Wide notice is active the BP Nouveau template pack is prepending a notification to the logged in user notifications list.

FYI: This feature was "missing in action" since the introduction of the bp_notifications_get_grouped_notifications_for_user() and its use inside the bp_notifications_get_notifications_for_user() function in replacement of bp_notifications_get_all_notifications_for_user().

Fixes #8547

File:

• trunk/src/bp-templates/bp-nouveau/includes/messages/loader.php (modified) (2 diffs)

trunk/src/bp-templates/bp-nouveau/includes/messages/loader.php

@@ -4 +4 @@
  *
  * @since 3.0.0
- * @version 3.0.0
+ * @version 10.0.0
  */
 
@@ -89 +89 @@
      */
     protected function setup_filters() {
-        // Enqueue specific styles
+        // Enqueue specific styles.
         add_filter( 'bp_nouveau_enqueue_styles', 'bp_nouveau_messages_enqueue_styles', 10, 1 );
 
-        // Register messages scripts
+        // Register messages scripts.
         add_filter( 'bp_nouveau_register_scripts', 'bp_nouveau_messages_register_scripts', 10, 1 );
 
-        // Localize Scripts
+        // Localize Scripts.
         add_filter( 'bp_core_get_js_strings', 'bp_nouveau_messages_localize_scripts', 10, 1 );
 
-        // Notices
-        add_filter( 'bp_messages_single_new_message_notification', 'bp_nouveau_format_notice_notification_for_user', 10, 1 );
-        add_filter( 'bp_notifications_get_all_notifications_for_user', 'bp_nouveau_add_notice_notification_for_user', 10, 2 );
+        // Notices.
+        add_filter( 'bp_core_get_notifications_for_user', 'bp_nouveau_add_notice_notification_for_user', 10, 2 );
 
-        // Messages
+        // Messages.
         add_filter( 'bp_messages_admin_nav', 'bp_nouveau_messages_adjust_admin_nav', 10, 1 );
     }