Fix: Don't wrongly warn about config duplicates

spnda · Nov 13, 2024 · 37fc1d9 · 37fc1d9
1 parent 88d35df
commit 37fc1d9
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/spigot/src/main/java/de/sean/blockprot/bukkit/config/DefaultConfig.java b/spigot/src/main/java/de/sean/blockprot/bukkit/config/DefaultConfig.java
@@ -106,12 +106,14 @@ private <T extends Enum<?>> void loadBlockListFromConfig(
             @NotNull String key, @NotNull final ArrayList<T> list, @NotNull final T[] enumValues, Function<T, Boolean> validateCallback) {
         List<?> configList = config.getList(key);
         if (configList == null) return;
-        ArrayList<String> stringList = configList
+        final var stringList = configList
             .stream()
             .filter(String.class::isInstance)
             .map(String.class::cast)
+            .distinct() // Remove duplicates
             .collect(Collectors.toCollection(ArrayList::new));
-        Set<T> newEnumValues = this.loadEnumValuesByName(enumValues, stringList);
+
+        final var newEnumValues = this.loadEnumValuesByName(enumValues, stringList);
         newEnumValues.removeIf((value) -> {
            if (!validateCallback.apply(value)) {
                BlockProt.getInstance().getLogger().warning("Caught invalid value passed to " + key + ": " + value.toString());