Fixed trivial issue NAS-2741. It wrote out useless warnings to the log

netarchivesuite · Apr 18, 2018 · 32ffbf4 · 32ffbf4
1 parent 9708b36
commit 32ffbf4
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/harvester/harvester-core/src/main/java/dk/netarkivet/harvester/datamodel/Domain.java b/harvester/harvester-core/src/main/java/dk/netarkivet/harvester/datamodel/Domain.java
@@ -818,11 +818,12 @@ public void setCrawlerTraps(List<String> regExps, boolean strictMode) {
                 		+ e.getDescription() + " . Please correct the expression.");
             }
         }
-        if (strictMode) 
-        	if (errMsgs.size() > 0) {
-            throw new ArgumentNotValid(errMsgs.size() +  " errors were found: " + StringUtils.conjoin(",", errMsgs));
-        } else {
-            log.warn(errMsgs.size() +  " errors were found: " + StringUtils.conjoin(",", errMsgs));
+        if (errMsgs.size() > 0) {
+            if (strictMode){ 
+                throw new ArgumentNotValid(errMsgs.size() +  " errors were found: " + StringUtils.conjoin(",", errMsgs));
+            } else {
+                log.warn(errMsgs.size() +  " errors were found: " + StringUtils.conjoin(",", errMsgs));
+            }
         }
         crawlerTraps = Collections.unmodifiableList(cleanedListOfCrawlerTraps);
         if (!crawlerTraps.isEmpty()) {