Closed
Bug 333614
Opened 19 years ago
Closed 17 years ago
timezone parameter should indicate the expected format (or even have pulldown)
Categories
(Bugzilla :: Administration, task)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.2
People
(Reporter: toby, Assigned: spam)
Details
Attachments
(1 file)
(deleted),
patch
|
wicked
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.0.1) Gecko/20060111 Firefox/1.5.0.1
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.0.1) Gecko/20060111 Firefox/1.5.0.1
When setting this parameter, the option text does not give any idea what format the timezone specification should be in (EST? GMT-5? -5?)
Reproducible: Always
Steps to Reproduce:
Browse to editparams.cgi
Actual Results:
User confusion.
Expected Results:
"Ah! I have to enter it like THAT." or, show menu with user's zone.
Comment 1•19 years ago
|
||
We may do this, but we'd probably wait until the timezone parameter actually does something. Right now all it does is display the exact string you put next to every time/date.
Comment 2•18 years ago
|
||
(In reply to comment #1)
> Right now all it does is display the exact string you put next
> to every time/date.
Not exactly. The time displayed at the top of buglists takes this parameter into account (I tested right now, i.e. the time displayed is different if 'timezone' is set to CET or PDT, for instance).
I don't want a static menu; it should look at %Time::Zone::dstZone, but this hash is not exported by Time::Zone, unfortunately. So I think an example would be enough in the param description would be enough.
Comment 3•18 years ago
|
||
(In reply to comment #2)
> I don't want a static menu; it should look at %Time::Zone::dstZone, but this
> hash is not exported by Time::Zone, unfortunately. So I think an example would
> be enough in the param description would be enough.
Hrm. There must be *some* programmatic way to get the list, though, from somewhere. Every OS stores on it *somewhere* the list of timezones, and has internal functions for getting the list. So I'd think that perl would have some way to get that information...
Assignee | ||
Comment 4•18 years ago
|
||
Attachment #250942 -
Flags: review?
Comment 5•17 years ago
|
||
Comment on attachment 250942 [details] [diff] [review]
patch for tip, add some examples
Okay, technically this works and is nice bandaid. But for complete solution we should probably switch to DateTime::TimeZone that can give us full list of timezones.
Attachment #250942 -
Flags: review? → review+
Comment 6•17 years ago
|
||
I'll let approvers make a decision on how to proceed with this..
Flags: approval?
Target Milestone: --- → Bugzilla 3.2
Updated•17 years ago
|
Assignee: administration → spam
Updated•17 years ago
|
Status: NEW → ASSIGNED
Flags: approval? → approval+
Comment 7•17 years ago
|
||
Checking in template/en/default/admin/params/core.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/admin/params/core.html.tmpl,v <-- core.html.tmpl
new revision: 1.9; previous revision: 1.8
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•