Ticket #8426: 8426.3.patch

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

@@ -215 +215 @@
  * @since 3.0.0
  *
  * @param int $thread_id ID of the thread being marked as read.
+ * @param int $num_rows The number of affected rows by the "mark read" update query.
  */
-function bp_messages_mark_notification_on_mark_thread( $thread_id ) {
+function bp_messages_mark_notification_on_mark_thread( $thread_id, $num_rows = 0 ) {
+        if ( ! $num_rows ) {
+                return;
+        }
+
         $thread_messages = BP_Messages_Thread::get_messages( $thread_id );
 
         foreach ( $thread_messages as $thread_message ) {
                 bp_notifications_mark_notifications_by_item_id( bp_loggedin_user_id(), $thread_message->id, buddypress()->messages->id, 'new_message' );
         }
 }
-add_action( 'messages_thread_mark_as_read', 'bp_messages_mark_notification_on_mark_thread' );
+add_action( 'messages_thread_mark_as_read', 'bp_messages_mark_notification_on_mark_thread', 10, 2 );
 
 /**
  * When a message is deleted, delete corresponding notifications.

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

@@ -625 +625 @@
                         bp_displayed_user_id() :
                         bp_loggedin_user_id();
 
-                $bp     = buddypress();
-                $retval = $wpdb->query( $wpdb->prepare( "UPDATE {$bp->messages->table_name_recipients} SET unread_count = 0 WHERE user_id = %d AND thread_id = %d", $user_id, $thread_id ) );
+                $bp       = buddypress();
+                $num_rows = $wpdb->query( $wpdb->prepare( "UPDATE {$bp->messages->table_name_recipients} SET unread_count = 0 WHERE user_id = %d AND thread_id = %d", $user_id, $thread_id ) );
 
                 wp_cache_delete( 'thread_recipients_' . $thread_id, 'bp_messages' );
                 wp_cache_delete( $user_id, 'bp_messages_unread_count' );
@@ -635 +635 @@
                  * Fires when messages thread was marked as read.
                  *
                  * @since 2.8.0
+                 * @since 8.0.0 Passes the number of affected rows into the action.
                  *
                  * @param int $thread_id The message thread ID.
+                 * @param int|false $num_rows The number of affected rows by the update query.
+                 *                            False on error.
                  */
-                do_action( 'messages_thread_mark_as_read', $thread_id );
+                do_action( 'messages_thread_mark_as_read', $thread_id, $num_rows );
 
-                return $retval;
+                return $num_rows;
         }
 
         /**
@@ -660 +663 @@
                         bp_displayed_user_id() :
                         bp_loggedin_user_id();
 
-                $bp     = buddypress();
-                $retval = $wpdb->query( $wpdb->prepare( "UPDATE {$bp->messages->table_name_recipients} SET unread_count = 1 WHERE user_id = %d AND thread_id = %d", $user_id, $thread_id ) );
+                $bp       = buddypress();
+                $num_rows = $wpdb->query( $wpdb->prepare( "UPDATE {$bp->messages->table_name_recipients} SET unread_count = 1 WHERE user_id = %d AND thread_id = %d", $user_id, $thread_id ) );
 
                 wp_cache_delete( 'thread_recipients_' . $thread_id, 'bp_messages' );
                 wp_cache_delete( $user_id, 'bp_messages_unread_count' );
@@ -670 +673 @@
                  * Fires when messages thread was marked as unread.
                  *
                  * @since 2.8.0
+                 * @since 8.0.0 Passes the number of affected rows into the action.
                  *
                  * @param int $thread_id The message thread ID.
+                 * @param int|false $num_rows The number of affected rows by the update query.
+                 *                            False on error.
                  */
-                do_action( 'messages_thread_mark_as_unread', $thread_id );
+                do_action( 'messages_thread_mark_as_unread', $thread_id, $num_rows );
 
-                return $retval;
+                return $num_rows;
         }
 
         /**