Merge "Preserve 'ooui' query string when overriding"

Merge "Special:Preferences: Create flag to enable OOjs UI"

Preserve 'ooui' query string when overriding

Change-Id: I32df23b427f18e6275beaa6cc10cd58dc3b6eb36

Localisation updates from https://translatewiki.net.

Change-Id: I9929e2ade53186340619aeceece3e1eed6166985

Merge "BotPasswords: Indicate when a password needs reset"

Merge "Fall back to default language when no i18n'ed licenses available"

Merge "tidy: Remove obsolete Depurate and Balancer drivers"

tidy: Remove obsolete Depurate and Balancer drivers

The Html5Depurate driver was intended to be used with an external Java
service, but it never gained traction due to deployment concerns.

The Html5Internal (Balancer) driver was originally intended for use with
the balanced templates proposal and could also handle tidying. But it was
tightly coupled to MediaWiki, so part of it was used as the basis of the
RemexHtml library. Remex most likely can also implement the balanced
templates proposal, so there isn't any reason to keep the Balancer code
around anymore,

Change-Id: I8542d69e9cdbf0e2fb7ebbb919933a64c1b8c293

Merge "jquery.makeCollapsible: Fix typo in documentation"

Merge "selenium: Minor clean-up in preparation for packaging"

selenium: Minor clean-up in preparation for packaging

This is functionally a no-op, purely refactoring (mostly style).

* Consistently require packages at the top of a file.
  (e.g. MWBot in edit.page.js).

* Remove unused .call(this) from mwbot interaction closures,
  which didn't use 'this'.

* Use Node.js regular Promise chaining with then(), instead of
  complex bluebird.coroutine generator function yields, which
  are intended to emulate async-await, but the syntax is quite
  error-prone for inexperienced developers and hard to debug.
  Once we require Node 7+ for the selenium tests, we can use
  async-await here natively, but until then, might as well use
  regular then() syntax, which we already use elsewhere in the
  tests, and is also what MWBot documentation uses.

* Also applied some minor whitespace changes for consistency
  among these files and other MediaWiki JS. E.g. no empty line
  before the first statement of a function. Add a new line between
  different methods, and between the end of a class and the
  export statement.

* Remove 'use strict' from test files. The patterns that would expose
  the bad non-strict behaviour are mostly already forbidden by ESLint,
  and the run-time optimisation to disable non-strict can't be noticed
  in tests (more useful in prod where e.g. the same process would run
  a function 1 million times). Main reason here is to keep things
  simple for new-comers and reduce boilerplate, given that these tests
  will mainly be worked on by browser-JS developers, not Node.js devs,
  and we don't currently use strict mode in our front-end code, either.

* Remove unused bluebird dependency.

Bug: T193088
Change-Id: I59f9211299e8e884c28c7733bcee3b7b28542610

jquery.makeCollapsible: Fix typo in documentation

Change-Id: Iaf53f0e460bc8684f0925c2f3963d988f8624aa4

Merge "HTMLTitleTextField: Allow this field to not be required"

Merge "RELEASE-NOTES-1.32: Follow-up f1d0d364c4d, explain this is alignment not quality loss"

Merge "mediawiki.util: Drop updateTooltipAccessKeys(), deprecated since 1.24"

RELEASE-NOTES-1.32: Follow-up f1d0d364c4d, explain this is alignment not quality loss

Change-Id: I03f6db941516a74c1f16aeb8158a6445ebc7b5a9

Merge "mediawiki.util: Drop escapeId(), deprecated since 1.30 and unused"

Merge "RELEASE-NOTES-1.32: Some cleanup"

Merge "Use BlockListPager::getLimitSelectList() in Special:BlockList and Special:AutoblockList"

Merge "Sort user groups in Special:Listusers"

Merge "jquery-tablesorter and sortable classes always added together"

jquery-tablesorter and sortable classes always added together

If absent, jquery-tablesorter should add the sortable class.
The sortable class should be used to avoid FOUC, but clients
may need time to update their interfaces to do so.

Bug: T194045
Change-Id: I76709bf30ab06ff5ee7fc18bed1f143002477529

Merge "rdbms: fix finalization stage errors in LBFactory::commitMasterChanges"

mediawiki.util: Drop updateTooltipAccessKeys(), deprecated since 1.24

Bug: T193901
Change-Id: I72d82b8c3a073c4ecde24f88bb5a2027327adb0c

mediawiki.util: Drop escapeId(), deprecated since 1.30 and unused

Bug: T193901
Change-Id: Ifef37d2f0ccf8acbe36900a25d2a51864083f11f

RELEASE-NOTES-1.32: Some cleanup

Change-Id: Id58d0d18116a82bccf6b9aabc695ae36a668d4e4

HTMLTitleTextField: Allow this field to not be required

