Fix "secondary constructor -> primary" in case of implicit super call

So #KT-16903 Fixed

idea/src/org/jetbrains/kotlin/idea/intentions/ConvertSecondaryConstructorToPrimaryIntention.kt

@@ -30,6 +30,7 @@ import org.jetbrains.kotlin.psi.psiUtil.endOffset
 import org.jetbrains.kotlin.psi.psiUtil.startOffset
 import org.jetbrains.kotlin.resolve.BindingContext
 import org.jetbrains.kotlin.resolve.DescriptorToSourceUtils
+import org.jetbrains.kotlin.util.kind
 
 class ConvertSecondaryConstructorToPrimaryInspection : IntentionBasedInspection<KtSecondaryConstructor>(
         ConvertSecondaryConstructorToPrimaryIntention::class,
@@ -162,17 +163,16 @@ class ConvertSecondaryConstructorToPrimaryIntention : SelfTargetingRangeIntentio
 
         element.moveParametersToPrimaryConstructorAndInitializers(constructor, parameterToPropertyMap, context, factory)
 
-        val delegationCall = element.getDelegationCall()
+        val argumentList = element.getDelegationCall().valueArgumentList
-        val argumentList = delegationCall.valueArgumentList
+        for (superTypeListEntry in klass.superTypeListEntries) {
-        if (!delegationCall.isImplicit && argumentList != null) {
+            val typeReference = superTypeListEntry.typeReference ?: continue
-            for (superTypeListEntry in klass.superTypeListEntries) {
+            val type = context[BindingContext.TYPE, typeReference]
-                val typeReference = superTypeListEntry.typeReference ?: continue
+            if ((type?.constructor?.declarationDescriptor as? ClassifierDescriptorWithTypeParameters)?.kind == ClassKind.CLASS) {
-                val type = context[BindingContext.TYPE, typeReference]
+                val superTypeCallEntry = factory.createSuperTypeCallEntry(
-                if ((type?.constructor?.declarationDescriptor as? ClassDescriptor)?.kind == ClassKind.CLASS) {
+                        "${typeReference.text}${argumentList?.text ?: "()"}"
-                    val superTypeCallEntry = factory.createSuperTypeCallEntry("${typeReference.text}${argumentList.text}")
+                )
-                    superTypeListEntry.replace(superTypeCallEntry)
+                superTypeListEntry.replace(superTypeCallEntry)
-                    break
+                break
-                }
             }
         }
 

idea/testData/intentions/convertSecondaryConstructorToPrimary/withBaseAliasImplicit.kt

@@ -0,0 +1,7 @@
+abstract class A<T>
+
+typealias AS = A<String>
+
+class C : AS {
+    <caret>constructor()
+}

idea/testData/intentions/convertSecondaryConstructorToPrimary/withBaseAliasImplicit.kt.after

@@ -0,0 +1,6 @@
+abstract class A<T>
+
+typealias AS = A<String>
+
+class C() : AS() {
+}

idea/testData/intentions/convertSecondaryConstructorToPrimary/withBaseClassImplicit.kt

@@ -0,0 +1,5 @@
+abstract class A
+
+class C : A {
+    <caret>constructor()
+}

idea/testData/intentions/convertSecondaryConstructorToPrimary/withBaseClassImplicit.kt.after

@@ -0,0 +1,4 @@
+abstract class A
+
+class C() : A() {
+}

idea/testData/intentions/convertSecondaryConstructorToPrimary/withInterfaceAlias.kt

@@ -0,0 +1,7 @@
+interface A<T>
+
+typealias AS = A<String>
+
+class C : AS {
+    <caret>constructor()
+}

idea/testData/intentions/convertSecondaryConstructorToPrimary/withInterfaceAlias.kt.after

@@ -0,0 +1,6 @@
+interface A<T>
+
+typealias AS = A<String>
+
+class C() : AS {
+}

idea/tests/org/jetbrains/kotlin/idea/intentions/IntentionTestGenerated.java

@@ -5636,12 +5636,24 @@ public class IntentionTestGenerated extends AbstractIntentionTest {
             doTest(fileName);
         }
 
+        @TestMetadata("withBaseAliasImplicit.kt")
+        public void testWithBaseAliasImplicit() throws Exception {
+            String fileName = KotlinTestUtils.navigationMetadata("idea/testData/intentions/convertSecondaryConstructorToPrimary/withBaseAliasImplicit.kt");
+            doTest(fileName);
+        }
+
         @TestMetadata("withBaseClass.kt")
         public void testWithBaseClass() throws Exception {
             String fileName = KotlinTestUtils.navigationMetadata("idea/testData/intentions/convertSecondaryConstructorToPrimary/withBaseClass.kt");
             doTest(fileName);
         }
 
+        @TestMetadata("withBaseClassImplicit.kt")
+        public void testWithBaseClassImplicit() throws Exception {
+            String fileName = KotlinTestUtils.navigationMetadata("idea/testData/intentions/convertSecondaryConstructorToPrimary/withBaseClassImplicit.kt");
+            doTest(fileName);
+        }
+
         @TestMetadata("withBaseClassNoArgs.kt")
         public void testWithBaseClassNoArgs() throws Exception {
             String fileName = KotlinTestUtils.navigationMetadata("idea/testData/intentions/convertSecondaryConstructorToPrimary/withBaseClassNoArgs.kt");
@@ -5672,6 +5684,12 @@ public class IntentionTestGenerated extends AbstractIntentionTest {
             doTest(fileName);
         }
 
+        @TestMetadata("withInterfaceAlias.kt")
+        public void testWithInterfaceAlias() throws Exception {
+            String fileName = KotlinTestUtils.navigationMetadata("idea/testData/intentions/convertSecondaryConstructorToPrimary/withInterfaceAlias.kt");
+            doTest(fileName);
+        }
+
         @TestMetadata("withModifiers.kt")
         public void testWithModifiers() throws Exception {
             String fileName = KotlinTestUtils.navigationMetadata("idea/testData/intentions/convertSecondaryConstructorToPrimary/withModifiers.kt");