Changeset 5880 for trunk/src/includes/replies/functions.php

Timestamp:

07/24/2015 11:21:28 AM (8 years ago)

Author:

netweb

Message:

Replies: In bbp_update_reply_walker() always update the topics last active time with bbp_update_topic_last_active_time() regardless of reply's published status.

This changeset including unit tests ensures that when calling reply actions spam/unspam, trash/untrash, approve/unapprove the reply's parent topic's last active time is updated to the time of the last published reply.

Props netweb. See #2838

File:

• trunk/src/includes/replies/functions.php (modified) (1 diff)

trunk/src/includes/replies/functions.php

@@ -973 +973 @@
                 }
 
-                // Only update if reply is published
-                if ( bbp_is_reply_published( $reply_id ) ) {
-                    bbp_update_topic_last_active_time( $ancestor, $topic_last_active_time );
-                }
+                // Update the topic last active time regardless of reply status.
+                // See https://bbpress.trac.wordpress.org/ticket/2838
+                bbp_update_topic_last_active_time( $ancestor, $topic_last_active_time );
 
                 // Counts