Currently term_view.inc does not execute the normal Drupal hooks, e.g. hook_taxonomy_term_view and hook_entity_view. In order for several other modules to work correctly, e.g. Metatag, these hooks must be executed.

Note: this will depend upon #1067120: Missing hook_taxonomy_term_view() or hook_entity_view() when viewing a term.

CommentFileSizeAuthor
#3 ctools-n1760584-3-d7.patch3.84 KBdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Related:

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Related Metatag issue: #1363476: Panels integration - ensure meta tags work OOTB on entity pages

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new ctools-n1760584-3-d7.patch3.84 KB

This patch is a little more involved than the other patches for node_view and user_view because we need to avoid loading all of the nodes taxonomy_term_page() usually loads until it's certain that the page processing is not being handled by a CTools or Page Manager plugin.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Note: that patch depends upon #1067120.

damienmckenna’s picture

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

This needs some additional work, it should also trigger the page title handling.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Never mind, I think I found the solution for the page titles in #1732538: Page title pattern ignored. Pushing this back for review.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review
brunorios1’s picture

brunorios1 commented
Status: Needs review » Reviewed & tested by the community

i tested and it worked in D7.

chrisroditis’s picture

chrisroditis commented

Working fine here too

liquidcms’s picture

liquidcms commented

and this patch works also (along with core patch due in D7.17)

jenlampton’s picture

jenlampton
she / her
commented

Patch also working for me.

merlinofchaos’s picture

merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Lots of testing on this (hooray, I don't see a lot of that!), so committed and pushed after a cursory review.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Referenced #1067120.