Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
menu system
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
15 Jul 2014 at 15:31 UTC
Updated:
12 Aug 2014 at 23:50 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
cilefen commentedComment #2
cilefen commentedComment #3
cilefen commentedComment #4
tim.plunkettPostponed on #2256521: [META] New plan, Phase 2: Implement menu links as plugins, including static admin links and views, and custom links with menu_link_content entity, all managed via menu_ui module.
Comment #5
effulgentsia commentedDoes this really need to be postponed on all 5 parts of the menu link conversion? #2301239: MenuLinkNG part1 (no UI or conversions): plugins (static + MenuLinkContent) + MenuLinkManager + MenuTreeStorage already landed, and I don't think this patch conflicts with any of the other parts.
Comment #6
tim.plunkettAh! If it doesn't conflict with the rest of the patches, then by all means.
Comment #7
effulgentsia commentedCool. Back to needs review then. On dreditor inspection, patch looks good to me, but I haven't checked if it's complete or reviewed the change record, so leaving to others to RTBC.
Comment #8
dawehnerThere aren't other menu links left:
One thing we should talk about in a follow up, the hook names no longer match the file names at all:
See #2304067: Adapt menu alter hooks to the new filenames
Comment #9
alexpottCommitted b560f34 and pushed to 8.x. Thanks!