Bugfix: detach variables from Realm before using in VariableManager

Waboodoo · May 22, 2024 · 617a1a9 · 617a1a9
1 parent c04b80b
commit 617a1a9
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/HTTPShortcuts/app/src/main/kotlin/ch/rmy/android/http_shortcuts/variables/VariableManager.kt b/HTTPShortcuts/app/src/main/kotlin/ch/rmy/android/http_shortcuts/variables/VariableManager.kt
@@ -14,10 +14,12 @@ import java.io.UnsupportedEncodingException
 import java.net.URLEncoder
 
 class VariableManager(
-    val variables: List<Variable>,
+    variables: List<Variable>,
     preResolvedValues: Map<VariableKey, String> = emptyMap(),
 ) : VariableLookup {
 
+    val variables: List<Variable>
+
     private val variablesById: Map<VariableId, Variable>
 
     private val variablesByKey: Map<VariableKey, Variable>
@@ -26,6 +28,7 @@ class VariableManager(
 
     init {
         val detachedVariables = variables.map { if (it.isManaged()) it.copyFromRealm() else it }
+        this.variables = variables
         variablesById = detachedVariables.associateBy { it.id }
         variablesByKey = detachedVariables.associateBy { it.key }