api_entreprise: fix mandataires’ first names delimiter (#76737) #218
No reviewers
Labels
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: entrouvert/passerelle#218
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/76737-api-entrepise-mandataire-social-first-names-delimiter"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -28,3 +28,3 @@
if any([attr not in mandataire for attr in ['prenom', 'nom', 'date_naissance']]):
return False
if normalize(mandataire['prenom'].split(',')[0]) != normalize(first_name):
if normalize(mandataire['prenom'].split(maxsplit=1)[0]) != normalize(first_name):
Assez étonné que ça ne nécessite pas le changement d'au moins un test.
Oui, on peut adapter les tests, y placer des données de mandataires avec plusieurs prénoms, pour relever la correction apportée. C’est à jour dans la branche.
c6de4202df
to76c68e12a4
Visiblement ce ne sont pas toujours des espaces non plus #75739-4:
Arf en effet, bien vu, j’ajoute les deux délimiteurs supportés.
76c68e12a4
toa5a36c983c
C’est dans la branche.
Mouais
Ça va poser souci pour 2804 prénoms connus de l'INSEE:
(Source: https://www.insee.fr/fr/statistiques/2540004?sommaire=4767262)
Je comprends pas bien où on va avec ces bidouilles, on pourrait juste remplacer les tirets par des espaces des deux cotés, enlever les autres diacritiques, et voir si depuis la donnée FC on match au moins chacun des prénoms API Entreprise de façon approchée, genre 90%, non ?
a5a36c983c
to6d55216e43
Ok, version à jour avec à la place un bête
prenom.replace(…).split(…)
au lieu de bricoler des regex.