Project

General

Profile

Development #39732

Création d'une demande : paramétrage

Added by Marie Kuntz 6 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-create_formdata-keep-user-on-parameter-s-view-when-c.patch View (2.6 KB) Benjamin Dauvergne, 11 Feb 2020 08:02 PM

0001-create_formdata-fix-typo-in-_resolve_formdef_slug-39.patch View (906 Bytes) Benjamin Dauvergne, 03 Apr 2020 04:16 PM

0002-create_formdata-keep-user-on-parameter-s-view-when-c.patch View (3.04 KB) Benjamin Dauvergne, 03 Apr 2020 04:16 PM


Related issues

Related to w.c.s. - Development #39657: création d'une demande, exposer par défaut Solution déployée 07 Feb 2020
Related to w.c.s. - Development #39543: Ajouter un bouton "Enregistrer et continuer les modifications" dans la vue d'admin des actions Nouveau 04 Feb 2020

Associated revisions

Revision 85997ba0 (diff)
Added by Benjamin Dauvergne 4 months ago

create_formdata: fix typo in _resolve_formdef_slug() (#39732)

Revision 5c38338e (diff)
Added by Benjamin Dauvergne 4 months ago

create_formdata: keep user on parameter's view when changing formdef_slug (#39732)

History

#1 Updated by Marie Kuntz 6 months ago

  • Subject changed from Création d'une demande to Création d'une demande : paramétrage

#2 Updated by Marie Kuntz 6 months ago

#3 Updated by Frédéric Péters 6 months ago

Oui il a été noté que c'était compliqué mais il y a #39543 qui a été créé.

#4 Updated by Frédéric Péters 6 months ago

  • Related to Development #39543: Ajouter un bouton "Enregistrer et continuer les modifications" dans la vue d'admin des actions added

#5 Updated by Benjamin Dauvergne 6 months ago

  • Assignee set to Benjamin Dauvergne

#6 Updated by Benjamin Dauvergne 6 months ago

En attendant un vrai bouton "Enregistrer et continuer...".

#7 Updated by Nicolas Roche 6 months ago

J'ai l'impression que la branche pointe sur #39638
... et que le patch ne s'applique pas.

#8 Updated by Benjamin Dauvergne 6 months ago

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).

#9 Updated by Thomas Noël 5 months ago

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'))

#11 Updated by Thomas Noël 4 months ago

  • Status changed from Solution proposée to Solution validée

#12 Updated by Benjamin Dauvergne 4 months ago

  • Status changed from Solution validée to 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)

#13 Updated by Frédéric Péters 4 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF