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:

Files: 
CommentFileSizeAuthor
#1 views_taxonomy_entity_uri-1806266-1.patch1.26 KBderhasi
PASSED: [[SimpleTest]]: [MySQL] 1,658 pass(es).
[ View ]

Comments

StatusFileSize
new1.26 KB
PASSED: [[SimpleTest]]: [MySQL] 1,658 pass(es).
[ View ]

And there is the patch.

Status:Active» Needs review

Status:Needs review» Reviewed & tested by the community

I was experiencing the issue in #1388684: Can't use [term:name] for a term path replacement and this patch fixed it for me.

+++ b/modules/taxonomy/views_handler_field_taxonomy.incundefined
@@ -65,9 +66,13 @@ class views_handler_field_taxonomy extends views_handler_field {
+      if (isset($uri['options'])) {
+        $this->options['alter'] = array_merge($this->options['alter'], $uri['options']);
+      }

This change feels odd, because we don't do that even in core, but yeah it seems to be better to do it.

I can report that this patch helped with my implementation of entity_path as well. Thanks!

This patch helped me too. Please merge it into master.

Status:Reviewed & tested by the community» Needs review

... need some response for #4 :(

Patch from #1 fixed the problem with entity_path for me too.

term fields support?

So you suggest to load the full taxonomy term?

#1 Patch works for me too

Issue summary:View changes
Status:Needs review» Reviewed & tested by the community

This patch applies cleanly, does what it announces, and fixes #1498560: Query pattern not used in taxonomy View for me.

Getttitin yah!