More correct detection of super property

idea/src/org/jetbrains/kotlin/idea/j2k/IdeaJavaToKotlinServices.kt

@@ -16,13 +16,23 @@
 
 package org.jetbrains.kotlin.idea.j2k
 
+import com.intellij.psi.PsiMethod
 import org.jetbrains.kotlin.j2k.*
 
-object IdeaJavaToKotlinServices: JavaToKotlinConverterServices {
+object IdeaJavaToKotlinServices : JavaToKotlinConverterServices {
     override val referenceSearcher: ReferenceSearcher
         get() = IdeaReferenceSearcher
+
+    override val superMethodsSearcher: SuperMethodsSearcher
+        get() = IdeaSuperMethodSearcher
+
     override val resolverForConverter: ResolverForConverter
         get() = IdeaResolverForConverter
+
     override val docCommentConverter: DocCommentConverter
         get() = IdeaDocCommentConverter
+}
+
+object IdeaSuperMethodSearcher : SuperMethodsSearcher {
+    override fun findDeepestSuperMethods(method: PsiMethod) = method.findDeepestSuperMethods().asList()
 }

idea/src/org/jetbrains/kotlin/idea/j2k/IdeaReferenceSearcher.kt

@@ -49,4 +49,4 @@ public object IdeaReferenceSearcher: ReferenceSearcher {
         val searchScope = GlobalSearchScope.getScopeRestrictedByFileTypes(GlobalSearchScope.projectScope(element.getProject()), *fileTypes.toTypedArray())
         return ReferencesSearch.search(element, searchScope).findAll()
     }
-}
+}

j2k/src/org/jetbrains/kotlin/j2k/JavaToKotlinConverterServices.kt

@@ -16,8 +16,11 @@
 
 package org.jetbrains.kotlin.j2k
 
+import com.intellij.psi.PsiMethod
+
 interface JavaToKotlinConverterServices {
     val referenceSearcher: ReferenceSearcher
+    val superMethodsSearcher: SuperMethodsSearcher
     val resolverForConverter: ResolverForConverter
     val docCommentConverter: DocCommentConverter
 }
@@ -25,8 +28,22 @@ interface JavaToKotlinConverterServices {
 object EmptyJavaToKotlinServices: JavaToKotlinConverterServices {
     override val referenceSearcher: ReferenceSearcher
         get() = EmptyReferenceSearcher
+
+    override val superMethodsSearcher: SuperMethodsSearcher
+        get() = SuperMethodsSearcher.Default
+
     override val resolverForConverter: ResolverForConverter
         get() = EmptyResolverForConverter
+
     override val docCommentConverter: DocCommentConverter
         get() = EmptyDocCommentConverter
-}
+}
+
+interface SuperMethodsSearcher {
+    fun findDeepestSuperMethods(method: PsiMethod): Collection<PsiMethod>
+
+    object Default : SuperMethodsSearcher {
+        // use simple findSuperMethods by default because findDeepestSuperMethods requires some service from IDEA
+        override fun findDeepestSuperMethods(method: PsiMethod) = method.findSuperMethods().asList()
+    }
+}

j2k/src/org/jetbrains/kotlin/j2k/propertyDetection.kt

@@ -255,17 +255,15 @@ private class PropertyDetector(
     }
 
     private fun createAccessorInfo(getOrSetMethod: PsiMethod, field: PsiField?, kind: AccessorKind, propertyName: String): AccessorInfo? {
-        //TODO: deepest method
         //TODO: multiple
-        for (superSignature in getOrSetMethod.hierarchicalMethodSignature.superSignatures) {
+        for (superMethod in converter.services.superMethodsSearcher.findDeepestSuperMethods(getOrSetMethod)) {
-            val method = superSignature.method
+            val containingClass = superMethod.containingClass!!
-            val containingClass = method.containingClass!!
             val superPropertyInfo: SuperPropertyInfo? = if (converter.inConversionScope(containingClass)) {
-                val propertyInfo = converter.propertyDetectionCache[containingClass][method]
+                val propertyInfo = converter.propertyDetectionCache[containingClass][superMethod]
                 if (propertyInfo != null) SuperPropertyInfo(propertyInfo.isVar) else null
             }
-            else if (method is KotlinLightMethod) {
+            else if (superMethod is KotlinLightMethod) {
-                val origin = method.getOrigin()
+                val origin = superMethod.getOrigin()
                 if (origin is JetProperty) SuperPropertyInfo(origin.isVar) else null
             }
             else {

j2k/testData/JavaApi.java

@@ -1,7 +1,10 @@
 package javaApi;
 
 import org.jetbrains.annotations.Nullable;
+
+import java.lang.String;
 import java.util.Set;
+import kotlinApi.KotlinClassWithProperties;
 
 public @interface Anon1 {
     String[] value();
@@ -103,4 +106,12 @@ public class JavaClassWithProperties {
 
     public int getValue4() { return 1; }
     public void setValue4(int value) { }
+}
+
+public class JavaClassDerivedFromKotlinClassWithProperties extends KotlinClassWithProperties {
+    @Override
+    public String getSomeVar1() { return ""; }
+
+    @Override
+    public void setSomeVar2(String value) { }
 }

j2k/testData/fileOrElement/dropAccessors/OverridesOfExternalCode.java

@@ -1,5 +1,6 @@
 import kotlinApi.KotlinClassWithProperties;
 import javaApi.JavaClassWithProperties;
+import javaApi.JavaClassDerivedFromKotlinClassWithProperties;
 
 import org.jetbrains.annotations.NotNull;
 
@@ -100,4 +101,13 @@ class C extends A {
     public String getSomeVar1() {
         return super.getSomeVar1();
     }
+}
+
+class D extends JavaClassDerivedFromKotlinClassWithProperties {
+    @Override
+    public String getSomeVar1() { return "a"; }
+
+    @Override
+    public void setSomeVar2(String value) { }
+
 }

j2k/testData/fileOrElement/dropAccessors/OverridesOfExternalCode.kt

@@ -2,8 +2,11 @@
 // ERROR: Property must be initialized
 // ERROR: Property must be initialized
 // ERROR: Property must be initialized
+// ERROR: Property must be initialized
+// ERROR: Property must be initialized
 import kotlinApi.KotlinClassWithProperties
 import javaApi.JavaClassWithProperties
+import javaApi.JavaClassDerivedFromKotlinClassWithProperties
 
 internal open class A : KotlinClassWithProperties() {
     override var someVar1: String
@@ -82,4 +85,16 @@ internal class C : A() {
         get() {
             return super.someVar1
         }
+}
+
+internal class D : JavaClassDerivedFromKotlinClassWithProperties() {
+    override var someVar1: String
+        get() {
+            return "a"
+        }
+
+    override var someVar2: String
+        set(value: String) {
+        }
+
 }