Secondary Avatar Link

[jag1989]

I've been trying to wrap any secondary avatar in a link to that profile, group, event etc. I would like to use this within the activity stream, so that users can just click on the avatars to reach a destination. Currently BuddyPress only has the first avatar linked. However I would like the secondary to be 'linkable', and any further avatars within the activity.

See Here: (http://buddypress.org/community/groups/how-to-and-troubleshooting/forum/topic/wrapping-secondary-avatar-with-link/)

After posting this Boone stated that because there are not enough filters within a particular hook to get the information I need.

Here's another quick example:

On a fresh BP install, if one navigated to their activity feed where there were two avatars per activity item, only the first one would be linked. However the second one would be mid sentence and have a link next to it. I wanted to move this avatar next to the first one, and have it linked to the appropriate destination.

[DJPaul]

I think this involves changing/adding filters in bp_get_activity_secondary_avatar()

[transom]

Add optional argument to bp_get_activity_secondary_avatar - (bool) "linked", if true, wraps the secondary avatar with a link to the corresponding object.

Add two new filters:

bp_get_activity_secondary_avatar_link - passed the URI of the computed link to the object
bp_get_activity_secondary_avatar_link_class - passed an empty string that is added as a class to the link

[transom]

Better handling of link class option

[transom]

Added new version of patch with better handling (use case) of the class attribute for the optional link applied to the secondary avatar. This patch removes the bp_get_activity_secondary_avatar_link_class and adds another optional argument "link_class". The value of the provided "link_class" argument is added to the secondary avatar link (default value is "")

[boonebgorges]

Thanks for the patch, transom. This approach generally looks good - linking the secondary avatar seems like a pretty common request, so I'm happy to add the 'linked' option in addition to the filters. I'll have a closer look in the upcoming weeks (unless another core dev gets to it first)

[johnjamesjacoby]

(In [6607]) Add ability to link secondary avatars in activity streams. Props jag1989. Fixes #4436.