Development #30618
champ commentaire : valider le gabarit si le champ semble en contenir un
0%
Description
En passant par ComputedExpressionWidget.validate_template
Cela va bloquer les commentaires qui contiendrait des crochets non-ezt. Ca ne me semble pas important. Ca bloquera aussi les {{ ou {% qui ne serait pas du Django : aucune importance.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 5 ans
Poussé l'idée dans http://git.entrouvert.org/wcs.git?h=wip/30618-commentfield-validation, semble bien fonctionner "en vrai" mais je n'arrive pas à écrire un test de syntax error tests/test_admin_pages.py
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0001-fields-add-validation-on-comment-field-30618.patch 0001-fields-add-validation-on-comment-field-30618.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Désolé, j'ai ajouté un commit sur la branche de Thomas, puis j'en ai créé une autre parce que j'ai eu des difficultés pour rebaser.
Dans le patch, j'ai ré-intégré le code de Thomas puis j'ai ajouté 3 tests.
Mis à jour par Thomas Noël il y a plus de 4 ans
Y'a un soucis avec "use_validation_function" qui n'a pas d'usage ici. L'idée dans mon patch https://git.entrouvert.org/wcs.git/commit/?h=wip/30618-commentfield-validation&id=4cf9444daac829a6f687556f1fb1e04f03cae273 était de l'avoir sur TextWidget::_parse pour éviter de faire la validation lors de l'appel fait par WysiwygTextWidget::_parse. Tu perds cela ici.
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0001-fields-add-validation-on-comment-field-30618.patch 0001-fields-add-validation-on-comment-field-30618.patch ajouté
Désolé, j'ai mal recopié.
Donc on valide après les modifications du contenu (_sanitizeHTML et unescape)
Mis à jour par Thomas Noël il y a presque 4 ans
Et zut j'ai completement laissé ce patch passer aux oubliettes :( Tu peux le rebaser, Nicolas ?
Mis à jour par Nicolas Roche il y a presque 4 ans
- Fichier 0001-fields-add-validation-on-comment-field-30618.patch 0001-fields-add-validation-on-comment-field-30618.patch ajouté
- Statut changé de En cours à Solution proposée
voilà
Mis à jour par Thomas Noël il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Nicolas Roche il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c0a09241c3723c9292de303b548efcd20f3c88c4 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Thu Sep 5 17:35:00 2019 +0200 fields: add validation on comment field (#30618)
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
fields: add validation on comment field (#30618)