Changeset 39180 for trunk/src/wp-includes/theme.php

Timestamp:

11/09/2016 05:44:14 AM (10 years ago)

Author:

westonruter

Message:

Customize: Prevent post_content and post_name from being modified when trashing customize_changeset posts.

See #30937.
Fixes #38719.

File:

• trunk/src/wp-includes/theme.php (modified) (2 diffs)

trunk/src/wp-includes/theme.php

@@ -2550 +2550 @@
  */
 function _wp_customize_publish_changeset( $new_status, $old_status, $changeset_post ) {
-        global $wp_customize;
+        global $wp_customize, $wpdb;
 
         $is_publishing_changeset = (
@@ -2601 +2601 @@
          */
         if ( ! wp_revisions_enabled( $changeset_post ) ) {
-                wp_trash_post( $changeset_post->ID );
+                $post = $changeset_post;
+                $post_id = $changeset_post->ID;
+
+                /*
+                 * The following re-formulates the logic from wp_trash_post() as done in
+                 * wp_publish_post(). The reason for bypassing wp_trash_post() is that it
+                 * will mutate the the post_content and the post_name when they should be
+                 * untouched.
+                 */
+                if ( ! EMPTY_TRASH_DAYS ) {
+                        wp_delete_post( $post_id, true );
+                } else {
+                        /** This action is documented in wp-includes/post.php */
+                        do_action( 'wp_trash_post', $post_id );
+
+                        add_post_meta( $post_id, '_wp_trash_meta_status', $post->post_status );
+                        add_post_meta( $post_id, '_wp_trash_meta_time', time() );
+
+                        $old_status = $post->post_status;
+                        $new_status = 'trash';
+                        $wpdb->update( $wpdb->posts, array( 'post_status' => $new_status ), array( 'ID' => $post->ID ) );
+                        clean_post_cache( $post->ID );
+
+                        $post->post_status = $new_status;
+                        wp_transition_post_status( $new_status, $old_status, $post );
+
+                        /** This action is documented in wp-includes/post.php */
+                        do_action( 'edit_post', $post->ID, $post );
+
+                        /** This action is documented in wp-includes/post.php */
+                        do_action( "save_post_{$post->post_type}", $post->ID, $post, true );
+
+                        /** This action is documented in wp-includes/post.php */
+                        do_action( 'save_post', $post->ID, $post, true );
+
+                        /** This action is documented in wp-includes/post.php */
+                        do_action( 'wp_insert_post', $post->ID, $post, true );
+
+                        /** This action is documented in wp-includes/post.php */
+                        do_action( 'trashed_post', $post_id );
+                }
         }
 }