Development #48546
attribute_kinds : normaliser les préfixes des numéros de téléphone locaux
0%
Description
Pour éviter de stocker du +33xxx ou 0033xxx.
Fichiers
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Bloqué par Development #48350: attribute_kinds: normaliser les numéros de téléphone posés via l'API ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-attribute_kinds-normalize-local-phone-numbers-prefix.patch 0001-attribute_kinds-normalize-local-phone-numbers-prefix.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-attribute_kinds-normalize-local-phone-numbers-prefix.patch 0001-attribute_kinds-normalize-local-phone-numbers-prefix.patch ajouté
+ migration pour normaliser les numéros déjà enregistrés
Mis à jour par Thomas Noël il y a plus de 3 ans
Ca ne fonctionne ici que pour la France, et uniquement métropolitaine. Et je pense que l'algo un peu simplet va difficilement tenir le choc d'autres pays (Authentic est au moins utilisé en vrai en Belgique).
Quid de faire un type d'attribut "phone_number_fr" avec une fonction de normalisation bien plus explicite (cf le filtre phonenumber_fr dans w.c.s.) ? Ca me semblerait moins hasardeux et ça ne toucherait à rien de l'existant (on a peut-être des sites qui veulent avoir des +33).
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Thomas Noël a écrit :
Ca ne fonctionne ici que pour la France, et uniquement métropolitaine. Et je pense que l'algo un peu simplet va difficilement tenir le choc d'autres pays (Authentic est au moins utilisé en vrai en Belgique).
C'est pas l'algo qui t'embête, il marcherait je suppose bien avec des préfixes Belges. C'est le fait que ça va migrer les +33 en Belgique alors qu'on voudrait certainement les conserver. Ça je comprends et le mieux c'est de rendre la configuration des préfixes vide par défaut et de transformer la migration en commande.
Quid de faire un type d'attribut "phone_number_fr" avec une fonction de normalisation bien plus explicite (cf le filtre phonenumber_fr dans w.c.s.) ? Ca me semblerait moins hasardeux et ça ne toucherait à rien de l'existant (on a peut-être des sites qui veulent avoir des +33).
Là c'était pour la présentation, ici c'est juste pour normaliser, en dehors du fait que la liste des préfixes devrait varier sur chaque installation, voir être une configuration explicite, je ne suis pas certain d'avoir besoin d'une fonction aussi complexe. Je ne cherche pas à vérifier que les numéros sont valides. Juste pour 99% des numéros locaux qu'ils aient toujours la même forme pour la recherche (en normalisant de ce coté aussi #48352).
Mis à jour par Thomas Noël il y a plus de 3 ans
En fait je trouve ce fonctionnement magique (caché) pas très bon. On n'a pas l'habitude nulle part de modifier les champs saisis par les usagers, je pense pas utile de créer un précédent.
En fait je ne comprends même pas à quoi ça sert, finalement... ça doit être ça mon problème :)
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Bloqué par Development #48350: attribute_kinds: normaliser les numéros de téléphone posés via l'API supprimé
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution proposée à Rejeté
Je rejette, je vais améliorer les choses uniquement coté recherche.
Ce que ça améliore c'est que si on a stocké +33699999999 et qu'on cherche 0699999999 et bien on ne trouve pas, mais je peut aussi traiter le problème uniquement coté recherche en cherchant les trois chaînes ou alors en enlevant automatiquement le préfixe (dans ce cas on cherchera toujours 699999999 et rien d'autre, ticket #48352.