Closed
Bug 646209
Opened 12 years ago
Closed 12 years ago
Bugzilla doesn't offer UTC timezone
Categories
(Bugzilla :: User Accounts, defect)
Bugzilla
User Accounts
Tracking
()
VERIFIED
FIXED
Bugzilla 4.0
People
(Reporter: bugzilla, Assigned: bugzilla)
Details
Attachments
(1 file)
410 bytes,
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; Linux i686) AppleWebKit/534.27 (KHTML, like Gecko) Ubuntu/10.04 Chromium/12.0.717.0 Chrome/12.0.717.0 Safari/534.27 Build Identifier: 4.0 If you go to user preferences, you can't select UTC as the timezone you want to see. Reproducible: Always Steps to Reproduce: 1. Edit user or site preferences, cannot pick UTC Actual Results: You should be able to pick UTC, so you can work around bug 452353. Expected Results: I got a list of timezones, but UTC was not among them. I suspect it is because the DateTime::TimeZone->all_names call doesn't return all names, and especially doesn't include UTC.
Assignee | ||
Comment 1•12 years ago
|
||
Here is my current workaround; I force 'UTC' to be added to the end of the list when the preferences are displayed. It works and allows me to select UTC and see the times in UTC even though the server time is PST.
![]() |
||
Comment 2•12 years ago
|
||
As for other similiar requests [1], the list of timezones comes from DateTime::TimeZone. Artificially adding UTC to the list won't help. It should be added to DateTime::TimeZone itself to make it work correctly. [1] https://bugzilla.mozilla.org/buglist.cgi?quicksearch=INV+prod%3ABugzilla+summary%3Atimezone
Status: UNCONFIRMED → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
Comment 3•12 years ago
|
||
Are you saying that some DateTime versions return UTC on some systems, and others don't? Because we should definitely have UTC as a choice, no matter what DateTime does.
Status: RESOLVED → UNCONFIRMED
Resolution: INVALID → ---
Assignee | ||
Comment 4•12 years ago
|
||
My version of DateTime (1.19) doesn't have UTC in the list of "all_names" but does have things that link to it. 1.22 and 1.31 (latest) don't have it either. It appears they do list SOME three-letter times, but we specifically grep them out: # Remove old formats, such as CST6CDT, EST, EST5EDT. @timezones = grep { $_ =~ m#.+/.+#} @timezones; So even if DateTime returned UTC (it doesn't) we'd not see it.
Assignee | ||
Comment 5•12 years ago
|
||
I've posted adding UTC to DateTime::TimeZone: https://rt.cpan.org/Public/Bug/Display.html?id=67070
Comment 6•12 years ago
|
||
Comment on attachment 522801 [details] [diff] [review] Minor patch to force UTC to be available Looks good to me.
Attachment #522801 -
Flags: review+
Updated•12 years ago
|
Assignee: user-accounts → bugzilla
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: approval4.0+
Flags: approval+
OS: Linux → All
Hardware: x86_64 → All
Target Milestone: --- → Bugzilla 4.0
Assignee | ||
Comment 7•12 years ago
|
||
Can someone commit this for me? According to https://wiki.mozilla.org/Bugzilla:Developers I should do it but I'm not a committer.
Comment 8•12 years ago
|
||
Sure. Thanks for the bug report and the patch! :-) Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/ modified Bugzilla/User/Setting/Timezone.pm Committed revision 7771. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/4.0/ modified Bugzilla/User/Setting/Timezone.pm Committed revision 7569.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•12 years ago
|
||
This is working after an upgrade to 4.0.1.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•