diff --git a/java/java-analysis-impl/src/com/intellij/codeInspection/RedundantSuppressInspectionBase.java b/java/java-analysis-impl/src/com/intellij/codeInspection/RedundantSuppressInspectionBase.java index 684e555f3c12..cf9e5c769732 100644 --- a/java/java-analysis-impl/src/com/intellij/codeInspection/RedundantSuppressInspectionBase.java +++ b/java/java-analysis-impl/src/com/intellij/codeInspection/RedundantSuppressInspectionBase.java @@ -18,10 +18,7 @@ package com.intellij.codeInspection; import com.intellij.analysis.AnalysisScope; import com.intellij.codeInsight.daemon.GroupNames; import com.intellij.codeInsight.daemon.impl.RemoveSuppressWarningAction; -import com.intellij.codeInspection.ex.GlobalInspectionContextBase; -import com.intellij.codeInspection.ex.GlobalInspectionToolWrapper; -import com.intellij.codeInspection.ex.InspectionToolWrapper; -import com.intellij.codeInspection.ex.LocalInspectionToolWrapper; +import com.intellij.codeInspection.ex.*; import com.intellij.codeInspection.reference.*; import com.intellij.codeInspection.ui.SingleCheckboxOptionsPanel; import com.intellij.openapi.diagnostic.Logger; @@ -303,8 +300,8 @@ public class RedundantSuppressInspectionBase extends GlobalInspectionTool { } protected InspectionToolWrapper[] getInspectionTools(PsiElement psiElement, @NotNull InspectionManager manager) { - return InspectionProjectProfileManager.getInstance(manager.getProject()).getCurrentProfile().getModifiableModel() - .getInspectionTools(psiElement); + // todo for what we create modifiable model here? + return new InspectionProfileModifiableModel(InspectionProjectProfileManager.getInstance(manager.getProject()).getCurrentProfile()).getInspectionTools(psiElement); } @Override diff --git a/java/java-tests/testSrc/com/intellij/codeInspection/OfflineInspectionResultViewTest.java b/java/java-tests/testSrc/com/intellij/codeInspection/OfflineInspectionResultViewTest.java index 321d587b0251..ab01e4be08dd 100644 --- a/java/java-tests/testSrc/com/intellij/codeInspection/OfflineInspectionResultViewTest.java +++ b/java/java-tests/testSrc/com/intellij/codeInspection/OfflineInspectionResultViewTest.java @@ -84,8 +84,8 @@ public class OfflineInspectionResultViewTest extends TestSourceBasedTestCase { @Override @NotNull - public InspectionProfileImpl getModifiableModel() { - return new InspectionProfileImpl("test") { + public InspectionProfileModifiableModel getModifiableModel() { + return new InspectionProfileModifiableModel(this) { @Override @NotNull public InspectionToolWrapper[] getInspectionTools(PsiElement element) { diff --git a/java/java-tests/testSrc/com/intellij/codeInspection/ex/InspectionProfileTest.java b/java/java-tests/testSrc/com/intellij/codeInspection/ex/InspectionProfileTest.java index 32b79b0c0dff..de9269ef60a2 100644 --- a/java/java-tests/testSrc/com/intellij/codeInspection/ex/InspectionProfileTest.java +++ b/java/java-tests/testSrc/com/intellij/codeInspection/ex/InspectionProfileTest.java @@ -215,8 +215,7 @@ public class InspectionProfileTest extends LightIdeaTestCase { ""); InspectionProfileImpl profile = createProfile(new InspectionProfileImpl("foo")); profile.readExternal(element); - InspectionProfileImpl model = profile.getModifiableModel(); - model.commit(); + profile.getModifiableModel().commit(); assertThat(profile.writeScheme()).isEqualTo(element); @@ -239,8 +238,7 @@ public class InspectionProfileTest extends LightIdeaTestCase { " \n" + ""); profile.readExternal(unusedProfile); - model = profile.getModifiableModel(); - model.commit(); + profile.getModifiableModel().commit(); assertEquals("\n" + "