Changeset 9886

Timestamp:

05/21/2015 07:34:40 PM (10 years ago)

Author:

johnjamesjacoby

Message:

Blogs: Updates to bp_get_blog_class()

• Allow custom classes to be passed in via array parameter
• Clean up comparisons and variable definitions

See #6384. See r9884, r9885.

File:

• trunk/src/bp-blogs/bp-blogs-template.php (modified) (3 diffs)

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

@@ -713 +713 @@
  *
  * @since BuddyPress (1.7.0)
- */
-function bp_blog_class() {
-    echo bp_get_blog_class();
+ *
+ * @param array $classes Array of custom classes
+ */
+function bp_blog_class( $classes = array() ) {
+    echo bp_get_blog_class( $classes );
 }
     /**
@@ -723 +725 @@
      *
      * @global BP_Blogs_Template $blogs_template
+     * @param array $classes Array of custom classes
      *
      * @return string Row class of the site.
      */
-    function bp_get_blog_class() {
+    function bp_get_blog_class( $classes = array() ) {
         global $blogs_template;
 
-        $classes     = array();
-        $pos_in_loop = (int) $blogs_template->current_blog;
-
-        // If we've only one site in the loop, don't bother with odd and even.
-        if ( $blogs_template->blog_count > 1 )
-            $classes[] = ( $pos_in_loop % 2 ) ? 'even' : 'odd';
-        else
+        // Add even/odd classes, but only if there's more than 1 group
+        if ( $blogs_template->blog_count > 1 ) {
+            $pos_in_loop = (int) $blogs_template->current_blog;
+            $classes[]   = ( $pos_in_loop % 2 ) ? 'even' : 'odd';
+
+        // If we've only one site in the loop, don't bother with odd and even
+        } else {
             $classes[] = 'bp-single-blog';
+        }
 
         /**
@@ -747 +751 @@
         $classes = apply_filters( 'bp_get_blog_class', $classes );
         $classes = array_merge( $classes, array() );
-
-        $retval = 'class="' . join( ' ', $classes ) . '"';
+        $retval  = 'class="' . join( ' ', $classes ) . '"';
+
         return $retval;
     }