Ticket #1974 (closed defect: fixed)
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: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="/public/.image_m.jpg" alt="image.jpg" title="image.jpg, juin 2014" />" title="image.jpg" width="1200" height="290" /></p>
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
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: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.