Closed (fixed)
Project:
Drupal core
Version:
9.3.x-dev
Component:
menu system
Priority:
Normal
Category:
Plan
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
16 Sep 2018 at 00:33 UTC
Updated:
15 Jul 2021 at 12:44 UTC
Jump to comment: Most recent
Comments
Comment #2
volegerHow about to move
template_preprocess_menu_local_task()andtemplate_preprocess_menu_local_action()to the
systemmodule as it contains base menu functionality?Comment #3
andypostMakes sense, also
menu_local_tabsis tricky - looks better to deprecate it in favour of inline code or render elementComment #4
andypostbtw Other deprecated functions needs work for proper deprecation (provide link & throw error)
Comment #5
berdirYeah, menu_local_tabs() is not used in core it seems and LocalTasksBlock calls it directly. I don't think this function is needed anymore, I'd just deprecate it in favor of using the block or using the theme function directly.
Comment #6
volegerFilled #3021718: Properly deprecate menu_local_tabs(), menu_secondary_local_tasks(), menu_primary_local_tasks()
Comment #7
andypostFiled #3021788: Move template_preprocess_menu_local_task() and template_preprocess_menu_local_action() to the core/includes/theme.inc
Comment #8
andypostComment #9
andypostfixed links in IS
Comment #10
andypostIt makes sense to file new issue to review menu entity usage
Comment #11
andypostBtw already deprecated functions needs tests which could be based on test class added in #3021718: Properly deprecate menu_local_tabs(), menu_secondary_local_tasks(), menu_primary_local_tasks()
Comment #12
volegerFilled #3021823: Properly deprecate functions in menu.inc
Comment #14
andypostDone #3021718: Properly deprecate menu_local_tabs(), menu_secondary_local_tasks(), menu_primary_local_tasks()
Comment #15
andypostComment #16
andypostComment #17
andypostComment #18
andypostComment #20
volegerUpdate version
Comment #21
volegerReparent
Comment #22
berdirI think it's just preprocess functions and one helper now? Moving preprocess to theme.inc might be possible in 9.x, without BC because the location of functions isn't part of BC.
Comment #23
andypostLast 2 child issues left, done #3021788: Move template_preprocess_menu_local_task() and template_preprocess_menu_local_action() to the core/includes/theme.inc
Comment #24
daffie commentedCan the 2 remaining issue: #3021804: Remove optional dependency on menu_ui module in \Drupal\views\Plugin\views\wizard\WizardPluginBase and #1882552: Deprecate menu_list_system_menus() and menu_ui_get_menus(), be done independent of each other or is one blocking the other. If so, which one should go first?
Comment #25
andypost@daffie I still think this 2 issues are separate, one affects views and only after it's fixed we can deprecate system menus
Both have own scope and I'd like if someone has idea how to join them
Comment #28
andypostlast 2 issues could go separate but will affect each other so better to get #3021804: Remove optional dependency on menu_ui module in \Drupal\views\Plugin\views\wizard\WizardPluginBase in first
Moreover there's another one blocked on it #3028612: Deprecate menu_ui_get_menus() and replace all of its usages (added to related)
Comment #29
andypostupdates IS a bit
Comment #30
voleger#1882552: Deprecate menu_list_system_menus() and menu_ui_get_menus() is the last one in this META issue
Comment #31
andypostThere's one more issue to work on #3028612: Deprecate menu_ui_get_menus() and replace all of its usages
Comment #32
volegerLooks like the meta issue is completed.
Comment #33
andypostYay! It's done