Changeset 8053 for trunk/bp-messages/bp-messages-classes.php

Timestamp:

03/06/2014 01:28:03 AM (11 years ago)

Author:

r-a-y

Message:

Cache the unread inbox count for a user.

This commit:

• Caches calls to BP_Messages_Thread::get_inbox_count()
• Invalidates the cache when a new message is sent or deleted
• Adds a unit test

See #5414

File:

• trunk/bp-messages/bp-messages-classes.php (modified) (4 diffs)

trunk/bp-messages/bp-messages-classes.php

@@ -301 +301 @@
         $sql = $wpdb->prepare( "UPDATE {$bp->messages->table_name_recipients} SET unread_count = 0 WHERE user_id = %d AND thread_id = %d", bp_loggedin_user_id(), $thread_id );
         $wpdb->query($sql);
+
+        wp_cache_delete( bp_loggedin_user_id(), 'bp_messages_unread_count' );
     }
 
@@ -315 +317 @@
         $sql = $wpdb->prepare( "UPDATE {$bp->messages->table_name_recipients} SET unread_count = 1 WHERE user_id = %d AND thread_id = %d", bp_loggedin_user_id(), $thread_id );
         $wpdb->query($sql);
+
+        wp_cache_delete( bp_loggedin_user_id(), 'bp_messages_unread_count' );
     }
 
@@ -383 +387 @@
 
     /**
-     * Gets the inbox message count for a user.
+     * Gets the unread message count for a user.
      *
      * @since BuddyPress (1.0.0)
@@ -397 +401 @@
         }
 
-        $sql = $wpdb->prepare( "SELECT SUM(unread_count) FROM {$bp->messages->table_name_recipients} WHERE user_id = %d AND is_deleted = 0 AND sender_only = 0", $user_id );
-        $unread_count = $wpdb->get_var( $sql );
-
-        if ( empty( $unread_count ) || is_wp_error( $unread_count ) ) {
-            return 0;
+        $unread_count = wp_cache_get( $user_id, 'bp_messages_unread_count' );
+
+        if ( false === $unread_count ) {
+            $unread_count = (int) $wpdb->get_var( $wpdb->prepare( "SELECT SUM(unread_count) FROM {$bp->messages->table_name_recipients} WHERE user_id = %d AND is_deleted = 0 AND sender_only = 0", $user_id ) );
+
+            wp_cache_set( $user_id, $unread_count, 'bp_messages_unread_count' );
         }