Previously we would always run the validation, and it would fail when
the field was empty, since empty string is not a valid title.

Respect the 'required' option (defined by HTMLFormField) and make it
default to true for compatibility with existing forms that might rely
on this.

Also add a TODO comment about a confusing special case in validation
code. I don't want to dig into that.

Change-Id: I93ad51ffe7bee597d2d127f4c5d6b2929ffc8f7e

Merge "Replace deprecated ObjectCache and CommentStore calls in filerepo"

rdbms: fix finalization stage errors in LBFactory::commitMasterChanges

If a pre-commit callback caused a new LoadBalancer object to be created,
that object will be in the "cursory" state rather than the "finalized"
state. If any callbacks run on an LB instance, make LBFactory iterate
over them all again to finalize these new instances.

Make LoadBalancer::finializeMasterChanges allow calls to
already-finalized instances for simplicity.

Bug: T193668
Change-Id: I4493e9571625a350c0a102219081ce090967a4ac

tests: Add some more @covers tags

Change-Id: I84b66879773d97593152b683ff69d034ed67aa3d

Don't globally disable PHPCS's prohibition of assert()

Whitelist the remaining usages of assert(), and reinstate the PHPCS sniff
that forbids usage of it. Add FIXME comments as well, so any casual readers
of the code will not think that the disabling and usage is intentional.

Change-Id: I7cabe715c0e6aa6a9ef3ffe5657f3de7fd8e662b

Merge "Special:ListFiles: Convert to use OOUI"

resourceloader: Deprecate OutputPage::addModuleScripts()

And the counter-part in ParserOutput as well.

This follows-up on 80e5b160e0985, which removed the last use of this
method. It also deprecated ResourceLoaderClientHtml::setModuleScripts()
but that isn't usually used directly. The public interfaces are
mainly in OutputPage/ParserOutput, which are now deprecated as well.

Bug: T188689
Change-Id: I90baafdfc089ac03c4c3abc12065c0b60d43994e

Special:ListFiles: Convert to use OOUI

Bug: T117743
Change-Id: I1e61dc80e83f0aac9ec6a4e98837ca730b6946de

Replace deprecated ObjectCache and CommentStore calls in filerepo

Change-Id: I0e863dc8e4bc6dac938a2ed437c2691aace19690

Localisation updates from https://translatewiki.net.

Change-Id: I4088291349d08d9aa1ea2510b04cebd16a2fb085

Merge "Remove deprecated SqlDataUpdate class"

Remove empty skins folder from phan config

Similar to extensions folder there is nothing to do

Change-Id: I5bd9a8f70fb020325c892f04e3eac7a3e46d772a

Merge "rdbms: set cpPosIndex to null in ChronologyProtector::shutdown on failure"

rdbms: set cpPosIndex to null in ChronologyProtector::shutdown on failure

This already happens on lock acquisition failure but not on write failure.
If the write failed, then there is not point timing out on the next request
after waiting "waitForPosStoreTimeout" seconds.

Also raise the initPositions() logging for missing positions to WARNING
and improve the wait loop log entries in that method.

Change-Id: Id738eb6f85c91dd05fadd2e6c3f19ae3204e2575

Fix class/function case mismatches

Change-Id: I25632d4db5a451975cb7a678372d8675c28c0897

Merge "rdbms: consolidate LBFactory code for running callbacks"

Localisation updates from https://translatewiki.net.

Change-Id: I98b5541df86b2be47d7d3cf74871bbb0a902f84e

Merge "mw.widgets.visibleLengthLimit: Do not show countdown until last 99 characters"

Remove deprecated SqlDataUpdate class

Deprecated since 1.28

Change-Id: I4dcbc8fcfdb99caa0469e7cf63745b94e19cd21d

skins: Move default style modules to getDefaultModules

This advances T140664 as well, because it encourages module
loading logic from the skin to be in this single method.

Update the tests for setupSkinUserCss(), to now target
getDefaultModules() instead, given setupSkinUserCss() no longer
provides these behaviours. Had to move where the mock object
was created so that it can be injected in the skin (previously
it could be passed as parameter). Besides, its generally best-
practice to create mocks and stubs within the test anyhow, not in
the data provider.

Bug: T140664
Depends-On: Ib2b19ba165a9d646a770702cdf1724509156b93e
Change-Id: I3404c1c2a7e8eae0b803b31f333cb9b837f43d4a

rdbms: consolidate LBFactory code for running callbacks

This makes sure rollbackMasterChanges() has the same looping logic
for consistency.

Also clarify some code comments.

Change-Id: Id34ba8ff7377c9f3842e2db675c8050a6e68407e

Localisation updates from https://translatewiki.net.

Change-Id: I8b952979bdd5b13e3d0267c106439f8c27e46433

Merge "Replace wfGetLB"

