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.
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.
Comment | File | Size | Author |
---|---|---|---|
#1 | taxonomy_manager-n567564.patch | 2.25 KB | DamienMcKenna |
Comments
Comment #1
DamienMcKennaHere's a patch that just simply sets the default value to TRUE.
Comment #2
DamienMcKennaHonestly 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..
Comment #3
mh86 CreditAttribution: mh86 commentedI agree, that we should disable the redirect by default or implement it differently with path_redirect
Comment #4
mh86 CreditAttribution: mh86 commentedAs 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)