jenkins-bot has submitted this change and it was merged. Change subject: resourceloader: Remove 'loaderScripts' option from FileModule ......................................................................
resourceloader: Remove 'loaderScripts' option from FileModule Not used in any of our public repositories. Bug: T65240 Change-Id: I1e9f741c3ef0f922129ecd10039228b58565bf62 --- M RELEASE-NOTES-1.27 M includes/resourceloader/ResourceLoaderFileModule.php M includes/resourceloader/ResourceLoaderModule.php M includes/resourceloader/ResourceLoaderStartUpModule.php M tests/phpunit/structure/ResourcesTest.php 5 files changed, 3 insertions(+), 60 deletions(-) Approvals: Gilles: Looks good to me, approved jenkins-bot: Verified diff --git a/RELEASE-NOTES-1.27 b/RELEASE-NOTES-1.27 index b3add69..3a45fae 100644 --- a/RELEASE-NOTES-1.27 +++ b/RELEASE-NOTES-1.27 @@ -92,6 +92,7 @@ * ProfilerOutputUdp was removed. Note that there is a ProfilerOutputStats class. * WikiPage::doDeleteArticleReal() and WikiPage::doDeleteArticle() now ignore the 2nd and 3rd arguments (formerly $id and $commit). +* Removed "loaderScripts" option from ResourceLoaderFileModule class. == Compatibility == diff --git a/includes/resourceloader/ResourceLoaderFileModule.php b/includes/resourceloader/ResourceLoaderFileModule.php index e4def4d..b46707c 100644 --- a/includes/resourceloader/ResourceLoaderFileModule.php +++ b/includes/resourceloader/ResourceLoaderFileModule.php @@ -74,15 +74,6 @@ protected $debugScripts = array(); /** - * @var array List of paths to JavaScript files to include in the startup module - * @par Usage: - * @code - * array( [file-path], [file-path], ... ) - * @endcode - */ - protected $loaderScripts = array(); - - /** * @var array List of paths to CSS files to always include * @par Usage: * @code @@ -195,8 +186,6 @@ * ), * // Scripts to include in debug contexts * 'debugScripts' => [file path string or array of file path strings], - * // Scripts to include in the startup module - * 'loaderScripts' => [file path string or array of file path strings], * // Modules which must be loaded before this module * 'dependencies' => [module name string or array of module name strings], * 'templates' => array( @@ -239,7 +228,6 @@ // Lists of file paths case 'scripts': case 'debugScripts': - case 'loaderScripts': case 'styles': $this->{$member} = (array)$option; break; @@ -391,18 +379,6 @@ } /** - * Get loader script. - * - * @return string|bool JavaScript code to be added to startup module - */ - public function getLoaderScript() { - if ( count( $this->loaderScripts ) === 0 ) { - return false; - } - return $this->readScriptFiles( $this->loaderScripts ); - } - - /** * Get all styles for a given context. * * @param ResourceLoaderContext $context @@ -551,8 +527,7 @@ $this->templates, $context->getDebug() ? $this->debugScripts : array(), $this->getLanguageScripts( $context->getLanguage() ), - self::tryForKey( $this->skinScripts, $context->getSkin(), 'default' ), - $this->loaderScripts + self::tryForKey( $this->skinScripts, $context->getSkin(), 'default' ) ); if ( $this->skipFunction ) { $files[] = $this->skipFunction; @@ -592,7 +567,6 @@ // - position (only used by OutputPage) 'scripts', 'debugScripts', - 'loaderScripts', 'styles', 'languageScripts', 'skinScripts', diff --git a/includes/resourceloader/ResourceLoaderModule.php b/includes/resourceloader/ResourceLoaderModule.php index 3dd7a4b..782af81 100644 --- a/includes/resourceloader/ResourceLoaderModule.php +++ b/includes/resourceloader/ResourceLoaderModule.php @@ -299,23 +299,10 @@ } /** - * Get the loader JS for this module, if set. - * - * @return mixed JavaScript loader code as a string or boolean false if no custom loader set - */ - public function getLoaderScript() { - // Stub, override expected - return false; - } - - /** * Get a list of modules this module depends on. * * Dependency information is taken into account when loading a module * on the client side. - * - * To add dependencies dynamically on the client side, use a custom - * loader script, see getLoaderScript() * * Note: It is expected that $context will be made non-optional in the near * future. diff --git a/includes/resourceloader/ResourceLoaderStartUpModule.php b/includes/resourceloader/ResourceLoaderStartUpModule.php index 4a672f2..8a2423c 100644 --- a/includes/resourceloader/ResourceLoaderStartUpModule.php +++ b/includes/resourceloader/ResourceLoaderStartUpModule.php @@ -163,13 +163,9 @@ * - array 'dependencies' * - string|null 'group' * - string 'source' - * - string|false 'loader' */ public static function compileUnresolvedDependencies( array &$registryData ) { foreach ( $registryData as $name => &$data ) { - if ( $data['loader'] !== false ) { - continue; - } $dependencies = $data['dependencies']; foreach ( $data['dependencies'] as $dependency ) { $implicitDependencies = self::getImplicitDependencies( $registryData, $dependency ); @@ -230,7 +226,6 @@ 'dependencies' => $module->getDependencies( $context ), 'group' => $module->getGroup(), 'source' => $module->getSource(), - 'loader' => $module->getLoaderScript(), 'skip' => $skipFunction, ); } @@ -240,22 +235,9 @@ // Register sources $out .= ResourceLoader::makeLoaderSourcesScript( $resourceLoader->getSources() ); - // Concatenate module loader scripts and figure out the different call - // signatures for mw.loader.register + // Figure out the different call signatures for mw.loader.register $registrations = array(); foreach ( $registryData as $name => $data ) { - if ( $data['loader'] !== false ) { - $out .= ResourceLoader::makeCustomLoaderScript( - $name, - $data['version'], - $data['dependencies'], - $data['group'], - $data['source'], - $data['loader'] - ); - continue; - } - // Call mw.loader.register(name, version, dependencies, group, source, skip) $registrations[] = array( $name, diff --git a/tests/phpunit/structure/ResourcesTest.php b/tests/phpunit/structure/ResourcesTest.php index ae0d325..3282665 100644 --- a/tests/phpunit/structure/ResourcesTest.php +++ b/tests/phpunit/structure/ResourcesTest.php @@ -232,7 +232,6 @@ 'lists' => array( 'scripts', 'debugScripts', - 'loaderScripts', 'styles', ), -- To view, visit https://gerrit.wikimedia.org/r/247475 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e9f741c3ef0f922129ecd10039228b58565bf62 Gerrit-PatchSet: 4 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Krinkle <krinklem...@gmail.com> Gerrit-Reviewer: Bartosz DziewoĆski <matma....@gmail.com> Gerrit-Reviewer: Gilles <gdu...@wikimedia.org> Gerrit-Reviewer: Krinkle <krinklem...@gmail.com> Gerrit-Reviewer: Legoktm <legoktm.wikipe...@gmail.com> Gerrit-Reviewer: TheDJ <hartman.w...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits