See T71084 for the motivation behind this work.
Classes that extend ActionPage can access configs using SpecialPage::getConfig.
Example from VoterEligibilityPage:
global $wgSecurePollUseNamespace;
...
if ( global $wgSecurePollUseNamespace ) {
...can be replaced with:
if ( $this->specialPage->getConfig()->get('SecurePollUseNamespace') ) {
...ActionPage classes that access global config variables:
| Project | Branch | Lines +/- | Subject | |
|---|---|---|---|---|
| mediawiki/extensions/SecurePoll | master | +13 -32 | Remove Global Vars from ActionPages |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T11968 Reduce use of global and public variables | |||
| Open | None | T270646 Use Config instead of globals in SecurePoll extension | |||
| Resolved | wikitrent | T270644 Stop accessing global config variables from SecurePoll ActionPages (OT) |
Change 655751 had a related patch set uploaded (by Wikitrent; owner: Wikitrent):
[mediawiki/extensions/SecurePoll@master] Remove Global Vars from ActionPages
Change 655751 merged by jenkins-bot:
[mediawiki/extensions/SecurePoll@master] Remove Global Vars from ActionPages