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.
Right now, the term context isn't working in site_template, and what I can see it's because the task uses the old context "term" instead of "entity:taxonomy_term".
I've changed all context to the new entity context and created a patch.
Comment | File | Size | Author |
---|---|---|---|
#4 | use-entity-context-1550294-4.patch | 1.48 KB | logaritmisk |
#2 | use-entity-context-1550294-2.patch | 614 bytes | logaritmisk |
#1 | use-entity-context-1550294-1.patch | 3.46 KB | logaritmisk |
Comments
Comment #1
logaritmisk CreditAttribution: logaritmisk commentedAnd the patch.
Comment #2
logaritmisk CreditAttribution: logaritmisk commentedNew patch which only fixes this bug, we should maybe create a new issue to update panels_everywhere to the new context standard.
Comment #3
Letharion CreditAttribution: Letharion commentedMerlin, I think this patch makes sense, and I'm looking over logaritmisk's shoulder and can see that it works, but strangely enough I can't reproduce this problem. Can you take a look? :)
Comment #4
logaritmisk CreditAttribution: logaritmisk commentedChanged title and fix the rest of the term context in site_template.inc.
Comment #5
merlinofchaos CreditAttribution: merlinofchaos commentedHm. This should probably actually be fixed in CTools so that the 'term' context ends up created as the proper context instead. I thought I had fixed that. Or maybe I wasn't able to. It's annoying that we're still having issues like this, though.
I know that the 'node' context has a special line in it, 'superceded by' => 'entity:node' that makes all 'node' contexts turn into 'entity:node' contexts.
I wonder why term contexts don't have this.
Comment #6
IRuslan CreditAttribution: IRuslan commentedWe still have this problem actual.
That patch fix it from panels_everywhere side.
It works for node, because we have entity type name as one of context's type. And for taxonomy_term it doesn't work, as keyword and entity type name don't match.
Comment #7
B-Prod CreditAttribution: B-Prod commentedThe patch in #4 fixes the issue.
Without the patch, no Taxonomy term context is available from default site template (for example in the relationship list, there is nohing related to a term entity).
With the patch, those expected contexts show up!
Comment #8
DamienMcKennaCommitted. Thanks for the patch, and patience.
Comment #12
DamienMcKennaI've (reverted and) committed this one again in order to fix the authorship.