Opened 2 years ago

Closed 2 years ago

#17448 closed defect (bug) (fixed)

Make sure $tz is not empty in get_gmt_from_date()

Reported by: johnjamesjacoby Owned by: nacin
Priority: normal Milestone: 3.2
Component: Date/Time Version:
Severity: normal Keywords: has-patch
Cc:

Description

It's possible that $tz might still be empty by default, which causes errors when posting new content.

See r17920 for where this was introduced.

Attachments (1)

17448.patch​ (828 bytes) - added by johnjamesjacoby 2 years ago.

Download all attachments as: .zip

Change History (5)

johnjamesjacoby — 2 years ago

comment:1   nacin — 2 years ago

  • Owner set to nacin
  • Status changed from new to accepted

comment:2   ryan — 2 years ago

Perhaps we need to revert back and just remove the class_exists() check.

comment:3   johnjamesjacoby — 2 years ago

Yep, that's what needs to happen.

My patch prevents the error, but results in the post_date and post_date_gmt being the same.

comment:4   ryan — 2 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In [17958]:

Revert part of [17920]. Support for empty tz must remain. Props johnjamesjacoby. fixes #17448

Note: See TracTickets for help on using tickets.