Changeset 41372

Timestamp:

09/11/2017 05:49:58 PM (9 years ago)

Author:

westonruter

Message:

Customize: Align behavior of WP_Customize_Manager::save_changeset_post() with wp_insert_post() by setting status to future if supplied status is publish but date is future.

Props dlh.
Fixes #41336.

Location:

trunk

Files:

• src/wp-includes/class-wp-customize-manager.php (modified) (1 diff)
• tests/phpunit/tests/customize/manager.php (modified) (1 diff)

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

@@ -2271 +2271 @@
                                 return new WP_Error( 'not_future_date' );
                         }
+                }
+
+                if ( ! empty( $is_future_dated ) && 'publish' === $args['status'] ) {
+                        $args['status'] = 'future';
                 }
 

trunk/tests/phpunit/tests/customize/manager.php

@@ -1414 +1414 @@
 
         /**
+         * Ensure that saving a changeset with a publish status but future date will change the status to future, to align with behavior in wp_insert_post().
+         *
+         * @ticket 41336
+         * @covers WP_Customize_Manager::save_changeset_post
+         */
+        function test_publish_changeset_with_future_status_when_future_date() {
+                $wp_customize = $this->create_test_manager( wp_generate_uuid4() );
+
+                $wp_customize->save_changeset_post( array(
+                        'date_gmt' => gmdate( 'Y-m-d H:i:s', strtotime( '+1 day' ) ),
+                        'status' => 'publish',
+                        'title' => 'Foo',
+                ) );
+
+                $this->assertSame( 'future', get_post_status( $wp_customize->changeset_post_id() ) );
+        }
+
+        /**
          * Ensure that save_changeset_post method bails updating an underlying changeset which is invalid.
          *