resourceloader: Move 'site' and 'user' logic to makeModuleResponse

* Keep this out of makeLoaderImplementScript() to keep it more generic
  and to simplify future refactoring.

* Remove now-broken test case that asserted that the output varies
  by global debug mode.

* Make the test responsible for wrapping in XmlJsCode. Previously
  this magically happened because the module name was "user" in the
  last test case.

* Make makeLoaderImplementScript protected. It's not used anywhere
  outside ResourceLoader and we should keep it that way.

Test plan:
* Verify output unchanged:
  - load.php?modules=user&only=scripts&user=Admin (raw code)
  - load.php?modules=user&user=Admin (implement with unwrapped string)
  - load.php?modules=jquery.client (implement with closure)

Change-Id: I527d01926fb6e4ce68c931695d830cdb9ceb608c

Merge "Avoid global state in DatabaseBase::factory()/query()"

Merge "Remove wfSetBit()/wfSetVar() calls in DatabaseBase"

Remove wfSetBit()/wfSetVar() calls in DatabaseBase

Change-Id: I4f1269d3a4e26c766f181208942042fdf768e0af

Merge "Remove wfRandomString() dependency in DatabaseBase"

Merge "Move LoadMonitorMySQL to LoadMonitor"

Merge "Move database constants to their own file"

Avoid global state in DatabaseBase::factory()/query()

Change-Id: Ibb4f1c0dafea071a1c34e0cd5b5c15b8b4bb7bc6

Remove wfRandomString() dependency in DatabaseBase

Change-Id: I9a889e120bd665683e53dfbc8729a7c484ea507c

Move database constants to their own file

Change-Id: I47e067687bbce7b89e02a0e732ee27368beddb61

Merge "Remove wfGetCaller() dependency from DatabaseBase"

Merge "resourceloader: Set a Title on context-created messages"

Merge "Deprecate jquery.arrowSteps module"

Move LoadMonitorMySQL to LoadMonitor

* The LoadMonitor interface is now ILoadMonitor.
* Nothing in this code was MySQL specific.
  Now any DB type can benefit from a LoadMonitor.

Change-Id: I272a72cd55a70f99a866d518d44cb3bcaca91b9e

Merge "Infrastructure for augmenting search results"

Infrastructure for augmenting search results

Bug: T117493
Change-Id: Ia5413a7846cc961026a2dc3542b619493bc76a23

Add one more level for GlobalTitleFail logging

htmlspecialchars/Message->__toString/Message->toString/Message->parseText/MessageCache->parse
is useless without the caller of htmlspecialchars().

Change-Id: I2dad5a487225a11b98149610f1c775057f8baad2

resourceloader: Set a Title on context-created messages

Ensure that messages created using ResourceLoaderContext::msg() have a
title set so they don't trigger the GlobalTitleFail log. We use a dummy
title since there is no real title we can use here, and the cache
doesn't vary on a title anyways. This particular title was picked
especially for Roan.

This patch should quiet the GlobalTitleFail logs for
MFResourceLoaderParsedMessageModule and VisualEditorDataModule.

Change-Id: I502faa22776e1cb34a6ef17be96567f121c80081

EditPage: Generate all messages using $this->context so we have titles

Change-Id: I5ba13f1677e1a1b4160e14357a483b5e1f11ea1c

Remove wfGetCaller() dependency from DatabaseBase

Change-Id: I3e240b2eb5c1f6a21f1bc974c3d28f5755c7451a

Merge "Replace WikiPage::doEdit() usages"

Merge "Allow quotes in $wgDebugComments HTML comments"

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

Replace WikiPage::doEdit() usages

Update/add some comments

Bug: T145737
Change-Id: Ib3b2b69906074cbb51917dc57204c989fd6c58b2

Merge "benchmarkParse: Add option to clear link cache"

Merge "Emit deprecation warnigns for Article::getText()"

Merge "Linker: Deprecate formatSize()"

Localisation updates from https://translatewiki.net.

Change-Id: Ic6504bfa03dfb50c40231151255bc0297f28275c

Allow quotes in $wgDebugComments HTML comments

Change-Id: Id1fbab9ad3d6d645b6efea1d1e66a18ab4fe0b89

Merge "Fix default "srvCache" BagOStuff in DatabaseBase"

Merge "Dependency inject $wgSharedTables into DatabaseBase"

Dependency inject $wgSharedTables into DatabaseBase

Change-Id: I3e659b56c587d2059fc3d3f922bc38dec7685df4

Fix default "srvCache" BagOStuff in DatabaseBase

Follow up to 4660b56f

Change-Id: I19975ce66ddc318fd494d5c4a3807c62d04fad70

Merge "Move updater/installer specific methods out of DatabaseBase"

Merge "Rename LBFactory => LBFactoryMW and make LBFactory in /libs"

