Metatag tokens for Context variables were not being substituted for the correct value. e.g. %movie:title

This patch substitutes matching tokens with the Context variable.

CommentFileSizeAuthor
#8 metatag-substitute-panels-context-variables-2408211-8.patch691 bytesMatthewHager
#7 metatag-substitute-panels-context-variables-2408211-7.patch707 bytesMatthewHager
#5 metatag_panels-substitute-context-variables-2408211-5.patch656 bytesinfinet
#1 metatag_panels-substitue-context-variables-2408211-2.patch668 bytesinfinet

Comments

infinet’s picture

infinet commented
StatusFileSize
new metatag_panels-substitue-context-variables-2408211-2.patch668 bytes

Patch attached

infinet’s picture

infinet commented

Patch attached

infinet’s picture

infinet commented
infinet’s picture

infinet commented
Status: Active » Needs review
infinet’s picture

infinet commented
StatusFileSize
new metatag_panels-substitute-context-variables-2408211-5.patch656 bytes

Removed unnecessary add_slashes()

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Project: Meta tags: Panels » Metatag
Component: Code » Panels integration

Please note that the separate Metatag Panels module has been deprecated in favor of the submodule that's bundled with Metatag.

MatthewHager’s picture

MatthewHager commented
StatusFileSize
new metatag-substitute-panels-context-variables-2408211-7.patch707 bytes

I've rolled a patch against latest dev in the submodule that is included with Metatag.

MatthewHager’s picture

MatthewHager commented
StatusFileSize
new metatag-substitute-panels-context-variables-2408211-8.patch691 bytes

New patch. Previous patch only worked if substitution happened at the beginning of the string.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed. Thanks!

  • DamienMcKenna committed aa7637b on 7.x-1.x authored by infinet 
    Issue #2408211 by infinet, MatthewHager: Context substitution added to...

Status: Fixed » Closed (fixed)

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