Changeset 24879 for branches/3.6/wp-admin/includes/post.php

Timestamp:

07/29/2013 07:06:10 PM (11 years ago)

Author:

markjaquith

Message:

Delete old autosave if new autosave has same content as the post.

Props nacin. Fixes #7392 for 3.6.

File:

• branches/3.6/wp-admin/includes/post.php (modified) (1 diff)

branches/3.6/wp-admin/includes/post.php

@@ -1321 +1321 @@
         $new_autosave['post_author'] = $post_author;
 
+        // If the new autosave is the same content as the post, delete the old autosave.
+        $post = get_post( $post_id );
+        $autosave_is_different = false;
+        foreach ( array_keys( _wp_post_revision_fields() ) as $field ) {
+            if ( normalize_whitespace( $new_autosave[ $field ] ) != normalize_whitespace( $post->$field ) ) {
+                $autosave_is_different = true;
+                break;
+            }
+        }
+
+        if ( ! $autosave_is_different ) {
+            wp_delete_post_revision( $old_autosave->ID );
+            return;
+        }
+
         return wp_update_post( $new_autosave );
     }
 
     // _wp_put_post_revision() expects unescaped.
-    $_POST = wp_unslash($_POST);
+    $post_data = wp_unslash( $_POST );
 
     // Otherwise create the new autosave as a special post revision
-    return _wp_put_post_revision( $_POST, true );
+    return _wp_put_post_revision( $post_data, true );
 }