Changeset 7966 for trunk/bp-groups/bp-groups-loader.php

Timestamp:

02/22/2014 09:21:21 PM (12 years ago)

Author:

r-a-y

Message:

Don't fetch item counts in profile navigation unnecessarily.

Previously, when a component's setup_nav() method is fired, the item
count for the component is fetched without any conditional checks.

This causes unneccessary queries to be fired when a user isn't logged
in or when we're not on a single user page.

This commit only fetches the item count in the profile nav if we're
currently on a single user page and/or if the current user has access
to the component nav item.

Fixes #5413

Props r-a-y, boonebgorges

File:

• trunk/bp-groups/bp-groups-loader.php (modified) (1 diff)

trunk/bp-groups/bp-groups-loader.php

@@ -347 +347 @@
         public function setup_nav( $main_nav = array(), $sub_nav = array() ) {
 
+                // Only grab count if we're on a user page
+                if ( bp_is_user() ) {
+                        $count    = bp_get_total_group_count_for_user();
+                        $class    = ( 0 === $count ) ? 'no-count' : 'count';
+                        $nav_name = sprintf( __( 'Groups <span class="%s">%s</span>', 'buddypress' ), esc_attr( $class ), number_format_i18n( $count ) );
+                } else {
+                        $nav_name = __( 'Groups', 'buddypress' );
+                }
+
                 // Add 'Groups' to the main navigation
-                $count    = bp_get_total_group_count_for_user();
-                $class    = ( 0 === $count ) ? 'no-count' : 'count';
                 $main_nav = array(
-                        'name'                => sprintf( __( 'Groups <span class="%s">%s</span>', 'buddypress' ), esc_attr( $class ), number_format_i18n( $count ) ),
+                        'name'                => $nav_name,
                         'slug'                => $this->slug,
                         'position'            => 70,