From c8ecbcf0883298d7fee1f3540acf0762bca496de Mon Sep 17 00:00:00 2001 From: Valentin Kipyatkov Date: Wed, 21 Jan 2015 15:58:24 +0300 Subject: [PATCH] Converted file to Kotlin (step 2) --- ...mportAction.java => JetAddImportAction.kt} | 22 +++++++------------ 1 file changed, 8 insertions(+), 14 deletions(-) rename idea/src/org/jetbrains/kotlin/idea/actions/{JetAddImportAction.java => JetAddImportAction.kt} (89%) diff --git a/idea/src/org/jetbrains/kotlin/idea/actions/JetAddImportAction.java b/idea/src/org/jetbrains/kotlin/idea/actions/JetAddImportAction.kt similarity index 89% rename from idea/src/org/jetbrains/kotlin/idea/actions/JetAddImportAction.java rename to idea/src/org/jetbrains/kotlin/idea/actions/JetAddImportAction.kt index a9df3b1674e..ded5fb5668b 100644 --- a/idea/src/org/jetbrains/kotlin/idea/actions/JetAddImportAction.java +++ b/idea/src/org/jetbrains/kotlin/idea/actions/JetAddImportAction.kt @@ -16,7 +16,6 @@ package org.jetbrains.kotlin.idea.actions -import com.google.common.collect.Lists import com.intellij.codeInsight.daemon.QuickFixBundle import com.intellij.codeInsight.daemon.impl.actions.AddImportAction import com.intellij.codeInsight.hint.QuestionAction @@ -29,7 +28,6 @@ import com.intellij.openapi.ui.popup.PopupStep import com.intellij.openapi.ui.popup.util.BaseListPopupStep import com.intellij.psi.PsiDocumentManager import com.intellij.psi.PsiElement -import com.intellij.psi.PsiFile import com.intellij.util.PlatformIcons import org.jetbrains.kotlin.idea.JetBundle import org.jetbrains.kotlin.idea.quickfix.ImportInsertHelper @@ -50,11 +48,7 @@ public class JetAddImportAction * @param imports Variants for resolution. */ (private val myProject: Project, private val myEditor: Editor, private val myElement: PsiElement, imports: Iterable) : QuestionAction { - private val possibleImports: List - - { - possibleImports = Lists.newArrayList(imports) - } + private val possibleImports = imports.toList() override fun execute(): Boolean { PsiDocumentManager.getInstance(myProject).commitAllDocuments() @@ -75,20 +69,20 @@ public class JetAddImportAction return true } - protected fun getImportSelectionPopup(): BaseListPopupStep { + protected fun getImportSelectionPopup(): BaseListPopupStep { return object : BaseListPopupStep(JetBundle.message("imports.chooser.title"), possibleImports) { override fun isAutoSelectionEnabled(): Boolean { return false } - override fun onChosen(selectedValue: FqName?, finalChoice: Boolean): PopupStep? { + override fun onChosen(selectedValue: FqName?, finalChoice: Boolean): PopupStep? { if (selectedValue == null) { - return PopupStep.FINAL_CHOICE + return null } if (finalChoice) { addImport(myElement, myProject, selectedValue) - return PopupStep.FINAL_CHOICE + return null } val toExclude = AddImportAction.getAllExcludableStrings(selectedValue.asString()) @@ -103,7 +97,7 @@ public class JetAddImportAction AddImportAction.excludeFromImport(myProject, selectedValue) } - return super.onChosen(selectedValue, finalChoice) + return null } } } @@ -137,8 +131,8 @@ public class JetAddImportAction ApplicationManager.getApplication().runWriteAction(object : Runnable { override fun run() { val file = element.getContainingFile() - if (!(file is JetFile)) return - ImportInsertHelper.getInstance().addImportDirectiveIfNeeded(selectedImport, file as JetFile) + if (file !is JetFile) return + ImportInsertHelper.getInstance().addImportDirectiveIfNeeded(selectedImport, file) } }) }