From ef2cc29b137919c4bac38bb916da9e061a09169f Mon Sep 17 00:00:00 2001
From: Max Medvedev <maxim.medvedev@jetbrains.com>
Date: Fri, 21 Sep 2012 17:05:53 +0400
Subject: [PATCH] IDEA-90089 Groovy: diamonds are not suggested in Smart Code
 Completion after "new" if declaration and assignment are split

---
 .../GroovySmartCompletionContributor.java     | 42 +++++++++++++++++--
 .../completion/GroovyCompletionTest.groovy    | 31 ++++++++++++++
 2 files changed, 69 insertions(+), 4 deletions(-)

diff --git a/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/completion/GroovySmartCompletionContributor.java b/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/completion/GroovySmartCompletionContributor.java
index 40e4154b8741..397989a17057 100644
--- a/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/completion/GroovySmartCompletionContributor.java
+++ b/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/completion/GroovySmartCompletionContributor.java
@@ -29,6 +29,7 @@ import com.intellij.psi.*;
 import com.intellij.psi.codeStyle.CodeStyleSettings;
 import com.intellij.psi.codeStyle.CodeStyleSettingsManager;
 import com.intellij.psi.search.GlobalSearchScope;
+import com.intellij.psi.tree.IElementType;
 import com.intellij.psi.util.PsiTreeUtil;
 import com.intellij.psi.util.TypeConversionUtil;
 import com.intellij.util.Consumer;
@@ -40,6 +41,7 @@ import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 import org.jetbrains.plugins.groovy.config.GroovyConfigUtils;
 import org.jetbrains.plugins.groovy.lang.completion.handlers.AfterNewClassInsertHandler;
+import org.jetbrains.plugins.groovy.lang.lexer.GroovyTokenTypes;
 import org.jetbrains.plugins.groovy.lang.psi.GrReferenceElement;
 import org.jetbrains.plugins.groovy.lang.psi.GroovyPsiElementFactory;
 import org.jetbrains.plugins.groovy.lang.psi.api.GroovyResolveResult;
@@ -48,9 +50,9 @@ import org.jetbrains.plugins.groovy.lang.psi.api.statements.GrVariable;
 import org.jetbrains.plugins.groovy.lang.psi.api.statements.GrVariableDeclaration;
 import org.jetbrains.plugins.groovy.lang.psi.api.statements.expressions.*;
 import org.jetbrains.plugins.groovy.lang.psi.api.statements.expressions.literals.GrLiteral;
+import org.jetbrains.plugins.groovy.lang.psi.dataFlow.types.TypeInferenceHelper;
 import org.jetbrains.plugins.groovy.lang.psi.expectedTypes.GroovyExpectedTypesProvider;
 import org.jetbrains.plugins.groovy.lang.psi.expectedTypes.TypeConstraint;
-import org.jetbrains.plugins.groovy.lang.psi.dataFlow.types.TypeInferenceHelper;
 import org.jetbrains.plugins.groovy.lang.psi.util.PsiUtil;
 import org.jetbrains.plugins.groovy.refactoring.GroovyRefactoringUtil;
 
@@ -220,7 +222,12 @@ public class GroovySmartCompletionContributor extends CompletionContributor {
     final GrExpression expression = PsiTreeUtil.getParentOfType(place, GrExpression.class);
     if (expression == null) return;
 
-    final Set<PsiType> types = GroovyExpectedTypesProvider.getDefaultExpectedTypes(expression);
+    GrExpression placeToInferType = expression;
+    if (expression.getParent() instanceof GrApplicationStatement && expression.getParent().getParent() instanceof GrAssignmentExpression) {
+      placeToInferType = (GrExpression)expression.getParent();
+    }
+
+    final Set<PsiType> types = GroovyExpectedTypesProvider.getDefaultExpectedTypes(placeToInferType);
     for (PsiType type : types) {
       if (type instanceof PsiArrayType) {
         final LookupItem item = PsiTypeLookupItem.createLookupItem(GenericsUtil.eliminateWildcards(type), place);
@@ -271,9 +278,36 @@ public class GroovySmartCompletionContributor extends CompletionContributor {
     if (!(parent instanceof GrNewExpression)) return null;
 
     final PsiElement pparent = parent.getParent();
-    if (!(pparent instanceof GrVariable)) return null;
 
-    return ((GrVariable)pparent).getDeclaredType();
+    if (pparent instanceof GrVariable) {
+      return ((GrVariable)pparent).getDeclaredType();
+    }
+    else if (pparent instanceof GrAssignmentExpression) {
+      GrAssignmentExpression assignment = (GrAssignmentExpression)pparent;
+      IElementType optoken = assignment.getOperationToken();
+
+      GrExpression lvalue = assignment.getLValue();
+      GrExpression rvalue = assignment.getRValue();
+
+      if (parent == rvalue && optoken == GroovyTokenTypes.mASSIGN) {
+        return lvalue.getNominalType();
+      }
+    }
+    else if (pparent instanceof GrApplicationStatement) {
+      PsiElement ppparent = pparent.getParent();
+      if (ppparent instanceof GrAssignmentExpression) {
+        GrAssignmentExpression assignment = (GrAssignmentExpression)ppparent;
+        IElementType optoken = assignment.getOperationToken();
+
+        GrExpression lvalue = assignment.getLValue();
+        GrExpression rvalue = assignment.getRValue();
+
+        if (pparent == rvalue && optoken == GroovyTokenTypes.mASSIGN) {
+          return lvalue.getNominalType();
+        }
+      }
+    }
+    return null;
   }
 
   @Override
diff --git a/plugins/groovy/test/org/jetbrains/plugins/groovy/completion/GroovyCompletionTest.groovy b/plugins/groovy/test/org/jetbrains/plugins/groovy/completion/GroovyCompletionTest.groovy
index 0bb35e3f3243..6e09ae6a2c53 100644
--- a/plugins/groovy/test/org/jetbrains/plugins/groovy/completion/GroovyCompletionTest.groovy
+++ b/plugins/groovy/test/org/jetbrains/plugins/groovy/completion/GroovyCompletionTest.groovy
@@ -21,6 +21,7 @@ import com.intellij.codeInsight.completion.CompletionType
 import com.intellij.codeInsight.completion.impl.CamelHumpMatcher
 import com.intellij.codeInsight.lookup.LookupElement
 import com.intellij.codeInsight.lookup.LookupElementPresentation
+import com.intellij.codeInsight.lookup.PsiTypeLookupItem
 import com.intellij.psi.codeStyle.CodeStyleSettingsManager
 import org.jetbrains.plugins.groovy.GroovyFileType
 import org.jetbrains.plugins.groovy.codeStyle.GroovyCodeStyleSettings
@@ -1467,4 +1468,34 @@ class Inheritor extends Base {
 }
 ''')
   }
+
+  void testDiamondCompletionInAssignmentCompletion() {
+    doSmartTest('''\
+class Foo<T> {}
+
+Foo<String> var
+var = new <caret>
+''', '''\
+class Foo<T> {}
+
+Foo<String> var
+var = new Foo<>()<caret>
+''')
+  }
+
+  void testDiamondCompletionInAssignmentCompletion2() {
+    myFixture.with {
+      configureByText('_a.groovy', '''\
+class Foo<T> {}
+
+Foo<String> var
+var = new <caret>Foo<String>()
+''')
+      complete(CompletionType.SMART)
+      assertEquals(1, lookupElements.length)
+
+      assertInstanceOf(lookupElements[0], PsiTypeLookupItem)
+      assertTrue((lookupElements[0] as PsiTypeLookupItem).myDiamond)
+    }
+  }
 }
\ No newline at end of file