Produits Entr'ouvert » w.c.s.

Je pense que le patch suivant ferait l'affaire mais je n'ai pas réfléchi aux implications qu'un tel comportement amèneraient.

--- a/wcs/workflows.py
+++ b/wcs/workflows.py
@@ -345,6 +345,8 @@ class WorkflowStatus:
         old_status = formdata.status
         for item in self.items:
             url = item.perform(formdata) or url
+            if formdata.status != old_status:
+                break
         if formdata.status != old_status:
             if not formdata.evolution:
                 formdata.evolution = []

Tu pourrais détailler un petit peu la situation dans laquelle tu es ?

Frédéric Péters a écrit :

Tu pourrais détailler un petit peu la situation dans laquelle tu es ?

Comme c'est moi qui ait demandé à Victor de faire ce ticket, je réponds (et j'avais la même idée que toi pour le patch, avec la même interrogation sur les conséquences).

Donc, pendant qu'on cherchait à imaginer un workflow d'évaluation de différentes réponses pour le cg14, on a fait un statut de workflow avec les actions suivantes :

1. jump vers ETAT 1 si condition C
2. jump vers ETAT 2 (sans condition, sous-entendu "sinon", si le saut précédent n'a pas été fait).

Mais en fait, ça finit toujours à l'ETAT 2, même si la condition C est vraie, car le 1er "saut" ne fait que changer le statut, que la seconde action s'empresse de mettre à ETAT 2. Et ce n'est pas ce qui est "attendu".

Voilà pour le contexte. Je regarderai les conséquences du break que tu proposes en cas de changement de statut dans une action, mais à première vue ça me parait être la bonne solution.