fix: expressions are properly included in getSchemaVariables

Closes #1186
bpmn-io · May 29, 2024 · 1e25516 · 1e25516
1 parent aaaaf35
commit 1e25516
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 0 deletions.
diff --git a/packages/form-js-viewer/src/util/getSchemaVariables.js b/packages/form-js-viewer/src/util/getSchemaVariables.js
@@ -22,6 +22,7 @@ const EXPRESSION_PROPERTIES = [
   'url',
   'dataSource',
   'columnsExpression',
+  'expression',
 ];
 
 const TEMPLATE_PROPERTIES = [

diff --git a/packages/form-js-viewer/test/spec/expressionField.json b/packages/form-js-viewer/test/spec/expressionField.json
@@ -0,0 +1,18 @@
+{
+  "id": "Form_1lywnv3",
+  "components": [
+    {
+      "computeOn": "change",
+      "label": "Expression",
+      "type": "expression",
+      "layout": {
+        "row": "Row_0duk8pr",
+        "columns": null
+      },
+      "id": "Field_01dikzb",
+      "key": "exp_expression",
+      "expression": "=flatten(data[selected = filter].dates)"
+    }
+  ],
+  "type": "default"
+}
diff --git a/packages/form-js-viewer/test/spec/util/GetSchemaVariables.spec.js b/packages/form-js-viewer/test/spec/util/GetSchemaVariables.spec.js
@@ -18,6 +18,7 @@ import groupsSchema from '../groups.json';
 import shipsExampleSchema from '../ships-example.json';
 import iframesSchema from '../iframes.json';
 import htmlSchema from '../html.json';
+import expressionFieldSchema from '../expressionField.json';
 
 describe('util/getSchemaVariables', () => {
   it('should include form field keys', () => {
@@ -204,4 +205,10 @@ describe('util/getSchemaVariables', () => {
       'separated2',
     ]);
   });
+
+  it('should include variables in complex templates', () => {
+    const variables = getSchemaVariables(expressionFieldSchema);
+
+    expect(variables).to.include.members(['exp_expression', 'data', 'selected', 'filter']);
+  });
 });