Changeset 28115 for branches/3.7/tests/phpunit/tests/admin/includesPost.php

Timestamp:

04/14/2014 08:12:42 AM (11 years ago)

Author:

nacin

Message:

Avoid stomping of bulk postdata inside the bulk_edit_posts() loop.

Merges [28113] to the 3.7 branch.

Reverts [27992] which did not fix it for authors and comment/ping status.

props dd32, DrewAPicture.
fixes #27792.

Location:

branches/3.7

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/admin/includesPost.php (modified) (1 diff)

branches/3.7/tests/phpunit/tests/admin/includesPost.php

@@ -135 +135 @@
         $this->assertEquals( 'draft', $updated->post_status );
     }
+
+    /**
+     * @ticket 27792
+     */
+    function test_bulk_edit_posts_stomping() {
+        $admin = $this->factory->user->create( array( 'role' => 'administrator' ) );
+        $users = $this->factory->user->create_many( 2, array( 'role' => 'author' ) );
+        wp_set_current_user( $admin );
+
+        $post1 = $this->factory->post->create( array(
+            'post_author'    => $users[0],
+            'comment_status' => 'open',
+            'ping_status'    => 'open',
+            'post_status'    => 'publish',
+        ) );
+
+        $post2 = $this->factory->post->create( array(
+            'post_author'    => $users[1],
+            'comment_status' => 'closed',
+            'ping_status'    => 'closed',
+            'post_status'    => 'draft',
+        ) );
+
+        $request = array(
+            'post_type'        => 'post',
+            'post_author'      => -1,
+            'ping_status'      => -1,
+            'comment_status'   => -1,
+            '_status'          => -1,
+            'post'             => array( $post1, $post2 ),
+        );
+
+        $done = bulk_edit_posts( $request );
+
+        $post = get_post( $post2 );
+
+        // Check that the first post's values don't stomp the second post.
+        $this->assertEquals( 'draft', $post->post_status );
+        $this->assertEquals( $users[1], $post->post_author );
+        $this->assertEquals( 'closed', $post->comment_status );
+        $this->assertEquals( 'closed', $post->ping_status );
+    }
+
 }