Move try to higher level

RADAR-base · Jul 8, 2024 · 2335d6a · 2335d6a
1 parent 1712147
commit 2335d6a
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/main/java/org/radarbase/appserver/service/transmitter/EmailNotificationTransmitter.java b/src/main/java/org/radarbase/appserver/service/transmitter/EmailNotificationTransmitter.java
@@ -29,12 +29,12 @@ public EmailNotificationTransmitter(
     @Override
     public void send(Notification notification) throws EmailMessageTransmitException {
         if (notification.isEmailEnabled()) {
-            if (notification.getUser().getEmailAddress() == null || notification.getUser().getEmailAddress().isBlank()) {
-                log.warn("Could not transmit a notification via email because subject {} has no email address.",
-                    notification.getUser().getSubjectId());
-                return;
-            }
             try {
+                if (notification.getUser().getEmailAddress() == null || notification.getUser().getEmailAddress().isBlank()) {
+                    log.warn("Could not transmit a notification via email because subject {} has no email address.",
+                        notification.getUser().getSubjectId());
+                    return;
+                }
                 emailSender.send(createEmailFromNotification(notification));
             } catch (Exception e) {
                 log.error("Could not transmit a notification via email", e);