From b77fed6766b649241e14db874da600c2d67a6c45 Mon Sep 17 00:00:00 2001
From: Andrew Kingston <aptkingston@ssl-mail.com>
Date: Thu, 12 Aug 2021 09:50:09 +0100
Subject: [PATCH] Add duplicate action to filter drawer

---
 .../FilterEditor/FilterDrawer.svelte          | 182 ++++++++++++++++++
 .../FilterEditor/FilterEditor.svelte          |  34 +---
 .../FilterEditor/LuceneFilterBuilder.svelte   | 151 ---------------
 3 files changed, 190 insertions(+), 177 deletions(-)
 create mode 100644 packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/FilterDrawer.svelte
 delete mode 100644 packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/LuceneFilterBuilder.svelte
diff --git a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/FilterDrawer.svelte b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/FilterDrawer.svelte
new file mode 100644
index 0000000000..c206492595
--- /dev/null
+++ b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/FilterDrawer.svelte
@@ -0,0 +1,182 @@
+<script>
+  import {
+    DatePicker,
+    Icon,
+    Button,
+    Select,
+    Combobox,
+    Input,
+    DrawerContent,
+    Layout,
+    Body,
+  } from "@budibase/bbui"
+  import DrawerBindableInput from "components/common/bindings/DrawerBindableInput.svelte"
+  import { generate } from "shortid"
+  import { OperatorOptions, getValidOperatorsForType } from "helpers/lucene"
+
+  export let schemaFields
+  export let filters = []
+  export let bindings = []
+
+  const BannedTypes = ["link", "attachment"]
+
+  $: fieldOptions = (schemaFields ?? [])
+    .filter(field => !BannedTypes.includes(field.type))
+    .map(field => field.name)
+
+  const addFilter = () => {
+    filters = [
+      ...filters,
+      {
+        id: generate(),
+        field: null,
+        operator: OperatorOptions.Equals.value,
+        value: null,
+        valueType: "Value",
+      },
+    ]
+  }
+
+  const removeFilter = id => {
+    filters = filters.filter(field => field.id !== id)
+  }
+
+  const duplicateFilter = id => {
+    const existingFilter = filters.find(filter => filter.id === id)
+    const duplicate = { ...existingFilter, id: generate() }
+    filters = [...filters, duplicate]
+  }
+
+  const onFieldChange = (expression, field) => {
+    // Update the field type
+    expression.type = schemaFields.find(x => x.name === field)?.type
+
+    // Ensure a valid operator is set
+    const validOperators = getValidOperatorsForType(expression.type).map(
+      x => x.value
+    )
+    if (!validOperators.includes(expression.operator)) {
+      expression.operator = validOperators[0] ?? OperatorOptions.Equals.value
+      onOperatorChange(expression, expression.operator)
+    }
+  }
+
+  const onOperatorChange = (expression, operator) => {
+    const noValueOptions = [
+      OperatorOptions.Empty.value,
+      OperatorOptions.NotEmpty.value,
+    ]
+    expression.noValue = noValueOptions.includes(operator)
+    if (expression.noValue) {
+      expression.value = null
+    }
+  }
+
+  const getFieldOptions = field => {
+    const schema = schemaFields.find(x => x.name === field)
+    return schema?.constraints?.inclusion || []
+  }
+</script>
+
+<DrawerContent>
+  <div class="container">
+    <Layout noPadding>
+      <Body size="S">
+        {#if !filters?.length}
+          Add your first filter column.
+        {:else}
+          Results are filtered to only those which match all of the following
+          constraints.
+        {/if}
+      </Body>
+      {#if filters?.length}
+        <div class="fields">
+          {#each filters as filter, idx}
+            <Select
+              bind:value={filter.field}
+              options={fieldOptions}
+              on:change={e => onFieldChange(filter, e.detail)}
+              placeholder="Column"
+            />
+            <Select
+              disabled={!filter.field}
+              options={getValidOperatorsForType(filter.type)}
+              bind:value={filter.operator}
+              on:change={e => onOperatorChange(filter, e.detail)}
+              placeholder={null}
+            />
+            <Select
+              disabled={filter.noValue || !filter.field}
+              options={["Value", "Binding"]}
+              bind:value={filter.valueType}
+              placeholder={null}
+            />
+            {#if filter.valueType === "Binding"}
+              <DrawerBindableInput
+                disabled={filter.noValue}
+                title={`Value for "${filter.field}"`}
+                value={filter.value}
+                placeholder="Value"
+                {bindings}
+                on:change={event => (filter.value = event.detail)}
+              />
+            {:else if ["string", "longform", "number"].includes(filter.type)}
+              <Input disabled={filter.noValue} bind:value={filter.value} />
+            {:else if filter.type === "options"}
+              <Combobox
+                disabled={filter.noValue}
+                options={getFieldOptions(filter.field)}
+                bind:value={filter.value}
+              />
+            {:else if filter.type === "boolean"}
+              <Combobox
+                disabled={filter.noValue}
+                options={[
+                  { label: "True", value: "true" },
+                  { label: "False", value: "false" },
+                ]}
+                bind:value={filter.value}
+              />
+            {:else if filter.type === "datetime"}
+              <DatePicker disabled={filter.noValue} bind:value={filter.value} />
+            {:else}
+              <DrawerBindableInput disabled />
+            {/if}
+            <Icon
+              name="Duplicate"
+              hoverable
+              size="S"
+              on:click={() => duplicateFilter(filter.id)}
+            />
+            <Icon
+              name="Close"
+              hoverable
+              size="S"
+              on:click={() => removeFilter(filter.id)}
+            />
+          {/each}
+        </div>
+      {/if}
+      <div>
+        <Button icon="AddCircle" size="M" secondary on:click={addFilter}>
+          Add filter
+        </Button>
+      </div>
+    </Layout>
+  </div>
+</DrawerContent>
+
+<style>
+  .container {
+    width: 100%;
+    max-width: 1000px;
+    margin: 0 auto;
+  }
+  .fields {
+    display: grid;
+    column-gap: var(--spacing-l);
+    row-gap: var(--spacing-s);
+    align-items: center;
+    grid-template-columns: 1fr 120px 120px 1fr auto auto;
+  }
+</style>
diff --git a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/FilterEditor.svelte b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/FilterEditor.svelte
index adcf7ec29b..58b767353b 100644
--- a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/FilterEditor.svelte
+++ b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/FilterEditor.svelte
@@ -1,19 +1,11 @@
 <script>
-  import {
-    notifications,
-    ActionButton,
-    Button,
-    Drawer,
-    Body,
-    DrawerContent,
-    Layout,
-  } from "@budibase/bbui"
+  import { notifications, ActionButton, Button, Drawer } from "@budibase/bbui"
   import { createEventDispatcher } from "svelte"
   import {
     getDatasourceForProvider,
     getSchemaForDatasource,
   } from "builderStore/dataBinding"
-  import LuceneFilterBuilder from "./LuceneFilterBuilder.svelte"
+  import FilterDrawer from "./FilterDrawer.svelte"
   import { currentAsset } from "builderStore"
 
   const dispatch = createEventDispatcher()
@@ -25,9 +17,6 @@
   let drawer
   let tempValue = value || []
 
-  $: numFilters = Array.isArray(tempValue)
-    ? tempValue.length
-    : Object.keys(tempValue || {}).length
   $: dataSource = getDatasourceForProvider($currentAsset, componentInstance)
   $: schema = getSchemaForDatasource($currentAsset, dataSource)?.schema
   $: schemaFields = Object.values(schema || {})
@@ -43,17 +32,10 @@
 <ActionButton on:click={drawer.show}>Define filters</ActionButton>
 <Drawer bind:this={drawer} title="Filtering">
   <Button cta slot="buttons" on:click={saveFilter}>Save</Button>
-  <DrawerContent slot="body">
-    <Layout noPadding>
-      <Body size="S">
-        {#if !numFilters}
-          Add your first filter column.
-        {:else}
-          Results are filtered to only those which match all of the following
-          constraints.
-        {/if}
-      </Body>
-      <LuceneFilterBuilder bind:value={tempValue} {schemaFields} {bindings} />
-    </Layout>
-  </DrawerContent>
+  <FilterDrawer
+    slot="body"
+    bind:filters={tempValue}
+    {bindings}
+    {schemaFields}
+  />
 </Drawer>
diff --git a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/LuceneFilterBuilder.svelte b/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/LuceneFilterBuilder.svelte
deleted file mode 100644
index be46169f57..0000000000
--- a/packages/builder/src/components/design/PropertiesPanel/PropertyControls/FilterEditor/LuceneFilterBuilder.svelte
+++ /dev/null
@@ -1,151 +0,0 @@
-<script>
-  import {
-    DatePicker,
-    ActionButton,
-    Button,
-    Select,
-    Combobox,
-    Input,
-  } from "@budibase/bbui"
-  import DrawerBindableInput from "components/common/bindings/DrawerBindableInput.svelte"
-  import { generate } from "shortid"
-  import { OperatorOptions, getValidOperatorsForType } from "helpers/lucene"
-
-  export let schemaFields
-  export let value
-  export let bindings = []
-
-  const BannedTypes = ["link", "attachment"]
-  $: fieldOptions = (schemaFields ?? [])
-    .filter(field => !BannedTypes.includes(field.type))
-    .map(field => field.name)
-
-  const addField = () => {
-    value = [
-      ...value,
-      {
-        id: generate(),
-        field: null,
-        operator: OperatorOptions.Equals.value,
-        value: null,
-        valueType: "Value",
-      },
-    ]
-  }
-
-  const removeField = id => {
-    value = value.filter(field => field.id !== id)
-  }
-
-  const onFieldChange = (expression, field) => {
-    // Update the field type
-    expression.type = schemaFields.find(x => x.name === field)?.type
-
-    // Ensure a valid operator is set
-    const validOperators = getValidOperatorsForType(expression.type).map(
-      x => x.value
-    )
-    if (!validOperators.includes(expression.operator)) {
-      expression.operator = validOperators[0] ?? OperatorOptions.Equals.value
-      onOperatorChange(expression, expression.operator)
-    }
-  }
-
-  const onOperatorChange = (expression, operator) => {
-    const noValueOptions = [
-      OperatorOptions.Empty.value,
-      OperatorOptions.NotEmpty.value,
-    ]
-    expression.noValue = noValueOptions.includes(operator)
-    if (expression.noValue) {
-      expression.value = null
-    }
-  }
-
-  const getFieldOptions = field => {
-    const schema = schemaFields.find(x => x.name === field)
-    return schema?.constraints?.inclusion || []
-  }
-</script>
-
-{#if value?.length}
-  <div class="fields">
-    {#each value as expression, idx}
-      <Select
-        bind:value={expression.field}
-        options={fieldOptions}
-        on:change={e => onFieldChange(expression, e.detail)}
-        placeholder="Column"
-      />
-      <Select
-        disabled={!expression.field}
-        options={getValidOperatorsForType(expression.type)}
-        bind:value={expression.operator}
-        on:change={e => onOperatorChange(expression, e.detail)}
-        placeholder={null}
-      />
-      <Select
-        disabled={expression.noValue || !expression.field}
-        options={["Value", "Binding"]}
-        bind:value={expression.valueType}
-        placeholder={null}
-      />
-      {#if expression.valueType === "Binding"}
-        <DrawerBindableInput
-          disabled={expression.noValue}
-          title={`Value for "${expression.field}"`}
-          value={expression.value}
-          placeholder="Value"
-          {bindings}
-          on:change={event => (expression.value = event.detail)}
-        />
-      {:else if ["string", "longform", "number"].includes(expression.type)}
-        <Input disabled={expression.noValue} bind:value={expression.value} />
-      {:else if expression.type === "options"}
-        <Combobox
-          disabled={expression.noValue}
-          options={getFieldOptions(expression.field)}
-          bind:value={expression.value}
-        />
-      {:else if expression.type === "boolean"}
-        <Combobox
-          disabled={expression.noValue}
-          options={[
-            { label: "True", value: "true" },
-            { label: "False", value: "false" },
-          ]}
-          bind:value={expression.value}
-        />
-      {:else if expression.type === "datetime"}
-        <DatePicker
-          disabled={expression.noValue}
-          bind:value={expression.value}
-        />
-      {:else}
-        <DrawerBindableInput disabled />
-      {/if}
-
-      <ActionButton
-        size="S"
-        quiet
-        icon="Close"
-        on:click={() => removeField(expression.id)}
-      />
-    {/each}
-  </div>
-{/if}
-<div>
-  <Button icon="AddCircle" size="M" secondary on:click={addField}>
-    Add expression
-  </Button>
-</div>
-
-<style>
-  .fields {
-    display: grid;
-    column-gap: var(--spacing-l);
-    row-gap: var(--spacing-s);
-    align-items: center;
-    grid-template-columns: 1fr 120px 120px 1fr auto;
-  }
-</style>
