Rename sanction and license options txt email templates to include po…

…stfix (#1038)
oncokb · Oct 23, 2023 · d29e9cb · d29e9cb
1 parent 09decc3
commit d29e9cb
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/org/mskcc/cbio/oncokb/domain/enumeration/MailType.java b/src/main/java/org/mskcc/cbio/oncokb/domain/enumeration/MailType.java
@@ -33,7 +33,7 @@ public enum MailType {
         .templateName("rejectionUsSanctionEmail")
         .description("User Rejection - US Sanction")
         .titleKey("email.reject.title")
-        .stringTemplateName("rejectionUsSanctionEmail.txt"))
+        .stringTemplateName("rejectionUsSanctionEmailString.txt"))
     , REJECT_ALUMNI_ADDRESS(new MailTypeBuilder()
         .templateName("alumniEmailAddress")
         .description("Reject - Registered under alumni email address")

diff --git a/src/main/java/org/mskcc/cbio/oncokb/service/MailService.java b/src/main/java/org/mskcc/cbio/oncokb/service/MailService.java
@@ -94,7 +94,7 @@ public void sendEmail(String to, String from, String cc, String subject, String
         MimeMessage mimeMessage = javaMailSender.createMimeMessage();
         MimeMessageHelper message = new MimeMessageHelper(mimeMessage, isMultipart, StandardCharsets.UTF_8.name());
         message.setTo(to);
-        if (cc != null) {
+        if (StringUtils.isNotEmpty(cc)) {
             message.setCc(cc);
         }
         message.setFrom(from);
@@ -190,7 +190,7 @@ public void sendEmailFromSlack(UserDTO user, String subject, String body, MailTy
                 ccAddress = defaultEmailSendFrom;
             }
             sendEmail(user.getEmail(), defaultEmailSendFrom, ccAddress, subject, body, null, false, false);
-            addUserMailsRecord(user, mailType, defaultEmailSendFrom, sendBy != null ? sendBy : "Through slack, unknown sender");
+            addUserMailsRecord(user, mailType, defaultEmailSendFrom, StringUtils.isNotEmpty(sendBy) ? sendBy : "Through slack, unknown sender");
             log.info("Sent email to User '{}'", user.getEmail());
         } catch (MailException | MessagingException e) {
             log.warn("Email could not be sent to user '{}'", user.getEmail(), e);

diff --git a/...sources/templates/mail/licenseOptions.txt → ...s/templates/mail/licenseOptionsString.txt b/...sources/templates/mail/licenseOptions.txt → ...s/templates/mail/licenseOptionsString.txt
diff --git a/...mplates/mail/rejectionUsSanctionEmail.txt → ...s/mail/rejectionUsSanctionEmailString.txt b/...mplates/mail/rejectionUsSanctionEmail.txt → ...s/mail/rejectionUsSanctionEmailString.txt