Changeset 5987 for trunk/bp-members/bp-members-functions.php

Timestamp:

04/13/2012 04:31:51 PM (14 years ago)

Author:

boonebgorges

Message:

'Total member count' cleanup:

• Introduces bp_core_get_active_member_count() to get an accurate count of members with recorded last_activity
• Points bp_get_total_member_count() to bp_core_get_active_member_count(), so that 'All Members' tab count on member directories matches the total count prepared by BP_Core_User::get_users(). Fixes #4061
• Changes 'All Members' count in activity/index.php to use bp_get_total_member_count(), for greater accuracy and consistency between activity and members components
• Introduces a cache-busting function to be fired when member counts change (new members, spam status changed, member deleted)

File:

• trunk/bp-members/bp-members-functions.php (modified) (4 diffs)

trunk/bp-members/bp-members-functions.php

@@ -487 +487 @@
 
 /**
+ * Returns the total number of members, limited to those members with last_activity
+ *
+ * @return int The number of active members
+ */
+function bp_core_get_active_member_count() {
+    global $wpdb;
+
+    if ( !$count = get_transient( 'bp_active_member_count' ) ) {
+        // Avoid a costly join by splitting the lookup
+        if ( is_multisite() ) {
+            $sql = $wpdb->prepare( "SELECT ID FROM $wpdb->users WHERE (user_status != 0 OR deleted != 0 OR user_status != 0)" );
+        } else {
+            $sql = $wpdb->prepare( "SELECT ID FROM $wpdb->users WHERE user_status != 0" );
+        }
+
+        $exclude_users = $wpdb->get_col( $sql );
+        $exclude_users_sql = !empty( $exclude_users ) ? $wpdb->prepare( "AND user_id NOT IN (" . implode( ',', wp_parse_id_list( $exclude_users ) ) . ")" ) : '';
+
+        $count = (int) $wpdb->get_var( $wpdb->prepare( "SELECT COUNT(user_id) FROM $wpdb->usermeta WHERE meta_key = 'last_activity' {$exclude_users_sql}" ) );
+        set_transient( 'bp_active_member_count', $count );
+    }
+
+    return apply_filters( 'bp_core_get_active_member_count', $count );
+}
+
+/**
  * Processes a spammed or unspammed user
  *
@@ -768 +794 @@
         return false;
 
+    do_action( 'bp_core_pre_delete_account', $user_id );
+
     // Specifically handle multi-site environment
     if ( is_multisite() ) {
@@ -773 +801 @@
         require( ABSPATH . '/wp-admin/includes/user.php' );
 
-        return wpmu_delete_user( $user_id );
+        $retval = wpmu_delete_user( $user_id );
 
     // Single site user deletion
     } else {
         require( ABSPATH . '/wp-admin/includes/user.php' );
-        return wp_delete_user( $user_id );
-    }
+        $retval = wp_delete_user( $user_id );
+    }
+
+    do_action( 'bp_core_deleted_account', $user_id );
+
+    return $retval;
 }
 
@@ -1149 +1181 @@
         $wpdb->query( $wpdb->prepare( "UPDATE $wpdb->users SET user_pass = %s WHERE ID = %d", $user['meta']['password'], $user_id ) );
 
-    // Delete the total member cache
-    wp_cache_delete( 'bp_total_member_count', 'bp' );
-
     do_action( 'bp_core_activated_user', $user_id, $key, $user );