Projet

Général

Profil

Bug #35387

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

Ajouté par Thomas Noël il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
13 août 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

cf #35299


Fichiers

Révisions associées

Révision d382a4c8 (diff)
Ajouté par Thomas Noël il y a plus de 4 ans

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

Historique

#2

Mis à jour par Thomas Noël il y a plus de 4 ans

#3

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée

Ack.

#4

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

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

Mis à jour par Thomas Noël il y a plus de 4 ans

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

Mis à jour par Thomas Noël il y a plus de 4 ans

  • Statut changé de Solution validée à 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

Mis à jour par Frédéric Péters il y a plus de 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF