jenkins-bot [Sun, 18 Mar 2018 22:19:35 +0000 (22:19 +0000)]
Merge "Set visiblity of some HashRing methods"
Kunal Mehta [Tue, 6 Mar 2018 01:14:07 +0000 (17:14 -0800)]
Use SPDX 3.0 license identifier
SPDX released version 3 of their license list (<https://spdx.org/licenses/>),
which changed the FSF licenses to explicitly end in -only or -or-later
instead of relying on an easy to miss + symbol.
Bug: T183858
Change-Id: I4cbb98ea2bbacf46a57401dbba2bdb076fca1d3a
Aaron Schulz [Sun, 18 Mar 2018 04:22:56 +0000 (21:22 -0700)]
Set visiblity of some HashRing methods
* getLiveRing() is now protected
* getLocation() is now final
Change-Id: I16002fe7187d8bbb6e36d4dd9a1302ee1d46a4bb
jenkins-bot [Sun, 18 Mar 2018 05:19:04 +0000 (05:19 +0000)]
Merge "Normalize PHPDoc attributes"
jenkins-bot [Sun, 18 Mar 2018 05:16:24 +0000 (05:16 +0000)]
Merge "Don't use deprecated NS_IMAGE*"
jenkins-bot [Sun, 18 Mar 2018 05:06:21 +0000 (05:06 +0000)]
Merge "RevisionStore: Properly encode timestamp"
jenkins-bot [Sun, 18 Mar 2018 04:58:06 +0000 (04:58 +0000)]
Merge "Remove internal use of deprecated $wgRequestTime"
jenkins-bot [Sun, 18 Mar 2018 04:46:03 +0000 (04:46 +0000)]
Merge "Remove @group from ParserTestRunner"
Kevin Israel [Fri, 12 Aug 2016 10:33:37 +0000 (06:33 -0400)]
Remove internal use of deprecated $wgRequestTime
* Use $_SERVER['REQUEST_TIME_FLOAT'] unconditionally in WebRequest.php
and libs/Timing.php. WebStart.php was doing this already without issue.
The key existst since PHP 5.4, for both Web and CLI (we require 5.5).
* In wfDebug() and wfReportTime(), use $_SERVER['REQUEST_TIME_FLOAT'] instead.
* In ApiFormatBase and MWDebug, use WebRequest::getElapsedTime() instead.
* In Maintenance.php, remove setting of $wgRequestTime.
* In rebuildFileCache.php, update mocking to $_SERVER['REQUEST_TIME_FLOAT']
so that we avoid re-introducing bug T24852.
Change-Id: I1b647da2862f815029caa533b592ec8a05b33806
jenkins-bot [Sun, 18 Mar 2018 04:06:05 +0000 (04:06 +0000)]
Merge "Rename ApiTestCaseUpload"
jenkins-bot [Sun, 18 Mar 2018 03:58:51 +0000 (03:58 +0000)]
Merge "Make IPTest::isNotIPAddress a test function"
Brad Jorsch [Sun, 18 Mar 2018 01:41:45 +0000 (21:41 -0400)]
RevisionStore: Properly encode timestamp
Timestamps in database queries need to be passed through
IDatabase::timestamp().
Change-Id: I1c163d80b2111a78bb10de49fe06d61f02c21d07
Max Semenik [Sat, 17 Mar 2018 05:30:19 +0000 (22:30 -0700)]
Normalize PHPDoc attributes
Change-Id: I83e686d099de0ff0aacda7e332972e1c7ee49f04
Max Semenik [Sat, 17 Mar 2018 00:26:17 +0000 (17:26 -0700)]
Don't use deprecated NS_IMAGE*
Change-Id: I3df00c8e55a79baa3f3727daeea6c9441113aebc
Timo Tijhof [Sat, 17 Mar 2018 02:51:26 +0000 (19:51 -0700)]
api: Restore fractions for default siprop=dbreplag response
Without this fix, the situation is that a query for 'siprop=dbreplag'
shows a db server with lag=0, while the siprop=dbreplag&sishowalldb=1
query shows the same server having non-zero lag. That's confusing.
Regression from r47865 (
88f35cdf6e).
Change-Id: Id4854c8453206a6f1f8c369dc89c3a8bd910f83f
jenkins-bot [Fri, 16 Mar 2018 23:09:41 +0000 (23:09 +0000)]
Merge "Remove #toc and #toctitle style definitions"
Derk-Jan Hartman [Tue, 20 Feb 2018 16:39:32 +0000 (17:39 +0100)]
Remove #toc and #toctitle style definitions
Styling by class is preferred over styling by id as a rule, and
this #toc .toc doubling up was in place since 2005.
(
28c9a747797dae738cd6fe1a97c29e1591564112)
13 years of cache protection seems adequate :)
Bug: T187808
Change-Id: I7f3a9df7f9fd1178986112b5265b1ae7c991d5b7
Aaron Schulz [Thu, 15 Mar 2018 21:29:29 +0000 (14:29 -0700)]
rdbms: small code cleanups to Database
Change-Id: I831b4ea1ced425b5008f3dd2802e16bb898f4b47
Umherirrender [Fri, 16 Mar 2018 17:40:38 +0000 (18:40 +0100)]
Rename ApiTestCaseUpload
Having *TestCase at end of the class name makes it easier to detect it
as TestCase class
Change-Id: Ic7ae7328e729c0f2d97afcf7c5be793d6a8df58f
Željko Filipin [Fri, 16 Mar 2018 15:55:06 +0000 (16:55 +0100)]
Selenium: configuration file for beta cluster
Bug: T185011
Change-Id: Icf8588f0e82acf09e6654dd5dbf33479e4ecbbb7
Umherirrender [Fri, 16 Mar 2018 16:54:14 +0000 (17:54 +0100)]
Make IPTest::isNotIPAddress a test function
Only functions starting with test* are running by phpunit
Change-Id: I91061fb3d5132c61e01e368fab0df0e511b1920d
Umherirrender [Fri, 16 Mar 2018 16:45:33 +0000 (17:45 +0100)]
Remove @group from ParserTestRunner
It should be used on phpunit tests only
Change-Id: I0f73690606c8b92ce65ed1324394f5523b8156f5
jenkins-bot [Fri, 16 Mar 2018 12:17:39 +0000 (12:17 +0000)]
Merge "Fix User::idFromName() ignoring cache for non-existent users."
Alexia E. Smith [Thu, 15 Mar 2018 15:24:21 +0000 (10:24 -0500)]
Fix User::idFromName() ignoring cache for non-existent users.
This fixes a database time out issue where User::idFromName()
is repeatedly called from Special:Import due to the user not existing.
The response is cached as null, but isset() will return false on a null key.
Bug: T189786
Change-Id: I78705089a25dfec84d3c75bedaf623b1e5ee82c4
jenkins-bot [Fri, 16 Mar 2018 11:21:56 +0000 (11:21 +0000)]
Merge "tests: fix Specified key was too long; in a schema"
Gergő Tisza [Fri, 16 Mar 2018 00:38:52 +0000 (17:38 -0700)]
Fix variable name in NewFilesPager::getQueryInfo
Bug: T189846
Change-Id: I11763ecbfd391deea0494386c0d7c1cb9861aa81
jenkins-bot [Thu, 15 Mar 2018 23:48:16 +0000 (23:48 +0000)]
Merge "Add cleanup for handlers cache."
Stanislav Malyshev [Thu, 15 Mar 2018 22:36:47 +0000 (15:36 -0700)]
Add cleanup for handlers cache.
This is useful if we want to override certain handler for testing
but have no way to remove it from cache otherwise.
Change-Id: Icb4f9360c6e7684d15e91a7009a5a3a3e2febb37
Antoine Musso [Thu, 15 Mar 2018 21:23:40 +0000 (22:23 +0100)]
tests: fix Specified key was too long; in a schema
MediaWikiTestCaseSchemaTest.sql comes with two varchar(255) combined
into a PRIMARY KEY.
That caused the testsuite to fail creating that imagelinks table with:
Error: 1071 Specified key was too long; max key length is 767 bytes
That is the limit for innodb and when using UTF8 it allows up to 3 bytes
per characters. Hence the key can be (255+255)*3 = 1530.
One can tune MySQL to bump the limit to ~ 3k with:
innodb_large_prefix = 1
Seems easier to just use smaller fields.
Change-Id: Ic4965b9eddc7ad9105c896e678ab9048ce0be8ef
Translation updater bot [Thu, 15 Mar 2018 21:03:48 +0000 (22:03 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: Ief5597ccece86f3f249cf7db7511be269572986c
jenkins-bot [Thu, 15 Mar 2018 16:07:28 +0000 (16:07 +0000)]
Merge "Fix field names and behavior in SlotRecord."
Volker E [Thu, 15 Mar 2018 04:19:34 +0000 (21:19 -0700)]
Remove `-webkit` vendor extension for browsers earlier than 2012
Removing `-webkit-gradient` value as it was just around for
Safari 4-5 and Chrome 2-9 both in released release range earlier
than 2012.
There's still the fallback color, which should be chosen wisely
anyways without cluttering our CSS output.
Change-Id: Iffb23fac44fc87cfb44f1056dfe1e4e1add444b4
Volker E [Thu, 15 Mar 2018 00:45:12 +0000 (17:45 -0700)]
Replace abandoned text color with WikimediaUI color palette one
Replacing leftover `#252525` with `#222` for code sanity.
Change-Id: I0406845950c65babe34672a6a59a390f56687e46
jenkins-bot [Thu, 15 Mar 2018 00:26:14 +0000 (00:26 +0000)]
Merge "resourceloader: Fix mw.loader to compute combined version in packed order"
jenkins-bot [Thu, 15 Mar 2018 00:16:19 +0000 (00:16 +0000)]
Merge "Convert MarkpatrolledAction to use OOUI"
jenkins-bot [Wed, 14 Mar 2018 23:13:41 +0000 (23:13 +0000)]
Merge "Revert "Let install.php detect and inject extensions""
jenkins-bot [Wed, 14 Mar 2018 22:33:58 +0000 (22:33 +0000)]
Merge "resourceloader: Make doRequest() an inline helper for batchRequest()"
Hashar [Wed, 14 Mar 2018 22:10:20 +0000 (22:10 +0000)]
Revert "Let install.php detect and inject extensions"
It is blatantly broken using non existing variables:
$installer and $options
This reverts commit
53926d9d6940eaa7f40086c9cf89af7bc726e73f.
Bug: T189567
Change-Id: Idd1f002bbd5d474f9fe386cd0e326e1fbfdec097
jenkins-bot [Wed, 14 Mar 2018 21:49:50 +0000 (21:49 +0000)]
Merge "resourceloader: Fix mw.loader to compute version for current request only"
Translation updater bot [Wed, 14 Mar 2018 20:55:25 +0000 (21:55 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: Id3b91ddc41a35c360ca3acb8fc2cf1c35321c967
jenkins-bot [Wed, 14 Mar 2018 20:46:58 +0000 (20:46 +0000)]
Merge "resourceloader: Fix broken code in load.php mock used to make a test fail"
jenkins-bot [Wed, 14 Mar 2018 20:12:40 +0000 (20:12 +0000)]
Merge "resourceloader: Simplify load.mock.php by removing includes dependency"
jenkins-bot [Wed, 14 Mar 2018 16:17:41 +0000 (16:17 +0000)]
Merge "WikiTextStructure: Exclude <style> tags"
jenkins-bot [Wed, 14 Mar 2018 14:09:01 +0000 (14:09 +0000)]
Merge "Correct PHP version in maintenance/dev/README"
jenkins-bot [Wed, 14 Mar 2018 10:18:08 +0000 (10:18 +0000)]
Merge "Fix typo accessable --> accessible"
jenkins-bot [Wed, 14 Mar 2018 09:57:18 +0000 (09:57 +0000)]
Merge "rdbms: allow construction of Database objects without connecting"
jenkins-bot [Wed, 14 Mar 2018 08:52:39 +0000 (08:52 +0000)]
Merge "Let install.php detect and inject extensions"
jenkins-bot [Wed, 14 Mar 2018 08:37:27 +0000 (08:37 +0000)]
Merge "Add Russian grammar forms to support Wikiversity"
Jayprakash12345 [Wed, 14 Mar 2018 05:24:09 +0000 (05:24 +0000)]
Fix typo accessable --> accessible
Change-Id: I239a15e1bc3a311d9284285a04094afae331111d
jenkins-bot [Wed, 14 Mar 2018 02:06:55 +0000 (02:06 +0000)]
Merge "rdbms: Add $join_conds to IDatabase::estimateRowCount()"
jenkins-bot [Wed, 14 Mar 2018 01:57:50 +0000 (01:57 +0000)]
Merge "rdbms: inject the mysql index name aliases into Database"
jenkins-bot [Wed, 14 Mar 2018 01:48:32 +0000 (01:48 +0000)]
Merge "resourceloader: Clean up and better document module list (un)packing"
Aaron Schulz [Thu, 8 Mar 2018 21:38:10 +0000 (13:38 -0800)]
rdbms: avoid throwing exceptions in Database::close() on reconnect
The check caused problems in reconnect() calls from rollback()
triggered by LBFactory::rollbackMasterChanges(). Since callbacks
are suppressed at that time, handleSessionLoss() does not consume
all of them, so the the call to open() triggered a close() call
that would error out since the callbacks are still there.
Only do that check if a connection was present beforehand.
Check for callback suppression before trying commit() too.
Also make writesOrCallbacksPending() check trxEndCallbacks.
Bug: T188875
Change-Id: Ia46d30d75132358a0b4f60e847937013781c1daa
Translation updater bot [Tue, 13 Mar 2018 20:58:50 +0000 (21:58 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: I627c60bb104653ae4203b4a89b28fc2fdee3f96b
daniel [Tue, 13 Mar 2018 19:32:18 +0000 (20:32 +0100)]
Avoid dumping big stack traces from unit tests.
Using assertFalse( $exception ) results in potentially very large stack
trace dumps, when $exception is an exception from deep inside MediaWiki.
Stack trace dumps include a dump of all parameters, resulting in console
output of (in my case) over 160 MB.
Change-Id: Ib78493d86531844a892586c071b2fa631ad765eb
Brad Jorsch [Tue, 13 Mar 2018 18:57:45 +0000 (14:57 -0400)]
WikiTextStructure: Exclude <style> tags
They aren't part of the article content, so they shouldn't be indexed
for search.
Bug: T189528
Change-Id: I3203f1f415eb821975098057d75c0e535b1fc76c
Leszek Manicki [Tue, 13 Mar 2018 12:39:14 +0000 (13:39 +0100)]
Correct PHP version in maintenance/dev/README
Change-Id: I1d52f60c20317efb3c36763fd47be09dc6dd57dd
Antoine Musso [Tue, 13 Mar 2018 09:16:15 +0000 (10:16 +0100)]
Let install.php detect and inject extensions
For CI, I went with a dirty trick to find extensions and either
wfLoadExtension() or include them. That has since been ported to
Installer::findExtensions() and LocalSettingsGenerator::getText().
The WebInstaller() relies on that to detect extensions and let the user
tick the ones to be installed.
Add --with-extensions to install.php so one can include all extensions
from the command line.
Note: The CliInstaller always wfLoad() any skin it can find.
Bug: T189567
Change-Id: Ic8aa6bea4e7294e54fd5b71df267c9e934f78fce
Translation updater bot [Mon, 12 Mar 2018 20:55:41 +0000 (21:55 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: I2deac4d88a30eab3503ff74147c92f429d554574
Aaron Schulz [Sat, 17 Feb 2018 22:09:02 +0000 (14:09 -0800)]
rdbms: inject the mysql index name aliases into Database
Also added LBFactory::setTableAlias() for consistency with this
Change-Id: Ie49003ff8fd5b99f75db9fae8fe0a184444254d4
jenkins-bot [Mon, 12 Mar 2018 18:51:47 +0000 (18:51 +0000)]
Merge "Switch Block::purgeExpired to use AutoCommitUpdate"
Brad Jorsch [Mon, 12 Mar 2018 16:15:14 +0000 (12:15 -0400)]
rdbms: Add $join_conds to IDatabase::estimateRowCount()
So queries with joins can be estimated.
Change-Id: I9163cf9005d2c2001a88bb102eb4142f0322b0df
jenkins-bot [Mon, 12 Mar 2018 12:12:04 +0000 (12:12 +0000)]
Merge "Fix typo acceptible --> acceptable"
Jayprakash12345 [Mon, 12 Mar 2018 10:53:20 +0000 (10:53 +0000)]
Fix typo acceptible --> acceptable
Change-Id: I9304d830744238e4733240dd65c285fed61d0416
zoranzoki21 [Sun, 4 Feb 2018 18:20:14 +0000 (19:20 +0100)]
Add Hijri calendar in Urdu
Allows users to select their preferred calendar from Preferences.
Bug: T145266
Change-Id: I7299d0f84f629da781b100c4db6b2a30a7d1bb86
jenkins-bot [Mon, 12 Mar 2018 03:06:30 +0000 (03:06 +0000)]
Merge "Remove "dash" case in preprocessToObj"
jenkins-bot [Mon, 12 Mar 2018 00:53:28 +0000 (00:53 +0000)]
Merge "Move phpunit @group from file comment to class comment"
Translation updater bot [Sun, 11 Mar 2018 20:56:39 +0000 (21:56 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: Iea4877827a12d5117578b8d1a6f1fefc53e1cdeb
Translation updater bot [Sat, 10 Mar 2018 20:54:04 +0000 (21:54 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: I556a81048fbf024a585f7f586b91762eea568cb3
Amir Sarabadani [Sat, 10 Mar 2018 19:43:03 +0000 (20:43 +0100)]
Convert MarkpatrolledAction to use OOUI
Bug: T189385
Change-Id: I9bd78166be5ab80e58498d2934531185404b17de
Umherirrender [Fri, 9 Mar 2018 21:56:41 +0000 (22:56 +0100)]
Move phpunit @group from file comment to class comment
Remove @group from non tests
Change-Id: Iae9ee3bc5f539a9b4ded8374006ab2993234450e
jenkins-bot [Sat, 10 Mar 2018 01:04:31 +0000 (01:04 +0000)]
Merge "Fix typo in word compatibility"
Aaron Schulz [Fri, 9 Mar 2018 11:32:30 +0000 (03:32 -0800)]
Cleanup some comments related to DB replication
Change-Id: Icaaa5144beaedfebb88530480733a1e916c935fb
Arlo Breault [Fri, 9 Mar 2018 22:46:53 +0000 (17:46 -0500)]
Remove "dash" case in preprocessToObj
This was introduced in
2877402 and removed in
186a182
Change-Id: Ibfa1ae1597bfc50ae6ea49402c7966ca042f12e5
Umherirrender [Fri, 9 Mar 2018 22:04:44 +0000 (23:04 +0100)]
Fix typo in word compatibility
Change-Id: Ic6ad14f5e156311cee3021cd427561dfcd68633a
Translation updater bot [Fri, 9 Mar 2018 21:00:22 +0000 (22:00 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: Ib13656ed8c5c45facc4166963cd4a7f32ade30bf
jenkins-bot [Fri, 9 Mar 2018 20:53:30 +0000 (20:53 +0000)]
Merge "Add ExternalUserNames::getLocal() to get local part of username"
jenkins-bot [Fri, 9 Mar 2018 20:05:34 +0000 (20:05 +0000)]
Merge "API: Update prefixsearch/opensearch docs"
Amir Sarabadani [Fri, 9 Mar 2018 19:32:41 +0000 (20:32 +0100)]
Add ExternalUserNames::getLocal() to get local part of username
Bug: T189320
Change-Id: I99908ced218c45ecbb23f390f065170292f153b4
Brad Jorsch [Fri, 9 Mar 2018 19:44:17 +0000 (14:44 -0500)]
API: Update prefixsearch/opensearch docs
The psnamespace parameter to action=query&list=prefixsearch is ignored
if pssearch has a valid namespace prefix. This should be documented.
Same for the similar parameters to action=opensearch.
Bug: T189139
Change-Id: I4914e5c915e6f8e5c0d5116f034ca671fe45a3f8
Brad Jorsch [Fri, 9 Mar 2018 19:11:36 +0000 (14:11 -0500)]
Remove useless use
A use declaration for a non-namespaced class in a non-namespaced context
causes a PHP warning.
Bug: T189302
Change-Id: I023e64c8194dd03cc3a1098e2d60c73f99bb02e3
Jayprakash12345 [Fri, 9 Mar 2018 13:06:58 +0000 (13:06 +0000)]
Remove deprecated function wfBaseConvert()
Bug: T189300
Change-Id: I5f05beb105dca8e8079e770df562c004a689c2ed
jenkins-bot [Fri, 9 Mar 2018 11:33:10 +0000 (11:33 +0000)]
Merge "Replace wfGetLBFactory"
jenkins-bot [Fri, 9 Mar 2018 10:45:53 +0000 (10:45 +0000)]
Merge "purgeExpiredUserrights: Show number of rows purged"
jenkins-bot [Fri, 9 Mar 2018 10:37:27 +0000 (10:37 +0000)]
Merge "Fix spelling mistake inside API i18n message"
Southparkfan [Thu, 8 Mar 2018 22:13:35 +0000 (22:13 +0000)]
Fix spelling mistake inside API i18n message
Change-Id: Ief2814e2fdd94b68eb6a52261b3aa4f47009bfc6
Aaron Schulz [Wed, 28 Feb 2018 20:56:34 +0000 (12:56 -0800)]
rdbms: allow construction of Database objects without connecting
* Database::factory() supports a $connect parameter, that defaults
to NEW_CONNECTED (current behavior) but can also be NEW_UNCONNECTED.
* Add tests asserting the type of various instances returned from
Database::factory().
* Clean up sqlite "conn" field handling to handle cases of it
not being set, just as other classes do.
* Add some comments about the return type of doQuery().
Change-Id: Ic0837cfdb35326c2045133d664abd29043d48c03
jenkins-bot [Fri, 9 Mar 2018 00:25:59 +0000 (00:25 +0000)]
Merge "rdbms: small cleanups to session loss handling"
jenkins-bot [Thu, 8 Mar 2018 23:15:23 +0000 (23:15 +0000)]
Merge "Update OOUI to v0.25.3"
Brad Jorsch [Thu, 8 Mar 2018 21:46:08 +0000 (16:46 -0500)]
Typo fix
Bug: T189251
Change-Id: I5e7af2629d566181f3280049b6847c0126850ff2
Aaron Schulz [Thu, 8 Mar 2018 20:40:07 +0000 (12:40 -0800)]
rdbms: small cleanups to session loss handling
Split the two callback runner calls in handleSessionLoss() into two
separate try/catch loops. Return the first exception, if any.
Also make the $recoverable check for connection loss in Database::query
slightly more readable by checking the positive rather than the negative.
Change-Id: I75935fa69e40450ac3983f0d3451ab3001650b6f
Translation updater bot [Thu, 8 Mar 2018 21:23:54 +0000 (22:23 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: I34c71561c72518697570e7f35bbf3e5ca6ea8e3d
jenkins-bot [Thu, 8 Mar 2018 20:28:42 +0000 (20:28 +0000)]
Merge "Job::insert: Hard deprecate (soft-deprecated in 1.21!)"
daniel [Tue, 6 Mar 2018 18:46:13 +0000 (19:46 +0100)]
Fix field names and behavior in SlotRecord.
The field names used in SlotRecord got out of sync with the changes made
to the database schema. Nobody noticed, because these fields are not yet
written to or read from the database.
This change also rewrites SlotRecordTest and adds several sanity checks
to SlotRecord, in preparation of the introduction of slot_origin.
Change-Id: Ifdf040174705bf88104b8c922c9d6d1120d48f3a
James D. Forrester [Tue, 6 Mar 2018 17:08:09 +0000 (17:08 +0000)]
Drop OpenSearch::getOpenSearchTemplate(), deprecated in 1.25
Change-Id: Ib76b96cf392b7f9fa38d28173dd2cd170e08a881
Timo Tijhof [Thu, 8 Mar 2018 04:10:13 +0000 (20:10 -0800)]
rdbms: Restore test for Database::setFlag()
Follows-up
b4eb1feed05b5, which inadvertendly replaced the setFlag()
test with the clearFlag() test.
Also move the test to the DatabaseTest.php file given it's only
executing and covering base class.
Change-Id: I2f4ed6c4eeba845eb67013e1ab7d2b2bde863119
Timo Tijhof [Wed, 7 Mar 2018 00:23:06 +0000 (16:23 -0800)]
resourceloader: Fix mw.loader to compute combined version in packed order
The 'version' param was being computed based on the order of the modules list
before we perform string compression. And this compression can change the order
for its optimisation purposes.
Specifically, when requesting modules like 'a', 'b.1', 'b.2' and 'c'. The
version was computed based on a + b.1 + b.2 + c (standard sort order),
whereas the optimised list is 'a,c|b.1,2', which expands to a + c + b.1 + b.2,
which makes hash validation fail.
Bug: T188076
Change-Id: I00d6985c054fecd88acf73041aa02878e83d62bc
Timo Tijhof [Thu, 1 Mar 2018 21:27:37 +0000 (13:27 -0800)]
resourceloader: Make doRequest() an inline helper for batchRequest()
Uses all the current variables, might as well just inline since
its only purpose was to avoid duplication of these three lines
of code within batchRequest()'s for-loop. It's used as private
function elsewhere internally.
Change-Id: Ib016ae412b32b4041d2bd8a74d22b88bf70417b4
jenkins-bot [Wed, 7 Mar 2018 23:54:06 +0000 (23:54 +0000)]
Merge "Self links should not be marked as legacy css"
jenkins-bot [Wed, 7 Mar 2018 23:01:04 +0000 (23:01 +0000)]
Merge "maintainance: Re-order OOUI script to be clearer, rename"