Bug #66102 [Com]: DateTimeZone::listIdentifiers does not return linked timezone

Edit report at https://bugs.php.net/bug.php?id=66102&edit=1 ID: 66102 Comment by: mal dot graty at idioplatform dot com Reported by: a dot servedio at technotraffic dot com Summary: DateTimeZone::listIdentifiers does not return linked timezone Status: Open Type: Bug Package: Date/time related Operating System: Linux, might be any PHP Version: Irrelevant Block user comment: N Private report: N New Comment: Yup, I agree with all of that. :) However it's these two comments specifically that makes me think that it's not expected to be returned from the listIdentifiers function: https://github.com/eggert/tz/commit/242980fc79ef5fb676383f91505f9cbce6ff74cb#commitcomment-5248692 and https://github.com/eggert/tz/issues/4#issuecomment-27102005 I interpret it as: for anyone already using America/Montreal they should see no change (i.e. can still be used for time calculations) but it should be omitted from the list because America/Toronto should be used in preference going forward. In my view DateTimeZone::listIdentifiers is correct in listing only distinct identifiers, keeping the list to a bare minimum to choose from (the full list is double the length). This also ensures that everything in the list has country and lat/lon info, something backward links do not have. For TZ validation, a try catch around the constructor seems like the best approach. However if you do need a complete list for some purpose, it can be derived from DateTimeZone::listAbbreviations as shown in the script below. Test script: --------------- // compile $abbrs = \DateTimeZone::listAbbreviations(); $usable_tzs = array_unique(array_filter( array_merge(array_map('strtoupper', array_keys($abbrs)), array_reduce( $abbrs, function ($tzs, $abbr) { return array_merge($tzs, array_map( function ($tz) { return $tz['timezone_id']; }, $abbr )); }, array() )), function ($tz) { return strlen($tz) > 1; } )); // validate foreach ($usable_tzs as $tz) { try { new DateTimeZone($tz); } catch (Exception $e) { echo $tz . ' is NOT a valid timezone' . PHP_EOL; } } // test var_dump(in_array('America/Montreal', $usable_tzs)); var_dump(count(DateTimeZone::listIdentifiers())); var_dump(count($usable_tzs)); Result: ---------------- bool(true) int(417) int(834) Previous Comments: ------------------------------------------------------------------------ [2014-08-09 09:25:42] a dot servedio at technotraffic dot com Hi, thank you for taking the time to answer. The iana maintainer is the same person in both my bug description link and the link you provide. Now reading both comments, it seems like the reasoning for the "backward" list isn't only for timezones that may be deprecated, but also for timezones that provides the same end result as another one and are being provided as "links" instead of distinctive entries. From the url you provide, he mentions "America/Montreal is redundant", not "deprecated". He also says "The TZ setting 'America/Montreal' should work as it did before, for all time stamps after 1970." in his answer on my link. Which is probably why <?php new \DateTimeZone("America/Montreal") ?> works. Which is probably also why the recent tzdata package in Linux will still create a fully usable /usr/share/zoneinfo/America/Montreal entry as of today, and that /etc/timezone can hold 'America/Montreal' no problem. ------------------------------------------------------------------------ [2014-08-09 03:57:30] mal dot graty at idioplatform dot com If I understand correctly, this is expected behaviour. America/Montreal is, in effect, deprecated. It will still work, for anything already using it, but going forward America/Toronto should be chosen instead. This decision was taken by the IANA, and isn't related to PHP. There have been discussions here (https://github.com/eggert/tz/commit/242980fc79ef5fb676383f91505f9cbce6ff74cb#commitcomment-5248365) and here (https://github.com/eggert/tz/issues/4) that explain the reasoning for doing so. tl;dr: Linked timezones function, but do not appear in DateTimeZone::listIdentifiers by design, since they are only linked for backward compatibility, and should not be used going forward where possible. ------------------------------------------------------------------------ [2013-11-15 12:23:55] a dot servedio at technotraffic dot com Description: ------------ America/Montreal timezone was changed from a distinct timezone (entry in zone.tab) to a linked timezone (linked to America/Toronto). This change comes from a iana update (2013e) introduced in september 2013. But other previous "linked" timezone don't show up in listIdentifiers as well. While not being a distinct timezone, these are still valid (See comment from TZ Maintainer at iana: https://github.com/eggert/tz/commit/45dcf69b45087cff50282d4da64b86a7d705ddf3#commitcomment-4602830) (issue occurs in both 5.3.10 and 5.5.5 as far as I know) Test script: --------------- // all valid timezones, except last 2 are timezone "links" $timezones = array('America/Toronto', 'America/Montreal', 'Asia/Istanbul'); // they are all valid here. No Exceptions thrown foreach($timezones as $tz) { new \DateTimeZone($tz); echo "Timezone object valid with $tz \n"; } // checking against identifiers list, will not find it for the last 2 timezones $tz_list = \DateTimeZone::listIdentifiers(); foreach($timezones as $tz) { echo "Timezone $tz "; echo in_array($tz, $tz_list) ? "in list\n" : "NOT in list\n"; } Expected result: ---------------- Timezone object valid with America/Toronto Timezone object valid with America/Montreal Timezone object valid with Asia/Istanbul Timezone America/Toronto in list Timezone America/Montreal in list Timezone Asia/Istanbul in list Actual result: -------------- Timezone object valid with America/Toronto Timezone object valid with America/Montreal Timezone object valid with Asia/Istanbul Timezone America/Toronto in list Timezone America/Montreal NOT in list Timezone Asia/Istanbul NOT in list ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=66102&edit=1