Changeset 13340 for branches/10.0/src/bp-messages/classes/class-bp-messages-thread.php

Timestamp:

10/11/2022 07:22:19 PM (3 years ago)

Author:

imath

Message:

BP Messages: make sure Threads unread count is consistent

[13102] introduced a regression considering the way we used to set a thread's unread count. This count is set using the list of recipient objects (which include an unread_count property for each user object) by checking for the requested user ID into this list to find the corresponding unread count.

Slicing this list into smaller parts to paginate recipients results introduced potential cases where the requested user ID wasn't found into this paginated list of recipients.

Before doing the slice operation, we need to set the requested user ID thread's unread count to be sure it's the right one and not a null value.

Props sjregan

Fixes #8750 (branch 10.0)

File:

• branches/10.0/src/bp-messages/classes/class-bp-messages-thread.php (modified) (3 diffs)

branches/10.0/src/bp-messages/classes/class-bp-messages-thread.php

@@ -201 +201 @@
         }
 
-        // Fetch the recipients.
+        // Fetch the recipients and set the displayed/logged in user's unread count.
         $this->recipients = $this->get_recipients( $thread_id, $r );
-
-        // Get the unread count for the user.
-        if ( isset( $this->recipients[ $r['user_id'] ] ) ) {
-            $this->unread_count = $this->recipients[ $r['user_id'] ]->unread_count;
-        }
 
         // Grab all message meta.
@@ -312 +307 @@
         }
 
+        // Set the unread count for the user.
+        if ( isset( $r['user_id'] ) && $r['user_id'] && isset( $recipients[ $r['user_id'] ]->unread_count ) ) {
+            $this->unread_count = (int) $recipients[ $r['user_id'] ]->unread_count;
+        }
+
         // Paginate the results.
         if ( ! empty( $recipients ) && $r['recipients_per_page'] && $r['recipients_page'] ) {
             $start      = ( $r['recipients_page'] - 1 ) * ( $r['recipients_per_page'] );
-            $recipients = array_slice( $recipients, $start, $r['recipients_per_page'] );
+            $recipients = array_slice( $recipients, $start, $r['recipients_per_page'], true );
         }
 
@@ -757 +757 @@
                 'ASC',
                 array(
+                    'user_id'             => $r['user_id'],
                     'update_meta_cache'   => false,
                     'recipients_page'     => $r['recipients_page'],