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.

Files: 
CommentFileSizeAuthor
#3 ctools-n1760584-3-d7.patch3.84 KBDamienMcKenna
PASSED: [[SimpleTest]]: [MySQL] 67 pass(es).
[ View ]

Comments

DamienMcKenna’s picture

DamienMcKenna’s picture

Status:Active» Needs review
StatusFileSize
new3.84 KB
PASSED: [[SimpleTest]]: [MySQL] 67 pass(es).
[ View ]

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

Note: that patch depends upon #1067120.

DamienMcKenna’s picture

Status:Needs review» Needs work

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

DamienMcKenna’s picture

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

Status:Needs work» Needs review
brunorios1’s picture

Status:Needs review» Reviewed & tested by the community

i tested and it worked in D7.

chrisroditis’s picture

Working fine here too

liquidcms’s picture

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

jenlampton’s picture

Patch also working for me.

merlinofchaos’s picture

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

Issue summary:View changes

Referenced #1067120.