resourceloader: Fix mw.loader to compute version for current request only
authorTimo Tijhof <krinklemail@gmail.com>
Thu, 1 Mar 2018 04:19:28 +0000 (20:19 -0800)
committerKrinkle <krinklemail@gmail.com>
Wed, 7 Mar 2018 21:09:08 +0000 (21:09 +0000)
commit15b0e653277c98e32004ed86d8b634d3a9d74ed3
tree62bf7a49a752984fbc4eb2b00fe66a71644315a7
parentb90bc351e3cd8d1e5e0dd7fe6f340c770201c06e
resourceloader: Fix mw.loader to compute version for current request only

Previously, the 'version' query parameter was computed before request-splitting
which meant that all requests within the same 'source/group'-batch carried the
same 'version' parameter. This was then consistently rejected on the server due
to it not batching the combined hash of modules for any given request.

In practice this happened very rarely (if at all) in production, because
urls don't usually hit anywhere near 2000 in common use.

Bug: T188076
Change-Id: I211523d4781623873887a05d048f56cccd28432c
resources/src/mediawiki/mediawiki.js
tests/qunit/data/load.mock.php
tests/qunit/suites/resources/mediawiki/mediawiki.loader.test.js