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

Timestamp:

02/26/2017 11:12:37 PM (8 years ago)

Author:

johnjamesjacoby

Message:

Engagements: Updates existing functions & unit tests:

• Correct tests so that post_author of 0 does not get included in the overall count
• Add user IDs to all topics & replies where voice counts are being tested
• Update voice-count update function to use the new user-relationships API
• Clean-up topic merge code to more efficiently migrate favorites, subscriptions, and engagements

See #3068.

File:

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

trunk/src/includes/replies/functions.php

@@ -1019 +1019 @@
                 bbp_update_topic_last_active_time( $ancestor, $topic_last_active_time );
 
+                // Always update voice counts
+                bbp_update_topic_voice_count( $ancestor );
+
                 // Only update reply count if we're deleting a reply, or in the dashboard.
                 if ( in_array( current_filter(), array( 'bbp_deleted_reply', 'save_post' ), true ) ) {
                     bbp_update_topic_reply_count(        $ancestor );
                     bbp_update_topic_reply_count_hidden( $ancestor );
-                    bbp_update_topic_voice_count(        $ancestor );
                 }