Changeset 3124

Timestamp:

05/09/2011 05:39:31 AM (15 years ago)

Author:

johnjamesjacoby

Message:

Fix bug in topic/reply revisions where log was 1 revision behind.

Location:

branches/plugin/bbp-includes

Files:

• bbp-reply-functions.php (modified) (2 diffs)
• bbp-topic-functions.php (modified) (2 diffs)

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

@@ -326 +326 @@
         if ( !is_wp_error( $bbp->errors ) || !$bbp->errors->get_error_codes() ) {
 
-            $reply_edit_reason = !empty( $_POST['bbp_reply_edit_reason'] ) ? esc_attr( strip_tags( $_POST['bbp_reply_edit_reason'] ) ) : '';
-
-            if ( !empty( $_POST['bbp_log_reply_edit'] ) && 1 == $_POST['bbp_log_reply_edit'] && $revision_id = wp_save_post_revision( $reply_id ) )
-                bbp_update_reply_revision_log( array( 'reply_id' => $reply_id, 'revision_id' => $revision_id, 'author_id' => bbp_get_current_user_id(), 'reason' => $reply_edit_reason ) );
-
             // Add the content of the form to $post as an array
             $reply_data = array(
@@ -340 +335 @@
             // Insert reply
             $reply_id = wp_update_post( $reply_data );
+
+            // Revisions
+            $reply_edit_reason = !empty( $_POST['bbp_reply_edit_reason'] ) ? esc_attr( strip_tags( $_POST['bbp_reply_edit_reason'] ) ) : '';
+
+            if ( !empty( $_POST['bbp_log_reply_edit'] ) && 1 == $_POST['bbp_log_reply_edit'] && $revision_id = wp_save_post_revision( $reply_id ) )
+                bbp_update_reply_revision_log( array( 'reply_id' => $reply_id, 'revision_id' => $revision_id, 'author_id' => bbp_get_current_user_id(), 'reason' => $reply_edit_reason ) );
 
             // Check for missing reply_id or error

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

@@ -308 +308 @@
         if ( !is_wp_error( $bbp->errors ) || !$bbp->errors->get_error_codes() ) {
 
-            $topic_edit_reason = !empty( $_POST['bbp_topic_edit_reason'] ) ? esc_attr( strip_tags( $_POST['bbp_topic_edit_reason'] ) ) : '';
-
-            if ( !empty( $_POST['bbp_log_topic_edit'] ) && 1 == $_POST['bbp_log_topic_edit'] && $revision_id = wp_save_post_revision( $topic_id ) )
-                bbp_update_topic_revision_log( array( 'topic_id' => $topic_id, 'revision_id' => $revision_id, 'author_id' => bbp_get_current_user_id(), 'reason' => $topic_edit_reason ) );
-
             // Stick status
             if ( !empty( $_POST['bbp_stick_topic'] ) && in_array( $_POST['bbp_stick_topic'], array( 'stick', 'super', 'unstick' ) ) ) {
@@ -343 +338 @@
             // Insert topic
             $topic_id = wp_update_post( $topic_data );
+
+            // Revisions
+            $topic_edit_reason = !empty( $_POST['bbp_topic_edit_reason'] ) ? esc_attr( strip_tags( $_POST['bbp_topic_edit_reason'] ) ) : '';
+
+            if ( !empty( $_POST['bbp_log_topic_edit'] ) && 1 == $_POST['bbp_log_topic_edit'] && $revision_id = wp_save_post_revision( $topic_id ) )
+                bbp_update_topic_revision_log( array( 'topic_id' => $topic_id, 'revision_id' => $revision_id, 'author_id' => bbp_get_current_user_id(), 'reason' => $topic_edit_reason ) );
 
             // Check for missing topic_id or error