#8113 create arrayDataStore map #979
|
@ -74,8 +74,11 @@ const props = defineProps({
|
||||||
type: Boolean,
|
type: Boolean,
|
||||||
default: false,
|
default: false,
|
||||||
},
|
},
|
||||||
|
mapKey: {
|
||||||
|
type: String,
|
||||||
|
default: '',
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
||||||
const emit = defineEmits(['onFetch', 'onPaginate', 'onChange']);
|
const emit = defineEmits(['onFetch', 'onPaginate', 'onChange']);
|
||||||
const isLoading = ref(false);
|
const isLoading = ref(false);
|
||||||
const mounted = ref(false);
|
const mounted = ref(false);
|
||||||
|
@ -84,7 +87,6 @@ const pagination = ref({
|
||||||
rowsPerPage: props.limit,
|
rowsPerPage: props.limit,
|
||||||
page: 1,
|
page: 1,
|
||||||
});
|
});
|
||||||
|
|
||||||
const arrayData = useArrayData(props.dataKey, {
|
const arrayData = useArrayData(props.dataKey, {
|
||||||
url: props.url,
|
url: props.url,
|
||||||
filter: props.filter,
|
filter: props.filter,
|
||||||
|
@ -96,6 +98,7 @@ const arrayData = useArrayData(props.dataKey, {
|
||||||
exprBuilder: props.exprBuilder,
|
exprBuilder: props.exprBuilder,
|
||||||
keepOpts: props.keepOpts,
|
keepOpts: props.keepOpts,
|
||||||
searchUrl: props.searchUrl,
|
searchUrl: props.searchUrl,
|
||||||
|
mapKey: props.mapKey,
|
||||||
});
|
});
|
||||||
const store = arrayData.store;
|
const store = arrayData.store;
|
||||||
|
|
||||||
|
|
|
@ -49,6 +49,7 @@ export function useArrayData(key = useRoute().meta.moduleName, userOptions) {
|
||||||
'exprBuilder',
|
'exprBuilder',
|
||||||
'searchUrl',
|
'searchUrl',
|
||||||
'navigate',
|
'navigate',
|
||||||
|
'mapKey',
|
||||||
];
|
];
|
||||||
if (typeof userOptions === 'object') {
|
if (typeof userOptions === 'object') {
|
||||||
for (const option in userOptions) {
|
for (const option in userOptions) {
|
||||||
|
@ -119,17 +120,11 @@ export function useArrayData(key = useRoute().meta.moduleName, userOptions) {
|
||||||
const { limit } = filter;
|
const { limit } = filter;
|
||||||
store.hasMoreData = limit && response.data.length >= limit;
|
store.hasMoreData = limit && response.data.length >= limit;
|
||||||
|
|
||||||
if (append) {
|
processData(response.data, { map: !!store.mapKey, append });
|
||||||
if (!store.data) store.data = [];
|
if (!append && !isDialogOpened()) updateRouter && updateStateParams();
|
||||||
for (const row of response.data) store.data.push(row);
|
|
||||||
} else {
|
|
||||||
store.data = response.data;
|
|
||||||
if (!isDialogOpened()) updateRouter && updateStateParams();
|
|
||||||
}
|
|
||||||
|
|
||||||
store.isLoading = false;
|
store.isLoading = false;
|
||||||
|
|
||||||
canceller = null;
|
canceller = null;
|
||||||
|
|
||||||
return response;
|
return response;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -288,6 +283,31 @@ export function useArrayData(key = useRoute().meta.moduleName, userOptions) {
|
||||||
router.replace(newUrl);
|
router.replace(newUrl);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
function processData(data, { map = true, append = true }) {
|
||||||
|
|||||||
|
if (!append) {
|
||||||
|
store.data = [];
|
||||||
|
store.map = new Map();
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!Array.isArray(data)) store.data = data;
|
||||||
|
else if (!map && append) for (const row of data) store.data.push(row);
|
||||||
jorgep
commented
Se podría usar ... pero he leído que en términos de rendimiento a la hora de manejar arrays muy grande es más rápido un for. Se podría usar **...** pero he leído que en términos de rendimiento a la hora de manejar arrays muy grande es más rápido un for.
|
|||||||
|
else
|
||||||
|
for (const row of data) {
|
||||||
|
const key = row[store.mapKey];
|
||||||
|
const val = { ...row, key };
|
||||||
|
if (store.map.has(key)) {
|
||||||
|
const { position } = store.map.get(key);
|
||||||
|
val.position = position;
|
||||||
|
store.map.set(key, val);
|
||||||
|
store.data[position] = val;
|
||||||
|
} else {
|
||||||
|
val.position = store.map.size;
|
||||||
|
store.map.set(key, val);
|
||||||
|
store.data.push(val);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
const totalRows = computed(() => (store.data && store.data.length) || 0);
|
const totalRows = computed(() => (store.data && store.data.length) || 0);
|
||||||
const isLoading = computed(() => store.isLoading || false);
|
const isLoading = computed(() => store.isLoading || false);
|
||||||
|
|
||||||
|
|
|
@ -441,6 +441,7 @@ watch(
|
||||||
<QPage class="column items-center q-pa-md">
|
<QPage class="column items-center q-pa-md">
|
||||||
<VnTable
|
<VnTable
|
||||||
data-key="advanceTickets"
|
data-key="advanceTickets"
|
||||||
|
:map-key="false"
|
||||||
jorgep
commented
Aquí se quiere tener duplicados. Aquí se quiere tener duplicados.
|
|||||||
ref="vnTableRef"
|
ref="vnTableRef"
|
||||||
url="Tickets/getTicketsAdvance"
|
url="Tickets/getTicketsAdvance"
|
||||||
search-url="advanceTickets"
|
search-url="advanceTickets"
|
||||||
|
|
|
@ -17,6 +17,7 @@ export const useArrayDataStore = defineStore('arrayDataStore', () => {
|
||||||
searchUrl: 'params',
|
searchUrl: 'params',
|
||||||
navigate: null,
|
navigate: null,
|
||||||
page: 1,
|
page: 1,
|
||||||
|
mapKey: 'id',
|
||||||
};
|
};
|
||||||
|
|
||||||
function get(key) {
|
function get(key) {
|
||||||
|
@ -46,6 +47,7 @@ export const useArrayDataStore = defineStore('arrayDataStore', () => {
|
||||||
function getDefaultState() {
|
function getDefaultState() {
|
||||||
return Object.assign(JSON.parse(JSON.stringify(defaultOpts)), {
|
return Object.assign(JSON.parse(JSON.stringify(defaultOpts)), {
|
||||||
data: ref(),
|
data: ref(),
|
||||||
|
map: ref(new Map()),
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -4,7 +4,11 @@ import VnPaginate from 'src/components/ui/VnPaginate.vue';
|
||||||
|
|
||||||
describe('VnPaginate', () => {
|
describe('VnPaginate', () => {
|
||||||
const expectedUrl = '/api/customers';
|
const expectedUrl = '/api/customers';
|
||||||
|
const defaultData = [
|
||||||
|
{ id: 1, name: 'Tony Stark' },
|
||||||
|
{ id: 2, name: 'Jessica Jones' },
|
||||||
|
{ id: 3, name: 'Bruce Wayne' },
|
||||||
|
];
|
||||||
let vm;
|
let vm;
|
||||||
beforeAll(() => {
|
beforeAll(() => {
|
||||||
const options = {
|
const options = {
|
||||||
|
@ -28,11 +32,7 @@ describe('VnPaginate', () => {
|
||||||
describe('paginate()', () => {
|
describe('paginate()', () => {
|
||||||
it('should call to the paginate() method and set the data on the rows property', async () => {
|
it('should call to the paginate() method and set the data on the rows property', async () => {
|
||||||
vi.spyOn(vm.arrayData, 'loadMore');
|
vi.spyOn(vm.arrayData, 'loadMore');
|
||||||
vm.store.data = [
|
vm.store.data = defaultData;
|
||||||
{ id: 1, name: 'Tony Stark' },
|
|
||||||
{ id: 2, name: 'Jessica Jones' },
|
|
||||||
{ id: 3, name: 'Bruce Wayne' },
|
|
||||||
];
|
|
||||||
|
|
||||||
await vm.paginate();
|
await vm.paginate();
|
||||||
|
|
||||||
|
@ -42,26 +42,25 @@ describe('VnPaginate', () => {
|
||||||
|
|
||||||
it('should call to the paginate() method and then call it again to paginate', async () => {
|
it('should call to the paginate() method and then call it again to paginate', async () => {
|
||||||
vi.spyOn(axios, 'get').mockResolvedValue({
|
vi.spyOn(axios, 'get').mockResolvedValue({
|
||||||
data: [
|
data: defaultData,
|
||||||
{ id: 1, name: 'Tony Stark' },
|
|
||||||
{ id: 2, name: 'Jessica Jones' },
|
|
||||||
{ id: 3, name: 'Bruce Wayne' },
|
|
||||||
],
|
|
||||||
});
|
});
|
||||||
vm.store.hasMoreData = true;
|
vm.store.hasMoreData = true;
|
||||||
await vm.$nextTick();
|
await vm.$nextTick();
|
||||||
|
|
||||||
vm.store.data = [
|
vm.store.data = defaultData;
|
||||||
{ id: 1, name: 'Tony Stark' },
|
|
||||||
{ id: 2, name: 'Jessica Jones' },
|
|
||||||
{ id: 3, name: 'Bruce Wayne' },
|
|
||||||
];
|
|
||||||
|
|
||||||
await vm.paginate();
|
await vm.paginate();
|
||||||
|
|
||||||
expect(vm.store.skip).toEqual(3);
|
expect(vm.store.skip).toEqual(3);
|
||||||
expect(vm.store.data.length).toEqual(6);
|
expect(vm.store.data.length).toEqual(6);
|
||||||
|
|
||||||
|
vi.spyOn(axios, 'get').mockResolvedValue({
|
||||||
jorgep
commented
Si son los mismos registros ahora no los va a volver a añadir. Si son los mismos registros ahora no los va a volver a añadir.
|
|||||||
|
data: [
|
||||||
|
{ id: 4, name: 'Peter Parker' },
|
||||||
|
{ id: 5, name: 'Clark Kent' },
|
||||||
|
{ id: 6, name: 'Barry Allen' },
|
||||||
|
],
|
||||||
|
});
|
||||||
await vm.paginate();
|
await vm.paginate();
|
||||||
|
|
||||||
expect(vm.store.skip).toEqual(6);
|
expect(vm.store.skip).toEqual(6);
|
||||||
|
@ -85,11 +84,7 @@ describe('VnPaginate', () => {
|
||||||
|
|
||||||
const index = 1;
|
const index = 1;
|
||||||
const done = vi.fn();
|
const done = vi.fn();
|
||||||
vm.store.data = [
|
vm.store.data = defaultData;
|
||||||
{ id: 1, name: 'Tony Stark' },
|
|
||||||
{ id: 2, name: 'Jessica Jones' },
|
|
||||||
{ id: 3, name: 'Bruce Wayne' },
|
|
||||||
];
|
|
||||||
|
|
||||||
await vm.onLoad(index, done);
|
await vm.onLoad(index, done);
|
||||||
|
|
||||||
|
@ -105,11 +100,7 @@ describe('VnPaginate', () => {
|
||||||
],
|
],
|
||||||
});
|
});
|
||||||
|
|
||||||
vm.store.data = [
|
vm.store.data = defaultData;
|
||||||
{ id: 1, name: 'Tony Stark' },
|
|
||||||
{ id: 2, name: 'Jessica Jones' },
|
|
||||||
{ id: 3, name: 'Bruce Wayne' },
|
|
||||||
];
|
|
||||||
|
|
||||||
expect(vm.pagination.page).toEqual(1);
|
expect(vm.pagination.page).toEqual(1);
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Se puede meter directamente el snippet en la fn fetch, pero , por legibilidad creo que es mejor dejarlo en esta fn.