Changeset 6100 for trunk/bp-groups/bp-groups-classes.php

Timestamp:

06/14/2012 08:12:59 PM (13 years ago)

Author:

djpaul

Message:

When a group is created or deleted, clear each group member's "total groups count" cache.
Addresses a bug where the counts never changed in an environment with a persistent object cache.

• See #4226
• Specify cache group for to wp_cache_delete() in groups_clear_group_user_object_cache().
• Add $user_ids parameter to the 'bp_groups_delete_group' action to save a repeat DB query later.
• Add $group object parameter to the 'groups_created_group' action.

File:

• trunk/bp-groups/bp-groups-classes.php (modified) (1 diff)

trunk/bp-groups/bp-groups-classes.php

@@ -173 +173 @@
 
         // Fetch the user IDs of all the members of the group
-        $user_ids = BP_Groups_Member::get_group_member_ids( $this->id );
-        $user_ids = implode( ',', (array) $user_ids );
+        $user_ids    = BP_Groups_Member::get_group_member_ids( $this->id );
+        $user_id_str = implode( ',', (array) $user_ids );
 
         // Modify group count usermeta for members
-        $wpdb->query( $wpdb->prepare( "UPDATE {$wpdb->usermeta} SET meta_value = meta_value - 1 WHERE meta_key = 'total_group_count' AND user_id IN ( {$user_ids} )" ) );
+        $wpdb->query( $wpdb->prepare( "UPDATE {$wpdb->usermeta} SET meta_value = meta_value - 1 WHERE meta_key = 'total_group_count' AND user_id IN ( {$user_id_str} )" ) );
 
         // Now delete all group member entries
         BP_Groups_Member::delete_all( $this->id );
 
-        do_action_ref_array( 'bp_groups_delete_group', array( &$this ) );
+        do_action_ref_array( 'bp_groups_delete_group', array( &$this, $user_ids ) );
 
         wp_cache_delete( 'bp_groups_group_' . $this->id, 'bp' );