Apostrophes, ampersands and less-than/greater-than signs in banner title and description display as HTML code rather than the symbol.

I found an explanation in a Google search, though I am not familiar enough to determine its validity:
Source: http://remmirath-en.blogspot.com/2011/09/marinelli-3-drupal-7-html-entit...
"!variable: Inserted as is. Use this for text that has already been sanitized.
@variable: Escaped to HTML using check_plain(). Use this for anything displayed on a page on the site.
The generation of banner title and description string is there made with the @ parameter. Unfortunately somewhere else in the code a further check_plain() call is made on the same strings. This second application of check_plain() function simply converts the symbols into HTML entities and this is the cause of the issue."

This link provides a fix as well that, again due to my lack of familiarity with the code, I have not tried myself:
"My solution was to replace the @ parameter with the ! one in t() function calls into marinelli_banner_markups()
This function in marinelli/logics/banners.inc
Here you are the code snippets:
'alt' => t('!image_desc', array('!image_desc'=>$banner['image_description'])),
'title' => t('!image_title', array('!image_title'=>$banner['image_title'])),
...
'longdesc' => t('!image_desc', array('!image_desc'=>$banner['image_description']))
"

Comments

hatcat’s picture

I'm having the same issue with the titles of articles from my feeds. Thanks for the info.

jonboykayaks’s picture

Thanks that fix on the website worked for me!

jonboykayaks’s picture

Status: Active » Needs review
jonboykayaks’s picture

Status: Needs review » Active
jeramy’s picture

I have now tested the fix as well and it does seem to correct the issue.

TheRobBrennan’s picture

Thank you! This worked for me, too.

Dade916’s picture

Thanks, the fix worked for me too.

mgifford’s picture

Issue tags: +longdesc

Could someone develop a proper patch for this?