bp_get_activity_avatar() for components do not work

[r-a-y]

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/

[johnjamesjacoby]

(In [3168]) Fixes #2578. (1.3 early bird special)

[johnjamesjacoby]

Primary avatars are always users, since groups and blogs don't do anything on their own. With that in mind, I introduced the bp_activity_secondary_avatar and bp_get_activity_secondary_avatar functions, that will behave according to the object and item_id for that activity.

This gives us functions to work with to tick off secondary activity avatars off the 1.3 when the time comes also. All it needs is a str_replace on the bp_activity_action to insert it in there, which can be done with a filter to get it off the ground.