From a206aad9043e6ae50c93c9664ce48963470adadf Mon Sep 17 00:00:00 2001
From: ThaumRystra <thaum@dicecloud.com>
Date: Sun, 3 Nov 2024 14:35:42 +0200
Subject: [PATCH] Prevented inserting duplicate creature variables documents

---
 .../engine/computation/writeComputation/writeScope.js | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/app/imports/api/engine/computation/writeComputation/writeScope.js b/app/imports/api/engine/computation/writeComputation/writeScope.js
index 1c2473e4..cf9af316 100644
--- a/app/imports/api/engine/computation/writeComputation/writeScope.js
+++ b/app/imports/api/engine/computation/writeComputation/writeScope.js
@@ -6,8 +6,17 @@ export default function writeScope(creatureId, computation) {
   if (!creatureId) throw 'creatureId is required';
   const scope = computation.scope;
   let variables = computation.variables;
+  // If the variables are not set, check if they can be fetched
   if (!variables) {
-    CreatureVariables.insert({ _creatureId: creatureId });
+    variables = CreatureVariables.findOne({
+      _creatureId: creatureId
+    });
+  }
+  // Otherwise create a new variables document
+  if (!variables) {
+    CreatureVariables.insert({
+      _creatureId: creatureId
+    });
     variables = {};
   }
   delete variables._id;