Page MenuHomePhabricator

Deprecate Skin::mapInterwikiToLanguage
Closed, ResolvedPublic

Description

According to https://codesearch.wmcloud.org/search/?q=mapInterwikiToLanguage&files=&excludeFiles=&repos= the public Skin::mapInterwikiToLanguage is used only by SkinLanguageHelper. Let's therefore move it to SkinLanguageHelper and deprecate it. This will help us with our overall goal of simplifying Skin code so that it's only responsibility is rendering the HTML of the page!

TODO

  • copy/paste mapInterwikiToLanguage from Skin.php into SkinLanguageHelper.php. Don't worry about the Skin method for now.
  • update the constructor for SkinLanguageHelper.php so that it doesn't take Skin
  • add wfDeprecated call to Skin::mapInterwikiToLanguage (which should now be unused
  • note its deprecation in RELEASE-NOTES-1.47

Event Timeline

Change #1284171 had a related patch set uploaded (by HakanIST; author: HakanIST):

[mediawiki/core@master] SkinLanguageHelper: Remove Skin dependency, use explicit DI

https://gerrit.wikimedia.org/r/1284171

Change #1284185 had a related patch set uploaded (by HakanIST; author: HakanIST):

[mediawiki/core@master] Skin: Deprecate mapInterwikiToLanguage

https://gerrit.wikimedia.org/r/1284185

Change #1284171 merged by jenkins-bot:

[mediawiki/core@master] SkinLanguageHelper: Replace Skin parameter with explicit dependencies

https://gerrit.wikimedia.org/r/1284171

Jdlrobson-WMF updated the task description. (Show Details)

Change #1284185 merged by jenkins-bot:

[mediawiki/core@master] Skin: Deprecate mapInterwikiToLanguage

https://gerrit.wikimedia.org/r/1284185