Repository navigation
Conversation
|
I second this! Why the hell aren't very popular/important time zones like CST6CDT or EST5EDT included, for crying out loud?? It works for php.ini and date(), why not DateTime? |
There was a problem hiding this comment.
Thank you for this! The hypocrisy of it all has caused me to fork my servers' own version of PHP just to have this functionality ;-)
|
This patch should not be merged. The exclusion of outdated timezones should not be in the default list because people should not be using them. And that is even more true for "timezones" like "CST6CDT" as they do not do correctly what you think they would do. |
|
There's still the issue that the change 1) broke backward compatibility, 2) occurred in a MINOR revision of 5.4, and 3) was NEVER announced in the changelog. Oh, and the fact that it breaks hard (critical error) on existing code. |
https://bugs.php.net/bug.php?id=64560
Include all timezones (with backwards compatibility) for default $what