Update and broaden @covers for PreprocessorTest

This class is intended to be an integration test of both preprocessor
implementations and their helper classes.

Change-Id: Iefbd6d8828bbc3278503a0f85efd7d1230a9d66c

Merge "LoginSignupSpecialPage: Restore old display of 'createacct-helpusername'"

Merge "Add urls from various adware to the CSP false positive list"

Add urls from various adware to the CSP false positive list

URLs are based on spam that comes into the Wikimedia log files,
based on testing on elwiki.

Change-Id: Iee86633abaae86c55764365042681bda1f2304be

LoginSignupSpecialPage: Restore old display of 'createacct-helpusername'

Note that this message is empty by default. You'll have to create it
on your wiki to see effects of this change.

Bug: T143923
Change-Id: I509c13775426b8d51f65c95af49dffb84ccaa0f4

Localisation updates from https://translatewiki.net.

Change-Id: Ia68c69742fbb7eae69e4160e923ec7fa459f7b67

Merge "Linker: Avoid Title in normaliseSpecialPage()"

Merge "debug: Interpolate log messages"

Merge "debug: Merge mediawiki.debug.init module into mediawiki.debug"

Merge "debug: Don't show git branch if on a detached HEAD"

mediawiki.inspect: Output size in bytes used in local storage

The current version only outputs the total size formated
human readable (kb, mb etc). Lets make it also output the size
in plain bytes so it can be formated depending on language.

Bug: T143770
Change-Id: Idda77fdae4e49a7cf46cce2fcd23c2fb2fe43f9d

Merge "mw.htmlform: Don't refer to OO.ui if it might not be loaded"

mw.htmlform.ooui.styles: Remove vertical margin from OOjs UI HTMLForms

Originally added in b524a4333f9c1b42bf6ac0f6caa3055bd8acf84a,
this is no longer needed for framed forms (with setWrapperLegend())
after e5bb1f68cf4154c1bc4389a4404808ebddb5cef9 in OOjs UI and was
never needed for non-framed forms at all.

Change-Id: If4b4b8776080a00685bbcb9026a3d5eb89e4167d

Merge "HTMLFormField: Don't display empty popup in OOUI mode if empty 'help' is given"

Merge "RestbaseVirtualRESTService: Support production URL layout"

Merge "Remove redundant UNIQUE from rev_page_id"

Linker: Avoid Title in normaliseSpecialPage()

Now that SpecialPage::getTitleValueFor() exists, use it so that we can
avoid returning a Title object uncessarily.

Change-Id: I5bf605baf2ecad62e189421a2e5c556cc0b6f6ac

Remove redundant UNIQUE from rev_page_id

The PRIMARY KEY is rev_id, so this is a waste and
makes the index slower to maintain (e.g. no change
buffering).

Bug: T142725
Change-Id: I63f817656ff5e62aa27caf607d70353cc99eb349

mw.htmlform: Don't refer to OO.ui if it might not be loaded

Dependencies are loaded dynamically here, before this code runs.
If there are no OOjs UI HTMLForm fields on the page, then OOjs UI
is not loaded, and the attempt to check if we're working with a
OO.ui.Widget would cause an exception. Invert the test to check
for jQuery, which is always loaded.

Follow-up to 89107070d14187915e760f8543579ec4d784620f.

Bug: T143850
Change-Id: Ibaa5a15fe367ed7470e3e8176e52b13442aeb985

Merge "PreprocessorTest: test both implementations"

PreprocessorTest: test both implementations

* Instead of only testing the configured preprocessor, test each in turn.
* Fix a test error when testing Preprocessor_Hash by removing <equals>
  tags -- only Preprocessor_Hash emits them, but they have no effect on
  the expansion.

Change-Id: I596f6b66fc636b767c447af3450556bfebe28241

Merge "Cleanups to SqlBagOStuff"

Merge "Optimize ChronologyProtector writes for multi-DC case"

Merge "Fix AuthManagerSpecialPage submit button logic"

Fix AuthManagerSpecialPage submit button logic

Bug: T143840
Change-Id: I7fe442f5044c3dce27f89d83b8003dc6e4170296

Cleanups to SqlBagOStuff

* Keep track of the custom LoadBalancer when it makes one.
* Use the custom LoadBalancer to wait for slaves if one
  was used, rather than the main singleton.
* Only wait on the slaves in the LoadBalancer if the
  main DBs are being used.

Change-Id: I11de814306c44f27e0c33b08b5921c0fd4cdc24f

Merge "resourceloader: Improve coverage in ResourceLoaderTest.php"

resourceloader: Improve coverage in ResourceLoaderTest.php

* Fix signature of makeLoaderSourcesScript() to match
  the change in behaviour since e103ba265.

* Consistently order providers before the test.

