diff --git a/idea/src/org/jetbrains/kotlin/idea/codeInsight/unwrap/KotlinUnwrapRemoveBase.java b/idea/src/org/jetbrains/kotlin/idea/codeInsight/unwrap/KotlinUnwrapRemoveBase.java index 58259f2f872..365fab1eac2 100644 --- a/idea/src/org/jetbrains/kotlin/idea/codeInsight/unwrap/KotlinUnwrapRemoveBase.java +++ b/idea/src/org/jetbrains/kotlin/idea/codeInsight/unwrap/KotlinUnwrapRemoveBase.java @@ -22,7 +22,7 @@ import com.intellij.psi.PsiWhiteSpace; import com.intellij.util.IncorrectOperationException; import org.jetbrains.annotations.NotNull; import org.jetbrains.kotlin.idea.KotlinBundle; -import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil; +import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil2; import org.jetbrains.kotlin.psi.KtBlockExpression; import org.jetbrains.kotlin.psi.KtElement; import org.jetbrains.kotlin.psi.KtExpression; @@ -44,7 +44,7 @@ public abstract class KotlinUnwrapRemoveBase extends AbstractUnwrapper + KotlinRefactoringUtil2.selectElement(editor, file, false, elementKinds) { expr -> if (expr != null) { selectTargetContainer(listOf(expr)) } diff --git a/idea/src/org/jetbrains/kotlin/idea/refactoring/introduce/introduceVariable/KotlinIntroduceVariableHandler.kt b/idea/src/org/jetbrains/kotlin/idea/refactoring/introduce/introduceVariable/KotlinIntroduceVariableHandler.kt index ad84ea475f6..50f6e90408d 100644 --- a/idea/src/org/jetbrains/kotlin/idea/refactoring/introduce/introduceVariable/KotlinIntroduceVariableHandler.kt +++ b/idea/src/org/jetbrains/kotlin/idea/refactoring/introduce/introduceVariable/KotlinIntroduceVariableHandler.kt @@ -719,11 +719,11 @@ object KotlinIntroduceVariableHandler : RefactoringActionHandler { if (file !is KtFile) return try { - KotlinRefactoringUtil.selectElement(editor, file, listOf(CodeInsightUtils.ElementKind.EXPRESSION)) { + KotlinRefactoringUtil2.selectElement(editor, file, listOf(CodeInsightUtils.ElementKind.EXPRESSION)) { doRefactoring(project, editor, it as KtExpression?, null, null) } } - catch (e: KotlinRefactoringUtil.IntroduceRefactoringException) { + catch (e: KotlinRefactoringUtil2.IntroduceRefactoringException) { showErrorHint(project, editor, e.message!!) } } diff --git a/idea/src/org/jetbrains/kotlin/idea/refactoring/safeDelete/KotlinOverridingDialog.java b/idea/src/org/jetbrains/kotlin/idea/refactoring/safeDelete/KotlinOverridingDialog.java index 046fd80c1e0..96853cfc590 100644 --- a/idea/src/org/jetbrains/kotlin/idea/refactoring/safeDelete/KotlinOverridingDialog.java +++ b/idea/src/org/jetbrains/kotlin/idea/refactoring/safeDelete/KotlinOverridingDialog.java @@ -37,7 +37,7 @@ import org.jetbrains.kotlin.descriptors.ClassDescriptor; import org.jetbrains.kotlin.descriptors.DeclarationDescriptor; import org.jetbrains.kotlin.idea.KotlinBundle; import org.jetbrains.kotlin.idea.caches.resolve.ResolutionUtils; -import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil; +import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil2; import org.jetbrains.kotlin.idea.util.IdeDescriptorRenderers; import org.jetbrains.kotlin.psi.KtElement; import org.jetbrains.kotlin.psi.KtNamedFunction; @@ -110,7 +110,7 @@ class KotlinOverridingDialog extends DialogWrapper { assert element instanceof PsiMethod : "Method accepts only kotlin functions/properties and java methods, but '" + element.getText() + "' was found"; - return KotlinRefactoringUtil.formatPsiMethod((PsiMethod) element, true, false); + return KotlinRefactoringUtil2.formatPsiMethod((PsiMethod) element, true, false); } @Override diff --git a/idea/src/org/jetbrains/kotlin/idea/refactoring/safeDelete/KotlinSafeDeleteProcessor.kt b/idea/src/org/jetbrains/kotlin/idea/refactoring/safeDelete/KotlinSafeDeleteProcessor.kt index 3c4faf8e5f9..8057d2a6fc6 100644 --- a/idea/src/org/jetbrains/kotlin/idea/refactoring/safeDelete/KotlinSafeDeleteProcessor.kt +++ b/idea/src/org/jetbrains/kotlin/idea/refactoring/safeDelete/KotlinSafeDeleteProcessor.kt @@ -39,7 +39,7 @@ import org.jetbrains.kotlin.descriptors.Modality import org.jetbrains.kotlin.idea.KotlinBundle import org.jetbrains.kotlin.idea.caches.resolve.analyze import org.jetbrains.kotlin.idea.core.deleteElementAndCleanParent -import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil +import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil2 import org.jetbrains.kotlin.idea.references.KtReference import org.jetbrains.kotlin.idea.search.usagesSearch.processDelegationCallConstructorUsages import org.jetbrains.kotlin.lexer.KtTokens @@ -260,10 +260,10 @@ class KotlinSafeDeleteProcessor : JavaSafeDeleteProcessor() { .mapTo(ArrayList()) { overridenDescriptor -> KotlinBundle.message( "x.implements.y", - KotlinRefactoringUtil.formatFunction(declarationDescriptor, true), - KotlinRefactoringUtil.formatClass(declarationDescriptor.containingDeclaration, true), - KotlinRefactoringUtil.formatFunction(overridenDescriptor, true), - KotlinRefactoringUtil.formatClass(overridenDescriptor.containingDeclaration, true) + KotlinRefactoringUtil2.formatFunction(declarationDescriptor, true), + KotlinRefactoringUtil2.formatClass(declarationDescriptor.containingDeclaration, true), + KotlinRefactoringUtil2.formatFunction(overridenDescriptor, true), + KotlinRefactoringUtil2.formatClass(overridenDescriptor.containingDeclaration, true) ) } } @@ -332,18 +332,18 @@ class KotlinSafeDeleteProcessor : JavaSafeDeleteProcessor() { when (element) { is KtParameter -> return element.toPsiParameters().flatMap { psiParameter -> - KotlinRefactoringUtil.checkParametersInMethodHierarchy(psiParameter) ?: emptyList() + KotlinRefactoringUtil2.checkParametersInMethodHierarchy(psiParameter) ?: emptyList() }.ifEmpty { listOf(element) } is PsiParameter -> - return KotlinRefactoringUtil.checkParametersInMethodHierarchy(element) + return KotlinRefactoringUtil2.checkParametersInMethodHierarchy(element) } if (ApplicationManager.getApplication()!!.isUnitTestMode) return Collections.singletonList(element) return when (element) { is KtNamedFunction, is KtProperty -> - KotlinRefactoringUtil.checkSuperMethods( + KotlinRefactoringUtil2.checkSuperMethods( element as KtDeclaration, allElementsToDelete, "super.methods.delete.with.usage.search" ) diff --git a/idea/tests/org/jetbrains/kotlin/idea/AbstractExpressionSelectionTest.java b/idea/tests/org/jetbrains/kotlin/idea/AbstractExpressionSelectionTest.java index dfc08d63e0e..c5fab00cbc0 100644 --- a/idea/tests/org/jetbrains/kotlin/idea/AbstractExpressionSelectionTest.java +++ b/idea/tests/org/jetbrains/kotlin/idea/AbstractExpressionSelectionTest.java @@ -21,7 +21,7 @@ import com.intellij.testFramework.LightCodeInsightTestCase; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; import org.jetbrains.kotlin.idea.codeInsight.CodeInsightUtils; -import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil; +import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil2; import org.jetbrains.kotlin.psi.KtFile; import org.jetbrains.kotlin.test.KotlinTestUtils; @@ -34,11 +34,11 @@ public abstract class AbstractExpressionSelectionTest extends LightCodeInsightTe final String expectedExpression = KotlinTestUtils.getLastCommentInFile((KtFile) getFile()); try { - KotlinRefactoringUtil.selectElement( + KotlinRefactoringUtil2.selectElement( getEditor(), (KtFile) getFile(), Collections.singletonList(CodeInsightUtils.ElementKind.EXPRESSION), - new KotlinRefactoringUtil.SelectElementCallback() { + new KotlinRefactoringUtil2.SelectElementCallback() { @Override public void run(@Nullable PsiElement element) { assertNotNull("Selected expression mustn't be null", element); @@ -47,7 +47,7 @@ public abstract class AbstractExpressionSelectionTest extends LightCodeInsightTe } ); } - catch (KotlinRefactoringUtil.IntroduceRefactoringException e) { + catch (KotlinRefactoringUtil2.IntroduceRefactoringException e) { assertEquals(expectedExpression, ""); } } diff --git a/idea/tests/org/jetbrains/kotlin/idea/AbstractSmartSelectionTest.java b/idea/tests/org/jetbrains/kotlin/idea/AbstractSmartSelectionTest.java index 39ebd4ad679..39f52eac916 100644 --- a/idea/tests/org/jetbrains/kotlin/idea/AbstractSmartSelectionTest.java +++ b/idea/tests/org/jetbrains/kotlin/idea/AbstractSmartSelectionTest.java @@ -20,7 +20,7 @@ import com.intellij.openapi.util.text.StringUtil; import com.intellij.testFramework.LightCodeInsightTestCase; import org.jetbrains.annotations.NotNull; import org.jetbrains.kotlin.idea.codeInsight.CodeInsightUtils; -import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil; +import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil2; import org.jetbrains.kotlin.psi.KtElement; import org.jetbrains.kotlin.psi.KtFile; import org.jetbrains.kotlin.test.KotlinTestUtils; @@ -34,12 +34,12 @@ public abstract class AbstractSmartSelectionTest extends LightCodeInsightTestCas configureByFile(path); String expectedResultText = KotlinTestUtils.getLastCommentInFile((KtFile) getFile()); - List elements = KotlinRefactoringUtil.getSmartSelectSuggestions( + List elements = KotlinRefactoringUtil2.getSmartSelectSuggestions( getFile(), getEditor().getCaretModel().getOffset(), CodeInsightUtils.ElementKind.EXPRESSION); List textualExpressions = new ArrayList(); for (KtElement element : elements) { - textualExpressions.add(KotlinRefactoringUtil.getExpressionShortText(element)); + textualExpressions.add(KotlinRefactoringUtil2.getExpressionShortText(element)); } assertEquals(expectedResultText, StringUtil.join(textualExpressions, "\n")); } diff --git a/idea/tests/org/jetbrains/kotlin/idea/refactoring/introduce/AbstractExtractionTest.kt b/idea/tests/org/jetbrains/kotlin/idea/refactoring/introduce/AbstractExtractionTest.kt index 2f9c7ce7dac..6d504e9fc25 100644 --- a/idea/tests/org/jetbrains/kotlin/idea/refactoring/introduce/AbstractExtractionTest.kt +++ b/idea/tests/org/jetbrains/kotlin/idea/refactoring/introduce/AbstractExtractionTest.kt @@ -37,7 +37,7 @@ import com.intellij.refactoring.util.occurrences.ExpressionOccurrenceManager import com.intellij.testFramework.fixtures.JavaCodeInsightTestFixture import com.intellij.testFramework.fixtures.LightCodeInsightFixtureTestCase import org.jetbrains.kotlin.idea.codeInsight.CodeInsightUtils -import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil +import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil2 import org.jetbrains.kotlin.idea.refactoring.introduce.extractFunction.EXTRACT_FUNCTION import org.jetbrains.kotlin.idea.refactoring.introduce.extractFunction.ExtractKotlinFunctionHandler import org.jetbrains.kotlin.idea.refactoring.introduce.extractionEngine.* @@ -115,7 +115,7 @@ abstract class AbstractExtractionTest() : KotlinLightCodeInsightFixtureTestCase( with (handler) { val target = (file as KtFile).findElementByCommentPrefix("// TARGET:") as? KtNamedDeclaration if (target != null) { - KotlinRefactoringUtil.selectElement(fixture.getEditor(), file, true, listOf(CodeInsightUtils.ElementKind.EXPRESSION)) { element -> + KotlinRefactoringUtil2.selectElement(fixture.getEditor(), file, true, listOf(CodeInsightUtils.ElementKind.EXPRESSION)) { element -> invoke(fixture.getProject(), fixture.getEditor(), element as KtExpression, target) } } diff --git a/idea/tests/org/jetbrains/kotlin/idea/refactoring/nameSuggester/KotlinNameSuggesterTest.kt b/idea/tests/org/jetbrains/kotlin/idea/refactoring/nameSuggester/KotlinNameSuggesterTest.kt index 2067f27f8e2..0bea7a50b22 100644 --- a/idea/tests/org/jetbrains/kotlin/idea/refactoring/nameSuggester/KotlinNameSuggesterTest.kt +++ b/idea/tests/org/jetbrains/kotlin/idea/refactoring/nameSuggester/KotlinNameSuggesterTest.kt @@ -22,7 +22,7 @@ import com.intellij.testFramework.fixtures.LightCodeInsightFixtureTestCase import org.jetbrains.kotlin.idea.caches.resolve.analyze import org.jetbrains.kotlin.idea.codeInsight.CodeInsightUtils import org.jetbrains.kotlin.idea.core.KotlinNameSuggester -import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil +import org.jetbrains.kotlin.idea.refactoring.KotlinRefactoringUtil2 import org.jetbrains.kotlin.idea.test.ConfigLibraryUtil import org.jetbrains.kotlin.idea.test.PluginTestCaseBase import org.jetbrains.kotlin.psi.KtExpression @@ -90,7 +90,7 @@ class KotlinNameSuggesterTest : LightCodeInsightFixtureTestCase() { if (withRuntime) { ConfigLibraryUtil.configureKotlinRuntimeAndSdk(myModule, PluginTestCaseBase.mockJdk()) } - KotlinRefactoringUtil.selectElement(myFixture.editor, file, listOf(CodeInsightUtils.ElementKind.EXPRESSION), object : KotlinRefactoringUtil.SelectElementCallback { + KotlinRefactoringUtil2.selectElement(myFixture.editor, file, listOf(CodeInsightUtils.ElementKind.EXPRESSION), object : KotlinRefactoringUtil2.SelectElementCallback { override fun run(element: PsiElement?) { val names = KotlinNameSuggester .suggestNamesByExpressionAndType(element as KtExpression, @@ -104,7 +104,7 @@ class KotlinNameSuggesterTest : LightCodeInsightFixtureTestCase() { } }) } - catch (e: KotlinRefactoringUtil.IntroduceRefactoringException) { + catch (e: KotlinRefactoringUtil2.IntroduceRefactoringException) { throw AssertionError("Failed to find expression: " + e.message) } finally {