Changeset 13102 for trunk/src/bp-messages/bp-messages-notifications.php

Timestamp:

09/09/2021 04:10:21 AM (4 years ago)

Author:

espellcaste

Message:

Adding support to the offset pagination to Messages and Recipients in a thread.

Previously, the BP_Messages_Thread query class would grab ALL messages created for a thread and ALL its recipients. This commit introduces support to the offset pagination
allowing one to paginate both messages and recipients using parameters with a specific limit and offset.

Props imath
Fixes #8508

File:

• trunk/src/bp-messages/bp-messages-notifications.php (modified) (1 diff)

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

@@ -285 +285 @@
     foreach ( $thread->get_recipients() as $recipient ) {
         foreach ( $message_ids as $message_id ) {
-            bp_notifications_delete_notifications_by_item_id( $recipient->user_id, (int) $message_id, buddypress()->messages->id, 'new_message' );
+            if ( ! empty( $recipient->user_id ) ) {
+                bp_notifications_delete_notifications_by_item_id(
+                    $recipient->user_id,
+                    (int) $message_id,
+                    buddypress()->messages->id,
+                    'new_message'
+                );
+            }
         }
     }