Ticket #32585: 32585.diff

src/wp-includes/post-functions.php

@@ -3042 +3042 @@
 
         // These variables are needed by compact() later.
         $post_content_filtered = $postarr['post_content_filtered'];
-        $post_author = empty( $postarr['post_author'] ) ? $user_id : $postarr['post_author'];
+        $post_author = isset( $postarr['post_author'] ) ? $postarr['post_author'] : $user_id;
         $ping_status = empty( $postarr['ping_status'] ) ? get_default_comment_status( $post_type, 'pingback' ) : $postarr['ping_status'];
         $to_ping = isset( $postarr['to_ping'] ) ? sanitize_trackback_urls( $postarr['to_ping'] ) : '';
         $pinged = isset( $postarr['pinged'] ) ? $postarr['pinged'] : '';

tests/phpunit/tests/post.php

@@ -1200 +1200 @@
                 // Teardown
                 wp_set_current_user( $old_uid );
         }
+
+        /**
+         * @ticket 32585
+         */
+        public function test_wp_insert_post_author_zero() {
+                $user_id = $this->factory->user->create( array( 'role' => 'administrator' ) );
+
+                wp_set_current_user( $user_id );
+
+                $post_id = $this->factory->post->create( array( 'post_author' => 0 ) );
+
+                $this->assertEquals( 0, get_post( $post_id )->post_author );
+        }
+
+        /**
+         * @ticket 32585
+         */
+        public function test_wp_insert_post_author_null() {
+                $user_id = $this->factory->user->create( array( 'role' => 'administrator' ) );
+
+                wp_set_current_user( $user_id );
+
+                $post_id = $this->factory->post->create( array( 'post_author' => null ) );
+
+                $this->assertEquals( $user_id, get_post( $post_id )->post_author );
+        }
 }