Fix add modifier for replacement case, add expect/actual order tests

compiler/frontend/src/org/jetbrains/kotlin/psi/addRemoveModifier.kt

@@ -83,10 +83,11 @@ internal fun addModifier(modifierList: KtModifierList, modifier: KtModifierKeywo
         }
         return
     }
-    if (modifierToReplace != null) {
+    if (modifierToReplace != null && modifierList.firstChild == modifierList.lastChild) {
         modifierToReplace.replace(newModifier)
     }
     else {
+        modifierToReplace?.delete()
         val newModifierOrder = MODIFIERS_ORDER.indexOf(modifier)
 
         fun placeAfter(child: PsiElement): Boolean {

idea/testData/intentions/changeVisibility/private/hasModifier3.kt.after

@@ -1 +1 @@
-anno<caret>tation private class Ann
+private annotation class Ann

idea/testData/intentions/changeVisibility/private/hasModifier4.kt.after

@@ -1 +1 @@
-annotation private cla<caret>ss Ann
+private annotation class Ann

idea/testData/multiModuleQuickFix/orderHeader/header/header.kt

@@ -0,0 +1,3 @@
+// "Replace with 'expect'" "true"
+
+public open <caret>header class My

idea/testData/multiModuleQuickFix/orderHeader/header/header.kt.after

@@ -0,0 +1,3 @@
+// "Replace with 'expect'" "true"
+
+public expect open class My

idea/testData/multiModuleQuickFix/orderHeader/jvm/My.kt

@@ -0,0 +1 @@
+actual class My

idea/testData/multiModuleQuickFix/orderHeader/jvm/My.kt.after

@@ -0,0 +1 @@
+actual class My

idea/testData/multiModuleQuickFix/orderImpl/header/header.kt

@@ -0,0 +1 @@
+header fun foo(n: Int): Int

idea/testData/multiModuleQuickFix/orderImpl/header/header.kt.after

@@ -0,0 +1 @@
+header fun foo(n: Int): Int

idea/testData/multiModuleQuickFix/orderImpl/jvm/foo.kt

@@ -0,0 +1,3 @@
+// "Replace with 'actual'" "true"
+
+<caret>impl public tailrec fun foo(n: Int): Int = if (n < 2) n else foo(n - 1)

idea/testData/multiModuleQuickFix/orderImpl/jvm/foo.kt.after

@@ -0,0 +1,3 @@
+// "Replace with 'actual'" "true"
+
+public actual tailrec fun foo(n: Int): Int = if (n < 2) n else foo(n - 1)

idea/testData/quickfix/override/overridingDelegatedMethod.kt.after

@@ -4,7 +4,7 @@ interface X {
 }
 
 interface XX : X {
-    override open fun foo() {
+    open override fun foo() {
 
     }
 }

idea/testData/quickfix/override/overridingFakeOverride.kt.after

@@ -4,7 +4,7 @@ interface X {
 }
 
 interface XX : X {
-    override open fun foo() {
+    open override fun foo() {
 
     }
 }

idea/tests/org/jetbrains/kotlin/idea/quickfix/QuickFixMultiModuleTest.kt

@@ -94,6 +94,16 @@ class QuickFixMultiModuleTest : AbstractQuickFixMultiModuleTest() {
         doMultiPlatformTest()
     }
 
+    @Test
+    fun testOrderHeader() {
+        doMultiPlatformTest()
+    }
+
+    @Test
+    fun testOrderImpl() {
+        doMultiPlatformTest()
+    }
+
     @Test
     fun testPackage() {
         doMultiPlatformTest()