feat: enable JEXLs for options in choice and multiple choice questions

projectcaluma · Sep 6, 2024 · 892b885 · 892b885
1 parent 194e42d
commit 892b885
Show file tree

Hide file tree

Showing 15 changed files with 190 additions and 87 deletions.
diff --git a/packages/form-builder/addon/components/cfb-form-editor/question-option.hbs b/packages/form-builder/addon/components/cfb-form-editor/question-option.hbs
@@ -0,0 +1,89 @@
+<li class="cfb-option-row" data-test-row={{concat "option-" (add @i 1)}}>
+  <ValidatedForm @model={{@row}} as |f|>
+    <div uk-grid class="uk-grid-small uk-flex uk-flex-top" id={{@row.slug}}>
+      <div class="uk-width-auto uk-flex uk-flex-middle">
+        {{#if @canReorder}}
+          <span
+            data-test-sort-handle
+            uk-icon="menu"
+            class="uk-sortable-handle uk-margin-small-right"
+            role="button"
+          ></span>
+        {{/if}}
+        {{#if (is-empty @row.id)}}
+          <button
+            data-test-delete-row
+            type="button"
+            class="uk-icon-button"
+            uk-icon="trash"
+            title={{t "caluma.form-builder.options.delete"}}
+            {{on "click" (fn @deleteRow @row)}}
+          >
+          </button>
+        {{else}}
+          <button
+            data-test-archive-row
+            type="button"
+            class="uk-icon-button"
+            uk-icon={{if @row.isArchived "refresh" "close"}}
+            title={{t
+              (concat
+                "caluma.form-builder.options."
+                (if @row.isArchived "restore" "archive")
+              )
+            }}
+            {{on
+              "click"
+              (fn (changeset-set @row "isArchived") (not @row.isArchived))
+            }}
+          >
+          </button>
+        {{/if}}
+      </div>
+      <div class="uk-width-expand">
+        <f.input
+          @name="label"
+          @inputName={{concat "option-" (add @i 1) "-label"}}
+          @required={{true}}
+          @disabled={{@row.isArchived}}
+          @submitted={{@submitted}}
+          @on-update={{@updateLabel}}
+        />
+      </div>
+      <div class="uk-width-1-4">
+        <f.input
+          @name="slug"
+          @inputName={{concat "option-" (add @i 1) "-slug"}}
+          @required={{true}}
+          @disabled={{not (is-empty @row.id)}}
+          @submitted={{@submitted}}
+          @renderComponent={{component
+            "cfb-slug-input"
+            hidePrefix=true
+            prefix=@model.slug
+            onUnlink=(fn (mut @row.slugUnlinked) true)
+          }}
+        />
+      </div>
+      <div class="uk-width-auto">
+        <button
+          data-test-toggle-jexl
+          type="button"
+          class="uk-icon-button"
+          uk-icon={{if this.showJexl "triangle-up" "triangle-down"}}
+          title={{t "caluma.form-builder.options.show-jexl"}}
+          {{on "click" (fn (mut this.showJexl) (not this.showJexl))}}
+        />
+      </div>
+      {{#if this.showJexl}}
+        <div class="uk-width-4-4">
+          <f.input
+            @label={{t "caluma.form-builder.question.isHidden"}}
+            @name="isHidden"
+            @renderComponent={{component "cfb-code-editor" language="jexl"}}
+          />
+        </div>
+      {{/if}}
+    </div>
+  </ValidatedForm>
+</li>
diff --git a/packages/form-builder/addon/components/cfb-form-editor/question-option.js b/packages/form-builder/addon/components/cfb-form-editor/question-option.js
@@ -0,0 +1,6 @@
+import Component from "@glimmer/component";
+import { tracked } from "@glimmer/tracking";
+
+export default class CfbFormEditorQuestionOption extends Component {
+  @tracked showJexl = false;
+}
diff --git a/packages/form-builder/addon/components/cfb-form-editor/question.js b/packages/form-builder/addon/components/cfb-form-editor/question.js
@@ -363,11 +363,11 @@ export default class CfbFormEditorQuestion extends Component {
       (changeset.get("options") || [])
         .filter((option) => option.get("isDirty"))
         .map(async (option) => {
-          const { label, slug, isArchived } = option;
+          const { label, slug, isArchived, isHidden } = option;
 
           await this.apollo.mutate({
             mutation: saveOptionMutation,
-            variables: { input: { label, slug, isArchived } },
+            variables: { input: { label, slug, isArchived, isHidden } },
           });
         }),
     );

diff --git a/packages/form-builder/addon/components/cfb-form-editor/question/options.hbs b/packages/form-builder/addon/components/cfb-form-editor/question/options.hbs
@@ -8,80 +8,14 @@
     class="uk-list uk-list-divider uk-form-controls uk-margin-small-top"
   >
     {{#each @value as |row i|}}
-      <li class="cfb-option-row" data-test-row={{concat "option-" (add i 1)}}>
-        <ValidatedForm @model={{row}} as |f|>
-          <div
-            uk-grid
-            class="uk-grid-small uk-flex uk-flex-top"
-            id={{row.slug}}
-          >
-            <div class="uk-width-auto uk-flex uk-flex-middle">
-              {{#if this.canReorder}}
-                <span
-                  data-test-sort-handle
-                  uk-icon="menu"
-                  class="uk-sortable-handle uk-margin-small-right"
-                  role="button"
-                ></span>
-              {{/if}}
-              {{#if (is-empty row.id)}}
-                <button
-                  data-test-delete-row
-                  type="button"
-                  class="uk-icon-button"
-                  uk-icon="trash"
-                  title={{t "caluma.form-builder.options.delete"}}
-                  {{on "click" (fn this.deleteRow row)}}
-                >
-                </button>
-              {{else}}
-                <button
-                  data-test-archive-row
-                  type="button"
-                  class="uk-icon-button"
-                  uk-icon={{if row.isArchived "refresh" "close"}}
-                  title={{t
-                    (concat
-                      "caluma.form-builder.options."
-                      (if row.isArchived "restore" "archive")
-                    )
-                  }}
-                  {{on
-                    "click"
-                    (fn (changeset-set row "isArchived") (not row.isArchived))
-                  }}
-                >
-                </button>
-              {{/if}}
-            </div>
-            <div class="uk-width-expand">
-              <f.input
-                @name="label"
-                @inputName={{concat "option-" (add i 1) "-label"}}
-                @required={{true}}
-                @disabled={{row.isArchived}}
-                @submitted={{@submitted}}
-                @on-update={{this.updateLabel}}
-              />
-            </div>
-            <div class="uk-width-1-4">
-              <f.input
-                @name="slug"
-                @inputName={{concat "option-" (add i 1) "-slug"}}
-                @required={{true}}
-                @disabled={{not (is-empty row.id)}}
-                @submitted={{@submitted}}
-                @renderComponent={{component
-                  "cfb-slug-input"
-                  hidePrefix=true
-                  prefix=@model.slug
-                  onUnlink=(fn (mut row.slugUnlinked) true)
-                }}
-              />
-            </div>
-          </div>
-        </ValidatedForm>
-      </li>
+      <CfbFormEditor::QuestionOption
+        @i={{i}}
+        @row={{row}}
+        @model={{@model}}
+        @canReorder={{this.canReorder}}
+        @deleteRow={{fn this.deleteRow row}}
+        @updateLabel={{this.updateLabel}}
+      />
     {{/each}}
     <li class="uk-text-center">
       <button

diff --git a/packages/form-builder/addon/gql/mutations/save-option.graphql b/packages/form-builder/addon/gql/mutations/save-option.graphql
@@ -4,6 +4,7 @@ mutation SaveOption($input: SaveOptionInput!) {
       id
       label
       slug
+      isHidden
     }
   }
 }
diff --git a/packages/form-builder/addon/gql/queries/form-editor-question.graphql b/packages/form-builder/addon/gql/queries/form-editor-question.graphql
@@ -76,6 +76,7 @@ query FormEditorQuestion($slug: String!) {
                 label
                 slug
                 isArchived
+                isHidden
               }
             }
           }
@@ -93,6 +94,7 @@ query FormEditorQuestion($slug: String!) {
                 label
                 slug
                 isArchived
+                isHidden
               }
             }
           }

diff --git a/packages/form-builder/translations/de.yaml b/packages/form-builder/translations/de.yaml
@@ -144,6 +144,7 @@ caluma:
       delete: "Option löschen"
       archive: "Option archivieren (verstecken)"
       restore: "Option wiederherstellen"
+      show-jexl: "JEXL anzeigen"
 
     notification:
       form:

diff --git a/packages/form-builder/translations/en.yaml b/packages/form-builder/translations/en.yaml
@@ -144,6 +144,7 @@ caluma:
       delete: "Delete option"
       archive: "Archive (hide) option"
       restore: "Restore option"
+      show-jexl: "Show JEXL"
 
     notification:
       form:

diff --git a/packages/form-builder/translations/fr.yaml b/packages/form-builder/translations/fr.yaml
@@ -144,6 +144,7 @@ caluma:
       delete: "Supprimer l'option"
       archive: "Archiver (masquer) l'option"
       restore: "Restaurer l'option"
+      show-jexl: "Afficher JEXL"
 
     notification:
       form:

diff --git a/packages/form/addon/gql/fragments/field.graphql b/packages/form/addon/gql/fragments/field.graphql
@@ -72,6 +72,7 @@ fragment SimpleQuestion on Question {
           slug
           label
           isArchived
+          isHidden
         }
       }
     }
@@ -89,6 +90,7 @@ fragment SimpleQuestion on Question {
           slug
           label
           isArchived
+          isHidden
         }
       }
     }

diff --git a/packages/form/addon/lib/field.js b/packages/form/addon/lib/field.js
@@ -339,7 +339,10 @@ export default class Field extends Base {
    * - DynamicMultipleChoiceQuestion
    *
    * This will also return the disabled state of the option. An option can only
-   * be disabled, if it is an old value used in a dynamic question.
+   * be disabled in any of the following cases:
+   * - it is a dynamic option which is no longer available
+   * - the option has been removed from the form using the form builder
+   * - the option is no longer visible due to its JEXL
    *
    * @property {null|Object[]} options
    */
@@ -352,9 +355,39 @@ export default class Field extends Base {
     const selected =
       (this.question.isMultipleChoice ? this.value : [this.value]) || [];
 
-    const options = this.question.options.filter(
-      (option) => !option.disabled || selected.includes(option.slug),
-    );
+    const options = this.question.options
+      .filter((option) => !option.disabled || selected.includes(option.slug))
+      .map((option) => {
+        if (!option.isHidden) {
+          return option;
+        }
+
+        try {
+          const isHidden = this.document.jexl.evalSync(
+            option.isHidden,
+            this.jexlContext,
+          );
+
+          if (selected.includes(option.slug) && isHidden) {
+            return {
+              ...option,
+              _isHidden: isHidden,
+              disabled: true,
+            };
+          }
+          return {
+            ...option,
+            _isHidden: isHidden,
+          };
+        } catch (error) {
+          throw new Error(
+            `Error while evaluating \`isHidden\` expression on Option\`${option.slug}\`: ${error.message}`,
+          );
+        }
+      })
+      .filter(
+        ({ _isHidden, disabled }) => !_isHidden || (_isHidden && disabled),
+      );
 
     const hasUnknownValue = !selected.every((slug) =>
       options.find((option) => option.slug === slug),

diff --git a/packages/form/addon/lib/question.js b/packages/form/addon/lib/question.js
@@ -141,11 +141,14 @@ export default class Question extends Base {
     const key = camelize(this.raw.__typename.replace(/Question$/, "Options"));
     const raw = this.isDynamic ? this[key] : this.raw[key];
 
-    return (raw?.edges || []).map(({ node: { label, slug, isArchived } }) => ({
-      label,
-      slug,
-      disabled: isArchived || false,
-    }));
+    return (raw?.edges || []).map(
+      ({ node: { label, slug, isArchived, isHidden } }) => ({
+        label,
+        slug,
+        disabled: isArchived || false,
+        isHidden,
+      }),
+    );
   }
 
   /**