Groups - Fatal error when joining a group and when the Activity component is disabled

[r-a-y]

In r11924, I conditionally loaded the bp-groups-activity.php file if the Activity component is active.

What I failed to check was the bp-groups-functions.php file.

When joining a group, there is a call to groups_record_activity() with no bp_is_active() check. This causes a fatal error if the Activity component is disabled:
​https://buddypress.org/support/topic/go-back-to-2-9-4-issues-with-3-0-0/#post-273490

😢

[r-a-y]

Fixes a couple of other instances where groups_record_activity() is called.

[r-a-y]

In 12123:

Groups: Ensure 'last_activity' meta is updated when the Activity component is disabled.

In r11924, the bp-groups-activity.php file is loaded only if the Activity
component is disabled.

However, the groups_update_last_activity() function was in this file.
This function is needed to set the last_activity group meta entry. To
ensure the last_activity group meta entry is always updated, this commit
moves the groups_update_last_activity() function over to
bp-groups-functions.php.

Anti-props r-a-y.

See #7870 (3.x branch).

[r-a-y]

In 12124:

Groups: Ensure 'last_activity' meta is updated when the Activity component is disabled.

In r11924, the bp-groups-activity.php file is loaded only if the Activity
component is disabled.

However, the groups_update_last_activity() function was in this file.
This function is needed to set the last_activity group meta entry. To
ensure the last_activity group meta entry is always updated, this commit
moves the groups_update_last_activity() function over to
bp-groups-functions.php.

Anti-props r-a-y.

See #7870 (trunk).

[r-a-y]

In 12125:

Groups: Wrap groups_record_activity() calls with bp_is_active( 'activity' )

In r11924, the bp-groups-activity.php file is loaded only if the Activity
component is enabled.

However, the groups_record_activity() function is located in this file.
Because this function now relies on the Activity component, if the Activity
component is disabled, a fatal error can occur during certain group
actions. This commit addresses the problem by checking for the Activity
component before calling on the function.

Anti-props r-a-y.

See #7870 (3.x branch).

[r-a-y]

In 12126:

Groups: Wrap groups_record_activity() calls with bp_is_active( 'activity' )

In r11924, the bp-groups-activity.php file is loaded only if the Activity
component is enabled.

However, the groups_record_activity() function is located in this file.
Because this function now relies on the Activity component, if the Activity
component is disabled, a fatal error can occur during certain group
actions. This commit addresses the problem by checking for the Activity
component before calling on the function.

Anti-props r-a-y.

See #7870 (trunk).