Changeset 5491

Timestamp:

09/10/2014 11:03:48 PM (11 years ago)

Author:

johnjamesjacoby

Message:

In bbp_split_topic_handler() remove $reply_position cheater code, and rely on bbp_update_reply_position() to recalculate the correct position in the hierarchy after wp_update_post() has finished. Props netweb. Fixes #2624.

File:

• trunk/src/includes/topics/functions.php (modified) (4 diffs)

trunk/src/includes/topics/functions.php

@@ -1669 +1669 @@
     if ( ! empty( $replies ) && ! is_wp_error( $replies ) ) {
 
-        // Calculate starting point for reply positions
-        switch ( $split_option ) {
-
-            // Get topic reply count for existing topic
-            case 'existing' :
-                $reply_position = bbp_get_topic_reply_count( $destination_topic->ID, true );
-                break;
-
-            // Account for new lead topic
-            case 'reply'    :
-                $reply_position = 1;
-                break;
-        }
-
         // Save reply ids
         $reply_ids = array();
@@ -1688 +1674 @@
         // Change the post_parent of each reply to the destination topic id
         foreach ( $replies as $reply ) {
-
-            // Bump the reply position each iteration through the loop
-            $reply_position++;
 
             // Update the reply
@@ -1698 +1681 @@
                 'post_name'   => false, // will be automatically generated
                 'post_parent' => $destination_topic->ID,
-                'menu_order'  => $reply_position,
                 'guid'        => ''
             ) );
@@ -1708 +1690 @@
             bbp_update_reply_topic_id( $reply->ID, $destination_topic->ID                           );
             bbp_update_reply_forum_id( $reply->ID, bbp_get_topic_forum_id( $destination_topic->ID ) );
+
+            // Adjust reply position 
+            bbp_update_reply_position( $reply->ID ); 
 
             // Adjust reply to values