Changeset 9676 for trunk/src/bp-xprofile/classes/class-bp-xprofile-group.php

Timestamp:

04/01/2015 11:14:16 PM (11 years ago)

Author:

johnjamesjacoby

Message:

XProfile: Introduce bp_xprofile_groups cache group, to replace bp cache group usage.

• Reduces field group cache keys down to just their ID, using a unique cache group
• Use all key instead of 'xprofile_groups_inc_empty`
• Use default_visibility_levels key instead of xprofile_default_visibility_levels

See #6341, #5733.

File:

• trunk/src/bp-xprofile/classes/class-bp-xprofile-group.php (modified) (7 diffs)

trunk/src/bp-xprofile/classes/class-bp-xprofile-group.php

@@ -82 +82 @@
                 global $wpdb;
 
-                $group = wp_cache_get( 'xprofile_group_' . $this->id, 'bp' );
+                $group = wp_cache_get( $id, 'bp_xprofile_groups' );
 
                 if ( false === $group ) {
-                        $bp = buddypress();
+                        $bp    = buddypress();
                         $group = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$bp->profile->table_name_groups} WHERE id = %d", $id ) );
+                        wp_cache_set( $id, $group, 'bp_xprofile_groups' );
                 }
 
@@ -94 +95 @@
 
                 $this->id          = $group->id;
-                $this->name        = stripslashes( $group->name );
-                $this->description = stripslashes( $group->description );
+                $this->name        = $group->name;
+                $this->description = $group->description;
                 $this->can_delete  = $group->can_delete;
                 $this->group_order = $group->group_order;
@@ -456 +457 @@
 
                         // If cached data is found, use it
-                        $cache_key  = 'xprofile_group_' . $group_id;
-                        $group_data = wp_cache_get( $cache_key, 'bp' );
+                        $group_data = wp_cache_get( $group_id, 'bp_xprofile_groups' );
                         if ( false !== $group_data ) {
                                 $groups[ $group_id ] = $group_data;
@@ -493 +493 @@
 
                                         // Cache previously uncached group data
-                                        $cache_key = 'xprofile_group_' . $gdata->id;
-                                        wp_cache_set( $cache_key, $gdata, 'bp' );
+                                        wp_cache_set( $gdata->id, $gdata, 'bp_xprofile_groups' );
                                 }
                         }
@@ -543 +542 @@
 
                 // Purge profile field group cache
-                wp_cache_delete( 'xprofile_groups_inc_empty', 'bp' );
+                wp_cache_delete( 'all', 'bp_xprofile_groups' );
 
                 $bp = buddypress();
@@ -608 +607 @@
                 global $wpdb;
 
-                $default_visibility_levels = wp_cache_get( 'xprofile_default_visibility_levels', 'bp' );
+                $default_visibility_levels = wp_cache_get( 'default_visibility_levels', 'bp_xprofile' );
 
                 if ( false === $default_visibility_levels ) {
@@ -628 +627 @@
                         }
 
-                        wp_cache_set( 'xprofile_default_visibility_levels', $default_visibility_levels, 'bp' );
+                        wp_cache_set( 'default_visibility_levels', $default_visibility_levels, 'bp_xprofile' );
                 }