* Simplify testRegisterValid() and remove needless @depends.

* Remove unused private method stripNoflip().

Coverage:

* Expand test coverage for register().

* Add tests for getModuleNames().

* Add tests for getModule().

* Expand test coverage for addSource().
  (case of invalid array)

* Expand test coverage for makeLoaderImplementScript().
  (case of unwrapped user script, and case of invalid scripts)

* Add tests for makeLoaderSourcesScript().

Change-Id: Ibca3e486fcd3664f171f135327a0f340ee6da9ee

Clear SiteStats process cache after DB update

This fixes a phpunit test error, wherein {{NUMBEROFFILES}} would give
the number of files in the host wiki, not in the temporary database,
when Scribunto was installed, due to a Scribunto phpunit data provider
calling SiteStats::pages().

Change-Id: Ic0d021a72addaa2a13a6b94fd34dccc423de3a8f

Merge "API: Log when too many values are passed for a multi-valued parameter"

debug: Interpolate log messages

Change-Id: Ib61a50863022db796d69264bf738fb0b22f776e9

debug: Merge mediawiki.debug.init module into mediawiki.debug

It just calls mw.Debug.init() on document ready, which can easily be
done inside mediawiki.debug.

Change-Id: Ifaa5a4884c9e430241a4bff51454bb96f468cd61

debug: Don't show git branch if on a detached HEAD

The short SHA1 is already shown with the MediaWiki version, and
displaying the full SHA1 again adds 40+ more characters. Mostly this
annoys me as it causes the toolbar to wrap earlier when making my
browser window smaller.

Change-Id: If7474a090466de708d65cef6dcc30a36df56621f

Merge "Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds"

Localisation updates from https://translatewiki.net.

Change-Id: If5f82b2d32ab93592dd54cf21cb5d36c52315936

Optimize ChronologyProtector writes for multi-DC case

Change-Id: Iecd218043814ac0963c67b16d043e697a6933741

Merge "Call ssl_set() in DatabaseMysqli if DBO_SSL is set"

Merge "Move Article checkLastModified() up to MediaWiki::performRequest"

Merge "Make Special:CreateAccount respect ?returnto parameter"

Make Special:CreateAccount respect ?returnto parameter

Change-Id: I074ee184ad1fec84ef2e93eea8397d932e96e0cd

Merge "Sync up with Parsoid parserTests."

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 47e085881094299aa652320aba1dfb5b99855690

Change-Id: I2da551dc9d8568c56decbabcad8bcf8d57680b47

API: Log when too many values are passed for a multi-valued parameter

Currently the API just throws away the excess values, no continuation.
This is probably not the best behavior. There is a warning, but clients
are notorious for just ignoring warnings.

We could improve the situation:
* Turn it into an error.
* Turn it into an error for most cases, but allow some to override if
  necessary.
* Allow some cases to make it an error, while keeping it a warning by
  default.

Before we can decide which option to pick, we should get an idea of how
often this is being hit and which things it's being hit for.

Bug: T41936
Change-Id: I52926f410c30d1cd7e7fcd86465b16519fb5cbd2

HTMLFormField: Don't display empty popup in OOUI mode if empty 'help' is given

Change-Id: I1aa68dcb9cdf1584f65436a641b119f0d61537ef

HTMLFormField: Move 'flatlist' handling to fields that use it and document

Change-Id: I5dc6ad71880a741c41757bc64d236971edfbabfa

Localisation updates from https://translatewiki.net.

Change-Id: Ib1a471eb8e9692fab1e5d66cf6711388ea2e20cc

Move Article checkLastModified() up to MediaWiki::performRequest

This lets revalidations via IMS headers run a bit faster.

Change-Id: I1f61086dea4c6bc460f6249ed7fda78316117a8d

Merge "VirtualRESTServiceClient management cleanups"

Merge "Avoid INSERT..SELECT in LocalFileDeleteBatch"

VirtualRESTServiceClient management cleanups

* Add getVirtualRESTServiceClient() to MediaWikiServices.
* Support auto-mounting services that are usable by the
  main MediaWikiServices instance.
* Support lazy-loading in mount(), where only class/args
  are set until the service is needed. This avoids excess
  overhead.

Change-Id: I5c22be59664b3f5716c957e2c3d7c8e70d5fdc6c

Merge "A few more DBLockManager fixes and cleanups"

Merge "Remove some unused hooks from hooks.txt"

Remove some unused hooks from hooks.txt

These were removed in I2b2c9693a.

Change-Id: I31c2ff3ce9d734fab94695bd3e8eb85cbf65803f

EditPage: Use context instead of globals (4/4)

Change-Id: Ie84bceb0ac0940a7d14b9790eb8ab81321b20629

EditPage: Use context instead of globals (3/4)

