This is a follow up to #351249: Finer control over the Parent Menu select box where we moved this setting to structure>content types.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 716792.patch | 1.63 KB | agentrickard |
| #1 | menu.patch | 995 bytes | bleen |
This is a follow up to #351249: Finer control over the Parent Menu select box where we moved this setting to structure>content types.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 716792.patch | 1.63 KB | agentrickard |
| #1 | menu.patch | 995 bytes | bleen |
Comments
Comment #1
bleen commentedhere's a quick patch
Comment #2
JODA-RE75VBW0D4 commented#1: menu.patch queued for re-testing.
Comment #3
agentrickardConfirmed.
Comment #4
dries commentedCare to provide a bit more background/information -- especially given that this patch is marked 'critical'?
Menu.module still seem to use this variable in:
./modules/menu/menu.module: $menu_name = variable_get('menu_default_node_menu', 'navigation');Comment #5
agentrickardThe background is all in #351249: Finer control over the Parent Menu select box, and that looks like an oversight in the current patch.
Revised to catch the one holdover usage of the old variable.
In effect, default menus are now content-type specific.
Comment #6
dries commentedCommitted to CVS HEAD. Thanks.
Comment #8
gábor hojtsyThis was not documented in the update guide, now added http://drupal.org/update/modules/6/7#menu_default_node_menu
Comment #9
gábor hojtsyThis change is missing an upgrade path. Critical issue opened at #761648: Menu D6->D7 upgrade doesn't maintain node-menu configuration (f.k.a.: Menu items disappear after upgrade or manual menu entry).