verdnatura/salix-front
verdnatura
/
salix-front
1
2
Fork 1
Code Issues Pull Requests 57 Releases Wiki Activity

#8113 create arrayDataStore map #979

Open
jorgep wants to merge 16 commits from 8113-preventDuplicateRecords into dev
pull from: 8113-preventDuplicateRecords
merge into: verdnatura:dev
Conversation 0 Commits 16 Files Changed 5 +2 -2
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit f32c08bd1d - Show all commits

2
src/components/ui/VnPaginate.vue
View File

@ -76,7 +76,7 @@ const props = defineProps({
}, },
mapKey: { mapKey: {
type: String, type: String,
default: 'id', default: '',
}, },
}); });
const emit = defineEmits(['onFetch', 'onPaginate', 'onChange']); const emit = defineEmits(['onFetch', 'onPaginate', 'onChange']);

2
src/composables/useArrayData.js
View File

@ -290,7 +290,7 @@ export function useArrayData(key = useRoute().meta.moduleName, userOptions) {
} }
if (!map && !append) store.data = data; if (!map && !append) store.data = data;
else if (!map && append) store.data.push(...data); else if (!map && append) for (const row of data) store.data.push(row);
jorgep commented 2024-11-25 16:29:17 +00:00
Review
Copy Link

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 else
data.forEach((row) => { data.forEach((row) => {
const key = row[store.mapKey]; const key = row[store.mapKey];