Changeset 6369

Timestamp:

03/07/2017 07:13:52 PM (8 years ago)

Author:

johnjamesjacoby

Message:

Engagements: Move engagement count updates to action hooks.

Props thebrandonallen. Fixes #3068.

Location:

trunk/src/includes

Files:

• core/actions.php (modified) (1 diff)
• replies/functions.php (modified) (1 diff)

trunk/src/includes/core/actions.php

@@ -310 +310 @@
 add_action( 'bbp_insert_reply', 'bbp_insert_reply_update_counts', 10, 3 );
 
+// Update topic voice counts.
+add_action( 'bbp_new_reply',        'bbp_update_topic_voice_count' );
+add_action( 'bbp_trashed_reply',    'bbp_update_topic_voice_count' );
+add_action( 'bbp_untrashed_reply',  'bbp_update_topic_voice_count' );
+add_action( 'bbp_spammed_reply',    'bbp_update_topic_voice_count' );
+add_action( 'bbp_unspammed_reply',  'bbp_update_topic_voice_count' );
+add_action( 'bbp_approved_reply',   'bbp_update_topic_voice_count' );
+add_action( 'bbp_unapproved_reply', 'bbp_update_topic_voice_count' );
+
+// Insert reply voice counts.
+add_action( 'bbp_insert_reply', 'bbp_update_topic_voice_count' );
+
 // Topic status transition helpers for replies
 add_action( 'bbp_trash_topic',   'bbp_trash_topic_replies'   );

trunk/src/includes/replies/functions.php

@@ -1023 +1023 @@
                 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 );
                 }