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

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

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"

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"

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"

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

sortable tables/mw-collapsible no longer causes page jump

Relatively straightforwardly - we move padding-right definition into
top styles to avoid a reflow

This turned out to be a little tricker than first imagined.
The code in jquery.makeCollapsible.js finds elements marked up
as collapsible and then 1) collapses them 2) inserts a toggle button.

To avoid a reflow on the page, we have to make sure the collapsed content
is collapsed prior to the code running and reserve space for the toggle
button.

The toggle button is tricky as it is subject to i18n. However we can
access this label via a LESS variable and use a pseudo element (which
is supported by the majority of modern browsers) to preserve the space
we need. When the JavaScript has loaded there is no need for that CSS
any more.

Note on slow connections, collapsed content will not be accessible until
the JavaScript has fully loaded, however this is arguably less of a problem
then the existing reflows.

Note, there are many many many usages of mw-collapsible, because of this
this does not claim to address all reflows in all usages - but it does provide
support for the more widespread usages.

Follow up patches can be added for specific situations - for example table
of contents toggle.

Bug: T42792
Bug: T42812
Depends-On:  I3388c3c4f91cdbab11e89cdc95973b688d3f1ce7
Change-Id: If9c8f0974e3a4b08e4a66d37f7f5adf67d73054e

Silence TransactionProfiler in MediaWiki::triggerSyncJobs

Change-Id: Id04e7d25cbf7a4d05dfd80c7a32965312978ef39

Merge "CSSMin: Add more edge cases with spaces to CSSMinTest"

CSSMin: Add more edge cases with spaces to CSSMinTest

Change-Id: Id5e884711ab69615273dd050d83c65a385aaa817

selenium: Apply relPath() to the exclude option for CirrusSearch

It seems relPath() is used for all other paths here. When I was
running a single test from the /tests/selenium, I noticed wdio
was logging to the console that it excluded ./extensions/CirrusSearch,
which doesn't make sense from that directory. The other paths
are fine.

Change-Id: Ic89ae0a91cdb6590b7190ea44969e653e9e2d6ea

Merge "Upgrade QUnit from 2.4.0 to 2.6.0"

Merge "selenium: Increase mochaOpts.timeout from 20s to 60s"

selenium: Increase mochaOpts.timeout from 20s to 60s

I was unable to get the tests passing locally in MediaWiki-Vagrant.
The main slowdown appears to be from MWBot logging-in for each
command anew. Which works fine, but just seems to be quite slow.

Change-Id: I3c0abc79cac48c26d952b0bbe73574aa48b2d70e

Declare LogFormatter::$parsedParameters property

And rework LogFormatterTest so it doesn't need to access that property.

Change-Id: I4d429430efe74dc083019a1a8bb07257785484b9

Merge "Cleanup the element matches in doBlockLevels a bit"

Remove $wgExperimentalHtmlIds and related code, deprecated in 1.30

Bug: T139744
Change-Id: Ia15d5ab6e7637fd40d5c3399822a3dbeb7b383b5

Merge "mediawiki.api: Check that query exists in api.getToken() response"

Merge "rdbms: improve log warnings in runMasterPostTrxCallbacks()"

mediawiki.api: Check that query exists in api.getToken() response

It's possible that response.query does not exist when requesting a
token. This patch removes the assumption and handles the scenario in the
same way as a missing token.

Additionally, responses with a queryless response are now deleted from the cache.

Corresponding console warning:

> jQuery.Deferred exception: Cannot read property 'tokens' of undefined
>   at apiPromise.then.promise.abort (/w/resources/src/mediawiki/api.js)

Change-Id: Ia29e0c0d657bf4b3d94f1d463b942451eebd68b4

Merge "Use OOUI form for 'clear' and 'raw edit' of watchlist"

rdbms: improve log warnings in runMasterPostTrxCallbacks()

Bug: T191282
Change-Id: Iba663c58224af920f90d7b401aab2eb21f921941

Upgrade QUnit from 2.4.0 to 2.6.0

Source
 https://code.jquery.com/qunit/qunit-2.6.0.js
 https://code.jquery.com/qunit/qunit-2.6.0.css

Changelog
 https://github.com/qunitjs/qunit/blob/2.6.0/History.md

Highlights:
* 2.4.1: Fixed various bugs in HTML interface.
* 2.5.0: Added assert.rejects.
* 2.5.1: Fixed HTML reporter to reset attributes on qunit-fixture.
* 2.6.0: Changed behaviour to fail if no test suites exist.

Change-Id: I24120a74094db358f02f9fc1935920c43a0a7ced

Localisation updates from https://translatewiki.net.

Change-Id: Ib363fb3f98d2272ffca8954758ee4d22db0c7e14

Merge "Fix typo in postgres patch-drop-ar_text.sql"

Merge "Have ClassCollector ignore PHP 7 anonymous classes"

Merge "rdbms: enforce and improve LBFactory/LoadBalancer callback handling"

Have ClassCollector ignore PHP 7 anonymous classes

PHP 7 introduces anonymous classes with a syntax like

 $instance = new class() extends Foo { ... };

ClassCollector is incorrectly detecting this as a class named
"() extends Foo". This patch fixes that by having it ignore "new class"
in much the same way it currently ignores "::class".

Change-Id: I4d1985a9c04be71f7bea6cb7b61dcea74f44a6e2

Merge "Make password reset & reset token buttons destructive"

Merge "Revert "Remove cssclass in the favor of 'type' => 'user'""

Merge "Fix selector to align all button types"

Fix MySQLism in populateRevisionLength.php

Use addQuotes() to make a quoted string instead of MySQL's nonstandard
use of double-quotes.

Change-Id: Ie2d017a33b694417a3db140b2798a8d55f911985

Make password reset & reset token buttons destructive

Change-Id: I068d5bf7a8882348866c9beae7d109c28ea17b4e

Fix selector to align all button types

Bug: T193474
Change-Id: I02dbe5d880567c6e7e59c867c3ab910600c667d0

Revert "Remove cssclass in the favor of 'type' => 'user'"

This reverts commit ba08a9146898a1904101209551fa7ef67a7b64e8.

Bug: T193378
Change-Id: I3ceec6d70cf12515985ee6b56458ae7132ef3d8b

mediawiki.util: Remove deprecated wikiGetlink()

* Deprecated since MediaWiki 1.22, with console warnings
  since 1.23 (released 5 years ago).
* All uses in core, tarball components, and wmf-deployed components
  have been fixed.
* All uses in sitewide and user scripts on public WMF wikis
  have been fixed (Wow!).

Bug: T57764
Change-Id: I926966a83daadfee3adbb9a32acbb5b710a29686

rdbms: enforce and improve LBFactory/LoadBalancer callback handling

* Handle the case where an onTransaction* callback for one handle
  adds more onTransaction* callbacks to a different handle. Instead
  of supporting only a short chain of such callbacks, try to resolve
  the whole chain by using a loop in LoadBalancer and LBFactory.
* Add sanity checks to enforce the proper call order of LoadBalancer
  transaction methods, such as those that execute callbacks. This is
  the order that LBFactory already uses. Use ROUND_ERROR for problems
  that can ruin the instance state. Such problems require rollback.
* Correct setTrxEndCallbackSuppression() calls in beginMasterChanges()
  that were making tests fail.
* Make Database handle callback suppression for FLUSHING_ALL_PEERS
  instead of making LoadBalancer/LBFactory have to manage it.
* Simplify finalizeMasterChanges() given that suppression does not
  actually effect runOnTransactionPreCommitCallbacks().
* Make dangling callback warning in Database::close work properly.
* Actually use $fname in flushReplicaSnapshots().
* Use DBTransactionError instead of DBExpectedError in some places
  where stages fail.
* Fix failing testGetScopedLock() unit tests so everything passes.

Add more comments to setTransactionListener and onTransactionIdle.

Change-Id: I6a25a6e4e5ba666e0da065a24846cbab7e786c7b

selenium: Run wdio directly without grunt

Doesn't seem to add anything, except complexity.

This has the benefit of matching more closely the way the README
recommends running individual tests.

Also add a check for 'chromedriver' before running it.
Normally the -e and pipefail would suffice, but because
it runs in the background, the error can be buried,
hard to find, or even not cause exit code to be set
properly. Thus, do a simple 'hash' check that will
print a useful command and exit cleanly.

Before:
> $ something &
> [1] 57922
> -bash: something: command not found
> [1]+  Exit 127                something
> $ echo $?
> 0

After:
> $ hash something
> -bash: hash: something: not found
> (exit: 1) $ echo $?
> 1

Change-Id: Id95f18927b5443defe679a77a82c5cbdd127c716

Localisation updates from https://translatewiki.net.

Change-Id: I4c835c6dea1acf1fcd455d5ec21db551d4ebbe28

Merge "RecentChange: Use constants for the $noudp parameter of save()"

Use OOUI form for 'clear' and 'raw edit' of watchlist

Change-Id: I2c53f66bc36b33d42d453b79056822b30796ea4e

Merge "Fix PHP Notice in Special:Statistics"

Merge "Fix alignment of checkbox label in feedback dialog"

Merge "Better class documentation"

Fix PHP Notice in Special:Statistics

Change-Id: I7a8a15537fce4b7c15e4e452fda268c395ad4d2e
Follow-up: I7736c5fb6032f1b7e8a60b49d422435a8c8e4962

Merge "jquery.ui: Use on() instead of deprecated bind() in jquery.ui.mouse"

Merge "media: Rename files to match the name of the class they define"

