Ticket #27792: 27792.4.diff

src/wp-admin/includes/post.php

@@ -433 +433 @@
         }
 
         $updated = $skipped = $locked = array();
+        $shared_post_data = $post_data;
+
         foreach ( $post_IDs as $post_ID ) {
+                // Start with fresh post data with each iteration.
+                $post_data = $shared_post_data;
+
                 $post_type_object = get_post_type_object( get_post_type( $post_ID ) );
 
                 if ( !isset( $post_type_object ) || ( isset($children) && in_array($post_ID, $children) ) || !current_user_can( 'edit_post', $post_ID ) ) {
@@ -482 +487 @@
                 $post_data['ID'] = $post_ID;
                 $post_data['post_ID'] = $post_ID;
 
-                $translated_post_data = _wp_translate_postdata( true, $post_data );
-                if ( is_wp_error( $translated_post_data ) ) {
+                $post_data = _wp_translate_postdata( true, $post_data );
+                if ( is_wp_error( $post_data ) ) {
                         $skipped[] = $post_ID;
                         continue;
                 }
 
-                $updated[] = wp_update_post( $translated_post_data );
+                $updated[] = wp_update_post( $post_data );
 
                 if ( isset( $post_data['sticky'] ) && current_user_can( $ptype->cap->edit_others_posts ) ) {
                         if ( 'sticky' == $post_data['sticky'] )

tests/phpunit/tests/admin/includesPost.php

@@ -136 +136 @@
                 wp_set_current_user( 0 );
         }
 
+        /**
+         * @ticket 27792
+         */
+        function test_bulk_edit_posts() {
+                $user_id = $this->factory->user->create( array( 'role' => 'administrator' ) );
+                $users = $this->factory->user->create_many( 2, array( 'role' => 'author' ) );
+                wp_set_current_user( $user_id );
+
+                $post1 = $this->factory->post->create( array(
+                        'post_author'    => $users[0],
+                        'comment_status' => 'open',
+                        'ping_status'    => 'open',
+                ) );
+
+                $post2 = $this->factory->post->create( array(
+                        'post_author'    => $users[1],
+                        'comment_status' => 'closed',
+                        'ping_status'    => 'closed',
+                ) );
+
+                $post2_clone = $post2;
+
+                $request = array(
+                        's'                => null,
+                        'post_status'      => 'all',
+                        'post_type'        => 'post',
+                        '_wpnonce'         => rand_str( 10 ),
+                        '_wp_http_referer' => '/wp-admin/edit.php',
+                        'action'           => 'edit',
+                        'm'                => 0,
+                        'cat'              => 0,
+                        'paged'            => 1,
+                        'model'            => 'list',
+                        'post_author'      => -1,
+                        'ping_status'      => null,
+                        '_status'          => -1,
+                        'sticky'           => -1,
+                        'post_format'      => 'image',
+                        'post'             => array( $post1, $post2_clone ),
+                        'action2'          => -1,
+                );
+
+                $done = bulk_edit_posts( $request );
+
+                $post2 = get_post( $post2 );
+                $post2_clone = get_post( $post2_clone );
+
+                $this->assertEquals( $post2->post_author, $post2_clone->post_author );
+                $this->assertEquals( 'closed', $post2_clone->comment_status );
+                $this->assertEquals( 'closed', $post2_clone->ping_status );
+
+                wp_set_current_user( 0 );
+        }
+
 }
+ No newline at end of file