azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Save change script configuration within a transaction

Browse Source 
#EA-214281 Fixed
This commit is contained in:
Vladimir Dolzhenko
2019-10-15 10:35:31 +02:00
parent 625dbfa699
commit 15997bf635
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
idea/idea-core/src/org/jetbrains/kotlin/idea/core/script/ScriptClassRootsManager.kt
+1 -1
View File
@@ -38,7 +38,7 @@ internal class ScriptClassRootsManager(val project: Project) {
} }
private fun checkInTransaction() { private fun checkInTransaction() {
check(concurrentTransactions.get() > 0) check(concurrentTransactions.get() > 0) { "Transaction is not started" }
} }
inline fun transaction(body: () -> Unit) { inline fun transaction(body: () -> Unit) {
idea/idea-core/src/org/jetbrains/kotlin/idea/core/script/ScriptConfigurationManagerImpl.kt
+3 -1
View File
@@ -236,7 +236,9 @@ class ScriptConfigurationManagerImpl internal constructor(private val project: P
if (oldConfiguration != null) { if (oldConfiguration != null) {
file.removeScriptDependenciesNotificationPanel(project) file.removeScriptDependenciesNotificationPanel(project)
} }
saveChangedConfiguration(file, newConfiguration, skipSaveToAttributes) rootsManager.transaction {
saveChangedConfiguration(file, newConfiguration, skipSaveToAttributes)
}
} else { } else {
debug(file) { debug(file) {
"configuration changed, notification is shown: old = $oldConfiguration, new = $newConfiguration" "configuration changed, notification is shown: old = $oldConfiguration, new = $newConfiguration"