Changeset 23401 for trunk/wp-includes/comment.php

Timestamp:

02/11/2013 06:08:14 PM (12 years ago)

Author:

ryan

Message:

Use microtime() instead of incrementors for last_changed to to avoid race conditions with cache evictions.

Props westi
fixes #23448

File:

• trunk/wp-includes/comment.php (modified) (3 diffs)

trunk/wp-includes/comment.php

@@ -252 +252 @@
         $last_changed = wp_cache_get( 'last_changed', 'comment' );
         if ( ! $last_changed ) {
-            $last_changed = 1;
+            $last_changed = microtime();
             wp_cache_set( 'last_changed', $last_changed, 'comment' );
         }
@@ -1293 +1293 @@
     do_action('wp_insert_comment', $id, $comment);
 
-    if ( function_exists( 'wp_cache_incr' ) ) {
-        wp_cache_incr( 'last_changed', 1, 'comment' );
-    } else {
-        $last_changed = wp_cache_get( 'last_changed', 'comment' );
-        wp_cache_set( 'last_changed', $last_changed + 1, 'comment' );
-    }
+    wp_cache_set( 'last_changed', microtime(), 'comment' );
 
     return $id;
@@ -2051 +2046 @@
         wp_cache_delete($id, 'comment');
 
-    if ( function_exists( 'wp_cache_incr' ) ) {
-        wp_cache_incr( 'last_changed', 1, 'comment' );
-    } else {
-        $last_changed = wp_cache_get( 'last_changed', 'comment' );
-        wp_cache_set( 'last_changed', $last_changed + 1, 'comment' );
-    }
+    wp_cache_set( 'last_changed', microtime(), 'comment' );
 }