This is really minor, but bugged me when installing from CVS. The list of active timezones is in the format:

America/New_York

This should be presented correctly, without the underscores. The patch leaves the underscore value intact as the array key returned by system_time_zones().

(For the record, the underscores are generated by the PHP function timezone_identifiers_list())

CommentFileSizeAuthor
system_time_zones.patch1.11 KBagentrickard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Reviewed & tested by the community

Tested the patch and it works fine. The underscores are removed and adjusting the time zone works as well.

Setting RTBC because this is really a small patch. Feel free to reverse.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.