[buddypress-trac] [BuddyPress] #2578: bp_get_activity_avatar() for components do not work

[buddypress-trac at lists.automattic.com]

#2578: bp_get_activity_avatar() for components do not work
----------------------+-----------------------------------------------------
 Reporter:  r-a-y     |       Owner:           
     Type:  defect    |      Status:  new      
 Priority:  normal    |   Milestone:  1.2.6    
Component:  Activity  |    Keywords:  has-patch
----------------------+-----------------------------------------------------
 If you take a look at the bp_get_activity_avatar() function, it appears
 that the code takes into consideration avatars for other components.

 eg. If you created a group activity update, the group avatar should show
 instead of the default user one

 However, there are some redundant checks for the user ID that breaks this
 functionality.

 The attached patch based on the BP 1.2 branch fixes this.

 The only thing I haven't taken into consideration is the link for the
 avatar.  Right now, it also defaults to the user's link, should it link to
 the component's home page?

 If so, I can create a patch for the bp_get_activity_user_link() function
 as well.

 ---

 Thanks to Anton for posting about this on the forums:
 http://buddypress.org/community/groups/creating-extending/forum/topic
 /group-avatar-in-activity-stream/

-- 
Ticket URL: <http://trac.buddypress.org/ticket/2578>
BuddyPress <http://buddypress.org/>
BuddyPress