Bugzilla doesn't offer UTC timezone

VERIFIED FIXED in Bugzilla 4.0

Status

()

Bugzilla
User Accounts
VERIFIED FIXED
7 years ago
6 years ago

People

(Reporter: Tom Dickson, Assigned: Tom Dickson)

Tracking

unspecified
Bugzilla 4.0
Bug Flags:
approval +
approval4.0 +

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
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

7 years ago
Created attachment 522801 [details] [diff] [review]
Minor patch to force UTC to be available

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

7 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
Last Resolved: 7 years ago
Resolution: --- → INVALID

Comment 3

7 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

7 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

7 years ago
I've posted adding UTC to DateTime::TimeZone:

https://rt.cpan.org/Public/Bug/Display.html?id=67070

Comment 6

7 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

7 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

7 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

7 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
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

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