Changeset 13244 for branches/10.0/src/bp-notifications/bp-notifications-cache.php

Timestamp:

02/22/2022 07:23:34 PM (3 years ago)

Author:

imath

Message:

Clear the notifications user cache when bulk updating/deleting items

This commit also groups PHP Unit tests about notifications cache functions into a specific file.

Props niftythree, oztaser.

Fixes #8637 (branch 10.0)

File:

• branches/10.0/src/bp-notifications/bp-notifications-cache.php (modified) (1 diff)

branches/10.0/src/bp-notifications/bp-notifications-cache.php

@@ -102 +102 @@
         $n = bp_notifications_get_notification( $where_args['id'] );
         bp_notifications_clear_all_for_user_cache( $n->user_id );
+
+        // Get the list of user IDs from notification IDs.
+    } elseif ( isset( $where_args['ids'] ) && $where_args['ids'] ) {
+        $ids = (array) $where_args['ids'];
+
+        $ns = BP_Notifications_Notification::get(
+            array(
+                'id' => $ids,
+            )
+        );
+
+        $user_ids = wp_list_pluck( $ns, 'user_id' );
+        $user_ids = array_unique( $user_ids );
+
+        foreach ( $user_ids as $user_id ) {
+            bp_notifications_clear_all_for_user_cache( $user_id );
+        }
     }
 }