diff --git a/java/java-impl/src/com/intellij/codeInsight/completion/JavaChainLookupElement.java b/java/java-impl/src/com/intellij/codeInsight/completion/JavaChainLookupElement.java index aba177fd4f2e..5c80671f0496 100644 --- a/java/java-impl/src/com/intellij/codeInsight/completion/JavaChainLookupElement.java +++ b/java/java-impl/src/com/intellij/codeInsight/completion/JavaChainLookupElement.java @@ -121,7 +121,7 @@ public class JavaChainLookupElement extends LookupElementDecorator= 0 && formatEnd >= 0) { - CodeStyleManager.getInstance(context.getProject()).reformatRange(context.getFile(), formatStart, formatEnd); + CodeStyleManager.getInstance(context.getProject()).reformatText(context.getFile(), formatStart, formatEnd); } } diff --git a/java/java-tests/testData/codeInsight/completion/normal/NoForceBraces.java b/java/java-tests/testData/codeInsight/completion/normal/NoForceBraces.java new file mode 100644 index 000000000000..a967eee64cf0 --- /dev/null +++ b/java/java-tests/testData/codeInsight/completion/normal/NoForceBraces.java @@ -0,0 +1,6 @@ +public class Foo { + void foo(boolean flag) { + if ('\\' == File.separach) + System.out.println(); + } +} diff --git a/java/java-tests/testData/codeInsight/completion/normal/NoForceBraces_after.java b/java/java-tests/testData/codeInsight/completion/normal/NoForceBraces_after.java new file mode 100644 index 000000000000..67bfc3131f7b --- /dev/null +++ b/java/java-tests/testData/codeInsight/completion/normal/NoForceBraces_after.java @@ -0,0 +1,8 @@ +import java.io.File; + +public class Foo { + void foo(boolean flag) { + if ('\\' == File.separatorChar) + System.out.println(); + } +} diff --git a/java/java-tests/testSrc/com/intellij/codeInsight/completion/NormalCompletionTest.groovy b/java/java-tests/testSrc/com/intellij/codeInsight/completion/NormalCompletionTest.groovy index f38adaf36b8d..2127814c8adb 100644 --- a/java/java-tests/testSrc/com/intellij/codeInsight/completion/NormalCompletionTest.groovy +++ b/java/java-tests/testSrc/com/intellij/codeInsight/completion/NormalCompletionTest.groovy @@ -406,6 +406,11 @@ public class NormalCompletionTest extends LightFixtureCompletionTestCase { doTest(); } + public void testNoForceBraces() { + codeStyleSettings.IF_BRACE_FORCE = CommonCodeStyleSettings.FORCE_BRACES_ALWAYS + doTest() + } + public void testExcludeStringBuffer() throws Throwable { CodeInsightSettings.getInstance().EXCLUDED_PACKAGES = [StringBuffer.name] as String[] try {