Project

General

Profile

Development #30618

champ commentaire : valider le gabarit si le champ semble en contenir un

Added by Thomas Noël 10 months ago. Updated 3 months ago.

Status:
Solution proposée
Priority:
Normal
Assignee:
Target version:
-
Start date:
13 Feb 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-fields-add-validation-on-comment-field-30618.patch View (4.19 KB) Nicolas Roche, 05 Sep 2019 06:06 PM

0001-fields-add-validation-on-comment-field-30618.patch View (6.54 KB) Nicolas Roche, 06 Sep 2019 04:53 PM

History

#1 Updated by Thomas Noël 10 months ago

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

#2 Updated by Nicolas Roche 3 months ago

  • Assignee set to Nicolas Roche

#3 Updated by Nicolas Roche 3 months ago

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.

#4 Updated by Thomas Noël 3 months ago

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.

#5 Updated by Nicolas Roche 3 months ago

Désolé, j'ai mal recopié.
Donc on valide après les modifications du contenu (_sanitizeHTML et unescape)

Also available in: Atom PDF