incompatibiliy with panel3 beta2 by providing a taxonomy/term/%

bennos - May 21, 2009 - 13:30
Project:Taxonomy Manager
Version:6.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

the new delegator module checks, that no other module has overwritten the taxonomy/term/% path.

http://drupal.org/node/469124

#1

mh86 - July 16, 2009 - 13:41
Status:active» fixed

I added a setting to the Taxonomy Manager to disable the overwriting of the taxonomy/term/% path by the Taxonomy Manager. The hook_menu_alter is used to redirect terms selected in the merging feature (which get deleted afterwards) to the resulting merged term, so this redirect feature is not essential for the Taxonomy Manager and users can now decide whether to active it or not.

#2

electricmonk - July 21, 2009 - 10:14

has this been committed to CVS?

#3

crea - July 22, 2009 - 22:10
Status:fixed» active

Latest dev snapshot is dated 2009-Jun-11 so it seems it's NOT in CVS (or in wrong branch)

#4

crea - July 22, 2009 - 22:36

#5

mh86 - July 23, 2009 - 07:27

As crea pointed out, this has been committed to the new Drupal-6--2 branch, a dev snapshot is now available on the project page

#6

mh86 - July 24, 2009 - 07:32
Status:active» fixed

marking issue as fixed, because code is committed to 6---2 branch and next release will contain this fix

#7

corona ronin - July 26, 2009 - 00:01

Hey I added the new 6-2x-dev version and I don't see the switch to turn off the overriding of taxonomy/%term/ page on the taxonomy manager module admin page. Can you please tell me where the menu item is?

Thanks!

#8

mh86 - July 26, 2009 - 09:56

you should find the setting 'Disable redirect of the taxonomy term page to merged terms' under Administer > Site Configuration > Taxonomy Manager

#9

corona ronin - July 26, 2009 - 17:40

Okay, found it, thanks...I didn't notice that secondary management menu...but it disables the redirect of merged terms...but it doesen't seem to disable the override of the taxonomy/%term page, which I have running through panels. I want to use taxonomy manager just to manage my taxonomy terms in an efficient fashion, I don't want it to have anything to do with my taxonomy/%term url links. How do I prevent it from overriding that page?

Thanks!

#10

mh86 - July 27, 2009 - 08:16

it definitely disables the override of the term page, code looks like this

/**
* Implementation of hook_menu_alter
*/
function taxonomy_manager_menu_alter(&$callbacks) {
  if (!variable_get('taxonomy_manager_disable_merge_redirect', 0)) {
    $callbacks['taxonomy/term/%']['page callback'] = 'taxonomy_manager_term_page';
  }
}

I don't know, what the panel module exactly does with the term page, but I don't see any reason why it shouldn't work now. Is there any error message you receive?

#11

crea - July 27, 2009 - 14:44

I installed 6.x-2.x dev and it seems to work as expected.

@corona ronin: Make sure you visit "admin/build/modules" page after enabling this setting, so menu system will rebuild. Until rebuilding menu this setting doesn't have effect.

#12

corona ronin - July 27, 2009 - 16:03

@crea, going to admin/build/modules seemed to fix the problem. I guess the menu setting needed to be reset. This is good stuff!

I'll post back if something else comes up...

Thanks!

#13

System Message - August 10, 2009 - 16:10
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.