Projet

Général

Profil

Bug #61391

[Contenu d'une fiche] Problème de caractère des contenus personnalisés : apostrophe devient « ' »

Ajouté par Anaïs Ecuvillon → en congés, retour le 30/04 il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
03 février 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non
Tags:

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

Lié à Publik - Development #58840: Améliorations sur la cellule « Contenu d'une fiche »Fermé

Actions

Révisions associées

Révision ce69ea73 (diff)
Ajouté par Lauréline Guérin il y a environ 2 ans

wcs: don't escape result of card cell custom fields (#61391)

Révision abae3b1a (diff)
Ajouté par Frédéric Péters il y a environ 2 ans

tests: add markup to string field value to check it's escaped properly (#61391)

Historique

#2

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é
#3

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.

#4

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.

#5

Mis à jour par Lauréline Guérin il y a environ 2 ans

  • Assigné à mis à Lauréline Guérin
#6

Mis à jour par Lauréline Guérin il y a environ 2 ans

(on a fait la même manip sur chrono pour #55220)

#7

Mis à jour par Thomas Noël il y a environ 2 ans

  • Statut changé de Solution proposée à Solution validée
#8

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 ?

#9

Mis à jour par Lauréline Guérin il y a environ 2 ans

ça ne sera pas interprété, on verra <b>plop</b>

#11

Mis à jour par Lauréline Guérin il y a environ 2 ans

Fred, ok pour toi ?

#12

Mis à jour par Frédéric Péters il y a environ 2 ans

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é).

#14

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)

#15

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)
#16

Mis à jour par Transition automatique il y a environ 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#17

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF