Extract Function: Skip callable reference selectors

#KT-13218 Fixed
This commit is contained in:
Alexey Sedunov
2016-08-08 18:57:53 +03:00
parent 6e3d1d726e
commit b793252a5f
5 changed files with 36 additions and 0 deletions
@@ -220,6 +220,7 @@ data class ExtractionData(
if (shouldSkipPrimaryReceiver && !(originalResolveResult.resolvedCall?.hasBothReceivers() ?: false)) continue
}
else {
if (newRef.getParentOfTypeAndBranch<KtCallableReferenceExpression> { callableReference } != null) continue
smartCast = originalContext[BindingContext.SMARTCAST, originalResolveResult.originalRefExpr]
possibleTypes = getPossibleTypes(originalResolveResult.originalRefExpr, originalResolveResult.resolvedCall, originalContext)
shouldSkipPrimaryReceiver = false
@@ -0,0 +1,13 @@
// WITH_RUNTIME
// PARAM_DESCRIPTOR: value-parameter arg: kotlin.String defined in Foo.Companion.bar
// PARAM_TYPES: kotlin.String
class Foo {
var arguments: Map<String, String>? = null
companion object {
fun bar(arg: String) = Foo().apply {
arguments = <selection>mapOf(Foo::arguments.name to arg)</selection>
}
}
}
@@ -0,0 +1,15 @@
// WITH_RUNTIME
// PARAM_DESCRIPTOR: value-parameter arg: kotlin.String defined in Foo.Companion.bar
// PARAM_TYPES: kotlin.String
class Foo {
var arguments: Map<String, String>? = null
companion object {
fun bar(arg: String) = Foo().apply {
arguments = map(arg)
}
private fun map(arg: String) = mapOf(Foo::arguments.name to arg)
}
}
@@ -831,6 +831,12 @@ public class ExtractionTestGenerated extends AbstractExtractionTest {
doExtractFunctionTest(fileName);
}
@TestMetadata("callableReferenceSelector.kt")
public void testCallableReferenceSelector() throws Exception {
String fileName = KotlinTestUtils.navigationMetadata("idea/testData/refactoring/extractFunction/basic/callableReferenceSelector.kt");
doExtractFunctionTest(fileName);
}
@TestMetadata("calleeExpression.kt")
public void testCalleeExpression() throws Exception {
String fileName = KotlinTestUtils.navigationMetadata("idea/testData/refactoring/extractFunction/basic/calleeExpression.kt");