bryce/mediawiki-extensions-UniversalLanguageSelector
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use $this->getConfig()->get instead of global variable

Browse Source 
Also use MainConfigNames.

Change-Id: I4d41783ef4055afa5299acac295aef079f453999
This commit is contained in:
Fomafix
2024-08-19 10:54:03 +00:00
parent 464a92813a
commit a2087c0492
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
data/LanguageNameIndexer.php
View File

@@ -26,6 +26,7 @@ require_once "$IP/maintenance/Maintenance.php";
use MediaWiki\Extension\CLDR\LanguageNames; use MediaWiki\Extension\CLDR\LanguageNames;
use MediaWiki\Languages\LanguageNameUtils; use MediaWiki\Languages\LanguageNameUtils;
use MediaWiki\MainConfigNames;
class LanguageNameIndexer extends Maintenance { class LanguageNameIndexer extends Maintenance {
public function __construct() { public function __construct() {
@@ -36,10 +37,8 @@ class LanguageNameIndexer extends Maintenance {
} }
public function execute() { public function execute() {
global $wgExtraLanguageNames;
// Avoid local configuration leaking to this script // Avoid local configuration leaking to this script
if ( $wgExtraLanguageNames !== [] ) { if ( $this->getConfig()->get( MainConfigNames::ExtraLanguageNames ) !== [] ) {
$this->fatalError( 'You have entries in $wgExtraLanguageNames. Needs to be empty for this script.' ); $this->fatalError( 'You have entries in $wgExtraLanguageNames. Needs to be empty for this script.' );
} }