Logging activities should adapt to the context they're displayed in

[imath]

Logging activities (see #8581) should be more dependent on the context they're displayed in:

1. In the Activity directory page: what they look today
2. In the User's activity page: logging activity about them should stay without content
3. In the Group's activity page: instead of the group's avatar/cover it should be the one of the author of the activity.

[imath]

In ​8604.patch, I suggest to avoid using generated content activities when viewing a group or a personal stream when the activity type is specific to the displayed group or specific to the displayed member.

About the Group's activity stream, I've changed my mind because my initial thought would require more templating work.

[imath]

Let's carry on working on this during next release

[imath]

Let's discuss about this during 11.0 dev cycle.

[dcavins]

This is definitely a nice change. It was not that helpful seeing the user/group banner in each item in a user or group's activity stream. Works for me!

[imath]

In 13308:

Activity: restrict rich logging items to the site-wide stream

10.0.0 introduced richer activities about user's interaction with their profile or with groups (eg: user A changed their profile image). Displaying these richer activities into the self user or self group streams is less helpful and we are now restricting this extended version of these activity types to the site-wide activity stream.

Props dcavins

Fixes #8604

[imath]

[13308] introduced a notice error, before trying to get groups_get_current_group()->id we need to make sure a group is displayed

[imath]

In 13326:

Improve the way we check the group context of logging activities

[13308] introduced a PHP warning due to a misuse of the
groups_get_current_group() function (my bad!). When no group is
displayed this function returns false, so checking if it's a
BP_Groups_Group object having an id property is wrong.

Fixes #8604