Bug #52527
grenoble-gru: gérer les cas ou le champ "adresse" du référentiel des PAVs est une chaîne de caractères
0%
Description
Cela devait être un dico.
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a environ 3 ans
- Fichier 0001-grenoble-gru-handle-case-when-adresse-field-is-strin.patch 0001-grenoble-gru-handle-case-when-adresse-field-is-strin.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 3 ans
On aurait presque envie de plutôt voir « if item['adresse'] == "None": continue » ...? Parce que j'imagine que c'est un bogue de leur côté et qu'il faut éviter de remonter des adresses plantées...?
Mis à jour par Serghei Mihai il y a environ 3 ans
Thomas Noël a écrit :
On aurait presque envie de plutôt voir « if item['adresse'] == "None": continue » ...? Parce que j'imagine que c'est un bogue de leur côté et qu'il faut éviter de remonter des adresses plantées...?
C'est sûrement un bogue et en remontant l'adresse telle quelle (quand c'est une string) ils feront le nécessaire de leur côté soit pour corriger les adresses, soit pour les exclure les PAVs sans adresse.
Mis à jour par Thomas Noël il y a environ 3 ans
J'ai peur alors que tu ne fasses qu'ajouter un bogue (des doublons) en permettant ce genre d'adresse... Si c'est pas un dict, fait juste un "continue", considère que l'entrée est pourrie.
Mis à jour par Serghei Mihai il y a environ 3 ans
- Fichier 0001-grenoble-gru-ignore-string-values-for-PAV-adresse-fi.patch 0001-grenoble-gru-ignore-string-values-for-PAV-adresse-fi.patch ajouté
D'acc
Mis à jour par Thomas Noël il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Si les cycles CPU étaient retenus sur les salaires, tu mettrais le « item['id'] = str(item['id']) » après le if/continue.
Et idéalement dans le test_get_pavs tu ajouteras à la fin un « assert len(response.json['data']) == 7 » et je t'offre un tour gratuit sur mon carbone.
</mode très pénible>
Mis à jour par Serghei Mihai il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Serghei Mihai
Pris en compte tes remarques et je t'offre un tour sur mon 20 pouces ;)
commit 42b061105feb41818d33233bb0d9f988c39a7ade (HEAD -> master, origin/main)
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Tue Mar 30 16:54:17 2021 +0200
grenoble-gru: ignore string values for PAV "adresse" field (#52527)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
grenoble-gru: ignore string values for PAV "adresse" field (#52527)