Changeset 41372

Timestamp:

09/11/2017 05:49:58 PM (7 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.
      *