Development #30419
arrêter avec la validation de regex html5 (?)
0%
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
Révisions associées
Historique
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)
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).
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
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?
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.
Mis à jour par Thomas Noël il y a environ 5 ans
- Fichier 0001-remove-regex-pattern-on-string-widgets-30419.patch 0001-remove-regex-pattern-on-string-widgets-30419.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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.
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é
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)
Mis à jour par Thomas Noël il y a plus de 4 ans
- Fichier 0001-form-remove-regex-pattern-on-string-widgets-30419.patch 0001-form-remove-regex-pattern-on-string-widgets-30419.patch ajouté
- Statut changé de Solution validée à Solution proposée
Le code a été refait depuis, nouveau patch, plus facile encore.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Ok.
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)
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
form: remove regex pattern on string widgets (#30419)