Merge "docs: mw.widgets.CategorySelector: Fix example code"

Rename LBFactory => LBFactoryMW and make LBFactory in /libs

The former extends the later with MW-specific logic.

Also removed a wf* method call from ChronologyProtector.

Change-Id: I325f59b7467ab9c2137731d1ce69816f5a020f03

Merge "Remove global state from DatabaseBase::__construct()"

Merge "Move Linker::formatTemplates() to separate class, remove global usage"

Merge "Fix broken wincache merge() implementation"

Merge "OutputPage: Don't set 'user' module state if filtered out"

Merge "Revert "Remove jquery.arrowSteps module""

Deprecate jquery.arrowSteps module

Bug: T144974
Change-Id: I765902082e6a71ca69983a2e67dcc856ad8b6ca8

Revert "Remove jquery.arrowSteps module"

It should probably be deprecated first after all.

This reverts commit 6fba9a7dc6ac969676c57f85b09bbdac9067a2bd.

Bug: T144974
Change-Id: I830fbfc2e453be6b18d43775a8aa1a366690907c

Merge "Set fallback language for lzh to zh-hant"

Change wording, change grammar in lang/en.json

Bug: T142756
Depends-On: I233ce58fd82a1dbc57b3ab58ce669b23a5781b57
Change-Id: I164d397d6ea41ea6b88bb0d84604b52419dadd0f

Fix broken wincache merge() implementation

Bug: T145664
Change-Id: I2f841779108df281e24003c586775b71aa176909

Cleaned up some un needed wording in the English i18n file

Changed messages:
* noindex-category
* morenotlisted

Change-Id: I233ce58fd82a1dbc57b3ab58ce669b23a5781b57

Remove global state from DatabaseBase::__construct()

Change-Id: Iba347fce8f0f531aa7e565bb8806260bcbf04a98

Set fallback language for lzh to zh-hant

Bug: T125373
Change-Id: Ie39d03efce0997c928ac0bc7f4a97ee8e047df45

Move updater/installer specific methods out of DatabaseBase

Change-Id: I995799fc15d2797ce7ab9ce2aca8beeef409447c

Emit deprecation warnigns for Article::getText()

Deprecated since 1.21.

Bug: T145747
Change-Id: I6fe620702fcc455dbe229abca1145b0c8ea3f8d2

benchmarkParse: Add option to clear link cache

For performance testing the lookup of links (e.g. LinkHolderArray),
allow resetting the LinkCache after every parse.

Change-Id: I75c2cad9a8cf1b41d192708773194f799673ce83

Remove unused LBFactoryFake class

Change-Id: I88acf72fc32c7e3c9bead81d11c5adfd8dff6151

Merge "Avoid MWException and wf* log methods in DatabaseBase"

Avoid MWException and wf* log methods in DatabaseBase

Change-Id: Idc418ae1088f87d6416e2552976d94f7d1e8f5db

Move various DB helper classes to /libs/rdbms

Change-Id: I0724f1acce4f6c43b1f0983fa119e628e7c53ba5

Move LoadBalancer to /libs/rdbms

Change-Id: Ic87f1cc84ba44be20f345944406e1d8a34be1c96

Change DatabaseBase => IDatabase in /db where possible

Change-Id: Ia0a049cd4294c5a39aa9ed228d4eb5b15736ea1f

Merge "Change $wiki => $domain in LoadBalancer"

Merge "Exception cleanups for LoadBalancer"

Change $wiki => $domain in LoadBalancer

Change-Id: I97f96383c501e9b3262c11d143795b82ae36aca5

phpunit: Simplify mock object syntax in includes/db/ tests

* Omit redundant `expects( $this->any() )`.
* Use willReturn() instead of `will( $this->returnValue() )`.

Change-Id: If90ac651748af8a78a720992240e40ba53cec79c

Exception cleanups for LoadBalancer

* Make the error message itself be dumb/raw.
* Make the exception exposer show the same GUI message.
* Remove overzelous wgLang check in MWExceptionRenderer.

Change-Id: Ifffff3b3cc785ea3080e4975efe33b3c2cf304d6

Merge "Move DBConnRef and ChronologyProtector to /libs/rdbms"

Merge "Move TransactionProfiler to /libs/rdbms"

Merge "Make WikiPage::getRevision() make use of Revision::newKnownCurrent()"

Move DBConnRef and ChronologyProtector to /libs/rdbms

Change-Id: If2c4b314a5c39311328843f534d91bf90823e179

Merge "More DB related class cleanups to prepare files for /libs"

Merge "Re-add parser tests to the ParserTests group"

Merge "Fix broken select() call in rebuildFileCache.php"

More DB related class cleanups to prepare files for /libs

