Skip to content

Commit

Permalink
disabled users should never get notifications #2419 #4475
Browse files Browse the repository at this point in the history
That means it's safe not to have this check.
  • Loading branch information
pdurbin committed Feb 26, 2021
1 parent e73c43b commit 8f08f7b
Showing 1 changed file with 0 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -103,10 +103,6 @@ public void sendNotification(AuthenticatedUser dataverseUser, Timestamp sendDate
}

public void sendNotification(AuthenticatedUser dataverseUser, Timestamp sendDate, Type type, Long objectId, String comment, AuthenticatedUser requestor, boolean isHtmlContent) {
if (dataverseUser.isDisabled()) {
logger.info("An attempt was made to send a " + type + " notification to a disabled user: " + dataverseUser);
return;
}
UserNotification userNotification = new UserNotification();
userNotification.setUser(dataverseUser);
userNotification.setSendDate(sendDate);
Expand Down

0 comments on commit 8f08f7b

Please sign in to comment.