Comments

voleger created an issue. See original summary.

voleger’s picture

How about to move template_preprocess_menu_local_task() and template_preprocess_menu_local_action()
to the system module as it contains base menu functionality?

andypost’s picture

Makes sense, also menu_local_tabs is tricky - looks better to deprecate it in favour of inline code or render element

andypost’s picture

btw Other deprecated functions needs work for proper deprecation (provide link & throw error)

berdir’s picture

Yeah, 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.

voleger’s picture

andypost’s picture

Issue summary: View changes
andypost’s picture

Issue summary: View changes

fixed links in IS

andypost’s picture

andypost’s picture

Btw 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()

voleger’s picture

Version: 8.7.x-dev » 8.8.x-dev

Drupal 8.7.0-alpha1 will be released the week of March 11, 2019, which means new developments and disruptive changes should now be targeted against the 8.8.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

andypost’s picture

Issue summary: View changes
andypost’s picture

Issue summary: View changes
andypost’s picture

Issue summary: View changes
andypost’s picture

Issue summary: View changes

Version: 8.8.x-dev » 8.9.x-dev

Drupal 8.8.0-alpha1 will be released the week of October 14th, 2019, which means new developments and disruptive changes should now be targeted against the 8.9.x-dev branch. (Any changes to 8.9.x will also be committed to 9.0.x in preparation for Drupal 9’s release, but some changes like significant feature additions will be deferred to 9.1.x.). For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

voleger’s picture

Version: 8.9.x-dev » 9.1.x-dev
Category: Task » Plan

Update version

berdir’s picture

I 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.

andypost’s picture

daffie’s picture

Can 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?

andypost’s picture

@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

Version: 9.1.x-dev » 9.2.x-dev

Drupal 9.1.0-alpha1 will be released the week of October 19, 2020, which means new developments and disruptive changes should now be targeted for the 9.2.x-dev branch. For more information see the Drupal 9 minor version schedule and the Allowed changes during the Drupal 9 release cycle.

Version: 9.2.x-dev » 9.3.x-dev

Drupal 9.2.0-alpha1 will be released the week of May 3, 2021, which means new developments and disruptive changes should now be targeted for the 9.3.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

andypost’s picture

last 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)

andypost’s picture

Issue summary: View changes

updates IS a bit

voleger’s picture

andypost’s picture

voleger’s picture

Status: Active » Reviewed & tested by the community

Looks like the meta issue is completed.

andypost’s picture

Status: Reviewed & tested by the community » Fixed

Yay! It's done

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.