Changeset 3501 for branches/plugin/bbp-includes/bbp-reply-functions.php

Timestamp:

09/09/2011 09:43:33 PM (14 years ago)

Author:

johnjamesjacoby

Message:

Fix issue when topics and replies are flagged as spam/trash where they would still update last active data for ancestors. Fixes #1631.
Fix issue when trashing and deleting a topic, replies may not be properly queried and updated. Fixes #1629.

File:

• branches/plugin/bbp-includes/bbp-reply-functions.php (modified) (4 diffs)

branches/plugin/bbp-includes/bbp-reply-functions.php

@@ -610 +610 @@
 
         // Set transient for throttle check (only on new, not edit)
-        if ( empty( $is_edit ) )
+        if ( empty( $is_edit ) ) {
             set_transient( '_bbp_' . bbp_current_author_ip() . '_last_posted', time() );
+        }
 
         // Website is optional
-        if ( !empty( $bbp_anonymous_website ) )
+        if ( !empty( $bbp_anonymous_website ) ) {
             update_post_meta( $reply_id, '_bbp_anonymous_website', $bbp_anonymous_website, false );
+        }
 
     } else {
-        if ( empty( $is_edit ) && !current_user_can( 'throttle' ) )
+        if ( empty( $is_edit ) && !current_user_can( 'throttle' ) ) {
             update_user_meta( $author_id, '_bbp_last_posted', time() );
+        }
     }
 
@@ -628 +631 @@
 
         // Subscribed and unsubscribing
-        if ( true == $subscribed && false == $subscheck )
+        if ( true == $subscribed && false == $subscheck ) {
             bbp_remove_user_subscription( $author_id, $topic_id );
 
         // Subscribing
-        elseif ( false == $subscribed && true == $subscheck )
+        } elseif ( false == $subscribed && true == $subscheck ) {
             bbp_add_user_subscription( $author_id, $topic_id );
+        }
     }
 
@@ -728 +732 @@
 
             // Last reply and active ID's
-            bbp_update_topic_last_reply_id  ( $ancestor, $reply_id  );
-            bbp_update_topic_last_active_id ( $ancestor, $active_id );
+            bbp_update_topic_last_reply_id ( $ancestor, $reply_id  );
+            bbp_update_topic_last_active_id( $ancestor, $active_id );
 
             // Get the last active time if none was passed
-            if ( empty( $last_active_time ) )
+            $topic_last_active_time = $last_active_time;
+            if ( empty( $last_active_time ) ) {
                 $topic_last_active_time = get_post_field( 'post_date', bbp_get_topic_last_active_id( $ancestor ) );
-            else
-                $topic_last_active_time = $last_active_time;
-
-            bbp_update_topic_last_active_time  ( $ancestor, $topic_last_active_time );
+            }
+
+            // Only update if reply is published
+            if ( bbp_is_reply_published( $reply_id ) ) {
+                bbp_update_topic_last_active_time( $ancestor, $topic_last_active_time );
+            }
 
             // Counts
@@ -754 +761 @@
             bbp_update_forum_last_active_id( $ancestor, $active_id );
 
-            if ( empty( $last_active_time ) )
+            // Get the last active time if none was passed
+            $forum_last_active_time = $last_active_time;
+            if ( empty( $last_active_time ) ) {
                 $forum_last_active_time = get_post_field( 'post_date', bbp_get_forum_last_active_id( $ancestor ) );
-            else
-                $forum_last_active_time = $last_active_time;
-
-            bbp_update_forum_last_active_time( $ancestor, $forum_last_active_time );
+            }
+
+            // Only update if reply is published
+            if ( bbp_is_reply_published( $reply_id ) ) {
+                bbp_update_forum_last_active_time( $ancestor, $forum_last_active_time );
+            }
 
             // Counts