Bug #21076
crash si une demande se trouve dans un status inconnu
0%
Description
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/wcs_olap/cmd.py", line 112, in job
feeder.feed()
File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 437, in feed
formdef_feeder.feed()
File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 859, in feed
self.do_data()
File "/usr/lib/python2.7/dist-packages/wcs_olap/feeder.py", line 658, in do_data
status = data.formdef.schema.workflow.statuses_map[status_id]
KeyError: u'6'
Ça peut malgré tout encore arriver dans w.c.s. quand on joue "trop" avec les workflows (écrasement ou joujou direct dans le pickle, etc).
Je pense qu'on pourrait juste ne pas considérer ces demandes.
Fichiers
Historique
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier 0001-ignore-formdata-with-unknown-status-21076.patch 0001-ignore-formdata-with-unknown-status-21076.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
On pourrait aussi inventer un statut avec le status_id en faisant .get(status_id, str(status_id)), au moins on verrait le souci remonté.
Mis à jour par Thomas Noël il y a plus de 6 ans
J'arrive pas à voir l'impact de ce que tu proposes, ça va pas déconner si le statut généré ne fait pas parti des data.formdef.schema.workflow.statuses_map ?
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Ouais tu as raison en plus je ne vois pas comment générer une valeur generic_status_id, le truc n'est pas défini (startpoint, endpoint, midpoint ?), laissons tombé, ack.
)
Mis à jour par Thomas Noël il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 1c0f6a3b4ceb8a1e7b9f7e07b5435ce6e45f432d
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed Jan 10 16:10:55 2018 +0100
ignore formdata with unknown status (#21076)
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de Résolu (à déployer) à En cours
+ self.logger.warning('%s.%s unknown status status_id',
+ data.formdef.schema.name, data.id, status_id)
Il y a trois paramètres mais seulement deux %s.
Mis à jour par Thomas Noël il y a plus de 6 ans
Frédéric Péters a écrit :
Il y a trois paramètres mais seulement deux %s.
Merci Frédéric ... fix poussé.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé