Project

General

Profile

Anomalie #2813

Ne pas manger les commentaires courts HTML

Added by Maïeul Rouquette about 8 years ago. Updated over 6 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
08/06/2012
Due date:
% Done:

0%

Resolution:
fixed
Navigateur:

Description

La fonction de compression HTML supprime tout les commentaires, sauf

<!--extra-->
.

Une pipeline pour définir d'autres exceptions pourrait être utile : c'est le cas notamment pour le plugin jeux. CF http://contrib.spip.net/Jeux-pour-SPIP-3#forum459955


Related issues

Related to Compresseur - Evolution #3076: Retirer la fonctionnalité de compression HTML Fermé 10/24/2013

History

#1 Updated by marcimat 🌻 about 8 years ago

Oué, c'est moi qui est ajouté cette exception déjà. J'avais eu des difficultés à comprendre le problème d'ailleurs :)

Tu crois que c'est réellement utile de mettre des exceptions et un pipeline ? ou peut être faut il ne pas modifier les commentaires html et les laisser tous ?

#2 Updated by b b about 8 years ago

Salut, je recommanderais plutôt de ne pas utiliser la compression HTML car celle-ci pose souvent problème. J'ai dernièrement rencontré un souci avec un plugin qui n'insérait pas le contenu prévu dans les pages du site car le preg_replace qui devait le faire ne matchait pas l'expression cherché à cause des sauts de lignes générés dans le source par la compression HTML.

Donc, à mon avis, plutôt que d'ajouter des exceptions autant éviter d'utiliser cette fonctionnalité.

#3 Updated by cedric - about 8 years ago

  • Subject changed from proposer une pipeline pour gérer des exceptions au commentaires HTML to Ne pas manger les commentaires courts HTML
  • Target version changed from 99 plus tard to 3.1

non, non faut surtout pas qu'on commence à ajouter un pipeline la-dessus.
Plutot modifier le compresseur html à la marge pour qu'il ne touche pas aux commentaires courts en un mot par exemple...

#4 Updated by b b about 8 years ago

Pour info, encore un bug généré par la compression HTML :

http://contrib.spip.net/GIS-4#forum461654

Dans ce cas la compression HTML s'applique sur le JSON généré par un squelette, ce qui rend le JSON invalide et bloque le fonctionnement du plugin qui l'utilise.

#5 Updated by cedric - over 7 years ago

bon on s'epargenerait de la maintenance en supprimant la compression HTML qui est quasi inutile quand on utilise gzip...

#6 Updated by Maïeul Rouquette over 7 years ago

je serais aussi d'avis de ne plus proposer cette fonctionalité...

#7 Updated by Maïeul Rouquette over 7 years ago

je ne sais pas, il me semble que le vrai problème est la syntaxe de de type de commentaire "pour insertion". On pourrait imaginer une syntaxe unifiée pour éviter les pb. Parce que certains commentaires ont vraiment vocation à disparaître (ceux qui … commentent)

#8 Updated by b b about 7 years ago

Voir aussi cet autre bug généré par la compression HTML #2787

#9 Updated by b b almost 7 years ago

Et hop, un ticket spécifique pour la proposition de retrait de cette fontionnalité #3076.

#10 Updated by b b almost 7 years ago

Je pense qu'on peut fermer le ticket puisque la fonctionnalité de compression du HTML n'est plus dispo, cf #3076.

#11 Updated by b b over 6 years ago

  • Status changed from Nouveau to Fermé
  • Resolution set to fixed

Pas de nouvelles, on ferme.

Also available in: Atom PDF