[buddypress-trac] [BuddyPress Trac] #5781: strtotime() is evil don't use it

buddypress-trac noreply at wordpress.org
Thu Sep 1 08:52:32 UTC 2016


#5781: strtotime() is evil don't use it
--------------------------+-----------------------
 Reporter:  r-a-y         |       Owner:  r-a-y
     Type:  defect (bug)  |      Status:  assigned
 Priority:  normal        |   Milestone:  2.7
Component:  Core          |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+-----------------------

Comment (by DJPaul):

 Good questions. I think it SHOULD but... if current xprofile is storing
 values in server timezone (or the timezone set in WP), if we switched to
 GMT, surely we'd have to run a conversion on update. If you want to put in
 the effort to do that, that's fab, otherwise let's not bother for now.
 This could be done in the future as a separate thing.

 > Activity dashboard index uses the localized timezone, whereas when you
 edit an activity item in the dashboard, the timezone is UTC.

 I know this comment is 2 years old but if still the case, we should
 convert it to local timezone on display, and then back again to GMT on
 save. I'm assuming this also will need to happen on the front-end as the
 underlying implementations are similar.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5781#comment:17>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list