Changeset 8676 for trunk/src/bp-core/bp-core-functions.php

Timestamp:

07/23/2014 06:07:39 PM (12 years ago)

Author:

r-a-y

Message:

Fix issues with bp_core_time_since().

Previously, when the $newer_date parameter isn't passed, we generated a
UNIX timestamp using strtotime() and bp_core_current_time(). strtotime()
uses the default timezone, which can cause timezone discrepencies if the
server time is not UTC.

This commit removes the strtotime() usage and refactors bp_core_current_time()
to accept current_time()'s 'type' parameter. This allows us to correctly
grab the UTC timestamp when setting bp_core_current_time()'s second
argument to 'timestamp'.

Commit also adds some unit tests.

Fixes #4310.

File:

• trunk/src/bp-core/bp-core-functions.php (modified) (2 diffs)

trunk/src/bp-core/bp-core-functions.php

@@ -772 +772 @@
  *
  * @param bool $gmt True to use GMT (rather than local) time. Default: true.
+ * @param string $type See the 'type' parameter in {@link current_time()}.
+          Default: 'mysql'.
  * @return string Current time in 'Y-m-d h:i:s' format.
  */
-function bp_core_current_time( $gmt = true ) {
-        // Get current time in MYSQL format
-        $current_time = current_time( 'mysql', $gmt );
+function bp_core_current_time( $gmt = true, $type = 'mysql' ) {
+        // Get current time
+        $current_time = current_time( $type, $gmt );
 
         return apply_filters( 'bp_core_current_time', $current_time );
@@ -841 +843 @@
          * work out time elapsed between two known dates.
          */
-        $newer_date = ( !$newer_date ) ? strtotime( bp_core_current_time() ) : $newer_date;
+        $newer_date = ( !$newer_date ) ? bp_core_current_time( true, 'timestamp' ) : $newer_date;
 
         // Difference in seconds