Changeset 7024 for trunk/src/includes/common/engagements.php

Timestamp:

12/11/2019 04:05:40 PM (5 years ago)

Author:

johnjamesjacoby

Message:

Engagements: fix bugs with caching implementation in User strategy.

This commit adds the $object_id to the end of the cache key, to avoid collisions with other IDs. It also loosens 2 explicit true comparisons because internal calls to update_meta() up the stack may return an integer.

See #3292. For 2.7, trunk.

File:

• trunk/src/includes/common/engagements.php (modified) (7 diffs)

trunk/src/includes/common/engagements.php

@@ -612 +612 @@
     private function get_cache_key( $meta_key = '', $object_id = 0 ) {
 
+        // No negative numbers in cache keys
+        $object_id = absint( $object_id );
+
         // Maybe guess at post type
         $post_type = ! empty( $object_id )
@@ -641 +644 @@
         }
 
-        // Return the old (pluralized) user option key
-        return $key;
+        // Return the old (pluralized) user option key with object ID appended
+        return "{$key}{$object_id}";
     }
 
@@ -661 +664 @@
         $cache_key = $this->get_cache_key( $meta_key, $object_id );
 
-        return wp_cache_get( $cache_key, 'bbpress_users' );
+        return wp_cache_get( $cache_key, 'bbpress_posts' );
     }
 
@@ -740 +743 @@
             $retval       = update_user_option( $user_id, $option_key, $object_ids );
 
-            // Delete cache, if successful
-            if ( true === $retval ) {
+            // Delete cache if successful (accounts for int & true)
+            if ( false !== $retval ) {
                 $this->cache_delete( $meta_key, $object_id );
             }
@@ -777 +780 @@
                 : delete_user_option( $user_id, $option_key );
 
-            // Delete cache, if successful
-            if ( true === $retval ) {
+            // Delete cache if successful (accounts for int & true)
+            if ( false !== $retval ) {
                 $this->cache_delete( $meta_key, $object_id );
             }
@@ -806 +809 @@
         $object_ids = $this->parse_comma_list( get_user_option( $option_key, $user_id ) );
 
-        // Attempt to delete the
+        // Attempt to delete the user option
         $retval = delete_user_option( $user_id, $option_key );
 
@@ -945 +948 @@
         $object_ids = $this->parse_comma_list( get_user_option( $option_key, $user_id ) );
 
-        // Maybe trick WP_Query into ".ID IN ()"
+        // Maybe trick WP_Query into ".ID IN (0)" to return no results
         if ( empty( $object_ids ) ) {
             $object_ids = array( 0 );