Posted by DamienMcKenna on September 3, 2009 at 5:28pm
3 followers
| Project: | Taxonomy Manager |
| Version: | 6.x-2.0 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Taxonomy_Manager is most useful for its management of taxonomy terms, most users would probably be more likely to use Views or Panels to override the default taxonomy/term/%term display, so it would be best if taxonomy_manager_disable_merge_redirect was set to TRUE by default. Alternatively, restructure it so that instead of disabling the merge_redirect value you're enabling it, i.e. the variable would become taxonomy_manager_enable_merge_redirect and set to FALSE by default.
Comments
#1
Here's a patch that just simply sets the default value to TRUE.
#2
Honestly a better option would be to rewrite this structure to use path_redirect when available rather than taking over the entire taxonomy/term/%term system just to handle legacy redirects..
#3
I agree, that we should disable the redirect by default or implement it differently with path_redirect
#4
As a first step to improve the compatibility with other modules, I committed your patch to disable the redirect by default. Thanks for that patch. So marking this issue as fixed.
But, as you said, there might be other options to implement this in a better way. So I'm opening another issue, where we can work on patch to improve the system. (see #569086: Improve redirect mechanism for merged terms)
#5
Automatically closed -- issue fixed for 2 weeks with no activity.