Project

General

Profile

Bug #35387

lille CLU : éviter un KeyError sur un service non activé

Added by Thomas Noël 9 days ago. Updated 7 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
13 Aug 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

cf #35299

0001-lille-urban-card-dont-set-subservice-if-no-service-3.patch View (1.06 KB) Thomas Noël, 13 Aug 2019 03:11 PM

Associated revisions

Revision d382a4c8 (diff)
Added by Thomas Noël 9 days ago

lille urban card: dont set subservice if no service (#35387)

History

#2 Updated by Thomas Noël 9 days ago

#3 Updated by Benjamin Dauvergne 9 days ago

  • Status changed from Solution proposée to Solution validée

Ack.

#4 Updated by Benjamin Dauvergne 9 days ago

Bon j'ai validé parce que ça supprime le bug, je ne sais pas si ça fait ce qu'on veut par contre vu que je ne comprends pas vraiment ce connecteur; à la lecture de l'autre ticket on a service_xxx: None, service_xxx_newsletter: "Oui" je ne sais pas ce qu'on attend dans ce cas.

#5 Updated by Thomas Noël 9 days ago

Benjamin Dauvergne a écrit :

Bon j'ai validé parce que ça supprime le bug, je ne sais pas si ça fait ce qu'on veut par contre vu que je ne comprends pas vraiment ce connecteur; à la lecture de l'autre ticket on a service_xxx: None, service_xxx_newsletter: "Oui" je ne sais pas ce qu'on attend dans ce cas.

Ce cas serait un bogue qu'ils doivent corriger dans le formulaire et/ou l'appel webservice.

Mais le cas actuel qui foire c'est :

  service_xxx: None
  service_xxx_newletter: None

et boum (parce qu'on peut pas éviter d'envoyer le service_xxx_newsletter). Donc mon patch fait juste rien quand service_xxx n'est pas Oui, et hop.

#6 Updated by Thomas Noël 9 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit d382a4c8502df3ca2367b575d3def6317ce25093
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Aug 13 15:06:38 2019 +0200

    lille urban card: dont set subservice if no service (#35387)

#8 Updated by Frédéric Péters 7 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF