Handling safe calls of multi-statement function

idea/idea-replacement-engine/src/org/jetbrains/kotlin/idea/replacement/introduceValue.kt

@@ -101,17 +101,29 @@ internal fun MutableReplacementCode.introduceValue(
         }
     }
     else {
-        //TODO: handle mainExpression == null and statementsBefore!
+        val useIt = !isNameUsed("it")
+        val name = if (useIt) Name.identifier("it") else suggestName { !isNameUsed(it) }
+        replaceUsages(name)
 
-        mainExpression = if (!isNameUsed("it")) {
+        mainExpression = psiFactory.buildExpression {
-            replaceUsages(Name.identifier("it"))
+            appendExpression(value)
-            psiFactory.createExpressionByPattern("$0?.let { $1 }", value, mainExpression!!)
+            appendFixedText("?.let{")
-        }
+
-        else {
+            if (!useIt) {
-            val name = suggestName { !isNameUsed(it) }
+                appendName(name)
-            replaceUsages(name)
+                appendFixedText("->")
-            psiFactory.createExpressionByPattern("$0?.let { $1 -> $2 }", value, name, mainExpression!!)
+            }
+
+            for (statement in statementsBefore) {
+                appendExpression(statement)
+                appendFixedText("\n")
+            }
+
+            appendExpression(mainExpression!!) //TODO: mainExpression == null
+
+            appendFixedText("}")
         }
+        statementsBefore.clear() //TODO: postInsertActions!
     }
 }
 

idea/testData/refactoring/inline/function/returnAtEnd/SafeCall.kt

@@ -0,0 +1,11 @@
+fun String.<caret>f(p: Int): Int {
+    println(p)
+    return hashCode() * p
+}
+
+fun f(s: String?) {
+    s?.f(1)
+    s?.substring(1)?.f(2)
+    val s1 = s?.f(3)
+    val s2 = s?.substring(1)?.f(4)
+}

idea/testData/refactoring/inline/function/returnAtEnd/SafeCall.kt.after

@@ -0,0 +1,19 @@
+fun f(s: String?) {
+    if (s != null) {
+        println(1)
+        s.hashCode() * 1
+    }
+    val substring = s?.substring(1)
+    if (substring != null) {
+        println(2)
+        substring.hashCode() * 2
+    }
+    val s1 = s?.let {
+        println(3)
+        it.hashCode() * 3
+    }
+    val s2 = s?.substring(1)?.let {
+        println(4)
+        it.hashCode() * 4
+    }
+}

idea/tests/org/jetbrains/kotlin/idea/refactoring/inline/InlineTestGenerated.java

@@ -138,6 +138,12 @@ public class InlineTestGenerated extends AbstractInlineTest {
                 doTest(fileName);
             }
 
+            @TestMetadata("SafeCall.kt")
+            public void testSafeCall() throws Exception {
+                String fileName = KotlinTestUtils.navigationMetadata("idea/testData/refactoring/inline/function/returnAtEnd/SafeCall.kt");
+                doTest(fileName);
+            }
+
             @TestMetadata("SingleStatement.kt")
             public void testSingleStatement() throws Exception {
                 String fileName = KotlinTestUtils.navigationMetadata("idea/testData/refactoring/inline/function/returnAtEnd/SingleStatement.kt");