[wp-trac] [WordPress Trac] #57035: Error in current_time() function when using timestamp and no value for gmt_offset

WordPress Trac noreply at wordpress.org
Tue Mar 28 14:04:33 UTC 2023


#57035: Error in current_time() function when using timestamp and no value for
gmt_offset
----------------------------+-----------------------
 Reporter:  Nick_theGeek    |       Owner:  audrasjb
     Type:  defect (bug)    |      Status:  reopened
 Priority:  normal          |   Milestone:  6.2
Component:  Date/Time       |     Version:  5.3
 Severity:  normal          |  Resolution:
 Keywords:  has-patch php8  |     Focuses:
----------------------------+-----------------------
Changes (by hellofromTonya):

 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 Reopening to revert [55054].

 == The discussion from comment:17:ticket:57998

 After discussing in Make/Core with the Release Squad and Committers, there
 is consensus to revert [55054] now to ship in 6.2.0.

 **Why revert?**

 1. The additional impacts shared by @Rarst and @reputeinfosystems
 highlighted that this regression can impact bookings, date permalinks, and
 potentially stores for users in these [https://www.timeanddate.com/time
 /time-zones-interesting.html partial-hour timezones].
 2. The impacted user base includes India, Australia, and others, i.e.
 large population areas.
 3. Even if a fast 6.2.1 were released, the impacts could already have been
 felt by business owners, customers, extenders, and users.
 4. And then there are the unknowns or as @desrosj noted:
 >I also think that impossible to anticipate the full impact of this
 change.

 **Why not fix in 6.2.0?**
 There's not enough time in the cycle to validate the fix truly fixes the
 issue without causing an additional issue or side effect. Thus, a fix is
 too risky this late in the cycle. More time is needed to discuss, add more
 test coverage, and consider other edge cases and scenarios.

 The revert will impact the original ticket that introduced the changeset,
 i.e. #57035.

 Once reverted, then the regression reported in this ticket is fixed as the
 code is restored to pre-6.2 state. Thus, this ticket can be closed work
 continuing in #57035 with this new knowledge.

 Huge props to @reputeinfosystems for reporting this regression and @Rarst
 and @reputeinfosystems for sharing the additional impacts!

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/57035#comment:17>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list