"Convert to primary constructor": do not move init section down

So #KT-19629 Fixed

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

@@ -18,6 +18,7 @@ package org.jetbrains.kotlin.idea.intentions
 
 import com.intellij.openapi.editor.Editor
 import com.intellij.openapi.util.TextRange
+import com.intellij.psi.search.searches.ReferencesSearch
 import org.jetbrains.kotlin.descriptors.*
 import org.jetbrains.kotlin.idea.caches.resolve.analyzeWithContent
 import org.jetbrains.kotlin.idea.inspections.IntentionBasedInspection
@@ -181,10 +182,19 @@ class ConvertSecondaryConstructorToPrimaryIntention : SelfTargetingRangeIntentio
         with (constructorInClass.replace(constructor)) {
             constructorCommentSaver.restore(this)
         }
-        element.delete()
 
-        if ((initializer.body as? KtBlockExpression)?.statements?.isNotEmpty() ?: false) {
+        if ((initializer.body as? KtBlockExpression)?.statements?.isNotEmpty() == true) {
-            klass.addDeclaration(initializer)
+            val hasPropertyAfterInitializer = element.parent.children.takeLastWhile { it !== element }.any {
+                it is KtProperty && ReferencesSearch.search(it, initializer.useScope).findFirst() != null
+            }
+            if (hasPropertyAfterInitializer) {
+                // In this case we must move init {} down, because it uses a property declared below
+                klass.addDeclaration(initializer)
+            } else {
+                klass.addDeclarationAfter(initializer, element)
+            }
         }
+
+        element.delete()
     }
 }

idea/testData/intentions/convertSecondaryConstructorToPrimary/init.kt

@@ -7,4 +7,6 @@ class ConvertToInit {
         foo()
         bar()
     }
+
+    fun baz() {}
 }

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

@@ -8,4 +8,5 @@ class ConvertToInit() {
         bar()
     }
 
+    fun baz() {}
 }

idea/testData/intentions/convertSecondaryConstructorToPrimary/varArg.kt

@@ -7,4 +7,6 @@ class WithVarArg {
     constructor(<caret>vararg zz: String) {
         x = listOf(*zz)
     }
+
+    fun foo() {}
 }

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

@@ -8,4 +8,5 @@ class WithVarArg(vararg zz: String) {
         x = listOf(*zz)
     }
 
+    fun foo() {}
 }