Project

General

Profile

Anomalie #2787

La compression HTML mange les retour ligne dans les formulaires

Added by nicolas - over 7 years ago. Updated over 6 years ago.

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

0%

Resolution:
fixed
Navigateur:

Description

En entrée dans un formulaire de forum :

Faisons un test.

- Premier item de liste.
- Deuxième item de liste.
- Troisième item de liste.

Un paragraphe.

Un autre.

Encore un autre.

En sortie :

<p>Faisons un test.</p><ul class="spip"><li> Premier item de liste.</li><li> Deuxième item de liste.</li><li> Troisième item de liste.
Un paragraphe.
Un autre.
Encore un autre.</li></ul>

Ce devrait à priori être ceci :

<p>Faisons un test.</p>
<ul class="spip"><li> Premier item de liste.</li><li> Deuxième item de liste.</li><li> Troisième item de liste.</li></ul>
<p>Un paragraphe.</p>
<p>Un autre.</p>
<p>Encore un autre.</p>

History

#1 Updated by cedric - over 7 years ago

  • Target version set to 90 a confirmer

je ne reproduis pas sur un site SPIP 3.0.3 conventionnel.

#2 Updated by nicolas - over 7 years ago

J'ai ça en base :

Faisons un test.
- Premier item de liste.
- Deuxième item de liste.
- Troisième item de liste.
Un paragraphe.
Un autre.
Encore un autre.

Les doubles retours chariot ont disparu.

J'utilise le plugin Comments.

#3 Updated by Ben . over 7 years ago

Coucou, Ne pas oublier d essayer de reproduire sur grml.eu avant de saisir un ticket

#4 Updated by nicolas - over 7 years ago

@Ben j'avoue que je n'y pense jamais

#5 Updated by cedric - over 7 years ago

je ne reproduis ni avec ni sans le plugin comments, ni avec ni sans auto_br activ.
Je pense que tu as autre chose qui pose problème sur ton site.

#6 Updated by nicolas - over 7 years ago

En désactivant tous les plugins, j'ai encore le problème, sachant que je n'ai pas de mes_options.php global ni de dossier "squelettes", tout est dans un plugin, que j'ai donc désactivé aussi.

J'ai bien les <p> dans la prévisualisation, mais les retours chariot doubles sont remplacés par un simple en base, et je me retrouve du coup avec des <br class="autobr"> au final.

Je suis sur SPIP 3.0.4 [19081] et ses extensions.

#7 Updated by cedric - over 7 years ago

On ne reproduit le problème nulle part. Il faut donc soupçonner une particularité du serveur, et pas d'autre solution que de debugguer sur place...
(var_dump sur le $_POST après prévisu, et voir à quel moment les doubles retour ligne disparaissent : lors du $_POST ? a l'insertion en base ?...)

#8 Updated by nicolas - over 7 years ago

J'ai exactement le même comportement en local sur un Mac avec MAMP et en prod chez AlwaysData, donc j'ai un doute que cela vienne d'une particularité du serveur.

Je vais creuser, je vous tiens au courant.

#9 Updated by nicolas - over 7 years ago

Je progresse un peu :

Mes doubles retour chariots sont déjà transformés en un seul dans le <input type="hidden" name="texte" value="…" /> du formulaire de prévisualisation alors que la prévisualisation en elle-même est bonne.

C'est donc la génération de ce champs caché qui vire un retour chariot sur deux.

#10 Updated by nicolas - over 7 years ago

Dans plugins-dist/forum/formulaires/forum.html, je place

<pre>#ENV{texte}</pre>
avant la ligne 24

J'obtiens ceci :

<pre>un exemple de commentaire

un autre

encore un
</pre>
<input type='hidden' name='texte' value="un exemple de commentaire
un autre
encore un
" />

Mes retours chariot sont perdus d'une ligne à l'autre…

J'ai envie de dire WTF!

#11 Updated by nicolas - over 7 years ago

OMG, J'AI TROUVÉ !!!

C'est la compression HTML qui se dit que deux retours chariots qui se suivent, ça ne sert à rien en HTML, autant n'en mettre qu'un seul.

Bin oui, mais faudrait éviter de toucher au contenu des <input> alors…

#12 Updated by cedric - over 7 years ago

  • Subject changed from Bug d'interprétation d'une liste suivie de paragraphes to La compression HTML mange les retour ligne dans les formulaires
  • Target version changed from 90 a confirmer to 3.0

ah oui ok, compris. Bien vu ! Je mets a jour le ticket donc

#13 Updated by nicolas - over 7 years ago

Super, merci !

#14 Updated by cedric - over 6 years ago

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

Corrigé (enfin !) par zone.spip.org/trac/spip-zone/changeset/72767

Also available in: Atom PDF