resourceloader: Remove redundant 'group' setting for StartupModule
authorTimo Tijhof <krinklemail@gmail.com>
Thu, 13 Jun 2019 18:33:46 +0000 (19:33 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Fri, 14 Jun 2019 14:46:10 +0000 (15:46 +0100)
This feaure exists for controlling cache fragmentation of modules
that are queued for loading. However, the startup module is never
"queued" for loading.

It's loading is hardwired into ResourceLoaderClientHtml and doesn't
use any batching and couldn't logically, given that by design it
has to be the first thing that loads and cannot be combined with
other code (because we need to be able to return early and control
what else gets loaded from there).

[0] https://www.mediawiki.org/wiki/ResourceLoader/Architecture#Balance

Change-Id: If8900c538ce623f6a9a68a3bbef0927b4b6ae346

includes/resourceloader/ResourceLoaderStartUpModule.php

index efed418..26e64d1 100644 (file)
@@ -425,11 +425,4 @@ class ResourceLoaderStartUpModule extends ResourceLoaderModule {
                // and hash it to determine the version (as used by E-Tag HTTP response header).
                return true;
        }
-
-       /**
-        * @return string
-        */
-       public function getGroup() {
-               return 'startup';
-       }
 }