Changeset 7918 for trunk/bp-blogs/bp-blogs-classes.php

Timestamp:

02/18/2014 12:50:11 PM (11 years ago)

Author:

boonebgorges

Message:

Pre-fetch blogmeta when in blog loops

This reduces overhead when querying for blog meta in the context of a
bp_has_blogs() template loop. Mirrors similar functionality in the activity,
groups, and xprofile components.

Also introduces an update_meta_cache parameter to the bp_has_blogs() stack, so
that plugin/theme authors can disable this pre-fetching.

Fixes #5398

File:

• trunk/bp-blogs/bp-blogs-classes.php (modified) (2 diffs)

trunk/bp-blogs/bp-blogs-classes.php

@@ -115 +115 @@
      * @param string|bool $search_terms Optional. Search by text stored in
      *        blogmeta (such as the blog name). Default: false.
+     * @param bool $update_meta_cache Whether to pre-fetch metadata for
+     *        blogs. Default: true.
      * @return array Multidimensional results array, structured as follows:
      *           'blogs' - Array of located blog objects
      *           'total' - A count of the total blogs matching the filter params
      */
-    public static function get( $type, $limit = false, $page = false, $user_id = 0, $search_terms = false ) {
+    public static function get( $type, $limit = false, $page = false, $user_id = 0, $search_terms = false, $update_meta_cache = true ) {
         global $bp, $wpdb;
 
@@ -162 +164 @@
         $paged_blogs = BP_Blogs_Blog::get_blog_extras( $paged_blogs, $blog_ids, $type );
 
+        if ( $update_meta_cache ) {
+            bp_blogs_update_meta_cache( $blog_ids );
+        }
+
         return array( 'blogs' => $paged_blogs, 'total' => $total_blogs );
     }