Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#3 | ctools-n1760584-3-d7.patch | 3.84 KB | DamienMcKenna |
Comments
Comment #1
DamienMcKennaRelated:
Comment #2
DamienMcKennaRelated Metatag issue: #1363476: Panels integration - ensure meta tags work OOTB on entity pages
Comment #3
DamienMcKennaThis 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.
Comment #4
DamienMcKennaNote: that patch depends upon #1067120.
Comment #5
DamienMcKennaThis needs some additional work, it should also trigger the page title handling.
Comment #6
DamienMcKennaNever mind, I think I found the solution for the page titles in #1732538: Page title pattern ignored. Pushing this back for review.
Comment #7
DamienMcKennaComment #8
brunorios1 CreditAttribution: brunorios1 commentedi tested and it worked in D7.
Comment #9
chrisroditis CreditAttribution: chrisroditis commentedWorking fine here too
Comment #10
liquidcms CreditAttribution: liquidcms commentedand this patch works also (along with core patch due in D7.17)
Comment #11
jenlamptonPatch also working for me.
Comment #12
merlinofchaos CreditAttribution: merlinofchaos commentedLots of testing on this (hooray, I don't see a lot of that!), so committed and pushed after a cursory review.
Comment #13.0
(not verified) CreditAttribution: commentedReferenced #1067120.