fixed some code smells

logicaldoc · Oct 3, 2023 · 3942d4d · 3942d4d
1 parent c336277
commit 3942d4d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/logicaldoc-core/src/main/java/com/logicaldoc/core/security/LoginThrottle.java b/logicaldoc-core/src/main/java/com/logicaldoc/core/security/LoginThrottle.java
@@ -288,7 +288,7 @@ private static void notifyBruteForceAttack(String suspectedUsername, String susp
 	}
 
 	private static List<User> getRecipients() throws PersistenceException {
-		List<User> recipients = new ArrayList<User>();
+		List<User> recipients = new ArrayList<>();
 		String setting = Context.get().getProperties().getProperty("throttle.alert.recipients", "");
 		if (StringUtils.isNotEmpty(setting)) {
 			UserDAO uDao = (UserDAO) Context.get().getBean(UserDAO.class);

diff --git a/...caldoc-gui/src/main/java/com/logicaldoc/gui/frontend/client/security/BruteForcePanel.java b/...caldoc-gui/src/main/java/com/logicaldoc/gui/frontend/client/security/BruteForcePanel.java
@@ -41,6 +41,8 @@
  */
 public class BruteForcePanel extends AdminPanel {
 
+	private static final String RECIPIENTS = "recipients";
+
 	private static final String ATTEMPTS = "attempts";
 
 	private static final String THROTTLE_IP_WAIT = "throttle.ip.wait";
@@ -147,10 +149,10 @@ private void initForm(Map<String, String> params) {
 			// Nothing to do
 		}
 
-		MultiComboBoxItem recipients = ItemFactory.newMultiComboBoxItem("recipients", "alertrecipients",
+		MultiComboBoxItem recipients = ItemFactory.newMultiComboBoxItem(RECIPIENTS, "alertrecipients",
 				new UsersDS(null, false, false),
 				params.get(THROTTLE_ALERT_RECIPIENTS) != null && !params.get(THROTTLE_ALERT_RECIPIENTS).trim().isEmpty()
-						? params.get(THROTTLE_ALERT_RECIPIENTS).trim().split("[,]")
+						? params.get(THROTTLE_ALERT_RECIPIENTS).trim().split(",")
 						: null);
 		recipients.setValueField("username");
 		recipients.setDisplayField("username");
@@ -264,9 +266,9 @@ public void onSave() {
 		params[5] = new GUIParameter(THROTTLE_USERNAME_DISABLEUSER,
 				"yes".equals(values.get("usernamedisableuser").toString()) ? "true" : "false");
 
-		if (values.get("recipients") != null) {
+		if (values.get(RECIPIENTS) != null) {
 			@SuppressWarnings("unchecked")
-			ArrayList<String> usernames = (ArrayList<String>) values.get("recipients");
+			ArrayList<String> usernames = (ArrayList<String>) values.get(RECIPIENTS);
 			params[6] = new GUIParameter(THROTTLE_ALERT_RECIPIENTS,
 					usernames.stream().collect(Collectors.joining(",")));
 		} else {