Changeset 10789

Timestamp:

05/19/2016 05:47:22 PM (9 years ago)

Author:

r-a-y

Message:

Notifications: Update meta cache in BP_Notifications_Notification::get().

We update the meta cache if update_meta_cache is true, which it is by
default. Commit also updates the DocBlock to outline a few missing
parameters.

See #6257, #7067.

File:

• trunk/src/bp-notifications/classes/class-bp-notifications-notification.php (modified) (2 diffs)

trunk/src/bp-notifications/classes/class-bp-notifications-notification.php

@@ -630 +630 @@
      *     @type int          $per_page          Number of items to show per page. Default:
      *                                           false (no pagination - all items).
+     *     @type array        $meta_query        Array of meta_query conditions. See WP_Meta_Query::queries.
+     *     @type array        $date_query        Array of date_query conditions. See first parameter of
+     *                                           WP_Date_Query::__construct().
+     *     @type bool         $update_meta_cache Whether to update meta cache. Default: true.
      * }
      * @return array Located notifications.
@@ -682 +686 @@
         $sql = "{$select_sql} {$from_sql} {$join_sql} {$where_sql} {$order_sql} {$pag_sql}";
 
-        return $wpdb->get_results( $sql );
+        $results = $wpdb->get_results( $sql );
+
+        // Update meta cache.
+        if ( true === $r['update_meta_cache'] ) {
+            bp_notifications_update_meta_cache( wp_list_pluck( $results, 'id' ) );
+        }
+
+        return $results;
     }