in wmf-config/mobile.php we have various lines like this:
$wgMFNoMobileCategory = $wmgMFNoMobileCategory; $wgMFNoMobilePages = $wmgMFNoMobilePages;
We often use variables starting with $wmg to trigger certain blocks of code in CommonSettings.php but I'm not sure how helpful our usages are.
It feels to me that we could probably remove all the lines in wmf-config/mobile.php in favour of wg prefixed variables in wmf-config/InitialiseSettings.php
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Switch MobileFrontend to extension registration | operations/mediawiki-config | master | +41 -71 |
| Status | Subtype | Assigned | Task | |
|---|---|---|---|---|
| · · · | ||||
| Resolved | None | T140852 Load all Wikimedia-deployed extensions and skins via extension registration | ||
| Resolved | Dereckson | T147234 Review/cleanup inappropriate usages of wmg in MobileFrontend site config | ||
| · · · |
This also appears to cause confusion when we deploy things:
https://gerrit.wikimedia.org/r/314627
So I'd suggest we fix this sooner rather than later :)
So, a bit of MediaWiki history:
To avoid such overwrite by the extension code, we used the wmg = wg trick.
As long as extensions use extension registration, we can instead directly use wg variables in IS.
Change 314748 had a related patch set uploaded (by Dereckson):
Switch to extension registration for Mobile extensions
Let's identify which Reading Web maintained extensions can benefit from this. If I'm not mistaken some extensions still use the old registration system.