Bug #66985 [ReO]: Some timezones are no longer valid in PHP 5.5.10

From: Date: Sun, 20 Jul 2014 00:43:59 +0000
Subject: Bug #66985 [ReO]: Some timezones are no longer valid in PHP 5.5.10
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-186752@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=66985&edit=1

 ID:                 66985
 Updated by:         rasmus@php.net
 Reported by:        evert at rooftopsolutions dot nl
 Summary:            Some timezones are no longer valid in PHP 5.5.10
 Status:             Re-Opened
 Type:               Bug
 Package:            Date/time related
 Operating System:   Any
 PHP Version:        5.5.10
 Block user comment: N
 Private report:     N

 New Comment:

Hrm.. In my simple test it worked. It was zone_search() that was failing. With that hack this works
fine:

$tz = new DateTimeZone('EST5EDT');
echo $tz->getName()."\n";
$date = new DateTime('2000-01-01', $tz);
$date = $date->modify('+6 months');
echo $date->format('Y-m-d') . "\n";

But true, looking closer at it, $tz->getTranstitions() doesn't return anything.


Previous Comments:
------------------------------------------------------------------------
[2014-07-19 00:02:31] yohgaki@php.net

Note: JST (Japanese Standard Time) is used to work with older PHP also.

------------------------------------------------------------------------
[2014-07-18 16:04:17] derick@php.net

It doesn't work, there are some other bits of code that only let allow through IDs with a / in
them. I've moved this to the top of my todo list, so hopefully get to it soon.

------------------------------------------------------------------------
[2014-07-18 15:51:51] rasmus@php.net

Derick, wouldn't the easiest solution just be to add these to timezonemap.h?
As in, for EST5EDT it would be something like:

{ "edt",   1, -14400, "EST5EDT"                       },
{ "est",   1, -18000, "EST5EDT"                       },

------------------------------------------------------------------------
[2014-07-16 17:01:09] evert at rooftopsolutions dot nl

OT: Bit dissapointed that it required a comment from Rasmus for this issue to be as much as
acknowledged. Didn't ask for an immediate fix, but getting ignored really sucks.

------------------------------------------------------------------------
[2014-07-16 14:30:48] rasmus@php.net

Yup, I looked at it too, I thought it would be as simple as adding them back into the array there,
but I see it isn't. The fact that it has been 4 patch releases isn't really an issue
though. People are migrating slowly to 5.5 still. For those who are already on current 5.5 they
either already fixed this by moving to a more precise tz identifier or they didn't have the
problem in the first place, so they won't be affected by the fix. And for the others who
aren't there yet it will only help them by removing a compatibility obstacle.

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=66985


--
Edit this bug report at https://bugs.php.net/bug.php?id=66985&edit=1


Thread (18 messages)

« previous php.bugs (#186752) next »