From 88c2af555cbf3583d248f7b13c5612bcd23c8d99 Mon Sep 17 00:00:00 2001
From: alexm <alexm@verdnatura.es>
Date: Mon, 27 May 2024 09:28:04 +0200
Subject: [PATCH] fix(ArrayData): refs #6825 router.replace and use
 filter.where

---
 src/components/VnTable/VnTable.vue            |   6 +-
 src/components/ui/VnFilterPanel.vue           |  55 +-
 src/composables/useArrayData.js               |  16 +-
 .../CustomerExtendedListBackup.vue            | 571 ------------------
 4 files changed, 46 insertions(+), 602 deletions(-)
 delete mode 100644 src/pages/Customer/ExtendedList/CustomerExtendedListBackup.vue

diff --git a/src/components/VnTable/VnTable.vue b/src/components/VnTable/VnTable.vue
index 703b24c48..044444c44 100644
--- a/src/components/VnTable/VnTable.vue
+++ b/src/components/VnTable/VnTable.vue
@@ -148,6 +148,7 @@ defineExpose({
                 :data-key="$attrs['data-key']"
                 :search-button="true"
                 v-model="params"
+                :disable-submit-event="true"
             >
                 <template #body>
                     <VnTableFilter
@@ -242,10 +243,7 @@ defineExpose({
                 </template>
                 <template #body-cell="{ col, row }">
                     <!-- Columns -->
-                    <QTd
-                        class="no-padding no-margin"
-                        :class="`text-${col.align ?? 'left'}`"
-                    >
+                    <QTd class="no-margin" :class="`text-${col.align ?? 'left'}`">
                         <VnTableColumn
                             :column="col"
                             :row="row"
diff --git a/src/components/ui/VnFilterPanel.vue b/src/components/ui/VnFilterPanel.vue
index cd5e96cee..e28085331 100644
--- a/src/components/ui/VnFilterPanel.vue
+++ b/src/components/ui/VnFilterPanel.vue
@@ -42,6 +42,10 @@ const props = defineProps({
         type: Array,
         default: () => [],
     },
+    disableSubmitEvent: {
+        type: Boolean,
+        default: false,
+    },
 });
 
 const emit = defineEmits(['refresh', 'clear', 'search', 'init', 'remove']);
@@ -58,7 +62,11 @@ onMounted(() => {
     if (params.value) userParams.value = JSON.parse(JSON.stringify(params.value));
     if (Object.keys(store.userParams).length > 0) {
         userParams.value = JSON.parse(JSON.stringify(store.userParams));
-        params.value = { ...params.value, ...userParams.value };
+        params.value = {
+            ...params.value,
+            ...userParams.value,
+            ...userParams.value?.filter?.where,
+        };
     }
     emit('init', { params: userParams.value });
 });
@@ -67,7 +75,7 @@ function setUserParams(params) {
     if (!params) {
         userParams.value = {};
     } else {
-        userParams.value = params;
+        userParams.value = typeof params == 'string' ? JSON.parse(params) : params;
     }
 }
 
@@ -86,7 +94,9 @@ watch(
 );
 
 const isLoading = ref(false);
-async function search() {
+async function search(evt) {
+    if (evt && props.disableSubmitEvent) return;
+
     store.filter.where = {};
     isLoading.value = true;
     const filter = { ...userParams.value, ...params.value };
@@ -141,23 +151,33 @@ async function clearFilters() {
 }
 
 const tagsList = computed(() => {
-    return Object.entries(userParams.value)
-        .filter(([key, value]) => value && !(props.hiddenTags || []).includes(key))
-        .map(([key, value]) => ({
-            label: key,
-            value: value,
-        }));
+    const tagList = [];
+    const params = {
+        ...userParams.value,
+    };
+    const where = params?.filter?.where;
+    if (where) {
+        Object.assign(params, where);
+    }
+    delete params.filter;
+    for (const key of Object.keys(params)) {
+        const value = params[key];
+        if (!value || (props.hiddenTags || []).includes(key)) continue;
+        tagList.push({ key, value });
+    }
+    return tagList;
 });
 
 const tags = computed(() =>
-    tagsList.value.filter((tag) => !(props.customTags || []).includes(tag.label))
+    tagsList.value.filter((tag) => !(props.customTags || []).includes(tag.key))
 );
 const customTags = computed(() =>
-    tagsList.value.filter((tag) => (props.customTags || []).includes(tag.label))
+    tagsList.value.filter((tag) => (props.customTags || []).includes(tag.key))
 );
 
 async function remove(key) {
-    userParams.value[key] = null;
+    delete userParams.value[key];
+    delete userParams.value.filter.where[key];
     params.value[key] = undefined;
     await arrayData.applyFilter({ params: userParams.value });
     emit('remove', key);
@@ -224,13 +244,13 @@ function formatValue(value) {
                 <div>
                     <VnFilterPanelChip
                         v-for="chip of tags"
-                        :key="chip.label"
-                        :removable="!unremovableParams.includes(chip.label)"
-                        @remove="remove(chip.label)"
+                        :key="chip.key"
+                        :removable="!unremovableParams.includes(chip.key)"
+                        @remove="remove(chip.key)"
                     >
                         <slot name="tags" :tag="chip" :format-fn="formatValue">
                             <div class="q-gutter-x-xs">
-                                <strong>{{ chip.label }}:</strong>
+                                <strong>{{ chip.key }}:</strong>
                                 <span>"{{ chip.value }}"</span>
                             </div>
                         </slot>
@@ -260,8 +280,9 @@ function formatValue(value) {
                         dense
                         icon="search"
                         rounded
-                        type="submit"
+                        :type="disableSubmitEvent ? 'button' : 'submit'"
                         unelevated
+                        @click="search()"
                     />
                 </QItemSection>
             </QItem>
diff --git a/src/composables/useArrayData.js b/src/composables/useArrayData.js
index e2465f5ba..ba6f62810 100644
--- a/src/composables/useArrayData.js
+++ b/src/composables/useArrayData.js
@@ -1,5 +1,5 @@
 import { onMounted, ref, computed } from 'vue';
-import { useRoute } from 'vue-router';
+import { useRouter, useRoute } from 'vue-router';
 import axios from 'axios';
 import { useArrayDataStore } from 'stores/useArrayDataStore';
 import { buildFilter } from 'filters/filterPanel';
@@ -13,6 +13,7 @@ export function useArrayData(key, userOptions) {
 
     const store = arrayDataStore.get(key);
     const route = useRoute();
+    const router = useRouter();
     let canceller = null;
 
     const page = ref(1);
@@ -189,15 +190,10 @@ export function useArrayData(key, userOptions) {
         if (store.userParams && Object.keys(store.userParams).length !== 0)
             query.params = JSON.stringify(store.userParams);
 
-        const url = new URL(window.location.href);
-        const { hash: currentHash } = url;
-        const [currentRoute] = currentHash.split('?');
-
-        const params = new URLSearchParams();
-        for (const param in query) params.append(param, query[param]);
-
-        url.hash = currentRoute + '?' + params.toString();
-        window.history.pushState({}, '', url.hash);
+        router.replace({
+            path: route.path,
+            query,
+        });
     }
 
     const totalRows = computed(() => (store.data && store.data.length) || 0);
diff --git a/src/pages/Customer/ExtendedList/CustomerExtendedListBackup.vue b/src/pages/Customer/ExtendedList/CustomerExtendedListBackup.vue
deleted file mode 100644
index a24d3211a..000000000
--- a/src/pages/Customer/ExtendedList/CustomerExtendedListBackup.vue
+++ /dev/null
@@ -1,571 +0,0 @@
-<script setup>
-import { ref, computed, onBeforeMount, onMounted } from 'vue';
-import { useI18n } from 'vue-i18n';
-import { useRouter } from 'vue-router';
-
-import { QBtn, QIcon } from 'quasar';
-import WorkerDescriptorProxy from 'src/pages/Worker/Card/WorkerDescriptorProxy.vue';
-import CustomerDescriptorProxy from 'src/pages/Customer/Card/CustomerDescriptorProxy.vue';
-import CustomerExtendedListActions from './CustomerExtendedListActions.vue';
-import CustomerExtendedListFilter from './CustomerExtendedListFilter.vue';
-import TableVisibleColumns from 'src/components/common/TableVisibleColumns.vue';
-import VnSubToolbar from 'src/components/ui/VnSubToolbar.vue';
-
-import { useArrayData } from 'composables/useArrayData';
-import { useStateStore } from 'stores/useStateStore';
-import { toDate } from 'src/filters';
-
-const { t } = useI18n();
-const router = useRouter();
-const stateStore = useStateStore();
-
-const arrayData = ref(null);
-
-onBeforeMount(async () => {
-    arrayData.value = useArrayData('CustomerExtendedList', {
-        url: 'Clients/extendedListFilter',
-        limit: 0,
-    });
-    await arrayData.value.fetch({ append: false });
-    stateStore.rightDrawer = true;
-});
-
-onMounted(() => {
-    const filteredColumns = columns.value.filter(
-        (col) => col.name !== 'actions' && col.name !== 'customerStatus'
-    );
-    allColumnNames.value = filteredColumns.map((col) => col.name);
-});
-
-const rows = computed(() => arrayData.value.store.data);
-
-const selectedCustomerId = ref(0);
-const selectedSalesPersonId = ref(0);
-const allColumnNames = ref([]);
-const visibleColumns = ref([]);
-
-const tableColumnComponents = {
-    customerStatus: {
-        component: QIcon,
-        props: (prop) => ({
-            name: !prop.row.isActive
-                ? 'vn:disabled'
-                : prop.row.isActive && prop.row.isFreezed
-                ? 'vn:frozen'
-                : '',
-            color: 'primary',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    id: {
-        component: QBtn,
-        props: () => ({ flat: true, color: 'blue' }),
-        event: (prop) => {
-            selectCustomerId(prop.row.id);
-        },
-    },
-    name: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    fi: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    salesPersonFk: {
-        component: QBtn,
-        props: () => ({ flat: true, color: 'blue' }),
-        event: (prop) => selectSalesPersonId(prop.row.salesPersonFk),
-    },
-    credit: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    creditInsurance: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    phone: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    mobile: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    street: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    countryFk: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    provinceFk: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    city: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    postcode: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    email: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    created: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    businessTypeFk: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    payMethodFk: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    sageTaxTypeFk: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    sageTransactionTypeFk: {
-        component: 'span',
-        props: () => {},
-        event: () => {},
-    },
-    isActive: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.isActive ? 'check' : 'close',
-            color: prop.row.isActive ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    isVies: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.isVies ? 'check' : 'close',
-            color: prop.row.isVies ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    isTaxDataChecked: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.isTaxDataChecked ? 'check' : 'close',
-            color: prop.row.isTaxDataChecked ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    isEqualizated: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.isEqualizated ? 'check' : 'close',
-            color: prop.row.isEqualizated ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    isFreezed: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.isFreezed ? 'check' : 'close',
-            color: prop.row.isFreezed ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    hasToInvoice: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.hasToInvoice ? 'check' : 'close',
-            color: prop.row.hasToInvoice ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    hasToInvoiceByAddress: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.hasToInvoiceByAddress ? 'check' : 'close',
-            color: prop.row.hasToInvoiceByAddress ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    isToBeMailed: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.isToBeMailed ? 'check' : 'close',
-            color: prop.row.isToBeMailed ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    hasLcr: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.hasLcr ? 'check' : 'close',
-            color: prop.row.hasLcr ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    hasCoreVnl: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.hasCoreVnl ? 'check' : 'close',
-            color: prop.row.hasCoreVnl ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    hasSepaVnl: {
-        component: QIcon,
-        props: (prop) => ({
-            name: prop.row.hasSepaVnl ? 'check' : 'close',
-            color: prop.row.hasSepaVnl ? 'positive' : 'negative',
-            size: 'sm',
-        }),
-        event: () => {},
-    },
-    actions: {
-        component: CustomerExtendedListActions,
-        props: (prop) => ({
-            id: prop.row.id,
-        }),
-        event: () => {},
-    },
-};
-
-const columns = computed(() => [
-    {
-        align: 'left',
-        field: '',
-        label: '',
-        name: 'customerStatus',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'id',
-        label: t('customer.extendedList.tableVisibleColumns.id'),
-        name: 'id',
-    },
-    {
-        align: 'left',
-        field: 'name',
-        label: t('customer.extendedList.tableVisibleColumns.name'),
-        name: 'name',
-    },
-    {
-        align: 'left',
-        field: 'fi',
-        label: t('customer.extendedList.tableVisibleColumns.fi'),
-        name: 'fi',
-    },
-    {
-        align: 'left',
-        field: 'salesPerson',
-        label: t('customer.extendedList.tableVisibleColumns.salesPersonFk'),
-        name: 'salesPersonFk',
-    },
-    {
-        align: 'left',
-        field: 'credit',
-        label: t('customer.extendedList.tableVisibleColumns.credit'),
-        name: 'credit',
-    },
-    {
-        align: 'left',
-        field: 'creditInsurance',
-        label: t('customer.extendedList.tableVisibleColumns.creditInsurance'),
-        name: 'creditInsurance',
-    },
-    {
-        align: 'left',
-        field: 'phone',
-        label: t('customer.extendedList.tableVisibleColumns.phone'),
-        name: 'phone',
-    },
-    {
-        align: 'left',
-        field: 'mobile',
-        label: t('customer.extendedList.tableVisibleColumns.mobile'),
-        name: 'mobile',
-    },
-    {
-        align: 'left',
-        field: 'street',
-        label: t('customer.extendedList.tableVisibleColumns.street'),
-        name: 'street',
-    },
-    {
-        align: 'left',
-        field: 'country',
-        label: t('customer.extendedList.tableVisibleColumns.countryFk'),
-        name: 'countryFk',
-    },
-    {
-        align: 'left',
-        field: 'province',
-        label: t('customer.extendedList.tableVisibleColumns.provinceFk'),
-        name: 'provinceFk',
-    },
-    {
-        align: 'left',
-        field: 'city',
-        label: t('customer.extendedList.tableVisibleColumns.city'),
-        name: 'city',
-    },
-    {
-        align: 'left',
-        field: 'postcode',
-        label: t('customer.extendedList.tableVisibleColumns.postcode'),
-        name: 'postcode',
-    },
-    {
-        align: 'left',
-        field: 'email',
-        label: t('customer.extendedList.tableVisibleColumns.email'),
-        name: 'email',
-    },
-    {
-        align: 'left',
-        field: 'created',
-        label: t('customer.extendedList.tableVisibleColumns.created'),
-        name: 'created',
-        format: (value) => toDate(value),
-    },
-    {
-        align: 'left',
-        field: 'businessType',
-        label: t('customer.extendedList.tableVisibleColumns.businessTypeFk'),
-        name: 'businessTypeFk',
-    },
-    {
-        align: 'left',
-        field: 'payMethod',
-        label: t('customer.extendedList.tableVisibleColumns.payMethodFk'),
-        name: 'payMethodFk',
-    },
-    {
-        align: 'left',
-        field: 'sageTaxType',
-        label: t('customer.extendedList.tableVisibleColumns.sageTaxTypeFk'),
-        name: 'sageTaxTypeFk',
-    },
-    {
-        align: 'left',
-        field: 'sageTransactionType',
-        label: t('customer.extendedList.tableVisibleColumns.sageTransactionTypeFk'),
-        name: 'sageTransactionTypeFk',
-    },
-    {
-        align: 'left',
-        field: 'isActive',
-        label: t('customer.extendedList.tableVisibleColumns.isActive'),
-        name: 'isActive',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'isVies',
-        label: t('customer.extendedList.tableVisibleColumns.isVies'),
-        name: 'isVies',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'isTaxDataChecked',
-        label: t('customer.extendedList.tableVisibleColumns.isTaxDataChecked'),
-        name: 'isTaxDataChecked',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'isEqualizated',
-        label: t('customer.extendedList.tableVisibleColumns.isEqualizated'),
-        name: 'isEqualizated',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'isFreezed',
-        label: t('customer.extendedList.tableVisibleColumns.isFreezed'),
-        name: 'isFreezed',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'hasToInvoice',
-        label: t('customer.extendedList.tableVisibleColumns.hasToInvoice'),
-        name: 'hasToInvoice',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'hasToInvoiceByAddress',
-        label: t('customer.extendedList.tableVisibleColumns.hasToInvoiceByAddress'),
-        name: 'hasToInvoiceByAddress',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'isToBeMailed',
-        label: t('customer.extendedList.tableVisibleColumns.isToBeMailed'),
-        name: 'isToBeMailed',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'hasLcr',
-        label: t('customer.extendedList.tableVisibleColumns.hasLcr'),
-        name: 'hasLcr',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'hasCoreVnl',
-        label: t('customer.extendedList.tableVisibleColumns.hasCoreVnl'),
-        name: 'hasCoreVnl',
-        format: () => ' ',
-    },
-    {
-        align: 'left',
-        field: 'hasSepaVnl',
-        label: t('customer.extendedList.tableVisibleColumns.hasSepaVnl'),
-        name: 'hasSepaVnl',
-        format: () => ' ',
-    },
-    {
-        align: 'right',
-        field: 'actions',
-        label: '',
-        name: 'actions',
-    },
-]);
-
-const stopEventPropagation = (event, col) => {
-    if (!['id', 'salesPersonFk'].includes(col.name)) return;
-    event.preventDefault();
-    event.stopPropagation();
-};
-
-const navigateToTravelId = (id) => router.push({ path: `/customer/${id}` });
-
-const selectCustomerId = (id) => (selectedCustomerId.value = id);
-
-const selectSalesPersonId = (id) => (selectedSalesPersonId.value = id);
-</script>
-
-<template>
-    <QDrawer v-model="stateStore.rightDrawer" side="right" :width="256" show-if-above>
-        <QScrollArea class="fit text-grey-8">
-            <CustomerExtendedListFilter
-                v-if="visibleColumns.length !== 0"
-                data-key="CustomerExtendedList"
-                :visible-columns="visibleColumns"
-            />
-        </QScrollArea>
-    </QDrawer>
-    <VnSubToolbar>
-        <template #st-actions>
-            <TableVisibleColumns
-                :all-columns="allColumnNames"
-                table-code="clientsDetail"
-                labels-traductions-path="customer.extendedList.tableVisibleColumns"
-                @on-config-saved="
-                    visibleColumns = ['customerStatus', ...$event, 'actions']
-                "
-            />
-        </template>
-    </VnSubToolbar>
-
-    <QPage class="column items-center q-pa-md">
-        <QTable
-            :columns="columns"
-            :rows="rows"
-            class="full-width q-mt-md"
-            row-key="id"
-            :visible-columns="visibleColumns"
-            @row-click="(evt, row, id) => navigateToTravelId(row.id)"
-        >
-            <template #body-cell="{ col, value }">
-                <QTd @click="stopEventPropagation($event, col)">
-                    {{ value }}
-                </QTd>
-            </template>
-            <template #body-cell-id="props">
-                <QTd @click="stopEventPropagation($event, props.col)">
-                    <component
-                        :is="tableColumnComponents[props.col.name].component"
-                        class="col-content"
-                        v-bind="tableColumnComponents[props.col.name].props(props)"
-                        @click="tableColumnComponents[props.col.name].event(props)"
-                    >
-                        <CustomerDescriptorProxy :id="props.row.id" />
-                        {{ props.row.id }}
-                    </component>
-                </QTd>
-            </template>
-            <template #body-cell-salesPersonFk="props">
-                <QTd @click="stopEventPropagation($event, props.col)">
-                    <component
-                        v-if="props.row.salesPerson"
-                        class="col-content"
-                        :is="tableColumnComponents[props.col.name].component"
-                        v-bind="tableColumnComponents[props.col.name].props(props)"
-                        @click="tableColumnComponents[props.col.name].event(props)"
-                    >
-                        <WorkerDescriptorProxy :id="props.row.salesPersonFk" />
-                        {{ props.row.salesPerson }}
-                    </component>
-                    <span class="col-content" v-else>-</span>
-                </QTd>
-            </template>
-            <template #body-cell-actions="props">
-                <QTd @click="stopEventPropagation($event, props.col)">
-                    <component
-                        :is="tableColumnComponents[props.col.name].component"
-                        class="col-content"
-                        v-bind="tableColumnComponents[props.col.name].props(props)"
-                        @click="tableColumnComponents[props.col.name].event(props)"
-                    />
-                </QTd>
-            </template>
-        </QTable>
-    </QPage>
-</template>
-
-<style lang="scss" scoped>
-.col-content {
-    border-radius: 4px;
-    padding: 6px;
-}
-</style>