Ticket #38539: 38539.1.diff

src/wp-includes/class-wp-customize-manager.php

@@ -959 +959 @@
                 // Posts & pages.
                 if ( ! empty( $posts ) ) {
                         foreach ( array_keys( $posts ) as $post_symbol ) {
-                                $posts[ $post_symbol ]['ID'] = wp_insert_post( wp_slash( array_merge(
-                                        $posts[ $post_symbol ],
-                                        array( 'post_status' => 'auto-draft' )
-                                ) ) );
+                                $r = $this->nav_menus->insert_auto_draft_post( $posts[ $post_symbol ] );
+                                if ( $r instanceof WP_Post ) {
+                                        $posts[ $post_symbol ]['ID'] = $r->ID;
+                                }
                         }
                         $this->set_post_value( 'nav_menus_created_posts', wp_list_pluck( $posts, 'ID' ) ); // This is why nav_menus component is dependency for adding posts.
                 }

src/wp-includes/class-wp-customize-nav-menus.php

@@ -734 +734 @@
          * @since 4.7.0
          *
          * @param array $postarr {
-         *     Abbreviated post array.
+         *     Post array. Note that post_status is overridden to be `auto-draft`.
          *
-         *     @var string $post_title Post title.
-         *     @var string $post_type  Post type.
+         *     @var string $post_title   Post title.
+         *     @var string $post_type    Post type.
+         *     @var string $post_name    Post name.
+         *     @var string $post_content Post content.
          * }
          * @return WP_Post|WP_Error Inserted auto-draft post object or error.
          */
@@ -745 +747 @@
                 if ( ! isset( $postarr['post_type'] ) || ! post_type_exists( $postarr['post_type'] )  ) {
                         return new WP_Error( 'unknown_post_type', __( 'Unknown post type' ) );
                 }
-                if ( ! isset( $postarr['post_title'] ) ) {
-                        $postarr['post_title'] = '';
+                if ( empty( $postarr['post_title'] ) ) {
+                        return new WP_Error( 'empty_title', __( 'Empty title' ) );
+                }
+                if ( ! empty( $postarr['post_status'] ) ) {
+                        return new WP_Error( 'status_forbidden', __( 'Status is forbidden' ) );
+                }
+
+                $postarr['post_status'] = 'auto-draft';
+
+                // Auto-drafts are allowed to have empty post_names, so it has to be explicitly set.
+                if ( ! isset( $postarr['post_name'] ) ) {
+                        $postarr['post_name'] = sanitize_title( $postarr['post_title'] );
                 }
 
                 add_filter( 'wp_insert_post_empty_content', '__return_false', 1000 );
-                $args = array(
-                        'post_status' => 'auto-draft',
-                        'post_type'   => $postarr['post_type'],
-                        'post_title'  => $postarr['post_title'],
-                        'post_name'   => sanitize_title( $postarr['post_title'] ), // Auto-drafts are allowed to have empty post_names, so we need to explicitly set it.
-                );
-                $r = wp_insert_post( wp_slash( $args ), true );
+                $r = wp_insert_post( wp_slash( $postarr ), true );
                 remove_filter( 'wp_insert_post_empty_content', '__return_false', 1000 );
 
                 if ( is_wp_error( $r ) ) {
@@ -1139 +1145 @@
                 $post_ids = $setting->post_value();
                 if ( ! empty( $post_ids ) ) {
                         foreach ( $post_ids as $post_id ) {
-                                wp_publish_post( $post_id );
+                                // Note that wp_publish_post() cannot be used because unique slugs need to be assigned.
+                                wp_update_post( array( 'ID' => $post_id, 'post_status' => 'publish' ) );
                         }
                 }
         }