Changeset 13320

Timestamp:

08/19/2022 03:06:51 PM (3 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',
+                )
+            );
+        }
     }
 }