Bug #39541
api entreprise, normalisation de date vs listes
0%
Description
Les dates sont vides pour l'OL :
{u'association': {u'adresse_siege': {u'code_insee': [u'69275', u'69275'], u'code_postal': [u'69150', u'69150'], u'commune': [u'DECINES-CHARPIEU', u'DECINES-CHARPIEU'], u'complement': u'3EME ETAGE 3EME ETAGE ', u'distribution': None, u'libelle_voie': [u'SIMONE VEIL', u'SIMONE VEIL'], u'numero_voie': [u'10', u'10'], u'type_voie': [u'AV', u'AV']}, u'civilite_dirigeant': None, u'code_civilite_dirigeant': None, u'code_etat': None, u'code_groupement': None, u'date_creation': None, u'date_declaration': None, u'date_dissolution': None, u'date_publication': None, u'etat': u'true', u'groupement': None, u'id': None, u'mise_a_jour': None, u'objet': None, u'siret': u'38507188100093', u'siret_siege_social': u'38507188100093', u'titre': u'OLYMPIQUE LYONNAIS'}}
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
- Sujet changé de api entreprise, normalisation de date vs None à api entreprise, normalisation de date vs listes
(non c'est le fait d'avoir des listes)
Mis à jour par Serghei Mihai il y a environ 4 ans
- Lié à Bug #39550: api_entreprise: revoir la manière de normaliser les dates ajouté
Mis à jour par Frédéric Péters il y a presque 4 ans
- Dupliqué par Support #44764: crash API Entreprise sur une adresse "tout en double" ajouté
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Dupliqué par Bug #45476: API entreprise - normalize_dates TypeError ajouté
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0001-api_entreprise-fix-normalize_dates-function-39541.patch 0001-api_entreprise-fix-normalize_dates-function-39541.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ca ne plantera plus sur un TypeError, mais je ne suis pas sûre que ce soit ok d'avoir une liste en résultat:
assert data['association']['adresse_siege']['code_postal'] == ['75014']
Mis à jour par Thomas Noël il y a plus de 3 ans
Je confirme que ça va pas le faire, la récupération dans w.c.s. n'aura pas d'intelligence, il faut toujours renvoyer une string sur les champs de l'adresse.
Comme API Entreprise renvoie du n'importe quoi sur l'adresse (cf "complement" qui est une concaténation de string et le reste non) je propose de ne chercher aucune logique dans l'affaire : quand c'est une liste, renvoyer le premier élément.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0001-api_entreprise-fix-normalize_dates-function-39541.patch 0001-api_entreprise-fix-normalize_dates-function-39541.patch ajouté
alors comme ça
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Lauréline Guerin a écrit :
alors comme ça
Sans doute que ça passe, mais je serai plus à l'aise si ça n'était fait que sur les dictionnaires dont le nom commence par "adresse" (j'ai l'impression qu'il n'y a que là qu'on rencontre ce bogue bizarre). Mais bon, à regarder les exemples fournis sur la colonne de droite de https://doc.entreprise.api.gouv.fr/ on pourrait se dire que jamais on ne doit avoir de "liste de string". Donc ok.
Ah, quand même : ça serait bien de profiter de l'occasion pour renommer cette fonction normalize_dates() en normalize_results() -- je valide quand même le patch sachant que mêem si tu oublies de renommer personne ne t'en voudra.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4e7820e7548fdb6145e5089f9bbc5372dbc71d11 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Tue Jul 28 15:18:28 2020 +0200 api_entreprise: fix normalize_dates function (#39541)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api_entreprise: fix normalize_dates function (#39541)