Now that dependency injection has made it into core: #1497230: Use Dependency Injection to handle object definitions we should reduce the number of things we put into the $GLOBALS array, starting with things that touch the site's language.

There are many uses of the $GLOBALS['lanaguage_interface'] that can now be updated to use the drupal_container()->get[LANGUAGE_TYPE_INTERFACE] that was provided in the recent patch.

CommentFileSizeAuthor
#1 1539622_1_local.patch2.4 KBcosmicdreams

Comments

cosmicdreams’s picture

cosmicdreams commented
StatusFileSize
new 1539622_1_local.patch2.4 KB

first try

cosmicdreams’s picture

cosmicdreams commented
Status: Active » Needs review

sending it to testbot to see where I'm at

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Component: language system » locale.module
Issue tags: +Dependency Injection (DI)
cosmicdreams’s picture

cosmicdreams commented

#1: 1539622_1_local.patch queued for re-testing.

cosmicdreams’s picture

cosmicdreams commented
Assigned: Unassigned » cosmicdreams

Assigning to me so I can show these to folks easier

robloach’s picture

robloach
he/him
commented
Priority: Normal » Minor
Status: Needs review » Reviewed & tested by the community

Easy pickins.

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

OK this makes sense. It'll be good when we have actual dependency injection instead of the drupal_container() calls, but since this is direct swap for $GLOBALS it seems fine to remove that bc layer. Committed/pushed to 8.x.

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