drupal_container() is deprecated, and all calls in the menu & menu_link modules module need to be replaced with Drupal::service(), except for where the module_handler service is requested, which needs to be replaced with Drupal::moduleHandler() (see #1957154: Replace calls to drupal_container()->get('module_handler') service with Drupal::moduleHandler())

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

thomas.fleming’s picture

Assigned: Unassigned » thomas.fleming
thomas.fleming’s picture

Status: Active » Needs review
FileSize
749 bytes
1.7 KB

Status: Needs review » Needs work

The last submitted patch, menu_link-drupal-service-2011104-1.patch, failed testing.

thomas.fleming’s picture

thomas.fleming’s picture

Status: Needs work » Needs review
thomas.fleming’s picture

#2: menu-drupal-service-2011104-1.patch queued for re-testing.

thomas.fleming’s picture

Attached is the combined patch for the menu and menu_link modules.

thomas.fleming’s picture

Assigned: thomas.fleming » Unassigned
ddrozdik’s picture

Status: Needs review » Reviewed & tested by the community

Applied and checked, looks good - RTBC

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

ddrozdik’s picture

Status: Fixed » Closed (fixed)