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

feat: refs #8077 sumDefaulter #1239

Merged
robert merged 9 commits from 8077-sumDefaulterFrontFix into dev 2025-01-24 10:55:38 +00:00
Conversation 0 Commits 9 Files Changed 3 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 13290aa44b - Show all commits

2
src/components/CrudModel.vue
View File

@ -114,7 +114,7 @@ onBeforeRouteLeave((to, from, next) => {
});
async function fetch(data) {
const rows = data[$attrs['key-data']] ?? data;
const rows = $attrs['key-data'] ? data[$attrs['key-data']] : data;
robert marked this conversation as resolved Outdated
robert commented 2025-01-22 07:35:21 +00:00
Outdated
Review
Copy Link

No se ha hecho de esta forma por que puede que data este vacío
const rows = data[$attrs['key-data']] ?? data;

No se ha hecho de esta forma por que puede que data este vacío const rows = data[$attrs['key-data']] ?? data;
jsegarra commented 2025-01-22 22:07:35 +00:00
Outdated
Review
Copy Link

Lo que no me acaba de convencer de esto es usar una clave dos veces.
Quizás si fuese en un componente, con 2 ocurrencias...en el limite, pero bien; sin embargo, se trata de CrudModel que es transversal para todos y por ello, te propongo estas líneas
const keyData = $attrs['key-data']; const rows = keyData ? data[keyData] : data;

Lo que no me acaba de convencer de esto es usar una clave dos veces. Quizás si fuese en un componente, con 2 ocurrencias...en el limite, pero bien; sin embargo, se trata de CrudModel que es transversal para todos y por ello, te propongo estas líneas `const keyData = $attrs['key-data']; const rows = keyData ? data[keyData] : data;`
resetData(rows);
emit('onFetch', rows);
return rows;