misc: add proper message in case of double submission (#66919) #975
No reviewers
Labels
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: entrouvert/wcs#975
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/66919-submitted-draft-exception-on-previous-page"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
412c79756d
toc0ad790738
WIP: misc: add proper message in case of double submission (#66919)to misc: add proper message in case of double submission (#66919)Deux petits trucs ramassés au passage, à prendre ou pas. Ack.
@ -1145,0 +1143,4 @@
# first step,
if not (get_session().user or get_session().anonymous_formdata_keys):
# that means we probably lost the session in mid-air.
get_session().message = ('error', _('Sorry, your session have been lost.'))
Petite coquille ici (your session has […]).
Merci j'ai ajouté un commit pour corriger cette erreur qui est là depuis 2015…
@ -1381,7 +1386,13 @@ class FormPage(Directory, TempfileDirectoryMixin, FormTemplateMixin):
if form.has_errors() or form.get_submit() is True:
if self.has_draft_support() and not honeypot_error:
# always save draft during server roundtrip, even if the form has errors
Du détail, mais question de lisibilité je laisserais ici un commentaire du genre de ce qui est plus bas pour l’autosave,
# save draft during server roundtrip
, je trouve lealways … even if …
moins justifié maintenant qu’on prend des précautions supplémentaires à la tentative de sauvegarde.Ok j'ai modifié le commentaire, sans arriver à reformuler l'idée que ça n'était pas grave que des données en erreur soient enregistrées.
c0ad790738
to0bec3c80f2
0bec3c80f2
to7bcef49a45