Minor: fixes after review
This commit is contained in:
@@ -347,7 +347,11 @@ private class SingleImportVariant(
|
|||||||
val descriptors: Collection<DeclarationDescriptor>
|
val descriptors: Collection<DeclarationDescriptor>
|
||||||
) : AutoImportVariant {
|
) : AutoImportVariant {
|
||||||
override val descriptorsToImport: Collection<DeclarationDescriptor>
|
override val descriptorsToImport: Collection<DeclarationDescriptor>
|
||||||
get() = listOf(descriptors.singleOrNull() ?: descriptors.minBy { if (it is ClassDescriptor) 0 else 1 }!!)
|
get() = listOf(
|
||||||
|
descriptors.singleOrNull()
|
||||||
|
?: descriptors.minBy { if (it is ClassDescriptor) 0 else 1 }
|
||||||
|
?: error("we create the class with not-empty descriptors always")
|
||||||
|
)
|
||||||
|
|
||||||
override val hint: String get() = excludeFqNameCheck.asString()
|
override val hint: String get() = excludeFqNameCheck.asString()
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -19,6 +19,7 @@ package org.jetbrains.kotlin.idea.imports
|
|||||||
import com.intellij.lang.ImportOptimizer
|
import com.intellij.lang.ImportOptimizer
|
||||||
import com.intellij.openapi.application.ApplicationManager
|
import com.intellij.openapi.application.ApplicationManager
|
||||||
import com.intellij.openapi.progress.ProgressIndicatorProvider
|
import com.intellij.openapi.progress.ProgressIndicatorProvider
|
||||||
|
import com.intellij.openapi.util.text.StringUtil
|
||||||
import com.intellij.psi.PsiElement
|
import com.intellij.psi.PsiElement
|
||||||
import com.intellij.psi.PsiFile
|
import com.intellij.psi.PsiFile
|
||||||
import org.jetbrains.kotlin.descriptors.*
|
import org.jetbrains.kotlin.descriptors.*
|
||||||
@@ -52,18 +53,8 @@ class KotlinImportOptimizer : ImportOptimizer {
|
|||||||
|
|
||||||
return object : ImportOptimizer.CollectingInfoRunnable {
|
return object : ImportOptimizer.CollectingInfoRunnable {
|
||||||
override fun getUserNotificationInfo(): String = if (remove == 0) "Rearranged imports"
|
override fun getUserNotificationInfo(): String = if (remove == 0) "Rearranged imports"
|
||||||
else buildString {
|
else "Removed $remove ${StringUtil.pluralize("import", remove)}" +
|
||||||
append("Removed ")
|
if (add > 0) ", added $add ${StringUtil.pluralize("import", add)}" else ""
|
||||||
append(remove)
|
|
||||||
append(" import")
|
|
||||||
if (remove > 1) append("s")
|
|
||||||
if (add > 0) {
|
|
||||||
append(", added ")
|
|
||||||
append(add)
|
|
||||||
append(" import")
|
|
||||||
if (add > 1) append("s")
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
override fun run() = replaceImports(ktFile, imports)
|
override fun run() = replaceImports(ktFile, imports)
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user