fix: prevent applicaiton deep update propagation

Exabyte-io · May 12, 2023 · 2c16856 · 2c16856
1 parent 1b74b4a
commit 2c16856
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/subworkflows/subworkflow.js b/src/subworkflows/subworkflow.js
@@ -108,10 +108,9 @@ export class Subworkflow extends BaseSubworkflow {
         // reset units if application name changes
         if (this.application.name !== application.name) this.setUnits([]);
         this._application = application;
-        // propagate application changes to all units
         this.units.forEach((unit) => {
             if (typeof unit.setApplication === "function") {
-                unit.setApplication(application);
+                unit.setApplication(application, false);
             }
         });
         this.setProp("application", application.toJSON());

diff --git a/src/units/execution.js b/src/units/execution.js
@@ -58,10 +58,12 @@ export class ExecutionUnit extends mix(BaseUnit).with(HashedInputArrayMixin) {
         return this.input.map((i) => new this.constructor.Template(i));
     }
 
-    setApplication(application) {
+    setApplication(application, propagateUpdates = true) {
         this._application = application;
         this.setProp("application", application.toJSON());
-        this.setExecutable(this.application.defaultExecutable);
+        if (propagateUpdates) {
+            this.setExecutable(this.application.defaultExecutable);
+        }
     }
 
     setExecutable(executable) {