Changeset 11578 for trunk/src/bp-messages/bp-messages-cache.php

Timestamp:

06/06/2017 05:46:21 PM (7 years ago)

Author:

dcavins

Message:

Show displayed user's messages on messages screens.

When a site administrator visits another user’s messages, she is shown
her own messages because the display logic for messages refers to the
logged-in user rather than the displayed user. This changeset updates
the logic to allow the site admin to view and interact with another
user’s messages.

Fixes #7096.

File:

• trunk/src/bp-messages/bp-messages-cache.php (modified) (2 diffs)

trunk/src/bp-messages/bp-messages-cache.php

@@ -75 +75 @@
  * @param int|array $thread_ids If single thread, the thread ID.
  *                              Otherwise, an array of thread IDs.
+ * @param int       $user_id    ID of the user that the threads were deleted for.
  */
-function bp_messages_clear_cache_on_message_delete( $thread_ids ) {
+function bp_messages_clear_cache_on_message_delete( $thread_ids, $user_id ) {
     // Delete thread and thread recipient cache.
     foreach( (array) $thread_ids as $thread_id ) {
@@ -84 +85 @@
 
     // Delete unread count for logged-in user.
-    wp_cache_delete( bp_loggedin_user_id(), 'bp_messages_unread_count' );
+    wp_cache_delete( $user_id, 'bp_messages_unread_count' );
 }
-add_action( 'messages_delete_thread', 'bp_messages_clear_cache_on_message_delete' );
+add_action( 'messages_delete_thread', 'bp_messages_clear_cache_on_message_delete', 10, 2 );
 
 /**