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

Move version check inside PersonalUrls hook

Browse Source 
Stop additional conditional hook if MW < 1.37, and move the check
to inside the hook itself

Bug: T299305
Change-Id: Ied1c5c0beee015d7b0a8d6bd750346f4a8a5aaa1
This commit is contained in:
Abijeet
2022-01-20 14:06:57 +05:30
committed by jenkins-bot
parent 18ebb4bafc
commit 6e6635bdbb
2 changed files with 11 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
includes/Hooks.php
View File

@@ -88,19 +88,7 @@ class Hooks implements
}
public static function setVersionConstant() {
global $wgHooks;
define( 'ULS_VERSION', '2020-07-20' );
// For MediaWiki < 1.37, there is no `user-interface-preferences` menu. We use
// the PersonalUrls hook to make sure the language button is added.
// In MediaWiki > 1.37, the personal urls was split out into multiple new menus,
// In the new format, the `user-interface-preferences` is the most relevant place to put
// this button. Using the SkinTemplateNavigation::Universal hook will ensure the button is
// added to the correct menu.
if ( version_compare( MW_VERSION, '1.37', '<' ) ) {
$wgHooks['PersonalUrls'][] = "UniversalLanguageSelector\\Hooks::onPersonalUrls";
}
}
/**
@@ -233,6 +221,16 @@ class Hooks implements
* @param SkinTemplate $skin
*/
public function onPersonalUrls( &$personal_urls, &$title, $skin ): void {
// For MediaWiki < 1.37, there is no `user-interface-preferences` menu. We use
// the PersonalUrls hook to make sure the language button is added.
// In MediaWiki >= 1.37, the personal urls was split out into multiple new menus,
// In the new format, the `user-interface-preferences` is the most relevant place to put
// this button. Using the SkinTemplateNavigation::Universal hook will ensure the button is
// added to the correct menu.
if ( version_compare( MW_VERSION, '1.37', '>=' ) ) {
return;
}
$personal_urls = $this->addPersonalBarTrigger(
$personal_urls,
$skin