From 6902845ec523e5aa107332b1eff53840d37695cf Mon Sep 17 00:00:00 2001 From: Anna Kozlova Date: Wed, 7 Oct 2015 15:43:48 +0200 Subject: [PATCH] extract method: don't search for the valid place to insert empty method when nullability state is calculated and containing class is not physical and used as a context only (IDEA-146102) --- .../refactoring/extractMethod/ExtractMethodProcessor.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/java/java-impl/src/com/intellij/refactoring/extractMethod/ExtractMethodProcessor.java b/java/java-impl/src/com/intellij/refactoring/extractMethod/ExtractMethodProcessor.java index e8a7c55ae61d..e8b339064ef0 100644 --- a/java/java-impl/src/com/intellij/refactoring/extractMethod/ExtractMethodProcessor.java +++ b/java/java-impl/src/com/intellij/refactoring/extractMethod/ExtractMethodProcessor.java @@ -640,7 +640,7 @@ public class ExtractMethodProcessor implements MatchProvider { if (classCopy == null) { return null; } - final PsiMethod emptyMethod = (PsiMethod)classCopy.add(generateEmptyMethod("name")); + final PsiMethod emptyMethod = (PsiMethod)classCopy.addAfter(generateEmptyMethod("name"), PsiTreeUtil.getParentOfType(classCopy, PsiMember.class)); prepareMethodBody(emptyMethod, false); if (myNotNullConditionalCheck || myNullConditionalCheck) { return Nullness.NULLABLE;