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.
As introduced in #897694: taxonomy_term_path not working drupal 7, render_link for taxonomy term fields is using entity_uri(). But there is the 'options' part missing in the current implementation.
Additionally it would help, for some entity_uri implementations (e.g. entity_path), if there was more information on the term, and not only the pseudo term object. For most purposes the term name would be fine. So I'd propose to add the name column to the field too. (see #1388684: Can't use [term:name] for a term path replacement).
I'll post a patch for that in the first comment.
May be related to:
Comment | File | Size | Author |
---|---|---|---|
#14 | views_taxonomy_entity_uri-1806266-1.patch | 1.26 KB | geek-merlin |
| |||
#1 | views_taxonomy_entity_uri-1806266-1.patch | 1.26 KB | derhasi |
Comments
Comment #1
derhasi CreditAttribution: derhasi commentedAnd there is the patch.
Comment #2
derhasi CreditAttribution: derhasi commentedComment #3
BWPanda CreditAttribution: BWPanda commentedI was experiencing the issue in #1388684: Can't use [term:name] for a term path replacement and this patch fixed it for me.
Comment #4
dawehnerThis change feels odd, because we don't do that even in core, but yeah it seems to be better to do it.
Comment #5
jordan8037310 CreditAttribution: jordan8037310 commentedI can report that this patch helped with my implementation of entity_path as well. Thanks!
Comment #6
siefca CreditAttribution: siefca commentedThis patch helped me too. Please merge it into master.
Comment #7
dawehner... need some response for #4 :(
Comment #8
ZoeN CreditAttribution: ZoeN commentedPatch from #1 fixed the problem with entity_path for me too.
Comment #9
divined CreditAttribution: divined commentedterm fields support?
Comment #10
dawehnerSo you suggest to load the full taxonomy term?
Comment #11
ashii CreditAttribution: ashii commented#1 Patch works for me too
Comment #12
geek-merlinThis patch applies cleanly, does what it announces, and fixes #1498560: Query pattern not used in taxonomy View for me.
Getttitin yah!
Comment #13
colanWe've recently switched our testing from the old qa.drupal.org to DrupalCI. Because of a bug in the new system, #2623840: Views (D7) patches not being tested, older patches must be re-uploaded. On re-uploading the patch, please set the status to "Needs Review" so that the test bot will add it to its queue.
If all tests pass, change the Status back to "Reviewed & tested by the community". We'll most likely commit the patch immediately without having to go through another round of peer review.
We apologize for the trouble, and appreciate your patience.
Comment #14
geek-merlinComment #16
dawehnerThank you all!