Problem

  • menu_enable() creates static menu links for dynamic menu entities.
  • These links need to synchronized manually in various spots of the code base and are generally not needed.

Related issue #1882552: Get rid of menu_list_system_menus()

Files: 
CommentFileSizeAuthor
#1 menu-item-1.jpg10.9 KBandypost

Comments

StatusFileSize
new10.9 KB

They are needed to select parent menu link all over core.
menu-item-1.jpg

Suppose we need deeper refactoring of menu link selection widget - for example:

[select menu] - ajax loads dependent list of it's menu-items
[select menu item]

Looks like there's only few forms to add menu links, so we should just unify this widget to MenuLinkFormController

menu_edit_item() - hopefully gone with #916388: Convert menu links into entities to form controller
menu_form_node_form_alter()
shortcut_link_edit()

Issue summary:View changes

Updated issue summary.