Changeset 7939

Timestamp:

02/20/2014 03:16:52 PM (11 years ago)

Author:

boonebgorges

Message:

Don't query for group extras if user is not logged in

Saves a couple needless database queries for unauthenticated users.

Fixes #5416

Props lenasterg for initial patch

File:

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

trunk/bp-groups/bp-groups-classes.php

@@ -1223 +1223 @@
 
         // Fetch the logged-in user's status within each group
-        $user_status_results = $wpdb->get_results( $wpdb->prepare( "SELECT group_id, is_confirmed, invite_sent FROM {$bp->groups->table_name_members} WHERE user_id = %d AND group_id IN ( {$group_ids} ) AND is_banned = 0", bp_loggedin_user_id() ) );
+        if ( is_user_logged_in() ) {
+            $user_status_results = $wpdb->get_results( $wpdb->prepare( "SELECT group_id, is_confirmed, invite_sent FROM {$bp->groups->table_name_members} WHERE user_id = %d AND group_id IN ( {$group_ids} ) AND is_banned = 0", bp_loggedin_user_id() ) );
+        } else {
+            $user_status_results = array();
+        }
 
         // Reindex
@@ -1256 +1260 @@
         }
 
-        $user_banned = $wpdb->get_col( $wpdb->prepare( "SELECT group_id FROM {$bp->groups->table_name_members} WHERE is_banned = 1 AND user_id = %d AND group_id IN ( {$group_ids} )", bp_loggedin_user_id() ) );
+        if ( is_user_logged_in() ) {
+            $user_banned = $wpdb->get_col( $wpdb->prepare( "SELECT group_id FROM {$bp->groups->table_name_members} WHERE is_banned = 1 AND user_id = %d AND group_id IN ( {$group_ids} )", bp_loggedin_user_id() ) );
+        } else {
+            $user_banned = array();
+        }
+
         for ( $i = 0, $count = count( $paged_groups ); $i < $count; ++$i ) {
             $paged_groups[$i]->is_banned = false;