Development #37251
assert previous_status, 'failed to compute previous status'
0%
Description
Quand on fait un saut vers "statut précédent" mais que celui-ci n'existe pas ou plus, boum :
[2019-10-28 09:40:04] exception caught Exception: type = '<type 'exceptions.AssertionError'>', value = 'failed to compute previous status' Stack trace (most recent call first): File "/usr/lib/python2.7/dist-packages/wcs/formdata.py", line 592, in jump_status 590 if status_id == '_previous': 591 previous_status = self.pop_previous_marked_status() > 592 assert previous_status, 'failed to compute previous status' 593 status_id = previous_status.id 594 status = 'wf-%s' % status_id
Le "assert" est assez violent, sans doute pourrait-on remplacer ça par un simple LoggerError ?
(ou bien ai-je oublié une raison qui faisait qu'on avait préféré ce assert ?)
Historique
Mis à jour par Thomas Noël il y a plus de 4 ans
Thomas Noël a écrit :
Le "assert" est assez violent
Je précise : ça fait une trace système... et rien d'autre, c-à-d que l'admin fonctionnel ne voit rien (en dehors des demandes qui ne sautent pas).
Mis à jour par Frédéric Péters il y a plus de 4 ans
Oui c'est #37169 (que je n'ai pas déplacé ici), j'y demande dans #37169#note-2 le déroulé pour reproduire.
Mis à jour par Frédéric Péters il y a plus de 4 ans
À propos de l'assert, #16524, tu y disais déjà :
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)
et de mon côté :
En fait ça me va plutôt bien de commencer par éclater sur ce genre de bug de workflow / de comportement; et ajouter des précautions quand on verra que c'est mal géré.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Nouveau à Fermé
Traité via #42854.