When this module is disabled, all of the containers are deleted. And its children get mixed into the containers parent. This is less then ideal since it destroys the IA hierarchy that user has built.

This is especially problematic because the upgrade path to Drupal 7 recommends disabling all contrib module before upgrading to Drupal 7. :-p

We can get around that by converting all the containers into regular custom menu items when menu_item_container is disabled. And (as long as the user hasn't edited the items) converting them back to containers when the module is re-enabled.

Comments

JohnAlbin’s picture

JohnAlbin
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin commented
Title: Support D7 upgrade path and module disabling » Convert containers to links when module disabled
Status: Active » Fixed

Ok. So its not quite so bad as I thought. The menu containers are not deleted, but they and all of their children are no longer displayed in the menu tree.

So, while the upgrade path is fine, disabling the module is still a problem.

Fixed. http://drupal.org/cvs?commit=487444

Status: Fixed » Closed (fixed)

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