Changeset 12126 for trunk/src/bp-groups/actions/create.php

Timestamp:

05/25/2018 11:12:06 PM (7 years ago)

Author:

r-a-y

Message:

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).

File:

• trunk/src/bp-groups/actions/create.php (modified) (1 diff)

trunk/src/bp-groups/actions/create.php

@@ -183 +183 @@
 
             // Once we completed all steps, record the group creation in the activity stream.
-            groups_record_activity( array(
-                'type' => 'created_group',
-                'item_id' => $bp->groups->new_group_id
-            ) );
+            if ( bp_is_active( 'activity' ) ) {
+                groups_record_activity( array(
+                    'type' => 'created_group',
+                    'item_id' => $bp->groups->new_group_id
+                ) );
+            }
 
             /**