drupal_container() is deprecated, and all calls in the overlay 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

rbayliss’s picture

Status: Active » Needs review
FileSize
3.54 KB
dcam’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll

#1 no longer applies.

chrisguindon’s picture

Attach is a reroll of patch 2011108-overlay-drupal-service-1.patch

This is my first patch reroll so I hope all is well.

chrisguindon’s picture

Status: Needs work » Needs review
dcam’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs reroll

Great job, @chrisguindon! I'm happy to RTBC your first reroll.

#3 looks good to me. The patch changes all instances of drupal_container() in the overlay module.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

ddrozdik’s picture

Status: Fixed » Closed (fixed)