[drupal-devel] [feature] _system_zonelist should return locale names

[TDobes]

Issue status update for http://drupal.org/node/21343

 Project:      Drupal
 Version:      cvs
 Component:    system.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  crunchywelch
 Updated by:   TDobes
 Status:       patch

It might be nice to have a more readable timezone list, but this might
cause some users to assume that Drupal deals with daylight savings
time.  For example, the majority of the "US Central" timezone (where I
live) switches for DST, so I would expect that, to most people, "US
Central" means "Central Prevailing Time" (CST or CDT depending on the
time of the year).  I can say for certain that, had I not previously
seen the code, this would be confusing to me.  On the other hand, the
numeric values (as we have now), suggest that a constant offset from
GMT is used.


If we really want to include these named timezones, I think we should
mention or indicate in some way that Drupal does not deal with DST. 
It's unfortunate that implementing international DST is so complicated.


So... I don't mind the named timezones, but I think that, if we do
commit this patch, it should be indicated in the UI that Drupal will
not adjust its clock automatically for daylight savings.




TDobes



Previous comments:
------------------------------------------------------------------------

April 25, 2005 - 23:59 : crunchywelch

Attachment: http://drupal.org/files/issues/system_6.patch (4.87 KB)

This patch adds locale names to the timezone offsets provided by
system.module.


Currently, the function returns an array with offsets for keys and
formatted 'local' times for the values. With this patch, the function
will return an associative array with two keys 'times' and 'locales'. 
Each key has an array with the offset values for keys, and thier
associated local time value, or locale name.


This patch provides a standard place to retrieve timezone values when
developing with them, improves useability when displaying timezone
select boxes, and updates the timezone offset values. This patch will
be utilized immediately by event.module, which currently uses an almost
equivelant function, event_zonelist.




------------------------------------------------------------------------

April 26, 2005 - 10:17 : Dries

Does anyone thinks is worth including in core?  If it is only useful for
one or two modules (eg. the event module), it would be easy enough to
include this table in those modules.




------------------------------------------------------------------------

April 26, 2005 - 11:01 : killes at www.drop.org

Yes, I think this patch is usefull. It makes the time zone selector
better readable and improves usability. This selector is used for both
the cetral system time and the user's time zone (if that is enabled).