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

buddypress-trac noreply at wordpress.org
Sat Jul 26 21:57:48 UTC 2014


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

Comment (by r-a-y):

 `02.patch` introduces `bp_core_strtotime()` to ensure that the timestamp
 is indeed UTC.

 There are some other things that I could use feedback on.

 * The admin dashboard uses a mixture of the localized date/time format and
 UTC dates.
    * Activity dashboard index uses the localized timezone, whereas when
 you edit an activity item in the dashboard, the timezone is UTC.
    * I mirrored this behavior for the Groups dashboard as well for now,
 but we should probably stick to UTC to avoid confusion.
 * I didn't touch the XProfile datebox fields because that probably doesn't
 need to be converted to UTC.

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


More information about the buddypress-trac mailing list