Instead of check_plain() we should use html_entity_decode(), as the patch does.

Patch against D6-3 and D7.

CommentFileSizeAuthor
d7-views-fix-tokens.patch784 bytesamitaibu
D6-views-fix-tokens.patch784 bytesamitaibu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Version: 6.x-3.x-dev » 7.x-3.x-dev

Settings correct version (although there's also a patch for D6).

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Version: 7.x-3.x-dev » 6.x-3.x-dev

Setting back to 6.x

merlinofchaos should have a look at it. I'm not sure whether he looks at 7.x issues

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

A related issue: #559056: Fields with apostrophes not displaying correctly

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented

Bump.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Code looks fine. Interesting stuff to read on php.net

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Seems to be okay. Committed to all branches.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.