Changeset 8005

Timestamp:

02/28/2014 01:49:15 PM (11 years ago)

Author:

boonebgorges

Message:

Normalize the cache keys and group name used for activity comments

wp_cache_get( $aid, 'bp_activity_comments' )

is more in line with single item caching across other components.

See r7956, r7997

See #5434

Location:

trunk/bp-activity

Files:

• bp-activity-classes.php (modified) (2 diffs)
• bp-activity-functions.php (modified) (3 diffs)

trunk/bp-activity/bp-activity-classes.php

@@ -895 +895 @@
         }
 
-        $comments = wp_cache_get( 'bp_activity_comments_' . $activity_id, 'bp' );
+        $comments = wp_cache_get( $activity_id, 'bp_activity_comments' );
 
         // We store the string 'none' to cache the fact that the
@@ -970 +970 @@
             }
 
-            wp_cache_set( 'bp_activity_comments_' . $activity_id, $cache_value, 'bp' );
+            wp_cache_set( $activity_id, $cache_value, 'bp_activity_comments' );
         }
 

trunk/bp-activity/bp-activity-functions.php

@@ -1244 +1244 @@
     ) );
 
-    // Clear the comment cache for this activity
-    wp_cache_delete( 'bp_activity_comments_' . $parent_id, 'bp' );
+    // Clear the comment and item cache for this activity
+    wp_cache_delete( $parent_id, 'bp_activity_comments' );
     wp_cache_delete( $parent_id, 'bp_activity' );
 
@@ -1663 +1663 @@
 
     // Clear the activity comment cache for this activity item
-    wp_cache_delete( 'bp_activity_comments_' . $activity->id, 'bp' );
+    wp_cache_delete( $activity->id, 'bp_activity_comments' );
 
     // If Akismet is active, and this was a manual spam/ham request, stop Akismet checking the activity
@@ -1703 +1703 @@
 
     // Clear the activity comment cache for this activity item
-    wp_cache_delete( 'bp_activity_comments_' . $activity->id, 'bp' );
+    wp_cache_delete( $activity->id, 'bp_activity_comments' );
 
     // If Akismet is active, and this was a manual spam/ham request, stop Akismet checking the activity