OutputPage: Remove appending of wgStyleVersion to legacy resources
authorTimo Tijhof <krinklemail@gmail.com>
Mon, 21 May 2018 21:21:28 +0000 (22:21 +0100)
committerKrinkle <krinklemail@gmail.com>
Wed, 23 May 2018 20:00:28 +0000 (20:00 +0000)
commit5d000f3b41c4d9083db38f6a4543fe89cac076a0
treece890778fbba0960a5bbe8e22f3d3c5742892ca0
parentdc77b2645242158f5b092aa63966cdb4b91d955f
OutputPage: Remove appending of wgStyleVersion to legacy resources

For addScriptFile(), just remove the appending of wgStyleVersion.
Going forward, anyone still using this, should simply append a query
parameter themselves in a way that is specific to that one url
(instead of relying on a generic global variable). Alternatively, one
could use OutputPage::transformResourcePath if the file is in /w/.

For addStyles(), also remove the appending of wgStyleVersion. Since this
method takes paths relative to /w/skins, we can easily update this to
automatically use transformResourcePath(), so that file-hash based query
parameters are automatically added.

Test Plan:
* Add calls to top of OutputPage::output():
  `$this->addStyle( 'Vector/README.md' );`
  `$this->addScriptFile( "{$GLOBALS['wgScriptPath']}/composer.json" );`
* Before, they are both inserted as `<link>` (head) and `<script>` (body)
  with a query parameter based on wgStyleVersion.
* After, the `<script>` (end of body) has no query.
  After, the stylesheet (head) has a SHA1 content hash as query.

Bug: T181318
Change-Id: Ie5ab5066ef7d07279086bde838d7305e9e4eabaf
includes/OutputPage.php