Développement #69906
api : faire que la recherche en "texte libre" sur les utilisateurs fonctionne sur un numéro international lorsqu’on tape un numéro local
0%
Description
Dans l’api on peut taper un /api/users/?q=061234… et actuellement c’est tout simplement l’usager qui possède ce numéro qui ressort.
Avec #69365 on migre les numéros de téléphone connus d’authentic au format international E.164 mais on souhaiterait bien sûr conserver cette fonctionnalité.
Il faudrait aussi que le numéro de téléphone renvoyé en tant que données de l’usager, en réponse à cette recherche, soit lui aussi au format local.
Files
Related issues
Associated revisions
History
Updated by Paul Marillonnet about 2 years ago
- Related to Développement #49212: Création de compte avec un numéro de téléphone mobile added
Updated by Paul Marillonnet about 2 years ago
- File 0001-api-let-free-text-search-accept-local-phone-numbers-.patch 0001-api-let-free-text-search-accept-local-phone-numbers-.patch added
- Status changed from Nouveau to Solution proposée
- Assignee set to Paul Marillonnet
- Patch proposed changed from No to Yes
Updated by Benjamin Dauvergne about 2 years ago
- Status changed from Solution proposée to En cours
Le premier parse ne produira aucune recherche (le else dépend du deuxième try/except). Il faudrait un cas qui passe dans le premier try, un cas except/try/else et un cas try/except/except, pour tester toutes les branches.
Updated by Benjamin Dauvergne about 2 years ago
Aussi j'ai l'impression que ça fait doublon avec la recherche sur le numéro faite juste avant via clean_number.
Updated by Paul Marillonnet about 2 years ago
Benjamin Dauvergne a écrit :
Le premier parse ne produira aucune recherche (le else dépend du deuxième try/except). Il faudrait un cas qui passe dans le premier try, un cas except/try/else et un cas try/except/except, pour tester toutes les branches.
Ah oui en effet j’avais loupé ça, je corrige.
Updated by Paul Marillonnet about 2 years ago
- File 0001-api-let-free-text-search-accept-local-phone-numbers-.patch 0001-api-let-free-text-search-accept-local-phone-numbers-.patch added
- Status changed from En cours to Solution proposée
Benjamin Dauvergne a écrit :
Aussi j'ai l'impression que ça fait doublon avec la recherche sur le numéro faite juste avant via clean_number.
Ok, je crois la manière correcte de procéder est :
· on cherche à parser un numéro globalement unique,
· si cela échoue, on cherche à parser un numéro composable depuis le pays par défaut dans la conf a2 (DEFAULT_COUNTRY_CODE),
· si cela échoue, on rebascule sur le clean_number standard.
Cette version dans le patche avec tous les embranchements testés.
Updated by Benjamin Dauvergne about 2 years ago
- Status changed from Solution proposée to Solution validée
Paul Marillonnet a écrit :
Benjamin Dauvergne a écrit :
Aussi j'ai l'impression que ça fait doublon avec la recherche sur le numéro faite juste avant via clean_number.
Ok, je crois la manière correcte de procéder est :
· on cherche à parser un numéro globalement unique,
· si cela échoue, on cherche à parser un numéro composable depuis le pays par défaut dans la conf a2 (DEFAULT_COUNTRY_CODE),
· si cela échoue, on rebascule sur le clean_number standard.Cette version dans le patche avec tous les embranchements testés.
Ça me va, par contre j'aime bien quand une même variable n'est pas réutilisée pour des types d'objet différents, ça rend la lecture difficile. Garder phone_number comme nom du résultat de .parse(), mais le résultat de .format_number() ou clean_number() devrait se nommer formatted_phone_number.
Updated by Paul Marillonnet about 2 years ago
Benjamin Dauvergne a écrit :
Ça me va, par contre j'aime bien quand une même variable n'est pas réutilisée pour des types d'objet différents, ça rend la lecture difficile. Garder
phone_numbercomme nom du résultat de.parse(), mais le résultat de.format_number()ouclean_number()devrait se nommerformatted_phone_number.
Fair enough, je fais la modif’ et j’attends Jenkins vert avant de pousser.
Updated by Paul Marillonnet almost 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 3e53e2ecf895f27fd33ee86f794c854d675c1554
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Mon Oct 17 14:42:20 2022 +0200
api: let free text search accept local phone numbers (#69906)
Updated by Transition automatique almost 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
api: let free text search accept local phone numbers (#69906)