Changeset 13320

Timestamp:

08/19/2022 03:06:51 PM (4 years ago)

Author:

dcavins

Message:

Membership Requests: Check if notifications are active.

Before attempting to delete notifications related to a
membership request, ensure that the notifications
component is active.

Props babyface, pycc, MickFrench.

File:

• branches/10.0/src/bp-members/bp-members-membership-requests.php (modified) (1 diff)

branches/10.0/src/bp-members/bp-members-membership-requests.php

@@ -246 +246 @@
  */
 function bp_members_membership_requests_delete_notifications_on_change( $signup_ids ) {
-        foreach ( $signup_ids as $signup_id ) {
-                BP_Notifications_Notification::delete(
-                        array(
-                                'item_id'          => $signup_id,
-                                'component_action' => 'membership_request_submitted',
-                        )
-                );
+        if ( bp_is_active( 'notifications' ) ) {
+                foreach ( $signup_ids as $signup_id ) {
+                        /**
+                         * We use this method instead of one of the notification functions
+                         * because we want to delete all notifications for all admins.
+                         */
+                        BP_Notifications_Notification::delete(
+                                array(
+                                        'item_id'          => $signup_id,
+                                        'component_action' => 'membership_request_submitted',
+                                )
+                        );
+                }
         }
 }