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.

CommentFileSizeAuthor
#1 taxonomy_manager-n567564.patch2.25 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

Status: Active » Needs review
FileSize
2.25 KB

Here's a patch that just simply sets the default value to TRUE.

DamienMcKenna’s picture

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..

mh86’s picture

I agree, that we should disable the redirect by default or implement it differently with path_redirect

mh86’s picture

Category: task » bug
Status: Needs review » Fixed

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)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.