Bug #23894
on ne loggue plus les statuts cibles invalides
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
17 mai 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
parce que dans 7e774f9f057f957b0aed27591e4f4aed5c4d282d #5739 on a retiré le [0] qui provoquait l'exception IndexError :
def get_target_status(self):
if not self.status:
- return None
+ return []
try:
- return [x for x in self.parent.parent.possible_status if x.id == self.status][0]
+ return [x for x in self.parent.parent.possible_status if x.id == self.status]
except IndexError:
get_publisher().get_app_logger().error(
...
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 6 ans
- Fichier 0001-really-log-invalid-status-detection-23894.patch 0001-really-log-invalid-status-detection-23894.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 6 ans
Plutôt que provoquer l'exception je serais plutôt pour avoir :
targets = [...]
if not targets:
log....
return targets
Mis à jour par Thomas Noël il y a presque 6 ans
- Fichier 0001-really-log-invalid-status-detection-23894.patch 0001-really-log-invalid-status-detection-23894.patch ajouté
Ouaip.
Mis à jour par Frédéric Péters il y a presque 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 579b1c17e31a1a0b6edcf91cc91190fe67d46ce1
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Thu May 17 15:16:55 2018 +0200
workflows: really log invalid status detection (#23894)
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
workflows: really log invalid status detection (#23894)