Skip to content

Commit 7ccf2f3

Browse files
imbanankoSpace Team
authored and
Space Team
committed
Remove deprecated methods from ScriptDefinitionProvider
^KTIJ-31294
1 parent b1867b6 commit 7ccf2f3

File tree

4 files changed

+1
-21
lines changed

4 files changed

+1
-21
lines changed

plugins/scripting/scripting-compiler-impl/src/org/jetbrains/kotlin/scripting/definitions/KotlinScriptDefinitionProvider.kt

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,6 @@ import com.intellij.openapi.project.Project
99
import kotlin.script.experimental.api.SourceCode
1010

1111
interface ScriptDefinitionProvider {
12-
@Deprecated("Migrating to configuration refinement", level = DeprecationLevel.ERROR)
13-
fun findScriptDefinition(fileName: String): KotlinScriptDefinition?
14-
15-
@Deprecated("Migrating to configuration refinement", level = DeprecationLevel.ERROR)
16-
fun getDefaultScriptDefinition(): KotlinScriptDefinition
17-
1812
fun isScript(script: SourceCode): Boolean
1913

2014
fun findDefinition(script: SourceCode): ScriptDefinition?

plugins/scripting/scripting-compiler-impl/src/org/jetbrains/kotlin/scripting/definitions/LazyScriptDefinitionProvider.kt

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -79,22 +79,11 @@ abstract class LazyScriptDefinitionProvider : ScriptDefinitionProvider {
7979
cachedDefinitions.firstOrNull { it.isScript(script) }
8080
}
8181

82-
@Suppress("OverridingDeprecatedMember", "DEPRECATION", "OVERRIDE_DEPRECATION")
83-
override fun findScriptDefinition(fileName: String): KotlinScriptDefinition? =
84-
if (nonScriptId(fileName)) {
85-
null
86-
} else {
87-
cachedDefinitions.map { it.legacyDefinition }.firstOrNull { it.isScript(fileName) }
88-
}
89-
9082
override fun isScript(script: SourceCode): Boolean = findDefinition(script) != null
9183

9284
override fun getKnownFilenameExtensions(): Sequence<String> =
9385
cachedDefinitions.map { it.fileExtension }
9486

95-
@Suppress("OverridingDeprecatedMember", "DEPRECATION", "OVERRIDE_DEPRECATION")
96-
override fun getDefaultScriptDefinition(): KotlinScriptDefinition = getDefaultDefinition().legacyDefinition
97-
9887
companion object {
9988
// TODO: find a common place for storing kotlin-related extensions and reuse values from it everywhere
10089
protected val nonScriptFilenameSuffixes = arrayOf(".${KotlinFileType.EXTENSION}", ".${JavaFileType.DEFAULT_EXTENSION}")

plugins/scripting/scripting-compiler-impl/src/org/jetbrains/kotlin/scripting/definitions/ScriptDefinitionsSource.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,5 @@
66
package org.jetbrains.kotlin.scripting.definitions
77

88
interface ScriptDefinitionsSource {
9-
109
val definitions: Sequence<ScriptDefinition>
1110
}

plugins/scripting/scripting-compiler-impl/src/org/jetbrains/kotlin/scripting/definitions/definitions.kt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,7 @@ fun PsiFile.isScript(): Boolean {
4141
// Sometimes - i.e. when event system is disabled for a view provider - requesting
4242
// virtual file directly from the viewProvider is the only way of obtaining it
4343
val virtualFile = virtualFile ?: originalFile.virtualFile ?: viewProvider.virtualFile
44-
if (virtualFile.isNonScript()) return false
45-
46-
return true
44+
return !virtualFile.isNonScript()
4745
}
4846

4947
fun PsiFile.findScriptDefinition(): ScriptDefinition? {

0 commit comments

Comments
 (0)