Using a Drupal 7.8 install on Mamp. Special characters, such as ampersand, are double encoded in page titles as well as Open Graph Protocol titles when Meta tags module is enabled. Disabling the module solves the problem. Possibly relevant (?): I tried the separate Page title module and the titles are properly encoded.

CommentFileSizeAuthor
#1 1284684-metatag-title-encoding.patch809 bytesdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Version: 7.x-1.0-alpha2 »
Status: Active » Needs review
StatusFileSize
new 1284684-metatag-title-encoding.patch809 bytes

Please test the following patch.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Nevermind I found that this was a little higher up: please test the latest development releases of the Metatag *and* Token modules in a day or so once they regenerate.

Vlad Stratulat’s picture

Vlad Stratulat commented

Thank you Dave. It's working!

marikomay’s picture

marikomay commented

Works for me. Thank you!

Status: Fixed » Closed (fixed)

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

rickdrup’s picture

rickdrup commented

Is there was any reason why this fix wasn't carried over into current (beta9) version of Metatag? I seem to be having the same problem as described above with beta9 verion of metatag and when I apply this patch to the current code, the issue appears to be fixed.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: » 7.x-1.x-dev
Issue summary: View changes

@rickdrup: The code has changed quite a bit in three years, please follow #2202031: Metatag and html tags encoding.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Closed (fixed) » Active

I'm sorry to tell you that this issue is open again in the latest 7.x-1.x-dev release!
The current stable release works.

The page title "Über uns" (German "Ü") seems to be double-encoded. Downgrade fixes it!

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Closed (fixed)

I've reproduced the problem and am handling it over in #2564483: Cannot translate metatags after last update anymore.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

FYI despite the fact that I had tests to make sure it was working correctly, the tests were not checking the output of double-encoded strings coming through tokens. Doh. So I've fixed that.