Changeset 2118

Timestamp:

11/25/2009 10:19:26 AM (16 years ago)

Author:

apeatling

Message:

Moving duplicated avatar ID's to a class instead. Fixes #1328 props r-a-y.

File:

• branches/1.1/bp-core/bp-core-avatars.php (modified) (3 diffs)

branches/1.1/bp-core/bp-core-avatars.php

@@ -78 +78 @@
     }
 
-    if ( !$css_id )
-        $css_id = $object . '-' . $item_id . '-avatar';
+    /* Add an identifying class to each item */
+    $class .= ' ' . $object . '-' . $item_id . '-avatar';
+
+    if ( !empty($css_id) )
+        $css_id = " id='{$css_id}'";
 
     if ( $width )
@@ -113 +116 @@
 
         if ( $avatar_url )
-            return apply_filters( 'bp_core_fetch_avatar', "<img src='{$avatar_url}' alt='{$alt}' id='{$css_id}' class='{$class}'{$html_width}{$html_height} />", $params );
+            return apply_filters( 'bp_core_fetch_avatar', "<img src='{$avatar_url}' alt='{$alt}' class='{$class}'{$css_id}{$html_width}{$html_height} />", $params );
     }
 
@@ -139 +142 @@
         $gravatar = apply_filters( 'bp_gravatar_url', 'http://www.gravatar.com/avatar/' ) . md5( $grav_email ) . '?d=' . $default_grav . '&amp;s=' . $grav_size;
 
-        return apply_filters( 'bp_core_fetch_avatar', "<img src='{$gravatar}' alt='{$alt}' id='{$css_id}' class='{$class}'{$html_width}{$html_height} />", $params );
+        return apply_filters( 'bp_core_fetch_avatar', "<img src='{$gravatar}' alt='{$alt}' class='{$class}'{$css_id}{$html_width}{$html_height} />", $params );
 
     } else {