developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 8 years ago

Closed 6 years ago

#1099 closed enhancement (wontfix)

Enlever htmlspecialchars() dans jMessage

Reported by: nesswaw Owned by:
Priority: low Milestone:
Component: jelix:plugins:tpl Version: 1.1.6
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Bonjour,

Simplement parce que si on veut mettre un lien dans un jMessage, le lien sera échappé à cause de htmlspecialchars().

Je propose de l'enlever, étant donné que c'est le développeur qui gère l'affichage des jMessage, si il a besoin d'échapper une variable, il le fera lui même.

???

Merci

Change History (2)

comment:1 Changed 8 years ago by laurentj

  • Component changed from jelix:utils to jelix:plugins:tpl
  • Priority changed from normal to low
  • Severity changed from normal to minor

Un framework, c'est fait pour simplifier la vie. Contraindre le développeur à penser à échapper lui même tout ses messages, ce n'est pas lui simplifier la vie. Surtout que dans 90% des cas, il ne va même pas y penser parce que pour beaucoup, ils ne savent même pas à quoi ça sert d'échapper. résultat : du code html invalide en bout de chaine.

Sans parler que ton problème ne se situe pas dans jMessage, mais dans le plugin de template pour afficher son contenu (jMessage ne sert qu'à stocker le message). Donc si tu as vraiment un souci pour afficher les messages comme tu le souhaites : fait ton plugin.

bref, hors de question de supprimer le htmlspecialchars.

On peut éventuellement améliorer jMessage pour indiquer le type de contenu que l'on stocke dans jMessage (parametre supplementaire pour dire si c'est du texte ou du html), et donc améliorer le plugin jmessage

comment:2 Changed 6 years ago by laurentj

  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.