Fix alignment of checkbox label in feedback dialog

Change-Id: I2af1fc716b1921928fa210f82d1b290be64142fd

Localisation updates from https://translatewiki.net.

Change-Id: I13de2b5cedae52d2b709e8a6a3e1764ce47412ac

Better class documentation

Change-Id: I25912f38bb483cd4636bff4ce219dc878a1046f3

Merge "resourceloader: Use plain arrays instead of $.Callbacks() in addEmbeddedCSS"

Localisation updates from https://translatewiki.net.

Change-Id: I45de1a7e52aa2b6531442a9e5deeaa359c6c9efd

Merge "mail: Refactor checks looking for PEAR libraries to be clearer"

Merge "Update wikimedia/html-formatter from 1.0.1 to 1.0.2"

Add since tag to wfRemoveDotSegments

Looked it up for another commit, so might aswell make it easier to find for others :)

Change-Id: I432b13b35481cbca9ed6e7328b3b9115fdd203ea

Merge "resourceloader: Remove mwPerformance stub and rename mwLoadStart mark"

Fix parameter doc type for matchEditToken

I noticed it while fixing some AbuseFilter errors. matchEditToken simply
calls getEditTokenObject, which accepts string|array as $salt.

Change-Id: I36497eb8797ee60759c5109b699643635bba9174

Update wikimedia/html-formatter from 1.0.1 to 1.0.2

https://github.com/wikimedia/html-formatter/releases/tag/1.0.2
https://github.com/wikimedia/html-formatter/compare/1.0.1...1.0.2

Bug: T193318
Change-Id: I5f7a3e2300f343f675124df399cfe12909f56e14
Depends-On: Ia69102af64a0102e05b96b10f63ffcf6cbf292fa

Merge "Change @throw to @throws"

Merge "Add missing use statement"

media: Rename files to match the name of the class they define

This will make jumping from file to file much easier in text
editors when file name lookups will autocomplete naturally when
looking for a class by name.

Change-Id: I4b2e55a2e72674b619d5a592866c8a019a2b0224

jquery.ui: Use on() instead of deprecated bind() in jquery.ui.mouse

Frequently seen on Beta Cluster due to indirect usage from
PageTriage extension.

Bug: T169385
Change-Id: Ia49ad6470aef1bcb1dd91c779dcd108a0a98d4dc

CRH Transliteration Pattern Matching Fixes

Refactor to match exceptions as patterns, not words
- break exception list to C2L and L2C pattern sets
- change main loop to break only on Roman numerals and transliterate
  everything else, rather than tokenizing on single-script words
  (this fixes the km² problem, too)
  - update word anchors from ^ and $ to \b
  - only process Roman numerals for L2C translit
  - add exception for single "Roman" character followed by a period
    which looks like an initial
- consolidate multi-step transliteration into regsConverter()
- remove regex support from main exception list to support strtr()
- re-organize some prefix/suffix/whole word patterns to the right place
- add tests for recently fixed use cases
- add support for many-to-one mappings in both directions
- update character classes, exception lists,  and regexes based on
  speaker feedback and example texts

Misc other fixes:
- fix some character classes errors
- remove unneeded character classes
- add tests for Roman numerals and quotes
- add tests for affixes and regexes

Bug: T188321
Bug: T189512
Change-Id: I056d36ff2b8f63b3998a5d3a442d8d539c15488d

Merge "Rewrite DumpRev to not always fatal out"

RecentChange: Use constants for the $noudp parameter of save()

It is a boolean parameter that is confusing for three reasons:

* It's a boolean parameter, given parameters are unnamed in PHP,
  these are always poor UX for call sites.

* It's negated ("noudp"). save(true) means no feeds events,
  save(false) [default] means sending events to feeds.

* To overcome this problem, typical use was to pass a free-form
  string that self-documents the intended behaviour,
  e.g. `save('pleasenoudp')`, any string casts to true.

Fix this by moving the booleans to constants and use those
instead. For compatiblity, keep the negation internally,
although it's hidden from regular usage. Also document
the string hack, deprecate it, and update callers.

Change-Id: Ia57c86b38bf50cb4ec580f42a6b1ca798fcf781a

Merge "EditPage: Remove isOouiEnabled(), deprecated since MW 1.30"

EditPage: Remove isOouiEnabled(), deprecated since MW 1.30

Change-Id: I0ca2960ca4da5f7ec5acb254b30254809857bafa

Rewrite DumpRev to not always fatal out

* The method fetchBlob() is private.
* Also remove all of the code duplication.

Change-Id: Ie307a937bcfd835a3239af23fb8deeb78704e0fe

Add missing use statement

Change-Id: Id14d97b5b74edf6c6bafb29b643ac9b9357bb681

Localisation updates from https://translatewiki.net.

Change-Id: I060031447212e6e82254ee570a569fc2fc9fa749