Hi!

Just wondering if it would be a good idea for taxonomy terms overview to consider the "uri callback" if altered e.g. forum.module altering the 'uri callback'. At the moment, I see links with a hardcoded taxonomy/term/$term->tid as the "href" in taxonomy_overview_terms.

Comments

javi-er’s picture

javi-er commented

I took a look to what you mention and the problem seems to be that the uri callback doesn't take in account url alias.

For instance if you replace the case you mention with the path retrieved from $term->uri() it will ignore the url alias (if any) defined for this term.

A test case for this is to enable forum module, go to admin/structure/taxonomy/forums edit the first term and add an alias.

As it is now, it will show the alias you set, if you change taxonomy_overview_terms to use the uri callback, the alias will be ignored and the view path for the term will be forum/[tid] .

I'm not sure which one is the right path, since forum/[tid] is defined by forum module with hook_entity_info_alter().

So I think the first question to make is which path to use for terms, the alias or the uri's.

dsdeiz’s picture

dsdeiz commented

Yeah, I forgot to mention that I tested this with the forum.module. If you view a forum node, the field for terms/forums seem to respect the "uri callback". But in the taxonomy overview page, the link points to taxonomy/term/[tid].

A test case for this is to enable forum module, go to admin/structure/taxonomy/forums edit the first term and add an alias.

This is probably because the alias was created for taxonomy/term/[tid] instead of forum/[tid].

I guess it's not really a big deal. It's nice that there's at least a specific page to show nodes in terms.

poker10’s picture

poker10 commented
Issue summary: View changes
Related issues: +#928188: taxonomy.admin.inc - is hardcoding taxonomy path

This seems to be a duplicate of: #928188: taxonomy.admin.inc - is hardcoding taxonomy path

poker10’s picture

poker10 commented
Status: Active » Closed (duplicate)