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

#8372 prevent double submit #1333

Merged
alexm merged 29 commits from 8372-fixDoubleRequest into dev 2025-02-13 12:40:17 +00:00
Conversation 1 Commits 29 Files Changed 6 +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 e1605e0610 - Show all commits

2
src/boot/qformMixin.js
View File

@ -44,7 +44,7 @@ export default {
return;
}
evt.preventDefault();
that.onSubmit(false);
that.onSubmit();
}
});
},

2
src/components/FormModel.vue
View File

@ -207,7 +207,7 @@ async function fetch() {
}
}
async function save(prevent = true) {
async function save(prevent = false) {
jorgep marked this conversation as resolved Outdated
alexm commented 2025-02-07 06:02:48 +00:00
Outdated
Review
Copy Link

duda, pq no poner save(prevent = true) y ya no hace falta pasar el true, solo los false?

duda, pq no poner `save(prevent = true)` y ya no hace falta pasar el true, solo los false?
jorgep commented 2025-02-07 08:57:44 +00:00
Outdated
Review
Copy Link

Lo probé, no funciona. Si se hace enter en un input normal, primero se dispara el evt @submit y luego entre en qFormMixin > evt keyup. El evt @submit de quasar siempre pasa como primer argumento la propiedad. Probé a ver si podía ponerle un .prevent para que no se disparé desde formModel pero así no funcionaba.

Lo probé, no funciona. Si se hace enter en un input normal, primero se dispara el evt @submit y luego entre en qFormMixin > evt keyup. El evt @submit de quasar siempre pasa como primer argumento la propiedad. Probé a ver si podía ponerle un **.prevent** para que no se disparé desde formModel pero así no funcionaba.
jorgep commented 2025-02-07 15:49:17 +00:00
Outdated
Review
Copy Link

Ahora creo va finito

Ahora creo va finito
if (prevent) return;
if ($props.observeFormChanges && !hasChanges.value)
return notify('globals.noChanges', 'negative');