Changeset 7898 for trunk/bp-groups/bp-groups-functions.php

Timestamp:

02/16/2014 09:47:18 PM (13 years ago)

Author:

boonebgorges

Message:

Introduce $args param to BP_Groups_Group constructor and groups_get_group()

This argument array can contain the following toggles:

• update_meta_cache (pre-fetch all group metadata for the queried group)
• populate_extras (pre-fetch last activity, admin/mod lists, and current user access/membership status in queried group)

Both items default to false, for backward compatibility

Allowing these pre-fetch toggles to be set to false allows us to avoid large
numbers of unnecessary database queries in certain cases, such as when
secondary avatars are populated on activity directories.

See #5398

File:

• trunk/bp-groups/bp-groups-functions.php (modified) (2 diffs)

trunk/bp-groups/bp-groups-functions.php

@@ -42 +42 @@
 function groups_get_group( $args = '' ) {
         $defaults = array(
-                'group_id'   => false,
-                'load_users' => false
+                'group_id'          => false,
+                'load_users'        => false,
+                'populate_extras'   => true,
+                'update_meta_cache' => true,
         );
 
@@ -52 +54 @@
 
         if ( !$group = wp_cache_get( $cache_key, 'bp' ) ) {
-                $group = new BP_Groups_Group( $group_id, true, $load_users );
+                $group_args = array(
+                        'populate_extras'   => $populate_extras,
+                        'update_meta_cache' => $update_meta_cache,
+                );
+
+                $group = new BP_Groups_Group( $group_id, $group_args );
                 wp_cache_set( $cache_key, $group, 'bp' );
         }