WordPress.org

Make WordPress Core

Opened 3 months ago

Last modified 3 weeks ago

#46483 reviewing enhancement

Eliminate use of date_default_timezone_set() for calculations in core

Reported by: Rarst Owned by: SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: minor Version:
Component: Date/Time Keywords: has-patch
Focuses: Cc:

Description

Core sets PHP timezone to UTC on boot and assumes it for Date/Time operations. Which we are in process of improving to eliminate reliance on this assumption in #44491

However there are also couple places in core where it changes PHP time zone back and forth for calculations:

  • in options to determine DST status
  • in PHPMailer (which I assume is just inherited from upstream)

We should get rid of these, calculations should not need to rely on manipulating global PHP configuration state.

Attachments (1)

settings-dst-detection.patch (973 bytes) - added by Rarst 3 weeks ago.

Download all attachments as: .zip

Change History (4)

This ticket was mentioned in Slack in #core-datetime by rarst. View the logs.


3 months ago

#2 @Rarst
3 weeks ago

  • Keywords has-patch added; needs-patch removed

#3 @SergeyBiryukov
3 weeks ago

  • Milestone changed from Awaiting Review to 5.3
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing
Note: See TracTickets for help on using tickets.