Projet

Général

Profil

Development #30419

arrêter avec la validation de regex html5 (?)

Ajouté par Frédéric Péters il y a environ 5 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
06 février 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Le message d'erreur du navigateur n'est pas satisfaisant et on ne peut rien y faire, mais aussi certaines regex n'y sont pas reconnues (je dis ça en ayant vu un message dans la console hier, sans plus savoir sur quel formulaire…).


Fichiers


Demandes liées

Lié à w.c.s. - Bug #31898: Une erreur sur un champ masqué empêche d'avancer mais ne s'affiche pasFermé01 avril 2019

Actions

Révisions associées

Révision 66e1afc7 (diff)
Ajouté par Thomas Noël il y a plus de 4 ans

form: remove regex pattern on string widgets (#30419)

Historique

#1

Mis à jour par Thomas Noël il y a environ 5 ans

"on ne peut rien y faire" : je pense que si, #15831, mais ça alourdit l'UI (ajouter à côté de la regex un input text "message de rappel du format attendu")...

Mais si "certaines regex n'y sont pas reconnues", c'est effectivement un sacré pépin, un seul exemple me suffira à être convaincu (c'est mon côté Saint Thomas)

#2

Mis à jour par Frédéric Péters il y a environ 5 ans

Mais si "certaines regex n'y sont pas reconnues", c'est effectivement un sacré pépin, un seul exemple me suffira à être convaincu (c'est mon côté Saint Thomas)

Je ne sais même plus si c'était avec Firefox ou Chromium… (et au pire si ce n'est pas reconnu j'imagine que c'est juste ignoré, ça n'empêche pas la saisie).

#3

Mis à jour par Frédéric Péters il y a environ 5 ans

Trouvé côté toodego, message console firefox :

Impossible de vérifier <input pattern='[0-9A-Za-z\-\ ]{0,10}'> car le modèle (pattern) n’est pas une expression régulière valide : invalid identity escape in regular expression

#5

Mis à jour par Serghei Mihai il y a environ 5 ans

L'alternative serait d'afficher une erreur de formulaire en pointant le champ qui ne match pas?

#6

Mis à jour par Frédéric Péters il y a environ 5 ans

Le comportement (re)deviendrait celui sans validation html5, le formulaire serait soumis, une erreur de validation serait détectée, le champ marqué en erreur, etc.

#7

Mis à jour par Thomas Noël il y a environ 5 ans

Je disais donc "un seul exemple me suffira à être convaincu" et donc oui, supprimons.

C'est en fait un "simple" revert de e3894bcfc87a7d31065775279e70db02fdf8a232 qu'il faut faire, je pense.

#8

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

  • Lié à Bug #31898: Une erreur sur un champ masqué empêche d'avancer mais ne s'affiche pas ajouté
#10

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

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

Yes, go le patch. (s'il s'applique encore ainsi)

#11

Mis à jour par Thomas Noël il y a plus de 4 ans

Le code a été refait depuis, nouveau patch, plus facile encore.

#12

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

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

Ok.

#13

Mis à jour par Thomas Noël il y a plus de 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 66e1afc797a68ce9f964311b1b9a62f41f030481
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Sat Sep 21 18:14:09 2019 +0200

    form: remove regex pattern on string widgets (#30419)

#14

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

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

Formats disponibles : Atom PDF