Development #39732
Création d'une demande : paramétrage
0%
Description
Quand on utilise l'action "création d'une demande", quand on valide le formulaire on se retrouve sur le statut, mais le paramétrage n'est pas terminé (il faut y retourner pour choisir les champs en correspondance). Ce serait pratique de rester sur la même page.
Fichiers
Demandes liées
Révisions associées
create_formdata: keep user on parameter's view when changing formdef_slug (#39732)
Historique
Mis à jour par Marie Kuntz il y a environ 4 ans
- Sujet changé de Création d'une demande à Création d'une demande : paramétrage
Mis à jour par Marie Kuntz il y a environ 4 ans
- Lié à Development #39657: création d'une demande, exposer par défaut ajouté
Mis à jour par Frédéric Péters il y a environ 4 ans
Oui il a été noté que c'était compliqué mais il y a #39543 qui a été créé.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Lié à Development #39543: Ajouter un bouton "Enregistrer et continuer les modifications" dans la vue d'admin des actions ajouté
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Fichier 0001-create_formdata-keep-user-on-parameter-s-view-when-c.patch 0001-create_formdata-keep-user-on-parameter-s-view-when-c.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En attendant un vrai bouton "Enregistrer et continuer...".
Mis à jour par Nicolas Roche il y a environ 4 ans
J'ai l'impression que la branche pointe sur #39638
... et que le patch ne s'applique pas.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Pas bien compris le souci, en tout cas branche rebasé sur master et j'ai changé ma façon de bloquer, je détecte directement si le widget mappings est actuellement visible (en testant si le hidden input added_elements est visible dans le POST).
Mis à jour par Thomas Noël il y a environ 4 ans
Pigé, c'est un peu astuce je trouve. On pourrait au moins ajouter un "if form.is_submitted():" pour faire comprendre ce bout de code ?
Genre (avec un commentaire en anglais yaourt approximatif) :
--- a/wcs/wf/create_formdata.py
+++ b/wcs/wf/create_formdata.py
@@ -232,8 +232,10 @@ class CreateFormdataWorkflowStatusItem(WorkflowStatusItem):
title=_('Mappings to new form fields'),
to_formdef=formdef,
value=self.mappings)
- if formdef_slug != self.formdef_slug and '%smappings$added_elements' % prefix not in get_request().form:
- form.get_widget('%smappings' % prefix).set_error(_('Please define new mappings'))
+ if form.is_submitted():
+ # do not validate form if formdef is changed and there is no mappings
+ if formdef_slug != self.formdef_slug and '%smappings$added_elements' % prefix not in get_request().form:
+ form.get_widget('%smappings' % prefix).set_error(_('Please define new mappings'))
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Fichier 0001-create_formdata-fix-typo-in-_resolve_formdef_slug-39.patch 0001-create_formdata-fix-typo-in-_resolve_formdef_slug-39.patch ajouté
- Fichier 0002-create_formdata-keep-user-on-parameter-s-view-when-c.patch 0002-create_formdata-keep-user-on-parameter-s-view-when-c.patch ajouté
Comm y dit Thomas.
Mis à jour par Thomas Noël il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 5c38338ea159e6463bdc11f5ef4c4b80222c6fb5
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Feb 11 19:55:58 2020 +0100
create_formdata: keep user on parameter's view when changing formdef_slug (#39732)
commit 85997ba06ca67b4a29f9918fd6c63f206c5b5d12
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Wed Feb 12 18:42:07 2020 +0100
create_formdata: fix typo in _resolve_formdef_slug() (#39732)Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
create_formdata: fix typo in _resolve_formdef_slug() (#39732)