Changeset 8593 for trunk/src/bp-blogs/bp-blogs-functions.php

Timestamp:

07/11/2014 02:13:05 PM (12 years ago)

Author:

johnjamesjacoby

Message:

Clean up bp_blogs_get_blogs():

• Use bp_parse_args() to allow just-in-time filtering of query arguments.
• Remove explode() usage.
• Add inline phpdoc to arguments.

File:

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

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

@@ -45 +45 @@
 function bp_blogs_get_blogs( $args = '' ) {
 
-        $defaults = array(
-                'type'              => 'active', // active, alphabetical, newest, or random
-                'user_id'           => false,    // Pass a user_id to limit to only blogs that this user has privilages higher than subscriber on
-                'include_blog_ids'  => false,
-                'search_terms'      => false,    // Limit to blogs that match these search terms
+        // Parse query arguments
+        $r = bp_parse_args( $args, array(
+                'type'              => 'active', // 'active', 'alphabetical', 'newest', or 'random'
+                'include_blog_ids'  => false,    // Array of blog IDs to include
+                'user_id'           => false,    // Limit to blogs this user can post to
+                'search_terms'      => false,    // Limit to blogs matching these search terms
                 'per_page'          => 20,       // The number of results to return per page
                 'page'              => 1,        // The page to return if limiting per page
-                'update_meta_cache' => true,
+                'update_meta_cache' => true      // Whether to pre-fetch blogmeta
+        ), 'blogs_get_blogs' );
+
+        // Get the blogs
+        $blogs = BP_Blogs_Blog::get(
+                $r['type'],
+                $r['per_page'],
+                $r['page'],
+                $r['user_id'],
+                $r['search_terms'],
+                $r['update_meta_cache'],
+                $r['include_blog_ids']
         );
 
-        $params = wp_parse_args( $args, $defaults );
-        extract( $params, EXTR_SKIP );
-
-        return apply_filters( 'bp_blogs_get_blogs', BP_Blogs_Blog::get( $type, $per_page, $page, $user_id, $search_terms, $update_meta_cache, $include_blog_ids ), $params );
+        // Filter and return
+        return apply_filters( 'bp_blogs_get_blogs', $blogs, $r );
 }