Changeset 34668

Timestamp:

09/28/2015 07:25:05 PM (9 years ago)

Author:

johnbillion

Message:

Correctly slash post fields when trashing and untrashing posts.

Fixes #27550
Props dmenard, Denis-de-Bernardy

Location:

trunk

Files:

• src/wp-includes/post-functions.php (modified) (2 diffs)
• tests/phpunit/tests/post/slashes.php (modified) (1 diff)

trunk/src/wp-includes/post-functions.php

@@ -2519 +2519 @@
 
     $post['post_status'] = 'trash';
-    wp_insert_post($post);
+    wp_insert_post( wp_slash( $post ) );
 
     wp_trash_post_comments($post_id);
@@ -2566 +2566 @@
     delete_post_meta($post_id, '_wp_trash_meta_time');
 
-    wp_insert_post($post);
+    wp_insert_post( wp_slash( $post ) );
 
     wp_untrash_post_comments($post_id);

trunk/tests/phpunit/tests/post/slashes.php

@@ -130 +130 @@
     }
 
+    /**
+     * @ticket 27550
+     */
+    function test_wp_trash_untrash() {
+        $post = array(
+            'post_title'   => $this->slash_1,
+            'post_content' => $this->slash_3,
+            'post_excerpt' => $this->slash_5,
+        );
+        $id = wp_insert_post( wp_slash( $post ) );
+
+        $trashed = wp_trash_post( $id );
+        $this->assertNotEmpty( $trashed );
+
+        $post = get_post( $id );
+
+        $this->assertEquals( $this->slash_1, $post->post_title );
+        $this->assertEquals( $this->slash_3, $post->post_content );
+        $this->assertEquals( $this->slash_5, $post->post_excerpt );
+
+        $untrashed = wp_untrash_post( $id );
+        $this->assertNotEmpty( $untrashed );
+
+        $post = get_post( $id );
+
+        $this->assertEquals( $this->slash_1, $post->post_title );
+        $this->assertEquals( $this->slash_3, $post->post_content );
+        $this->assertEquals( $this->slash_5, $post->post_excerpt );
+    }
 }