[buddypress-trac] [BuddyPress] #4573: bp-activity is off by 5 Hours

[buddypress-trac]

#4573: bp-activity is off by 5 Hours
--------------------------+------------------------------
 Reporter:  mgkoller      |       Owner:
     Type:  defect (bug)  |      Status:  reopened
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Activity      |     Version:  1.6.1
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+------------------------------

Comment (by shanebp):

 >Activity time seen via WP Admin is also 5 hours ahead of local time
 I can confirm that it is 'off' by the difference between local time and
 gmt.
 In a vanilla install of trunk-6829

 I think all these 'time is off' issues might be related to
 function bp_core_current_time( $gmt = true ) {  // in bp-core-
 functions.php

 Why is $gmt = true ?
 Wouldn't setting it to false solve a lot of problems?
 And be easier than filtering it against 'local time' - which doesn't seem
 to happen, at least in bp-activity-admin.php

 I use current_time('timestamp',0) whenever I need to stamp something.
 Works great for all client installs.

 Maybe bp_core_current_time() is passed a 'false' sometimes but not others?

 I can spend some time on this issue.
 But first, I'd like confirmation that I'm not missing something.

 So, filter it or just default $gmt to local or... ?

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4573#comment:4>
BuddyPress <http://buddypress.org/>
BuddyPress