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

This task a part of #2001206: Replace drupal_container() with Drupal::service()

Files: 
CommentFileSizeAuthor
#3 taxonomy-2014025-3.patch1.49 KBkgoel
PASSED: [[SimpleTest]]: [MySQL] 58,981 pass(es). View
#2 taxonomy-2014025-2.patch1.5 KBkgoel
PASSED: [[SimpleTest]]: [MySQL] 55,300 pass(es). View

Comments

kgoel’s picture

Assigned: Unassigned » kgoel
kgoel’s picture

Status: Active » Needs review
FileSize
1.5 KB
PASSED: [[SimpleTest]]: [MySQL] 55,300 pass(es). View
kgoel’s picture

FileSize
1.49 KB
PASSED: [[SimpleTest]]: [MySQL] 58,981 pass(es). View

Replaced Drupal::service('request') with \Drupal::request()

dcam’s picture

Status: Needs review » Reviewed & tested by the community

#3 looks good to me. I didn't find any additional uses of drupal_container() in the taxonomy 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)