Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
22 Mar 2010 at 22:40 UTC
Updated:
11 Apr 2011 at 16:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
jhodgdonHmmm...
The Menu System page on http://api.drupal.org/api/group/menu/7 is about menu.inc, which deals with hook_menu() and associated actions. That is, it's about page routing in Drupal (defining paths and what should happen when a visitor goes to that path).
The menu_save() and menu_delete() functions have to do with the Menu module, which is about creating lists of links to be displayed as navigation. So those two functions do not actually belong in the Menu System topic page.
What we should probably do, however, is add something to the Menu System topic page to point new folks to menu.module if they are interested in manipulating navigation menus.
Also, the text here: http://api.drupal.org/api/drupal/modules--menu--menu.module/7 should probably be expanded a bit. For one thing, it should say "navigation menus", and maybe it should point to the menu system topic as well?
Comment #2
jpsoto commentedThanks for your kind answer ...
Indeed, I was aware of such a difference.
I wanted to point up a difficulty to find out how create/delete a menu (container). No docs describe it. I had to look for into the devel module because it creates a custom menu.
Another tip. Seemingly, the proper place to call for menu_save() is the hook_enable; as devel module does it. However, devel module calls for menu_delete() in the hook_uninstall; instead of the hook_disable (counterpart of the hook_enable).
Anyway, thanks for your patience ... JP
Comment #3
jpsoto commentedComment #4
jhodgdonWhy have you postponed this issue? Let's leave it at "active" please.
Comment #5
jhodgdonHere's a patch that hopefully clarifies this distinction.
Comment #6
jpsoto commented@jhodgon
thanks for clarifying this issue ... I suppose the use of the term *menu* to deal with page routing has historical roots.
May be, in the future, *page routing* will have got its own proper naming.
Thanks again ...
Comment #7
kim-day commentedThis patch looks good so I changed to RTBC.
Comment #8
jhodgdonNeeds to go to d8 and then d7
Comment #9
dries commentedCommitted to my local 8.x tree. Will push this later tonight. Moving to 7.x in the mean time. Thanks Jennifer.
Comment #10
webchickCommitted to 7.x. Thanks!