Function that adds 'buddypress" to body tag is too greedy

[dcavins]

I've been adding the class buddypress to my themes manually via the functions.php file:

if ( function_exists( 'bp_is_blog_page' ) && !bp_is_blog_page() ) {
        $classes[] = 'buddypress';
      }

and it looks like similar functionality has been added to bp-core-template with 1.7. However, the new implementation sets 'buddypress' nearly all the time in my installation:

// Add BuddyPress class if we are within a BuddyPress page
		if ( !empty( $bp_classes ) ) {
			$bp_classes[] = 'buddypress';
		}

There are a lot of false positives that populate $bp_classes, like being logged in or visiting the home page (even if you're not logged in).

I definitely think adding 'buddypress' to the body class where appropriate is really helpful for theme developers.

Thanks for all of the excellent work in 1.7.

-David

[DJPaul]

I don't think the body is meant to have the BuddyPress class on non-BP pages

[boonebgorges]

r6920 r6921