Bug #13846
permettre à un statut de contenir deux actions "ajouter un formulaire"
0%
Description
Pour le moment on ne peut pas parce que les widgets se trouvent nommés pareils (et donc "ValueError: form already has 'f1' widget"), il faudrait voir pour préfixer ces noms par le nom de variable du formulaire.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Dupliqué par Development #45375: Trace sur l'affichage de plusieurs formulaires de worflow ajouté
Mis à jour par Frédéric Péters il y a presque 4 ans
- Dupliqué par Bug #45379: ValueError: form already has 'f1' widget ajouté
Mis à jour par Frédéric Péters il y a presque 4 ans
- Dupliqué par Bug #45440: ValueError: form already has 'submit' widget ajouté
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0001-add-test-reproducing-the-issue.patch 0001-add-test-reproducing-the-issue.patch ajouté
- Assigné à
Lauréline Guérinsupprimé
J'ajoute juste un test qui reproduit le problème (mes connaissances actuelles de w.c.s sont insuffisantes pour pouvoir proposer un fix qui tient la route)
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0002-workflow-prefix-fields-in-FormWorkflowStatusItem-138.patch 0002-workflow-prefix-fields-in-FormWorkflowStatusItem-138.patch ajouté
- Fichier 0001-misc-split-form_pages-tests.patch 0001-misc-split-form_pages-tests.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
je ne suis pas sûre d'être partie dans la bonne direction
Mis à jour par Frédéric Péters il y a plus de 3 ans
L'idée est correcte mais je crains que ça ne foire sur la partie évaluation live des conditions qui ne va pas retrouver ses petits.
Je me dis qu'il faut faire le tour,
for field in self.formdef.fields:
try:
field.id = '%s_%s' % (self.varname, int(field.id))
également dans evaluate_live_form.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0002-workflow-prefix-fields-in-FormWorkflowStatusItem-138.patch 0002-workflow-prefix-fields-in-FormWorkflowStatusItem-138.patch ajouté
- Fichier 0001-misc-split-form_pages-tests.patch 0001-misc-split-form_pages-tests.patch ajouté
J'ai complété le test pour passer par evaluate_live_form
Le test passe sans modifier le code, mais par sécurité j'ai ajouté la moulinette quand même
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Dupliqué par Bug #50195: ValueError: form already has 'f1' widget ajouté
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 081ff5911431cf1af7b2edec862fc043fcadd2b1
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri Nov 13 09:44:24 2020 +0100
workflow: prefix fields in FormWorkflowStatusItem (#13846)
commit 40f6ae25518eba22e7009e8f472cc8d63dc5f7dc
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Thu Jul 30 15:15:44 2020 +0200
misc: split form_pages tests
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
workflow: prefix fields in FormWorkflowStatusItem (#13846)