Remove redundant dependency on 'mediawiki.notify'

As this extension supports MW 1.34 (and MW 1.33) in the master
branch, apply the usual ternary PHP registration and ternary
tricks.

Bug: T233676
Change-Id: I4d18bab53460b397d4f50485c4a149185a1db7b3
This commit is contained in:
Timo Tijhof
2020-05-27 23:23:57 +01:00
committed by jenkins-bot
parent f9f90b7040
commit 5b7e12705a
2 changed files with 58 additions and 43 deletions

View File

@@ -28,6 +28,7 @@
"MakeGlobalVariablesScript": "UniversalLanguageSelectorHooks::addVariables",
"PersonalUrls": "UniversalLanguageSelectorHooks::addPersonalBarTrigger",
"ResourceLoaderGetConfigVars": "UniversalLanguageSelectorHooks::addConfig",
"ResourceLoaderRegisterModules": "UniversalLanguageSelectorHooks::onResourceLoaderRegisterModules",
"SkinTemplateOutputPageBeforeExec": "UniversalLanguageSelectorHooks::onSkinTemplateOutputPageBeforeExec",
"UserGetLanguageObject": "UniversalLanguageSelectorHooks::getLanguage"
},
@@ -219,22 +220,6 @@
"localBasePath": "resources",
"remoteExtPath": "UniversalLanguageSelector/resources"
},
"ext.uls.ime": {
"scripts": "js/ext.uls.ime.js",
"dependencies": [
"ext.uls.common",
"ext.uls.preferences",
"ext.uls.mediawiki",
"ext.uls.messages",
"jquery.ime",
"mediawiki.notify"
],
"messages": [
"uls-ime-helppage"
],
"localBasePath": "resources",
"remoteExtPath": "UniversalLanguageSelector/resources"
},
"ext.uls.init": {
"targets": [
"desktop"
@@ -341,33 +326,6 @@
},
"remoteExtPath": "UniversalLanguageSelector/resources"
},
"ext.uls.setlang": {
"styles": [
"css/ext.uls.dialog.less",
"css/ext.uls.setlang.less"
],
"scripts": [
"js/ext.uls.dialog.js",
"js/ext.uls.setlang.js"
],
"dependencies": [
"mediawiki.api",
"mediawiki.ui.button",
"mediawiki.notify",
"mediawiki.Uri"
],
"messages": [
"ext-uls-setlang-error",
"ext-uls-setlang-unknown-error",
"ext-uls-setlang-heading",
"ext-uls-setlang-message",
"ext-uls-setlang-accept",
"ext-uls-setlang-cancel",
"ext-uls-setlang-loading"
],
"localBasePath": "resources",
"remoteExtPath": "UniversalLanguageSelector/resources"
},
"ext.uls.webfonts": {
"scripts": "js/ext.uls.webfonts.js",
"dependencies": [

View File

@@ -165,6 +165,63 @@ class UniversalLanguageSelectorHooks {
$out->addModules( 'ext.uls.setlang' );
}
/**
* @param ResourceLoader $resourceLoader
*/
public static function onResourceLoaderRegisterModules( ResourceLoader $resourceLoader ) {
global $wgVersion;
// Support: MediaWiki 1.34
$hasOldNotify = version_compare( $wgVersion, '1.35', '<' );
$tpl = [
'localBasePath' => dirname( __DIR__ ) . '/resources',
'remoteExtPath' => 'UniversalLanguageSelector/resources',
];
$modules = [
"ext.uls.ime" => $tpl + [
"scripts" => "js/ext.uls.ime.js",
"dependencies" => array_merge( [
"ext.uls.common",
"ext.uls.preferences",
"ext.uls.mediawiki",
"ext.uls.messages",
"jquery.ime",
], $hasOldNotify ? [ 'mediawiki.notify' ] : [] ),
"messages" => [
"uls-ime-helppage"
],
],
"ext.uls.setlang" => $tpl + [
"styles" => [
"css/ext.uls.dialog.less",
"css/ext.uls.setlang.less"
],
"scripts" => [
"js/ext.uls.dialog.js",
"js/ext.uls.setlang.js"
],
"dependencies" => array_merge( [
"mediawiki.api",
"mediawiki.ui.button",
"mediawiki.Uri"
], $hasOldNotify ? [ 'mediawiki.notify' ] : [] ),
"messages" => [
"ext-uls-setlang-error",
"ext-uls-setlang-unknown-error",
"ext-uls-setlang-heading",
"ext-uls-setlang-message",
"ext-uls-setlang-accept",
"ext-uls-setlang-cancel",
"ext-uls-setlang-loading"
],
],
];
$resourceLoader->register( $modules );
}
/**
* Add some tabs for navigation for users who do not use Ajax interface.
* Hook: PersonalUrls