ID: 46111
Comment by: yoarvi at gmail dot com
Reported By: jason at eventshop dot com dot au
Status: Verified
Bug Type: Date/time related
Operating System: *
PHP Version: 5.*, 6CVS (2009-05-10)
New Comment:
The failures in PHP 5.3 are the following:
Africa/Dar_es_Salaam
Africa/Porto-Novo
America/Blanc-Sablon
America/Port-au-Prince
America/Port_of_Spain
Antarctica/DumontDUrville
Antarctica/McMurdo
The regex for tz in parse_date.re doesn't account for hyphens, or for
lower-case following an _ or for consecutive upper case letters.
Applying the following patch and regenerating parse_date.c fixes the
problem:
Index: ext/date/lib/parse_date.re
===================================================================
--- ext/date/lib/parse_date.re (revision 293574)
+++ ext/date/lib/parse_date.re (working copy)
@@ -854,7 +854,7 @@
second = minute | "60";
secondlz = minutelz | "60";
meridian = ([AaPp] "."? [Mm] "."?) [\000\t ];
-tz = "("? [A-Za-z]{1,6} ")"? | [A-Z][a-z]+([_/][A-Z][a-z]+)+;
+tz = "("? [A-Za-z]{1,6} ")"? | [A-Z][a-z]+([_/-][A-Za-z]+)+;
tzcorrection = "GMT"? [+-] hour24 ":"? minute?;
daysuf = "st" | "nd" | "rd" | "th";
Previous Comments:
------------------------------------------------------------------------
[2009-05-10 21:19:37] [email protected]
The list is a bit shorter with PHP_5_3 than with PHP_5_2..some
progress?
:)
------------------------------------------------------------------------
[2008-09-18 06:01:34] jason at eventshop dot com dot au
Description:
------------
When converting a date/time/timezone string using strtotime(), it
returns false for several of the supposedly valid timezones, such as
"Adelaide/ACT", "Israel", "US/Arizona", etc.
All other timezones work correctly.
Reproduce code:
---------------
$timezones = timezone_identifiers_list();
foreach ($timezones as $zone) {
$date_string = "2008-01-01 13:00:00 " . $zone;
if (!strtotime($date_string)) {
echo "<br />" . $zone;
}
}
Expected result:
----------------
No result should be displayed.
Actual result:
--------------
Africa/Dar_es_Salaam
Africa/Porto-Novo
America/Argentina/ComodRivadavia
America/Blanc-Sablon
... etc ...
US/Pacific
US/Pacific-New
US/Samoa
W-SU
Zulu
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=46111&edit=1