Merge "Add Ambonese Malay (abs) to Names.php"

Merge "resourceloader: Improve coverage of SkinModule"

Merge "API: Add ApiParseTest case for 'styles' in getDefaultModules"

resourceloader: Improve coverage of SkinModule

* Ignore getLogoData() which is a one-line proxy to ::getLogo,
  that can't really be tested because static methods can't be stubbed.
  It exists so that this method can be stubbed instead. The actual
  method is already covered.

* Simplify @covers for getStyles() to allow indirect coverage within
  the class because it's intended as a higher-level integration tests.
  The other tests in the suite still cover a specific method only.

Change-Id: I1445a016c1f12a6d8ceaaf745023a28cf20e5371

Merge "jsjpegmeta: Move export patch to a separate file"

Merge "CLDRPluralRuleParser: Move from src/ to lib/ without local patch"

Merge "moment: Put src files for moment in their own directory"

jsjpegmeta: Move export patch to a separate file

Keep the main file in src/ for now, because it's got some whitespace
changes and a closure-wrap that make it differ from upstream's
version. Those changes are hard to review both for diffing and for
functional differences due to its odd-looking scope assignments that
I'd rather not change as part of this.

Change-Id: I248831cfa984432d0a30aa923a9bcd98029b05c4

Merge "Use proper namespace name on Special:RelatedChanges"

BotPasswords: Indicate when a password needs reset

Certain things, such as changing the account's main login credentials,
causes all bot passwords to be invalidated. This state should be
indicated in Special:BotPasswords, and the API when login fails.

Bug: T193829
Change-Id: Ib12929fed861742c9f2f76702c9ac3254e8a5d97

Add Ambonese Malay (abs) to Names.php

Bug: T193566
Change-Id: I2c3b3812f5f2e4bb66c48da2b442194003eabb67

Merge "Add 'logid' parameter to Special:Log"

CLDRPluralRuleParser: Move from src/ to lib/ without local patch

Follows-up 648667ac9f, which didn't move this module because
it had a local patch for exposing via module.exports and mw.lib
(instead of its default 'pluralRuleParser' global).

Restore the file back to a clean copy from upstream, and perform
the export via a separately concatenated file instead, using the
same pattern we already use for 'oojs' and 'moment'.

Change-Id: I27ee80dc34e0ad5206cf9c1ce68be3ec8811ecf8

moment: Put src files for moment in their own directory

Change-Id: Id4eaf32186e4a9414bb25cb6c08dc18ae4a8ad92

Add 'logid' parameter to Special:Log

Make it possible to get a URL for a single log entry,
as an alternative to using Special:Redirect/logid/123.

The existing single-log-entry link of Special:Redirect/logid/123
is also simplified to be a redirect to this new parameter.

Bug: T191608
Bug: T187638
Change-Id: I5f2e52531cd2ba617a25570e40aa8c5168e284d9

Merge "resourceloader: Improve test coverage of ResourceLoader::register()"

resourceloader: Improve test coverage of ResourceLoader::register()

Verify that calling register() twice does not throw, but warns,
and that the last registration wins.

This behaviour was actually surprising to me because it used to
throw, and I'd assume that when we added the warning, the behaviour
would go from fatal to non-fatal, but keep that the last one is
at fault/unsupported.

Perhaps b1ea0612 / d3e3bcfd6 (T116628) should've added a return
statement. Oh well, we can consider changing that later, but at
least test for it.

Change-Id: I955132868146ea5bf88c9b9e648c84d8196cb1f9

Merge "Use AutoCommitUpdate in LinksUpdate::doUpdate"

API: Add ApiParseTest case for 'styles' in getDefaultModules

Follows-up a01d8be82c.

Bug: T140664
Change-Id: Ic4151a548884d10a2302af49d89e9e36fd6766fd

Use AutoCommitUpdate in LinksUpdate::doUpdate

The hook handlers are likely to write to secondary databases, in which
case it is better to wrap the callback in its own transaction round.

This lowers the chance of pending write warnings happening in
runMasterTransactionIdleCallbacks() as well as DBTransactionError
exceptions in LBFactory due to recursion during commit.

Bug: T191282
Bug: T193668
Change-Id: Ie207ca312888b6bb076f783d41f05b701f70a52e

Skin: Remove the cache-compat remainings of mediawiki.sectionAnchor

Follows-up 5205405385397782b5. Two 2 years seems long enough :)

Bug: T18691
Change-Id: Id2f17312cbfe3ee9dda68bf8f9bcf94e18b003c6

Merge "Silence TransactionProfiler in MediaWiki::triggerSyncJobs"

Merge "Make DeferredUpdates avoid running during LBFactory::commitMasterChanges"

Merge "resourceloader: Remove remnants of "dependencies as a function""

Merge "resourceloader: Remove creation of dynamic-styles marker"

resourceloader: Remove remnants of "dependencies as a function"

This hasn't been used in a while, at least two years from what
I could find. It also doesn't have tests, and numerous paths
that possibly should handle it, don't.

This was originally implemented for TMH as a way to dynamically
change what it loads based on the browser environment.

However, this has been addressed by two other aspects already:

* mw.loader.using() - During development of ResourceLoader, this
  method was unstable for a short while. During that pre-release
  time, dynamic dependencies was added as workaround for TMH.
  It being stable addresses TMH's use case of loading *different*
  modules (with callback) based on environment factors.
  This allows complete mutation of the array passed: remove,
  replace, and add operations. This can be used to completely
  change was a module does based on browser environment,
  e.g. load a decoder for X instead of Y, but places the logic
  in the startup module, which is inefficient.
  TMH has switched long ago to using using() instead.

* 'skipFunction' - Introduced in more recent years, this was
  added to more efficiently address the 'remove' use case.
  E.g. where the 'needs' are consistent between browsers, but
  a subset of the needs may be available natively without
  needing a polyfil. The skipFunction allows once to mark
  a module as "ready" without actually loading anything.
  This has been used for efficient and conditional loading of
  DOM2, JSON and ES5 polyfills. (All of which have been removed
  since, but that's a separate story.)

Bug: T192623
Change-Id: Ia97df72a0c3f0fc0dc917fce0a94213b23db4daa

Make DeferredUpdates avoid running during LBFactory::commitMasterChanges

Bug: T193668
Change-Id: I50890ef17ea72481a14c4abcd93ae58b93f15d28

RELEASE-NOTES-1.31: More cleanup

Change-Id: I645d6591424fb116439e85c28eaf52142583ba0b

Merge "RELEASE-NOTES-1.31: Add notes for I94377faa51"

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: Ie50ed1c556acb37d00c94b55877196df3b30e9d5

Merge "RELEASE-NOTES-1.31: Add notes for I8228cdd3b"

Merge "RELEASE-NOTES-1.31: Add notes for Ia76dd6db363"

mediawiki.jqueryMsg: Remove deprecated window.gM() method

* Deprecated with warnings since MediaWiki 1.23 (released 2013).
* Zero results for 'gM(' in mwgrep (sitewide scripts on public WMF wikis).

Bug: T135148
Change-Id: I3b97355012fc8741c7f4953f4b249d70beb96b01

Merge "CRH Transliteration Pattern Matching Fixes"

Special:Preferences: Create flag to enable OOjs UI

This reverts commit 808e45d13d400256d36cfcd95e79a567197d9a8b.

Bug: T117781
Change-Id: I152b82bcd647d97062eb82cd2d1064609124f9bc

Merge "Use OOUI form for edit watchlist"

Merge "selenium: Run wdio directly without grunt"

Merge "sortable tables/mw-collapsible no longer causes page jump"

RELEASE-NOTES-1.31: Add notes for I94377faa51

Change-Id: Icb7223864fd2bb3c5f2c961f3526d4f70e34bf7f

RELEASE-NOTES-1.31: Add notes for I8228cdd3b

Change-Id: I8a7faa3486055c137c6918aa75481aa64a02b856

RELEASE-NOTES-1.31: Add notes for Ia76dd6db363

Change-Id: Id661a75205b2c799afb02485ac2e408869eab093

RELEASE-NOTES-1.31: Wrap to 80 chars and fix up

Change-Id: I0a090a3ac057021ac2e3544f5e05b2fadd0b9155
(cherry picked from commit ccc3d1fc0570c6e3057562b1583bd743aef5d9fb)

Merge "Make DeferredUpdates re-throw exceptions during testing."

Make DeferredUpdates re-throw exceptions during testing.

This is a stop-gap solution for the problem described in T190178.
A better solution would probably include changing the behavior
of LegacyLogger.

Bug: T190178
Change-Id: I433be04b8ee725becd174e567270aa674d2661df

Merge "Remove $wgExperimentalHtmlIds and related code, deprecated in 1.30"

Replace wfGetLB

@deprecated since 1.27

Change-Id: Ibdd49fdfc0d1511503e1ed2173a592c612996c53

Use OOUI form for edit watchlist

Change-Id: I2e2aa099e7941e1685dcaca37842b12a3469d5f3

Merge "Remove three deprecated functions from the Revision class."

Localisation updates from https://translatewiki.net.

Change-Id: Ib865977cd3bd5c326b9005397a165bc250c24ea5

Remove three deprecated functions from the Revision class.

This commit removes Revision::getRawUser(), Revision::getRawUserText(), and Revision::getRawComment(). All three were deprecated in 1.25 and are only used in unmaintained extensions.

Bug: T61113
Change-Id: I729985b791df89bc7e577fb823e647c48549e637