The fix for 2180031 changed the value of the 'sanitize' option to that is used by tokens to do token replacements during metatag processing. In our use case, we need the [node:summary] and [node:description] tokens to pass through _text_sanitize() in order to invoke text filter processing. My fix is to convert the sanitize default value to a setting. But, does the really need to be a per-metatag setting?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dnmurray created an issue. See original summary.

dnmurray’s picture

DamienMcKenna’s picture

Status: Active » Needs review
metalbote’s picture

Thanks for this patch.

I encountered the problem with email adresses in [node:summary], which were formerly obfuscated by spamspan and now were displayed as plain in metatag description...

Worked as aspected and should be integrated as option as soon as possible, because many won't notice the side effects of fix 2180031 on customer pages.

DamienMcKenna’s picture

Status: Needs review » Fixed
Parent issue: » #2699045: Plan for Metatag 7.x-1.15 release

This seems to be useful for certain circumstances and doesn't break anything, so ok, cool, thanks!

Status: Fixed » Closed (fixed)

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