Change-Id: Id4f905c8db35d5c64375f55e2040d40b04f18dee

EditPage: Use context instead of globals (2/4)

Change-Id: I3b727fa5cb42ca2a7abf1a54fe4f2b4996f5150c

EditPage: Use context instead of globals (1/4)

Change-Id: Ie4e9c492679a75d753b69fb09f510564b9e7f0a8

EditPage: Fix some doc blocks

Change-Id: I0399df658b3b9c000b72dacbdc4ea18678a32db4

EditPage: Add getContext() function

EditPage already has access to a IContextSource object via
$this->mArticle->getContext(), but no code ever uses it. Add a
$this->getContext() helper function and $this->context member variable
so developers are aware that RequestContext is available and should be
used instead of globals. This is the first step to de-globalifying
EditPage.

Change-Id: I17130bdaf214e9bbe0577a0ee5564ca4760c99e1

Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds

This is in intended to replace the DataUpdate transaction round logic.
It could also be useful for doing transaction rounds in maintenance
scripts.

Also renamed $db => $conn in a few LB methods for consistency.

Change-Id: If21c2ba5e8bac48c250b96137279e7edaa8289f7

Merge "EditPage: Allow the 'save' button's label to be 'publish' for public wikis"

Merge "EditPage: Show a different label for the button on create vs. modify"

Merge "HTMLMultiSelectField: Add 'dropdown' option for 'mw-chosen' behavior and document"

Merge "Upgrade justinrainbow/json-schema to ~3.0"

Upgrade justinrainbow/json-schema to ~3.0

The release between 1.6.1 and 3.0.0 has a huge amount of code
maintenance changes, as well as internal optimization and some
"visible" changes (as well as the one mentioned in the linked
task).

However, it's a version jump over 2 major versions, which is,
by it's definition a major change ;). Nonetheless, the (for us)
important api has changed marginally: Instead of using the
JsonSchema\Uri\UriRetriever class to retrieve the schema, we now use
the $ref keyword to reference the json schema file (which also is
an internal optimization). In this way, we let the json-schema library
decide, how to resolve a ref (and the schema) instead of relying
on the UriRetriever api to be public and stable.

The versions also include various bug fixes (which, as far as I know,
doesn't apply to us).

I tested this change with various combinations of valid and invalid
extension.json schemas (version 2 as well as version 3). Given that
there were no major changes to the schema interpretation itself, and
the good test coverage of the library, there shouldn't be a high risk
because of this change.

The full list of changes can be found at:
https://github.com/justinrainbow/json-schema/compare/1.6.1...3.0.0

as well as the changelogs of the single versions:
https://github.com/justinrainbow/json-schema/releases

Bug: T141281
Depends-On: I5687286da9f7fa2bb2b84699fa43ab3c2547fe03
Change-Id: Ie37e2ebc48684783abf8d99d2f775ee6a5988da7

CloneDatabase: Simplify callback structure

Change-Id: I4d5184fd7417e61e9a111bd414f8c62539229ef9

Implement NumericUppercaseCollation

This collation orders text with numbers "naturally", so that
'Foo 1' < 'Foo 2' < 'Foo 12'.

Note that this only works in terms of sequences of digits, and the
behavior for decimal fractions or pretty-formatted numbers may be
unexpected.

This is only expected to work mostly correctly for English-language
text. Consider it a proof of concept. You probably want to use
an UCA collation with '-u-kn' suffix rather than this.

Bug: T8948
Change-Id: Ie268f2d92c5c75d0aaecf54ede2bdda1af3b309d

Merge "content: Refactor normalization of line endings code"

content: Refactor normalization of line endings code

The code that normalizes line endings ("\r\n" and "\r" to "\n") and
trims trailing whitespace is buried in Parser::preSaveTransform(), and
was duplicated to TextContent in 96b6afb31dfcff, as non-wikitext content
models should still be normalizing line endings.

This splits the duplicated code into
TextContent::normalizeLineEndings(), and utilize it in the Parser.
Additionally, expand the documentation of
TextContent::preSaveTransform() to document that subclasses should make
sure they normalize line endings during the PST stage.

And remove a useless rtrim() call from WikitextContent that did nothing.

Change-Id: I9094c671d4bbd23d75436f8f1d682d6dd6e6d2fc

Merge "Avoid INSERT..SELECT in doArticleDeleteReal()"

Remove commit() calls from JobQueueDB

These are not safe for the common case where the local DB
handle is used for the queue (and other table writes).

Change-Id: Ic24a05c18bf31e49bf7e9a3c058deb5d35271511

Avoid INSERT..SELECT in LocalFileDeleteBatch

That construct has poor locking characteristics in terms of
auto-inc columns as well as not allowing such inserts concurrently
for statement-based replication. Also, the INSERT..SELECT did not
have an ORDER BY, which could lead to fa_id drift with statement
based replication.

Change-Id: Iaacb75d9931b4cd24b70bdcaadd0e3979c7e9c90

Avoid INSERT..SELECT in doArticleDeleteReal()

That construct has poor locking characteristics in terms of
auto-inc columns as well as not allowing such inserts concurrently
for statement-based replication. Also, the INSERT..SELECT did not
have an ORDER BY, which could lead to ar_id drift with statement
based replication.

Change-Id: I9396869e474bc082fa6161b60afa3a5247df773b

Merge "Tell users that js/css subpages are public"

Merge "Reduce problems caused by $wgRunJobsAsync"

Fix broken lockmanager-fail-releaselock status messages

Change-Id: Icb0cfa6e38bc81c35430023afe50dd94ef3b2013

Merge "Expose form field objects in HTMLForm"

Merge "Make login/signup footer available to AuthChangeFormFields hook"

Merge "Improve default behavior for HTMLForm::canDisplayErrors"

Merge "mw.widgets.CategoryCapsuleItemWidget: Debug logging for "queue[title] is undefined""

mw.widgets.CategoryCapsuleItemWidget: Debug logging for "queue[title] is undefined"

Bug: T139130
Change-Id: Icd852a0b0d5cc42863965e303c410d1be50ff364

Merge "SpecialMyLanguage: Use page language instead of wiki language for redirect target check"

SpecialMyLanguage: Use page language instead of wiki language for redirect target check

With the change, named in Follow up, it's possible for site owners to
allow to change the language of a page using a special page.
Theoretically, any page can have another or a different page language,
depending on, if the language was changed using the special page or not.
For Special:MyLanguage it isn't enough anymore to check, if the current
user language is the same as the default content language. It has to
check, if the page language (which can potentionally differ from the
default content language) is the same as the user language.

The problem:
If content language is the same as the user language, Special:MyLanguage
currently redirects to the "base page" of a page ("Testpage" instead of
"Testpage/de"), no matter, if the page language of the base part is
another one as the default content language. This can result in the
problem, that Special:MyLanguage redirects to a page, that has a
different language as the user language, even if a subpage with the user
language code exists. This is fixed with this change.

Follow up: I0f82b146fbe948f917c1

Bug: T121834
Change-Id: Ic9fc9049813c153111829d37a2c248dc0768e0fb

Merge "Introduce {{#time: xit}} for days in the month in Iranian calendar"

Merge "User namespace localisation update for Slovak"

OutputPage.php: Reuse existing variable $user

Follows-up to 81c291f2

Change-Id: Id32daf74549c8af886a46119b30ff29ab2a6ac94

Merge "objectcache: Add missing @covers to unit tests"

objectcache: Add missing @covers to unit tests

* HashBagOStuff: 100%
* CachedBagOStuff: 64%
* MultiWriteBagOStuff: 33%

Change-Id: I50bb8f5eda7eabadb5fd4b841af42b3bbcaf9611

Reduce problems caused by $wgRunJobsAsync

* Use getCanonicalURL() to avoid links with the wrong host (e.g.
  when it is virtual) and to avoid getting redirects.
* Also disable this setting when post-send execution is already
  available, by default.
* Bump the socket timeout slightly.

Bug: T107290
Bug: T68485
Change-Id: I56c43193fa6583cc0c8209ff59cf20c986a799a3

Code cleanups to SqlBagOStuff

* Refactor local DB usage check into usesMainDB() method.
* Avoid using the db member of DBError instances.

Change-Id: I7350f5a471c551492094bfaf545ebc222eb6f7dd

Merge "Pingback: Tweak docs a tiny bit to point to mw.org better"

A few more DBLockManager fixes and cleanups

* Do not do the connection init step if the same DB handle as
  wfGetDB( DB_MASTER ) is being used to avoid clobbering it.
* Remove begin(), since only one of the subclasses wants
  transactions. That one now uses startAtomic() now.
* Make getConnection() throw an error for bad config instead
  of return null, which was not documented or expected.

Change-Id: Ib09a7972d6569c29e83e329a8f7f9f47a393b896

Merge "Move EnqueueableDataUpdate to a separate file"

Merge "Release notes for all the previous security patches"

Merge "Remove redundant isLoggedIn() call"

Release notes for all the previous security patches

Change-Id: Ie93338b7d41a90f3ffdfa1b41891994935c965c7

Merge "SECURITY: Move 'UserGetRights' call before application of Session::getAllowedUserRights()"

Merge "Remove commit() hack from User::addToDatabase()"

Merge "Various database class cleanups"

Move EnqueueableDataUpdate to a separate file

Change-Id: Iabc291cd1f3c5390ca8bcc5da64a0ff01a082575