NPE & SOE on array access converting

plugins/groovy/src/org/jetbrains/plugins/groovy/lang/psi/impl/PsiImplUtil.java

@@ -448,9 +448,10 @@ public class PsiImplUtil {
            TypesUtil.isAssignable(PsiType.INT, argTypes[0], manager, resolveScope);
   }
 
-  public static GroovyResolveResult getIndexPropertyMethodCandidate(PsiType thisType,
+  public static GroovyResolveResult getIndexPropertyMethodCandidate(@Nullable PsiType thisType,
                                                                     PsiType[] argTypes,
                                                                     GroovyPsiElement place) {
+    if (thisType == null) return GroovyResolveResult.EMPTY_RESULT;
     GroovyResolveResult[] candidates = ResolveUtil.getMethodCandidates(thisType, "getAt", place, argTypes);
     if (candidates.length != 1) {
       final GrTupleType tupleType = new GrTupleType(argTypes, JavaPsiFacade.getInstance(place.getProject()), place.getResolveScope());

plugins/groovy/src/org/jetbrains/plugins/groovy/refactoring/convertToJava/ExpressionGenerator.java

@@ -758,7 +758,7 @@ public class ExpressionGenerator extends Generator {
     final GrNamedArgument[] namedArgs = argList.getNamedArguments();
 
     if (PsiImplUtil.isSimpleArrayAccess(thisType, argTypes, manager, resolveScope)) {
-      expression.accept(this);
+      selectedExpression.accept(this);
       builder.append("[");
       final GrExpression arg = exprArgs[0];
       arg.accept(this);

plugins/groovy/test/org/jetbrains/plugins/groovy/refactoring/convertToJava/CodeBlockGenerationTest.groovy

@@ -150,4 +150,7 @@ public final class Matcher {
   void testAnonymousFromMap() {doTest()}
   void testClosure() {doTest()}
   void _testListAsArray() {doTest()}
+
+  void testUnresolvedArrayAccess() {doTest()}
+  void testArrayAccess() {doTest()}
 }

plugins/groovy/testdata/refactoring/convertGroovyToJava/codeBlock/arrayAccess.groovy

@@ -0,0 +1,2 @@
+int[] foo = [1, 2, 3]
+print foo[2]

plugins/groovy/testdata/refactoring/convertGroovyToJava/codeBlock/arrayAccess.java

@@ -0,0 +1,2 @@
+java.lang.Integer[] foo = new java.lang.Integer[]{1, 2, 3};
+print(foo[2]);

plugins/groovy/testdata/refactoring/convertGroovyToJava/codeBlock/unresolvedArrayAccess.groovy

@@ -0,0 +1,2 @@
+def foo
+print(foo[1])

plugins/groovy/testdata/refactoring/convertGroovyToJava/codeBlock/unresolvedArrayAccess.java

@@ -0,0 +1,2 @@
+java.lang.Object foo;
+print(foo.getAt(1));