Bug #61391
[Contenu d'une fiche] Problème de caractère des contenus personnalisés : apostrophe devient « ' »
0%
Description
Pour améliorer l'affichage de nos références, j'ai voulu utilisé l'affichage d'un contenu personnalisé genre {{ card.fields.client }} https://www.entrouvert.com/manage/pages/177/
L'affichage des apostrophes (') s'affiche ' https://www.entrouvert.com/references/client/57/
Fichiers
Demandes liées
Révisions associées
tests: add markup to string field value to check it's escaped properly (#61391)
Historique
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Lié à Development #58840: Améliorations sur la cellule « Contenu d'une fiche » ajouté
Mis à jour par Thomas Noël il y a environ 2 ans
- Sujet changé de [Contenu d'une fiche] Problème de caractère des contenus personnalisés à [Contenu d'une fiche] Problème de caractère des contenus personnalisés : apostrophe devient « ' »
Un peu bizarre... Il faut ajouter un « |safe », je l'ai fait pour {{ card.fields.client|safe }}
Mais je ne sais pas si c'est vraiment «normal», je m'étonne un peu de la présence du ' ... il faudrait sans doute creuser pour savoir d'où il vient.
Mis à jour par Anaïs Ecuvillon → en congés, retour le 30/04 il y a environ 2 ans
Thomas Noël a écrit :
Un peu bizarre... Il faut ajouter un « |safe », je l'ai fait pour {{ card.fields.client|safe }}
Ok en attendant j'ai ajouté |safe partout pour que ce soit propre pour nos futurs clients.
Et j'ai ajouté une cellule visible que par Debug EO pour que je puisse contrôler quand ce sera résolu.
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Fichier 0001-wcs-don-t-escape-result-of-card-cell-custom-fields-6.patch 0001-wcs-don-t-escape-result-of-card-cell-custom-fields-6.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
(on a fait la même manip sur chrono pour #55220)
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a environ 2 ans
Euh euh pas sûr là si un usager écrit <b>plop</b> dans wcs, ça va s'afficher comment ?
Mis à jour par Lauréline Guérin il y a environ 2 ans
ça ne sera pas interprété, on verra <b>plop</b>
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Fichier Capture d’écran de 2022-03-10 12-00-13.png Capture d’écran de 2022-03-10 12-00-13.png ajouté
mon test en local
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-tests-add-markup-to-string-field-value-to-check-it-s.patch 0001-tests-add-markup-to-string-field-value-to-check-it-s.patch ajouté
Je pense c'est ok mais qu'on gagnerait à ajouter au moins une ligne de commentaire aux tests,
assert PyQuery(result).find('.label').text() == 'Field E' assert PyQuery(result).find('.value p:first-child').text().strip() == 'lorem<strong>ipsum' - assert PyQuery(result).find('.value p:last-child').text().strip() == 'hello world' + assert PyQuery(result).find('.value p:last-child').text().strip() == "hello'world"
Au dessus de ça avoir
# check multiline text field is rendered with multiple paragraphs # (first line "lorem<strong>ipsum" and last line ("hello'world") # and the content is kept properly escaped.
~~
Par ailleurs après ton patch j'ajouterais bien du balisage dans la valeur de "fielda" (type string), pour ne pas avoir à mélanger à la fois la vérification "division en paragraphes" et la vérification "échappement correct". (attaché).
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Fichier 0002-tests-add-markup-to-string-field-value-to-check-it-s.patch 0002-tests-add-markup-to-string-field-value-to-check-it-s.patch ajouté
- Fichier 0001-wcs-don-t-escape-result-of-card-cell-custom-fields-6.patch 0001-wcs-don-t-escape-result-of-card-cell-custom-fields-6.patch ajouté
- Statut changé de Solution validée à Solution proposée
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
(je revalide sans lire)
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit abae3b1a702624c534a143a93ec001fe57aa6f15 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Mar 14 20:50:52 2022 +0100 tests: add markup to string field value to check it's escaped properly (#61391) commit ce69ea73d2bebbd430109a23f5e55545f2a1942c Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Thu Mar 10 10:50:46 2022 +0100 wcs: don't escape result of card cell custom fields (#61391)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
wcs: don't escape result of card cell custom fields (#61391)