Development #54663
schéma input json, possibilité d'associer une description à un motif de validation
0%
Description
Actuellement côté CMIS ça affiche
filename : string /[\w!\#\$%\&\+\-\^_`\~;\[\]\{\}\+=\~\.]+$/
et ça serait agréable de ne pas avoir à déchiffrer ça, qu'il puisse être écrit "caractères spéciaux autorisés (!#$%&+-^_`~;[]{}+=~)".
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-misc-allow-pattern-description-in-json-schemas-54663.patch 0001-misc-allow-pattern-description-in-json-schemas-54663.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Mis à jour par Frédéric Péters il y a plus de 2 ans
Ça serait pas mal aussi aux endroit où on a une expression terriblement moche pour des booléens, passerelle/contrib/toulouse_smart/schemas.py
'pattern': '[Oo][Uu][Ii]|[Nn][Oo][Nn]|[Tt][Rr][Uu][Ee]|[Ff][Aa][Ll][Ss][Ee]|1|0',
passerelle/contrib/caluire_axel/schemas.py,
'pattern': '[Oo]|[Nn]|[Tt][Rr][Uu][Ee]|[Ff][Aa][Ll][Ss][Ee]|1|0',
mais peut-être que ça serait pour un autre ticket, qui unifierait l'expression de booléen au passage.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-misc-allow-pattern-description-in-json-schemas-54663.patch 0001-misc-allow-pattern-description-in-json-schemas-54663.patch ajouté
Frédéric Péters a écrit :
Ça serait pas mal aussi aux endroit où on a une expression terriblement moche pour des booléens
Fait, ça ne mange pas de pain.
Mis à jour par Thomas Noël il y a plus de 2 ans
Ca me chiffonne qu'on ne puisse plus du tout voir la regex. Pas possible en tooltip ou truc du genre. Parce que par exemple quand on dit "Les valeurs "0", "1", "O", "N", "true" ou "false" sont autorisées" c'est bien, mais ça ne dit pas que True, False, TRUE, FALSE, o, n... passent aussi (entre autres).
Mis à jour par Frédéric Péters il y a plus de 2 ans
Mais on s'en fout des mille variations et de l'info que tRuE peut passer. (perso)
Qu'on donne une indication de ce qui va passer sûr, et qu'on soit en douce plus souple que ce qu'on indique, c'est pour moi très bien.
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
Mais on s'en fout des mille variations et de l'info que tRuE peut passer. (perso)
Qu'on donne une indication de ce qui va passer sûr, et qu'on soit en douce plus souple que ce qu'on indique, c'est pour moi très bien.
En l'occurence j'imagine la personne qui voit qu'il faut "true" et se dit qu'elle ne peut pas envoyer "True" alors que ça aurait été facile, et se retrouve à faire du if/else inutile, mais bon, bref, oui, allez, je comprends aussi ta position et je m'y range.
Je valide donc !
Mis à jour par Frédéric Péters il y a plus de 2 ans
On pourrait ajouter que c'est insensible à la casse; ça resterait plus intelligible que l'expression.
(je laisse la validation mais perso pas enthousiaste sur les chaines en français).
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-misc-allow-pattern-description-in-json-schemas-54663.patch 0001-misc-allow-pattern-description-in-json-schemas-54663.patch ajouté
- Statut changé de Solution validée à Solution proposée
Avec indication que insensible à la casse, et chaîne en anglais pour caluire_axel (pour toulouse_smart je la laisse par soucis d'homogénéité avec les autres chaînes du fichier qui sont en français (sûrement parce que la possibilité de traduire a été ajoutée ultérieurement avec #54661)).
Mis à jour par Nicolas Roche il y a plus de 2 ans
sûrement parce que la possibilité de traduire a été ajoutée ultérieurement avec #54661
Mea-culpa, le ticket a été commencé avant mais fini après.
Ok pour le rendu sur touloutse_smart.
Pour axel, les booléens ne sont pas encore utilisé à Caluire, mais il le sont à Toulouse.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit fe8af9ff0a6d3fd4720191dc1010c960d64f9214 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Thu Sep 23 11:59:59 2021 +0200 misc: allow pattern description in json schemas (#54663)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: allow pattern description in json schemas (#54663)