Dotclear

Ticket #1974 (closed defect: fixed)

Opened 11 years ago

Last modified 11 years ago

Wiki image with link insertion broken

Reported by: philippe Owned by: nikrou
Priority: high Milestone: 2.7
Component: module:clearbricks Version: dev
Severity: major Keywords:
Cc:

Description (last modified by philippe) (diff)

If you insert an image with a link to original image using wiki syntax, the wiki code is correct but the link gets deleted on saving post

[((url|texte alternatif))|url]

produces

<p><img src="url" alt="texte alternatif" /></p>

<a href="url"></a> is missing

Change History

comment:1 Changed 11 years ago by philippe

  • Description modified (diff)

comment:2 Changed 11 years ago by philippe

Sorry, I made a mistake in ticket description. This code in wiki syntax :

[((/public/.image_m.jpg|image.jpg||image.jpg, juin 2014))|/public/image.jpg||image.jpg]

produces the following html

<p><img src="/public/image.jpg" alt="<img src=&quot;/public/.image_m.jpg&quot; alt=&quot;image.jpg&quot; title=&quot;image.jpg, juin 2014&quot; />" title="image.jpg" width="1200" height="290" /></p>
Last edited 11 years ago by philippe (previous) (diff)

comment:3 Changed 11 years ago by philippe

Si je modifie dotclear/inc/libs/clearbricks/text.wiki2xhtml sur la ligne 168 pour désactiver le paramètre active_auto_img destiné à la fonction parseLink le problème disparaît.

Dans parseLink ce paramètre semble destiné à ajouter la dimension d'une image si elle est contenue dans un lien. Comme la syntaxe wiki n'a jamais, à ma connaissance, ajouté les dimensions des images contenues dans les liens, j'en déduis peut-être à tort que ce bout de code (lignes 892 à 913) n'était jamais utilisé auparavant par Dotclear, et qu'il l'est maintenant ?

Je ne sais pas si ma solution casse autre chose, c'est très probable :p

À vous les sorciers

Last edited 11 years ago by philippe (previous) (diff)

comment:4 Changed 11 years ago by franck

Je ne pense vraiment pas que ça vienne de ça, vu qu'on a touché à rien de ce côté depuis des lustres.

Par contre, j'ai ajouté il y a peu 'em' et 'strong' à la liste à la ligne 863, afin de permettre ces balises dans les liens (un ticket de Julien Mudry il me semble), et visiblement il est plus que possible que ça foute le dawa !

À tester si quelqu'un a un peu de temps pour ça ;-)

comment:5 Changed 11 years ago by philippe

Oui, Franck, et j'avais d'ailleurs essayé, en voyant les différences entre les deux fichiers, de supprimer em et strong des balises dans les liens, mais ça ne corrige rien.

C'est pourquoi j'ai suivi la piste de active_auto_img, car dans tous les cas la fonction initWikiPost de dotclear/inc/core/class.dc.core.php initialise ce paramètre à 0 ligne 196. Ce qui pourrait prouver que ce paramètre n'est plus passé correctement à CB, non ?

comment:6 Changed 11 years ago by franck

Ah mais dans ce cas, il va falloir chercher côté DC et pas côté Clearbricks (cela dit, doit aussi y'avoir un blème côté CB avec ce flag positionné à 1).

comment:7 Changed 11 years ago by philippe

Je ne trouve pas de raison à cela dans DC, ce n'est pas faute d'avoir cherché, mais peut-être qu'un sorcier y verra plus clair.

À mon niveau, comme on ne se sert pas de cette fonction dans DC, forcer le paramètre à 0 dans CB résout bêtement le souci, mais je comprends que tu cherches à comprendre :D

comment:8 Changed 11 years ago by philippe

  • Severity changed from normal to major
  • Milestone changed from A definir to 2.7

comment:9 Changed 11 years ago by franck

Peut-être lié au ticket #1987 ?

comment:10 Changed 11 years ago by franck

  • Owner changed from team to nikrou

comment:11 Changed 11 years ago by franck

Je pense que ce problème est un effet pervers du ticket ouvert par prospere à propos de l'instanciation de wiki2xhtml dans dcLegacyEditor, instanciation qui n'était pas suivie des réglages idoines des options de wiki2xhtml, comme c'est normalement fait dans initWikiPost().

Régler le ticket #1987 devrait au passage régler celui-ci.

comment:12 Changed 11 years ago by franck

  • Status changed from new to closed
  • Resolution set to fixed

Comme prévu (et après tests), régler le ticket #1987 a bien résolu celui-ci, je le ferme.

Note: See TracTickets for help on using tickets.

Sites map