Bonjour
Dans le remote.xml, les entités < et > sont doublencodées — elles apparaissent en < et >.
Il faudrait que le & soit doublencodé de la même manière, à savoir & au lieu de, actuellement, &.
Ainsi, la déxmlentitisation d'un message produirait de manière simple un xml valide, alors qu'actuellement le xml produit de cette manière ne l'est pas, vu qu'il y a un & hors entité seul dans le document.
Par exemple, au post suivant:<b> bold </b> < &
Le remote.xml contient: <b> bold </b> &lt; &
Alors qu'il serait mieux qu'il contienne:<b> bold </b> &lt; &amp;
En vous remerciant pour votre attention, je vous prie d'agréer, cher développeur, l'expression de mes plopifications confraternelles.
# pull request
Posté par hermenegilde . Évalué à 3 (+0/-0).
Alors je proposerais bien un pull request pour virer le gsub dans la ligne idoine mais je me dis que s'il est là, à cet endroit, doit y avoir une raison.
Certainement pour les & dans les url, ce qui devrait être normalement supporté par les brouteurs divers et variés.
[^] # Commentaire supprimé
Posté par Anonyme . Évalué à -2 (+0/-0).
Ce commentaire a été supprimé par l’équipe de modération.
[^] # Re: pull request
Posté par Pierre Tramonson . Évalué à 4 (+0/-0).
Elle a ptete été recodée à l'arrache, mais elle présente de prime abord moins de trous dans le slip que ses anciennes versions.
[^] # Re: pull request
Posté par Bruno Michel (site web personnel) . Évalué à 4 (+0/-0).
Je n'arrive pas à retrouver pourquoi le gsub est là. Je l'ai enlevé (cf https://github.com/nono/linuxfr.org/commit/8f5f1a99788ca1c83730440fbc8ea3348eb29038) et on va voir ce que cela donne. Ça doit corriger cette entrée de suivi et si ça pose des problèmes, j'aviserais à ce moment-là.
[^] # Re: pull request
Posté par hermenegilde . Évalué à 2 (+0/-0).
Super, merci!
Envoyer un commentaire
Suivre le flux des commentaires
Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.