Produits Entr'ouvert » w.c.s.

Pour les développeurs de workflow je pense que ce serait bien d'afficher le contenu de la pile, dans la sidebar sous un titre "Liste des marqueurs" quand il y en a, dans l'ordre inverse (le dernier poussé en premier).

Je ne comprends pas la modification dans formdata.get_status (le retrait de status_id = status.split('-')[1] ... comment ça peut marcher sans ça ?)

« get_previous_marked_status(self) » pourrait être renommée « pop_previous_marked_status(self) »

Dans cette même fonction, on pourrait vérifier que le status_id trouvé fait bien partie des possible_status ; dans le cas d'un workflow modifié entre temps ?

assert previous_status, 'failed to compute previous status' : c'est un debug ? pour moi on devrait juste ne rien faire dans ce cas (juste un log d'erreur)

Je remplacerais le update + del ici :

        if self.workflow_data:
            d.update(self.workflow_data)
            # pass over workflow data to:
            #  - attach an extra url attribute to uploaded files
            #  - remove "private" attributes
            for k, v in self.workflow_data.items():
                if k[0] == '_':
                    del d[k]
                    continue
            ...

par

        if self.workflow_data:
            for k, v in self.workflow_data.items():
                if k[0] == '_':
                    continue
                d[k] = v
                if isinstance(v, Upload):
                    #  attach an extra url attribute to uploaded files: 
                    ...

dans wcs/workflows.py, « has_markers = False » : ne sert jamais, remplacé par un break/else:continue/break de professionnel

« Set marker on status » j'aurai voulu un truc qui se traduise par « Poser un marqueur pour un retour vers le statut actuel »

Lorsqu'un saut est configuré comme retour (saut vers «Previously Marked Status»), que fait-on par rapport au choix "poser un marqueur". On l'interdit ? (je pense que oui, il faudrait interdire cette combinaison).

Ok pour cette mouture.

Reste un soucis : dans la liste des actions d'un workflows, et surtout l'affichage SVG, on pourrait voir les deux appels "aller" et "retour". Je pense qu'en modifiant "render_as_line" ça pourrait suffire.

• self.set_marker_on_status → on ajoute "(puis retour)" (ou autre ?)
• self.status == '_previous' → on ajoute "(retour)"

--- a/wcs/workflows.py
+++ b/wcs/workflows.py
@@ -1899,12 +1899,22 @@ class ChoiceWorkflowStatusItem(WorkflowStatusJumpItem):

     def render_as_line(self):
         if self.label:
+            more = ''
+            if self.set_marker_on_status:
+                more += ' ' + _('(and return)')
+            if self.status == '_previous':
+                more += ' ' + _('(return)')
             if self.by:
-                return _('Change Status "%(label)s" by %(by)s') % \
-                        { 'label' : self.label,
-                          'by' : self.render_list_of_roles(self.by) }
+                return _('Change Status "%(label)s" by %(by)s%(more)s') % {
+                        'label' : self.label,
+                        'by' : self.render_list_of_roles(self.by),
+                        'more': more
+                    }
             else:
-                return _('Change Status "%s"') % self.label
+                return _('Change Status "%(label)s"%(more)s') % {
+                        'label': self.label,
+                        'more': more
+                }
         else:
             return _('Change Status (not completed)')

et bon, déjà c'est pas joli comme code, mais surtout, le SVG ne suit pas le render_as_line... zut.

Ack