Fix signup notifications for deleted accounts (#1355)

MbinOrg · Jan 14, 2025 · 8a3d435 · 8a3d435
1 parent 1aae911
commit 8a3d435
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/MessageHandler/Notification/SentNewSignupNotificationHandler.php b/src/MessageHandler/Notification/SentNewSignupNotificationHandler.php
@@ -38,6 +38,11 @@ public function doWork(MessageInterface $message): void
         if (!$user) {
             throw new UnrecoverableMessageHandlingException('user not found');
         }
-        $this->signupNotificationManager->sendNewSignupNotification($user);
+
+        if (!$user->isAccountDeleted() && !$user->isSoftDeleted() && null === $user->markedForDeletionAt) {
+            // only send notifications for new accounts if the account is not deleted,
+            // this is necessary because we create dummy accounts to block the username when an account is deleted
+            $this->signupNotificationManager->sendNewSignupNotification($user);
+        }
     }
 }