A problem that was absent in Drupal 6 version

Thanks for the great work!

CommentFileSizeAuthor
#4 htmltrue-1245104-4.patch835 bytesfmitchell

Comments

Morn’s picture

Morn commented

I confirm this bug.

drupalmonkey’s picture

drupalmonkey commented

I have this bug as well.

W.M.’s picture

W.M. commented

Solution posted here http://drupal.org/node/1257618#comment-4897964 solves the issue perfectly!

The file link.module needs to be patched.

fmitchell’s picture

fmitchell commented
Status: Active » Needs review
StatusFileSize
new htmltrue-1245104-4.patch835 bytes

please see the attached patch

romenov’s picture

romenov commented

I'm not sure if this is the way to go .. I did some testing and to me it appears that the sanitize function it not required for the title field ..

When I enter in my field:

Link 1 & <test><strong>

The source becomes:

Link 1 &amp;amp; &lt;strong&gt;

When I remove the following function:
link.module @ line 409:

$title = filter_xss(token_replace($title, array('node' => $token_node)),
                        array('b', 'br', 'code', 'em', 'i', 'img', 'span', 'strong', 'sub', 'sup', 'tt', 'u'));

The output comes:

Link 1 &amp; &lt;test&gt;&lt;strong&gt;

Which seems correct to me, so the title data seems to be already cleaned up before it passes through the link_field_prepare_view field ..

drupalmonkey’s picture

drupalmonkey commented
Status: Needs review » Closed (fixed)

Geir19 is correct in #3, this is fixed in the other issue he links.

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented

thx 4 info, drupalmonkey. much appreciated! Awesome project by the way (museum)!

but please use fixed instead of closed (fixed).
-> http://drupal.org/node/156119

thanks 2 all