Projet

Général

Profil

Development #73345

fields : fournir une validation plus contraignante des numéros de téléphone

Ajouté par Paul Marillonnet il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
12 janvier 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Numéros maintenant saisis par l’usager à l’aide du PhoneField introduit à l’occasion de #49212.
Il y a eu à un moment l’idée que la validation de tels numéros est une science un peu inexacte, et qu’elle doit rester assez souple dans a2.
Néanmoins cette souplesse actuellement est excessive et l’usager peut taper des parties locales de numéro à 2 voire un seul chiffre.
La manière de remédier à cela est de ne plus se contenter de parser le numéro, mais d’utiliser les bouts d’interface de phonenumbers pour une validation plus complète, à savoir phonenumbers.is_possible_number(). ou même encore phonenumbers.is_valid_number().


Fichiers


Demandes liées

Lié à Publik - Development #49212: Création de compte avec un numéro de téléphone mobileEn cours01 octobre 2021

Actions

Révisions associées

Révision 043c7abf (diff)
Ajouté par Paul Marillonnet il y a plus d'un an

forms: provide stricter PhoneField validation (#73345)

Historique

#1

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Assigné à mis à Paul Marillonnet
#2

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Lié à Development #49212: Création de compte avec un numéro de téléphone mobile ajouté
#4

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Sujet changé de fields : fournir une validation plus contraignante des numéros de téléphone mobile à fields : fournir une validation plus contraignante des numéros de téléphone
  • Description mis à jour (diff)
#5

Mis à jour par Paul Marillonnet il y a plus d'un an

Finalement, ceinture et bretelles.
Une première tentative avec is_possible_number, qui donne une première erreur de validation, puis is_valid_number, plus restrictif encore.

#6

Mis à jour par Benjamin Dauvergne il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée

Je mettrai le même message d'erreur, qu'on essaie pas de trouver des traductions différentes alors qu'on a aucune idée de la différence entre les deux tests.

#7

Mis à jour par Paul Marillonnet il y a plus d'un an

Benjamin Dauvergne a écrit :

Je mettrai le même message d'erreur, qu'on essaie pas de trouver des traductions différentes alors qu'on a aucune idée de la différence entre les deux tests.

Ok, dans ce cas je vais juste laisser is_valid_number, qui est plus restrictif (¬is_possible_number → ¬is_valid_number).

#8

Mis à jour par Benjamin Dauvergne il y a plus d'un an

Si tu es certain que is_possible_number inclut is_valid_number ok, sinon j'aurai bien vu un and.

#9

Mis à jour par Paul Marillonnet il y a plus d'un an

Benjamin Dauvergne a écrit :

Si tu es certain que is_possible_number inclut is_valid_number ok, sinon j'aurai bien vu un and.

Je me suis planté dans mon précédent message (que j’ai édité ensuite pour le corriger), c’est is_valid_number qui inclut is_possible_number.

#10

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)
commit 043c7abf6d3c93fbbc0e679e75259d7c00d6581f
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Mon Jan 16 08:54:08 2023 +0100

    forms: provide stricter PhoneField validation (#73345)
#12

Mis à jour par Transition automatique il y a plus d'un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#13

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF