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"

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

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

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

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

Change @throw to @throws

Phpcs does not look at private functions

Change-Id: I9adedf1f1c488e74db7de89e97cb6a481bf7d567

Merge "selenium: Clean up wdio.conf.js"

Merge "Use PRC_AUTOPATROLLED for log entries and category entries"

Merge "Revert "wikibits: Remove redundant stubs for document.write""

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

Bug: T192623
Change-Id: I00c9571d353980d853cc8e92b2b508ad7271ab87

Merge "Fix warning "ob_end_flush(): failed to delete and flush buffer" during uploads"

Remove bogus @throws from HTTPFileStreamer

Change-Id: I28928a6f01b107b2aed8ab6fbeb013410a8f9718

Merge "Skins: getDefaultStyles can now define render blocking CSS"

Skins: getDefaultStyles can now define render blocking CSS

This optimisation attempts to minimise loading the styles in places
they are not needed.

The logic is kept inside Skin::getDefaultModules to avoid fragmentation
of where modules get defined.

Update ApiParse to avoid repetition of code.

Bug: T42792
Bug: T42812
Change-Id: I59f02a7bab3baa9d43f6bc2ef1f549d9d31d8456

Merge "rdbms: Replace reportConnectionError() with direct throws"

Localisation updates from https://translatewiki.net.

Change-Id: I17a9d121043e13c6919378a0a88ad36d9f647ee8

Fix typo in postgres patch-drop-ar_text.sql

Change-Id: I6d479d123848325b8501275ef4b98fc81cd99505

Revert "wikibits: Remove redundant stubs for document.write"

This reverts commit 7f2c6c401644c06b0bd30e311c6247bcaebec1bc.

Bug: T193191
Change-Id: Ia6bdc1e339fe0f6265094de6d3e0a5cea0919cd6

Use PRC_AUTOPATROLLED for log entries and category entries

These actions mark edits as patrolled automatically, so the
correct rc_patrolled value to use is PRC_AUTOPATROLLED rather than
PRC_PATROLLED. The code for log entries checks the autopatrol right, and
the code for category entries has a comment that says "just like log
entries".

Bug: T190408
Bug: T184791
Change-Id: Id994af8cd3833a862a09389431256aba35c2e6d9

selenium: Clean up wdio.conf.js

* Remove redundant user/pass for Jenkins, which is unused now
  that Jenkins jobs set it through ENV instead.
  See also T188520 and Ibb388741fee9 (integration/config).

* Use simple default operator (||) instead of elaborate
  undefined check.

* Document that the inline user/password defaults relate to
  MediaWiki vagrant.

* Set 'logPath' for Jenkins based on LOG_PATH env (from mw-set-env.sh),
  instead of assuming where it will be relative to mediawiki-core.

* Put new lines between blocks and functions.

* Document --no-sandbox hack from 4ff5652501b6cd.

Change-Id: I4c7856d260ded8c94e991a27863b80c9eea2e33e

mail: Refactor checks looking for PEAR libraries to be clearer

I don't think this makes any difference towards the linked bug, but
it makes it more obvious that MediaWiki doesn't try to use any PEAR
classes before attempting to require them.

Bug: T186456
Change-Id: I342f41903a15f13e72e76464499351412a9afa34

Merge "Filter autodiscovered parser tests in ConvertExtensionToRegistration"

Merge "import: Don't try duplicate detection when the sha1 is unavailable"

rdbms: Replace reportConnectionError() with direct throws

When reading through DatabaseMysqlBase::open(), it was not
obvious that execution would not continue after the conditional
`!$this->conn` block, given it ends in a method call, without
return or throw. I considered adding a return statement after it
for clarity, but it seems in this case it might make more sense
to throw directly given $error here has already gone through a
fallback to getLastError() a few lines up.

Replace the other three calls to reportConnectionError() as well,
which previously passed a useful string that was overwritten
with lastError(). Instead, log both. And make their call to
queryLogger->error() match the previous ones to have an 'error' as well.

This leaves reportConnectionError() as being unused, except for
a call from LoadBalancer. That call was problematic because
it was inside a conditional for IDatabase, but the method isn't
part of that interface. Replace it with a direct throw as well.

Deprecate the method as its now unused in core, and also remove its
'# New method' comment which hasn't made sense since r75341 (16cded8b32).

Change-Id: I0f2ef00ba44bf7090a3ce54edeb8c7e8e543e46a

tests: Assert that the user who deleted a page is logged as such

These tests are rather basic, but they're better than the nothing that
seems to be the current state. Revision deletion is still untested and
suppression is only very lightly tested. More tests should be added.

Bug: T188479
Bug: T188773
Change-Id: Iba0f41ffddc6af45e0a4be86176708160f4821f4

Merge "Introduce ResourceLoaderLessVarFileModule"

Merge "RELEASE-NOTES: Move a deprecation to "Deprecations" section"

Merge "phpunit: Clear job queues in MediaWikiTestCase::setUp()"

RELEASE-NOTES: Move a deprecation to "Deprecations" section

Change-Id: Ibe4ce1ed4d209f3d111524074c9e6cbd5b889aa6

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

Merge "Make setSubmitProgressive() Deprecate"

Merge "Removed unused Module mediawiki.userSuggest"

Localisation updates from https://translatewiki.net.

Change-Id: I6b22a6cadf7bfb166074c8ce29d1de9e6c9d16a3

Introduce ResourceLoaderLessVarFileModule

This ResourceLoader module provides a way to ship
messages to CSS variables.

We will need this going forward to deal with flash of
unstyled content in various JavaScript based UIs that
are subject to i18n such as table sorting and
collapsible elements.

To avoid overhead of hitting the database to fetch and
transform localisation messages we make use of the MessageBlobStore
making use of `messages` definition already inside
ResourceLoaderFileModule. Given this resource is only intended
for render blocking styles without JavaScript this should be okay
(although if requested in JavaScript will also ship associated
messages)

Bug: T42812
Change-Id: I2bf12cdc848478889acbe9a7a970e46f8aefa287

Cleanup the element matches in doBlockLevels a bit

The MARKER_PREFIX is removed since `unstripGeneral()` happens before we
get here.

Change-Id: Ic668784fd8bbaa8395cd5449c83a993abda141eb

Merge "mail: Always quote uncoded names in MailAddress"

Merge "ApiSandbox: Resize layout to fit content, and remove fullscreen mode"

Merge "Fix unclear comments on various ChangeTags functions"

ApiSandbox: Resize layout to fit content, and remove fullscreen mode

Avoids unpleasant nested scrollbars, and hacky fullscren mode.

Make toolbar position:sticky to keep buttons visible.

Bug: T129157
Change-Id: Ia3c3176c1bd7cac6af2c35f53d820f1272df72a5

Merge "TraditionalImageGallery factor getCaptionHtml into own function"

TraditionalImageGallery factor getCaptionHtml into own function

Better design and also want to override this in Wikibase.

Bug: T192869
Change-Id: I24de654ba9a14b8fe0925007e6ceeabca277c075

Fix unclear comments on various ChangeTags functions

These functions tell you whether *a user* can perform the relevant
actions. They are not meant to be used by extensions, except when
the extension is performing these actions on behalf of a user.

Change-Id: I8437ec2d66c151f504755964e20fda808d35de3b

Merge "PHPVersionCheck: Properly check for `xml` extension in PHP 7.2+"

PHPVersionCheck: Properly check for `xml` extension in PHP 7.2+

In 7.2, the `utf8_encode` function was moved into core PHP, and is no
longer a part of the `xml` extension. Use a different function to check
for the existence of it.

Change-Id: Ib50c1a59dd9c2a8777b07b2f7a3dd638a55832c6

Generalize ResourceLoader 'excludepage' functionality

There has long been a hack for previewing edits to user JS/CSS, where
OutputPage would pass an 'excludepage' parameter to
ResourceLoaderUserModule to tell it not to load one particular page and
would instead embed that page statically. That's nice, but there are
other places where we could use the same thing.

This patch generalizes it:
* DerivativeResourceLoaderContext may now contain a callback for mapping
  titles to replacement Content objects.
* ResourceLoaderWikiModule::getContent() uses the overrides, and
  requests embedding when they're used. All subclasses in Gerrit should
  pick it up automatically.
* OutputPage gains methods for callers to add to the override mapping,
  which it passes on to RL. It loses a bunch of the special casing it
  had for the 'user' and 'user.styles' modules.
* EditPage sets the overrides on OutputPage when doing the preview, as
  does ApiParse for prop=headhtml. TemplateSandbox does too in I83fa0856.
* OutputPage::userCanPreview() gets less specific to editing user CSS
  and JS, since RL now handles the embedding based on the actual
  modules' dependencies and EditPage only requests it on preview.

ApiParse also gets a new hook to support TemplateSandbox's API
integration (used in I83fa0856).

Bug: T112474
Change-Id: Ib9d2ce42931c1de8372e231314a1f672d7e2ac0e

Merge "Align to WikimediaUI themed widgets"

Merge "rdbms: make IDatabase::onTransaction* methods pass the DB handle for convenience"

Merge "Update OOUI to v0.26.5"

Merge "Special:PrefixIndex: Fix form legend"

Update OOUI to v0.26.5

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.26.5

Depends-on: Ieed078f4113e5ac4770c9e5d1f778a02c18ac6b7
Change-Id: Ia5e491f765935c2bf4bfeac2d02f1978f1a03b50

rdbms: make IDatabase::onTransaction* methods pass the DB handle for convenience

Change-Id: Ia45a26830d62326b103593268fbf34c907783c90

mail: Always quote uncoded names in MailAddress

Otherwise, names that contain whitespace (but no special characters
that need Q-encoding) will be sent as-is in an email header. While
that's valid by RFC 1036, RFC 5322 and others, some mail programs
fail to correctly process spaces. Avoid these problems by simply
quoting the name always - except for cases which are Q-encoded,
which must remain unquoted to be valid (and also cannot contain
spaces, thus unaffected).

Before:
> From: Foo Bar <fb@example.org>

After:
> From "Foo Bar" <fb@example.org>

Update test to work with the new criteria.

Bug: T191931
Change-Id: I4b2a0a80e8b43ecc943939fc5ebaa356a4f840e0

Merge "resourceloader: Consider having dependencies as non-empty in WikiModule::isKnownEmpty"