diff --git a/java/compiler/impl/src/com/intellij/compiler/options/ValidationConfigurable.java b/java/compiler/impl/src/com/intellij/compiler/options/ValidationConfigurable.java index 68a99bc134de..bf0eba94bb83 100644 --- a/java/compiler/impl/src/com/intellij/compiler/options/ValidationConfigurable.java +++ b/java/compiler/impl/src/com/intellij/compiler/options/ValidationConfigurable.java @@ -101,7 +101,7 @@ public class ValidationConfigurable implements SearchableConfigurable, Configura @Override public boolean isModified() { List selectedElements = myValidators.getMarkedElements(); - List markedValidators = getMarkedValidators(); + List markedValidators = getMarkedValidators(ContainerUtil.concat(selectedElements, myValidators.getElements(false))); if (markedValidators.size() != selectedElements.size()) { return true; } @@ -134,15 +134,14 @@ public class ValidationConfigurable implements SearchableConfigurable, Configura @Override public void reset() { myValidateBox.setSelected(myConfiguration.isValidateOnBuild()); - final List validators = getValidators(); - Collections.sort(validators, Comparator.comparing(Compiler::getDescription)); - myValidators.setElements(validators, false); - myValidators.markElements(getMarkedValidators()); + final List allValidators = getValidators(); + Collections.sort(allValidators, Comparator.comparing(Compiler::getDescription)); + myValidators.setElements(allValidators, false); + myValidators.markElements(getMarkedValidators(allValidators)); myExcludedConfigurable.reset(); } - private List getMarkedValidators() { - final List validators = getValidators(); + private List getMarkedValidators(@NotNull List validators) { return ContainerUtil.mapNotNull(validators, (NullableFunction)validator -> myConfiguration.isSelected(validator) ? validator : null); }