Projet

Général

Profil

Development #43556

La validation "Numéro de téléphone (France)" valide des numéros invalides

Ajouté par Mikaël Ates (de retour le 29 avril) il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
03 juin 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Le contrôle limite la chaîne saisie à 0 puis 9 caractères parmi chiffres, '.' et \r\n\t\f\v.

Cela autorise la saisie de valeurs comme 06 1234567 qui n'est pas un numéro de téléphone français valide.

Peut-être faudrait-il limiter la saisie à 0 puis 9 chiffres.


Fichiers

Révisions associées

Révision 2122f9d9 (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

misc: check for correct number of digits in French phone numbers (#43556)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Assigné à mis à Frédéric Péters
#2

Mis à jour par Frédéric Péters il y a presque 4 ans

Voilà et oui on continuera à valider des numéros qui n'existent pas, facilement genre 0012345678.

#3

Mis à jour par Mikaël Ates (de retour le 29 avril) il y a presque 4 ans

Mais on pourrait réduire à 0[1-9]{9} non ?

#4

Mis à jour par Frédéric Péters il y a presque 4 ans

Non. Et là tu essaies de répondre à l'exemple invalide que je donne, puis on en trouvera un autre, etc. Il faut accepter qu'il y aura des numéros mauvais.

#5

Mis à jour par Frédéric Péters il y a presque 4 ans

(Non parce qu'on doit autoriser des zéros plus loin, et non parce que sur du préremplissage depuis des données d'authentic on peut avoir des espaces etc.).

#6

Mis à jour par Emmanuel Cazenave il y a presque 4 ans

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

Mis à jour par Mikaël Ates (de retour le 29 avril) il y a presque 4 ans

Et là tu essaies de répondre à l'exemple invalide que je donne

J'ai écrit un peu vite l'expression qui est fausse, mais il me semble que l'on usait plutôt classiquement des expressions rationnelles :
  • ^0[1-9][0-9]{8}$ téléphone français
  • ^0[6-7][0-9]{8}$ téléphone français mobile

puis on en trouvera un autre, etc. Il faut accepter qu'il y aura des numéros mauvais.

Sûrement, l'idée c'était juste de réduire les possibilités.

non parce que sur du préremplissage depuis des données d'authentic on peut avoir des espaces etc

Il me semble que c'est plus ou moins équivalent avec la situation actuelle parce qu'à l'enregistrement, authentic, avec le type de champs téléphone, va enlever espaces, tirets et points mais autoriser un '+' en préfixe et 20 chiffres.

#8

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)

J'oubliais qu'authentic n'avait pas de validation des numéros français.

commit 2122f9d9553882c95d7188ea545dc994a63fa9a7
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jun 3 09:54:58 2020 +0200

    misc: check for correct number of digits in French phone numbers (#43556)
#9

Mis à jour par Frédéric Péters il y a presque 4 ans

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

Formats disponibles : Atom PDF