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

#6772 - Reload data section when change Id #192

Merged
jsegarra merged 34 commits from 6772_reload_sections into dev 2024-08-27 15:17:45 +00:00
Conversation 7 Commits 34 Files Changed 4 +22 -19
2 changed files with 22 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 5d2df03223 - Show all commits

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

@ -94,25 +94,7 @@ watch(
);
watch(
alexm commented 2024-04-24 05:47:02 +00:00
Outdated
Review
Copy Link

Esta part entonces deuria estar igual pero en FormModel per exemple??
Si es així igual valdria la pena ficar-ho en ArrayData y en una prop poder habilitar-ho o deshabilitar-ho?

Esta part entonces deuria estar igual pero en FormModel per exemple?? Si es així igual valdria la pena ficar-ho en ArrayData y en una prop poder habilitar-ho o deshabilitar-ho?
jsegarra commented 2024-04-24 06:06:06 +00:00
Outdated
Review
Copy Link

Mmm...el fallo no está en los formularios, ya que en FormModel no se han reportado errores.
Por otra parte, en que casos querrías deshabitar la funcionalidad de que al cambiar la URL cambie la ventana?

Mmm...el fallo no está en los formularios, ya que en FormModel no se han reportado errores. Por otra parte, en que casos querrías deshabitar la funcionalidad de que al cambiar la URL cambie la ventana?
jsegarra commented 2024-05-16 07:23:34 +00:00
Outdated
Review
Copy Link

Mover la funcionalidad a ArrayData
la funcion fecth cambiar a fetch({append:false});
Añadir condición para decidir que casos no se quiere. Por defecto si se quiere
Probar en formModel

Mover la funcionalidad a ArrayData la funcion fecth cambiar a fetch({append:false}); Añadir condición para decidir que casos no se quiere. Por defecto si se quiere Probar en formModel
() => route.params.id,
() => {
if (route.path === props.url) return;
//Así fuerzo al reinicio
store.data = null;
if (!arrayData.store?.filter?.where) {
//Cuando el cambio viene por VnSearchbar
if (route.params.id) store.url = store.url.replace(/(\d+)/, route.params.id);
} else {
// Cuando el usuario cambia en la URL
const keyFk = Object.keys(arrayData.store.filter.where).find((key) =>
key.endsWith('Fk')
);
if (keyFk) {
arrayData.store.filter.where[keyFk] = route.params.id;
}
}
fetch();
}
() => arrayData.reloadRoute()
);
async function fetch() {
await arrayData.fetch({ append: false });

21
src/composables/useArrayData.js
View File

@ -34,6 +34,26 @@ export function useArrayData(key, userOptions) {
setOptions();
}
function reloadRoute() {
if (route.path === store.url) return;
//Así fuerzo al reinicio
store.data = null;
if (!store?.filter?.where) {
//Cuando el cambio viene por VnSearchbar
alexm commented 2024-04-18 10:27:53 +00:00
Outdated
Review
Copy Link

Haciendo return store.url = store.url.replace(/(\d+)/, route.params.id); te evitas el else y queda un poco mas legible

Haciendo `return store.url = store.url.replace(/(\d+)/, route.params.id);` te evitas el else y queda un poco mas legible
jsegarra commented 2024-04-18 11:17:28 +00:00
Outdated
Review
Copy Link

Resuelto: 3359d5b6ae

Resuelto: 3359d5b6aefe593d73e2226f662cee620fad5c6a
if (route.params.id) store.url = store.url.replace(/(\d+)/, route.params.id);
} else {
// Cuando el usuario cambia en la URL
const keyFk = Object.keys(store.filter.where).find((key) =>
key.endsWith('Fk')
);
if (keyFk) {
store.filter.where[keyFk] = route.params.id;
}
}
fetch({ append: false });
}
function setOptions() {
const allowedOptions = [
'url',
@ -210,5 +230,6 @@ export function useArrayData(key, userOptions) {
totalRows,
updateStateParams,
isLoading,
reloadRoute,
};
}