* Cleaned up some related logging bits.
* Avoid some wf* methods in LoadBalancer.

Change-Id: I5da97174441c086e61ffc120889c068e241ebfd2

Make WikiPage::getRevision() make use of Revision::newKnownCurrent()

Change-Id: Id06319b31bc523a5d514930c3d609af4d991453b

Merge "Move IDatabase/ILoadBalancer to /libs/rdbms"

Merge "mediawiki.Upload.BookletLayout: Fix typo in comment"

Re-add parser tests to the ParserTests group

It was omitted due to the new way in which parser test TestCase objects
are constructed. Should fix Jenkins double-execution of parser tests.

Change-Id: I8131c3b13f2e08f784bce46fee16051c14761304

Move IDatabase/ILoadBalancer to /libs/rdbms

Change-Id: I71f980b1024dc2fbca9cafbad2c9daebb6f28c9b

Make SiteStatsUpdate implement MergeableUpdate

Change-Id: I53c9bf133f0707f2fa203ddd1a7fcb65be8b1329

mediawiki.Upload.BookletLayout: Fix typo in comment

Follows-up 33041b752d.

Change-Id: I50747f140d6c0a42385aa94c8827f8ff7354628d

resourceloader: Fix WikiModule preload to support localised titles

Follows-up 6f8dc27ca2 and dbd11e04aa. You'd expect a bug in preloading
to fallback to fetching it on-demand but due to a title format mismatch
the preload method did use the same title format for the cache key, but
not the same title format for discovering the results. As such, it set
the right cache key to an empty array.

* Make relevant methods testable and mockable.
* Add regression test.
* Change call to array_interect_key to use the same format as
  fetchTitleInfo(): Normalised title keys from Title::getPrefixedText.
  Previously, the intersect used the declared titles from getPages() which
  are not localised - causing an empty array to be returned from the
  intersect on wikis where the namespace name is localised.

Bug: T145673
Change-Id: Ibe788157724d73c727b9e2127b6828db32ca9420

Fix broken select() call in rebuildFileCache.php

A few other small cleanups

Change-Id: Ie62cf93feb5a57f629a893c7ea298ed902b52789

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

Localisation updates from https://translatewiki.net.

Change-Id: Iee268ac310eb4a09e6df11ca71a9a71a7d8b19e5

Merge "Move DBError to /libs/rdbms/exception"

Merge "Add MWExceptionRenderer class and decouple DBError"

Merge "Reduce WikiPage::getContentModel() queries via caching"

Merge "Add caching to Title::loadRestrictions()"

Move DBError to /libs/rdbms/exception

Change-Id: I2ad40db66a68f03fb34a63278a652e0505195076

Add MWExceptionRenderer class and decouple DBError

* This handles the work of showing exceptions so that
  MWException does not have too.
* Simplify the DBError classes to regular Exception
  classes. Lots of pointless prettification has been
  removed, but DBConnectionError still gets the usual
  special treatment of a fallback page and Google form.
* Remove hacky file cache fallback code that probably
  did not work.
* Make MWExceptionHandler::report() wrap
  MWExceptionExposer::output().
* Make MWException::runHooks() wrap
  MWExceptionExposer::runHooks().

Change-Id: I5dfdc84e94ddac65417226cf7c84513ebb9f9faa

ParserOptions: Remove setters for magic links options

This will allow clients to assume that the global configuration state
($wgEnableMagicLinks) is the same as the state in ParserOptions, fixing
some consistency issues in wfEscapeWikiText() and Scribunto. The
abstraction for the ParserOptions getters is still useful because it
keeps the global state out of the Parser.

Change-Id: Ic2c447443963eb336c786cacf9dfd041a2df3b77

Merge "API: Deprecate use of ApiPurge via GET"

Merge "API: Expose $wgEnableMagicLinks in meta=siteinfo"

Merge "Fix login API for users with @ in their usernames"

API: Deprecate use of ApiPurge via GET

This emits a warning and logs feature usage, and also pretends that POST
is required in the help and paraminfo output.

Bug: T145649
Change-Id: Ied8d47a947662158fddab383c764bf70c5ab0112

Move TransactionProfiler to /libs/rdbms

Change-Id: I13173facc9f101bca69e3bf3e85f292f8468e855

Merge "Move LoadMonitor classes to libs/rdbms/loadmonitor"

Move LoadMonitor classes to libs/rdbms/loadmonitor

Change-Id: Ib18108f24ff8b9d05dc324bad132f597b3e2ddef

Merge "Replace some MWExceptions with natives ones in /db"

Merge "Special:ChangeContentModel: Run EditFilterMergedContent hook"

Reduce CategoryMembershipChangeJob lock timeout

Bug: T133801
Change-Id: Ic5db3983a8b67b9138b602188a52d60dd95fcb59