fix: check activity settings only when user account exists

When identification method is email and person has no account,
notification was being blocked by activity check for non-existent user.
Now checks if user exists before verifying activity notification settings.

Signed-off-by: Vitor Mattos <[email protected]>

Author: vitormattos

lib/Listener/MailNotifyListener.php

@@ -59,20 +59,27 @@ protected function sendSignMailNotification(
 			if ($identifyMethod->getEntity()->isDeletedAccount()) {
 				return;
 			}
-			if ($this->isNotificationDisabledAtActivity($identifyMethod->getEntity()->getIdentifierValue(), SendSignNotificationEvent::FILE_TO_SIGN)) {
-				return;
-			}
 			$email = '';
 			if ($identifyMethod->getName() === 'account') {
+				$userId = $identifyMethod->getEntity()->getIdentifierValue();
 				$email = $this->userManager
-					->get($identifyMethod->getEntity()->getIdentifierValue())
+					->get($userId)
 					->getEMailAddress();
 			} elseif ($identifyMethod->getName() === 'email') {
 				$email = $identifyMethod->getEntity()->getIdentifierValue();
 			}
 			if (empty($email)) {
 				return;
 			}
+
+			$users = $this->userManager->getByEmail($email);
+			if (count($users) === 1) {
+				$userId = $users[0]->getUID();
+				if ($this->isNotificationDisabledAtActivity($userId, SendSignNotificationEvent::FILE_TO_SIGN)) {
+					return;
+				}
+			}
+
 			$isFirstNotification = $this->signRequestMapper->incrementNotificationCounter($signRequest, 'mail');
 			if ($isFirstNotification) {
 				$this->mail->notifyUnsignedUser($signRequest, $email);