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

Timestamp:

02/22/2022 07:23:34 PM (5 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 );
+                }
         }
 }