email only managers refs #65

VEuPathDB · Nov 18, 2023 · 6a49e0e · 6a49e0e
1 parent 3bc0c3c
commit 6a49e0e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/main/java/org/veupathdb/service/access/model/Config.java b/src/main/java/org/veupathdb/service/access/model/Config.java
@@ -11,7 +11,7 @@ public class Config extends Options
 
   @CommandLine.Option(
     names = "--enable-email",
-    defaultValue = "true", // Permanently enable e-mail, ignoring env var. Testing can be done with fake smtp.
+    defaultValue = "${env:ENABLE_EMAIL}", // set true to permanently enable e-mail, ignoring env var. Testing can be done with fake smtp.
     arity = "1"
   )
   @SuppressWarnings("FieldMayBeFinal")

diff --git a/src/main/java/org/veupathdb/service/access/service/user/EndUserPatchService.java b/src/main/java/org/veupathdb/service/access/service/user/EndUserPatchService.java
@@ -98,6 +98,7 @@ public void applySelfPatch(
         .orElseThrow();
       Optional<String> userEmail = AccountRepo.Select.getInstance().selectEmailByUserId(row.getUserId());
       final var ccs = ProviderRepo.Select.byDataset(row.getDatasetId(), 100L, 0L).stream()
+        .filter(ProviderRow::isManager)
         .map(UserRow::getEmail)
         .toArray(String[]::new);
       EndUserRepo.Update.self(row, userID);