Bug #23033
mauvais marqueur posé lors d'un saut automatique
0%
Description
Ajouter une action globale avec une action "saut vers le statut X", qui pose un marqueur.
Utiliser l'action sur une demande dans un statut Y.
Bogue : ce marqueur se retrouve être X (la cible) au lieu du statut actuel de la demande (Y). Conséquence, aucun "retour" possible.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 6 ans
Certainement par ce que, dans wcs/wf/jump.py, sur le perform :
formdata.status = 'wf-%s' % wf_status[0].id self.handle_markers_stack(formdata)
alors qu'il faudrait gérer le marker avant le changement de statut
Mis à jour par Thomas Noël il y a environ 6 ans
- Assigné à mis à Thomas Noël
Confirmé en local : ça se produit sur tout saut automatique, qu'il soit dans une action globale ou pas.
Mis à jour par Thomas Noël il y a environ 6 ans
Dans une optique TDD, je veux bien qu'on me confirme que le test était mauvais :
(venv) thomas@zepo:~/.../wcs/tests [master|✚ 1…528]$ git diff diff --git a/tests/test_backoffice_pages.py b/tests/test_backoffice_pages.py index 136c2531..36de056f 100644 --- a/tests/test_backoffice_pages.py +++ b/tests/test_backoffice_pages.py @@ -3651,7 +3651,7 @@ def test_workflow_jump_previous_auto(pub): resp = resp.follow() # jumped and got back - assert formdef.data_class().get(formdata.id).status == 'wf-south' + assert formdef.data_class().get(formdata.id).status == 'wf-north' assert not formdef.data_class().get(formdata.id).workflow_data['_markers_stack'] def test_backoffice_fields(pub):
- status north = auto-jump to south (mark)
- status south = clic-jump back to mark
et donc on devrait bien revenir à north après le clic sur button_back
Mis à jour par Thomas Noël il y a environ 6 ans
- Fichier 0001-wf-jump-store-marker-before-jump-not-after-23033.patch 0001-wf-jump-store-marker-before-jump-not-after-23033.patch ajouté
- Statut changé de Nouveau à En cours
- Priorité changé de Normal à Haut
- Patch proposed changé de Non à Oui
Le test était bon mais incomplet, il fallait tester le chemin suivi par le formulaire et vérifier qu'il y a bien retour sur north (qui renvoie le formulaire sur south).
Et donc, la correction est bien celle prévue.
Mis à jour par Thomas Noël il y a environ 6 ans
- Sujet changé de saut avec marqueur en action globale : mauvais marqueur à mauvais marqueur posé lors d'un saut automatique
Mis à jour par Thomas Noël il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 5af4dd6d5ed609b6b38a0db61e8ba6c97c36131f (tag: v2.69) Author: Thomas NOEL <tnoel@entrouvert.com> Date: Mon Apr 9 15:24:56 2018 +0200 wf/jump: store marker before jump, not after (#23033)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
wf/jump: store marker before jump, not after (#23033)