Hello, metatag does not show a title if it is overwritten in a contextual filter. I tried [current-page:title] and [view:title]... Neither of the tokens works. Any Ideas?

CommentFileSizeAuthor
#7 metatag-2908527.patch1.83 KBlapek
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SchieneIII created an issue. See original summary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Component: Integration with other module » Views integration
Issue tags: +Needs tests

Yeah, this seems like a bug, sorry about that.

alanyong’s picture

alanyong CreditAttribution: alanyong commented

Is this issue fixed ? I updated latest version 8.x-1.5 and seems still got the same issue. For now I have to create multiple views filter with categories to get page title. It's ain't convenience at all.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

This problem won't be fixed until this issue is marked "Fixed". Right now nobody has worked on it, nobody has uploaded a patch to fix the problem, so it hasn't been fixed yet.

alanyong’s picture

alanyong CreditAttribution: alanyong commented
Status: Active » Needs work

I updated Drupal version to 8.5.6 and it seems worked. I can anyone have this problem can try to update Drupal version and test [view:title]. But I'm not quite sure this is the exact solution to fix this bug.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Active

FYI the status "needs work" is for when there's a patch available but the patch needs further work, it isn't a general statement for saying "someone needs to work on this".

lapek’s picture

lapek CreditAttribution: lapek commented
Status: Active » Needs work
FileSize
metatag-2908527.patch1.83 KB

This seems to work, but could possibly be done in a better way?

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review

Thanks for putting together the patch. Lets have the testbot give it a try.

DrColossos’s picture

DrColossos CreditAttribution: DrColossos at CYLEDGE commented
Status: Needs review » Reviewed & tested by the community

we are using this and it works as expected

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: » #3078078: Plan for Metatag 8.x-1.11
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs tests

Committed. Thanks.

  • DamienMcKenna committed fc4a77a on 8.x-1.x authored by lapek 
    Issue #2908527 by lapek, DamienMcKenna, christian.rolf, DrColossos:...

Status: Fixed » Closed (fixed)

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