From c0b6dffffcdf7842db4672edd025112a7b45fa2c Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Tue, 17 Dec 2019 02:25:03 +0000 Subject: [PATCH] Date/Time: In `wp_insert_post()`, when checking the post date to set `future` or `publish` status, use a proper delta comparison. [3525] allowed a difference up to 59 seconds between the post date/time and the current time to consider the post published instead of scheduled, but that didn't take start of a new minute into account. Rapidly creating post fixtures in unit tests could encounter a one-second discrepancy between `current_time( 'mysql' )` and `gmdate( 'Y-m-d H:i:s' )`, returning values like `2019-12-16 23:43:00` vs. `2019-12-16 23:42:59`, respectively, and setting the post to a `future` status instead of `publish`. [45851], while working as intended, made the issue somewhat more likely to occur. This caused all sorts of occasional random failures in various tests on Travis, mostly on PHP 7.1. Fixes #48145. Built from https://develop.svn.wordpress.org/trunk@46968 git-svn-id: http://core.svn.wordpress.org/trunk@46768 1a063a9b-81f0-0310-95a4-ce76da25c4cd --- wp-includes/post.php | 7 ++++--- wp-includes/version.php | 2 +- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/wp-includes/post.php b/wp-includes/post.php index 812be58ed8..6ffd960c72 100644 --- a/wp-includes/post.php +++ b/wp-includes/post.php @@ -3753,13 +3753,14 @@ function wp_insert_post( $postarr, $wp_error = false ) { } if ( 'attachment' !== $post_type ) { + $now = gmdate( 'Y-m-d H:i:s' ); + if ( 'publish' === $post_status ) { - // String comparison to work around far future dates (year 2038+) on 32-bit systems. - if ( $post_date_gmt > gmdate( 'Y-m-d H:i:59' ) ) { + if ( strtotime( $post_date_gmt ) - strtotime( $now ) >= MINUTE_IN_SECONDS ) { $post_status = 'future'; } } elseif ( 'future' === $post_status ) { - if ( $post_date_gmt <= gmdate( 'Y-m-d H:i:59' ) ) { + if ( strtotime( $post_date_gmt ) - strtotime( $now ) < MINUTE_IN_SECONDS ) { $post_status = 'publish'; } } diff --git a/wp-includes/version.php b/wp-includes/version.php index 09956555cf..bd85ac9d30 100644 --- a/wp-includes/version.php +++ b/wp-includes/version.php @@ -13,7 +13,7 @@ * * @global string $wp_version */ -$wp_version = '5.4-alpha-46967'; +$wp_version = '5.4-alpha-46968'; /** * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.