Merge branch 'release_24.0' into release_24.1

galaxyproject · Aug 30, 2024 · b10e8f8 · b10e8f8
2 parents 12ce6af + 0c9cc61
commit b10e8f8
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/client/src/components/Workflow/Editor/Forms/FormTool.vue b/client/src/components/Workflow/Editor/Forms/FormTool.vue
@@ -141,6 +141,9 @@ export default {
             return !!this.configForm?.id;
         },
         inputs() {
+            // TODO: Refactor
+            // This code contains a computed side-effect and prop mutation.
+            // Both should be refactored
             const inputs = this.configForm.inputs;
             Utils.deepEach(inputs, (input) => {
                 if (input.type) {
@@ -150,10 +153,14 @@ export default {
                         input.info = `Data input '${input.name}' (${extensions})`;
                         input.value = { __class__: "RuntimeValue" };
                     } else {
-                        input.connectable = ["rules"].indexOf(input.type) == -1;
-                        input.collapsible_value = {
-                            __class__: "RuntimeValue",
-                        };
+                        const isRules = input.type === "rules";
+                        input.connectable = !isRules;
+                        input.collapsible_value = isRules
+                            ? undefined
+                            : {
+                                  __class__: "RuntimeValue",
+                              };
+
                         input.is_workflow =
                             (input.options && input.options.length === 0) ||
                             ["integer", "float"].indexOf(input.type) != -1;