Updated: Comment #0

Problem/Motivation

Follow-up from #1550866: Remove obsolete drupal_language_initialize().
Seems this method is not needed any more, because is not executed

Proposed resolution

Just remove the unused method

Remaining tasks

wait for #1550866: Remove obsolete drupal_language_initialize()
and file patch

User interface changes

no

API changes

LanguageManagerInterface::init() is removed

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

Title: Get rid of LanguageManagerInterface::init() and implementations as obsolete » Remove obsolete LanguageManagerInterface::init() method
Berdir’s picture

Status: Postponed » Active
Issue tags: +Novice
g3r4’s picture

Assigned: Unassigned » g3r4

I'll work on a patch for this

g3r4’s picture

I modified a couple of files to get rid of the init(), the interface and the implementation

g3r4’s picture

Status: Active » Needs review
Berdir’s picture

Status: Needs review » Reviewed & tested by the community

Thanks, nothing left in there and nothing calls it anymore, good to go I think.

g3r4’s picture

Great ! Thanks @Berdir

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

  • catch committed 7f8c7c0 on 8.x
    Issue #2214057 by g3r4: Remove obsolete LanguageManagerInterface::init...

Status: Fixed » Closed (fixed)

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