Changeset 11851 for trunk/src/bp-notifications/bp-notifications-functions.php

Timestamp:

02/08/2018 03:20:59 PM (7 years ago)

Author:

boonebgorges

Message:

Notifications: User count query should be separately fetched and cached.

Previously, notification counts were calculated by fetching the entire
notification objects.

Props m_uysl.
See #7130.

File:

• trunk/src/bp-notifications/bp-notifications-functions.php (modified) (1 diff)

trunk/src/bp-notifications/bp-notifications-functions.php

@@ -608 +608 @@
  */
 function bp_notifications_get_unread_notification_count( $user_id = 0 ) {
-    $notifications = bp_notifications_get_all_notifications_for_user( $user_id );
-    $count         = ! empty( $notifications ) ? count( $notifications ) : 0;
+    if ( empty( $user_id ) ) {
+        $user_id = ( bp_displayed_user_id() ) ? bp_displayed_user_id() : bp_loggedin_user_id();
+    }
+
+    $count = wp_cache_get( $user_id, 'bp_notifications_unread_count' );
+    if ( false === $count ) {
+        $count = BP_Notifications_Notification::get_total_count( array(
+            'user_id' => $user_id,
+            'is_new'  => true,
+        ) );
+        wp_cache_set( $user_id, $count, 'bp_notifications_unread_count' );
+    }
 
     /**