Development #48864
sms : être plus souple sur les numéros
0%
Description
On devrait accepter ce qu'on accepte coté authentic et normaliser de la même manière.
validate_phone_number = RegexValidator(
r'^\+?\d{,20}$',
message=_('Phone number can start with a + and must contain only digits.'))
def clean_number(number):
cleaned_number = re.sub(r'[-.\s/]', '', number)
validate_phone_number(cleaned_number)
return cleaned_number
Fichiers
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a environ 3 ans
- Fichier 0001-sms-allow-space-on-input-destiation-number-48864.patch 0001-sms-allow-space-on-input-destiation-number-48864.patch ajouté
- Statut changé de Nouveau à Information nécessaire
- Patch proposed changé de Non à Oui
Pas sûr de comprendre : j'ai l'impression que la regex proposée dans le ticket ne permet pas les espaces, alors que c'est ce qui est demandé dans le ticket lié.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
On doit laisser passer ce qu'authentic laisse passer, donc [.-\s] et nettoyer ça si derrière OVH ou autre n'aime pas (c'est ce dernier point qui me laissait dire qu'il fallait faire exactement comme authentic, sans savoir si c'est vraiment nécessaire, si OVH accepte ^+?([.-\s0-9]*)$ tant mieux, mais je me disais que plutôt que de faire différemment pour chaque backend, autant trouver une méthode unique.
Mis à jour par Nicolas Roche il y a environ 3 ans
- Fichier 0001-sms-accept-same-destination-numbers-as-authentic-doe.patch 0001-sms-accept-same-destination-numbers-as-authentic-doe.patch ajouté
- Statut changé de Information nécessaire à Solution proposée
- Assigné à mis à Nicolas Roche
Compris (j'ai mal lu le code d'a2).
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Ok, je vois que clean_numbers() fait déjà le reste du boulot.
Mis à jour par Nicolas Roche il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit d4741e3d800271380d89a6b96a94919dc23e6dd5
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Fri Jan 29 16:14:13 2021 +0100
sms: accept same destination numbers as authentic (#48864)
Mis à jour par Nicolas Roche il y a environ 3 ans
rq: Ce patch a temporairement cassé le build sur le test test_orange.py::test_send_msg,
mais je n'ai pas réussi à reproduire, ni localement, ni sur jenkins.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
Ça n'a rien, je pense qu'il y a un non déterminisme dans la queryset Job.objects.all() aucun ordre n'étant défini (.order_by('id') suffirait).
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
sms: accept same destination numbers as authentic (#48864)