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

From: Date: Sun, 17 Aug 2014 10:12:35 +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-187155@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:         derick@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:

I have a fix in the works for this. Code works, but need some more test cases and checking for
memory leaks.


Previous Comments:
------------------------------------------------------------------------
[2014-07-20 00:43:59] rasmus@php.net

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.

------------------------------------------------------------------------
[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.

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


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 (#187155) next »