[wp-trac] [WordPress Trac] #9588: time() is not GMT time()

WordPress Trac wp-trac at lists.automattic.com
Wed Dec 2 21:30:08 UTC 2009


#9588: time() is not GMT time()
--------------------------------+-------------------------------------------
  Reporter:  Denis-de-Bernardy  |        Type:  defect (bug)           
    Status:  closed             |    Priority:  normal                 
 Milestone:  2.9                |   Component:  Date/Time              
   Version:  2.8                |    Severity:  normal                 
Resolution:  fixed              |    Keywords:  has-patch needs-testing
--------------------------------+-------------------------------------------
Changes (by Denis-de-Bernardy):

  * status:  reopened => closed
  * resolution:  => fixed


Comment:

 Replying to [comment:23 Otto42]:
 > Depends, are any of the 14 patches you posted before still using time()
 where they should be using gmtime()? If so, they still need correcting,
 since the timezone will change once the real local timezone gets loaded
 (which happens when the gmt_offset option is used).
 >
 > I think those patches you posted before still need to be examined, and
 those replaced with gm* functions if that is the underlying assumption
 behind them.

 I'd agree if php4 wasn't unmaintained for over 2 years. re-closing...

 it's minor edge cases. we've lived with this bug for years, the patches
 initially got dismissed; we can live with it until php4 is completely
 phased out.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/9588#comment:24>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list