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

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""

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

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

docs: mw.widgets.CategorySelector: Fix example code

* jsduck was converting '#content' to a link, so switched that
  to 'body' instead
* Loading the required module so that the code can be copy pasted

Change-Id: I93148ff7480394c952b10db5a25004134fa07fe4

Replace some MWExceptions with natives ones in /db

Change-Id: I94532d09216926d401e94c61acd60fdc6241a2a0

Merge "refreshLinks: Add --namespace option"

refreshLinks: Add --namespace option

This allows limiting refreshing data to pages in a single namespace.

Change-Id: I309058df98b638beb32adb1d663455a0c4aa1cec

Set cpPosTime cookie for the non-redirect case as well

This is useful if the ChronologyProtector store cannot sync
among datacenters and waitForReplication() is DC-local.
It only applies if the user stays on the domain, however.

Change-Id: I800cade7b87a0f02d6a429e70924c54189a4a7af

Use cpPosTime cookie for same-domain redirects on DB change

This follows-up a3dacac90f to show the URL parameter in less cases.

Change-Id: Ibe015352962fb3ee14d5aa322f0e748ef4cba5c1

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

On pages where the 'user' module is filtered out (e.g.
on Special:Preferences) it would export state 'loading' (or state
'ready') eventhough the actual call to makeResourceLoader() later
in getBottomScripts() would be a no-op due to filtering.

This would cause either an indefinite state of "loading" or a
state "ready" that wasn't true.

This restores status quo as it was before 80e5b160 and 3e7a50d5f.

Test plan:
* Logged-in with non-empty user page common.js.
* View Special:Preferences.
* Verify in <head> source code, or via mw.loader.getState('user')
  that it has state "registered" (the default initial state) and
  not state "loading" or "ready".

Change-Id: I9b360d7e12703bddb80793aef47296fd63032c3d

Add ILoadBalancer interface

Remove redundant LoadBalancer docs (those without @since).

Change-Id: I981177b6854cfdbea4a51b6db7e365dac0da258a

Merge "Convert LoadBalancer/Database class logging to PSR"

Convert LoadBalancer/Database class logging to PSR

Reorder DatabaseBase constructor/factory to the top.

Change-Id: I2ffff950b9170d6ff15dcd46b64dd366f1e441a8

Merge "Partially reverting I8e684f06 to restore some legacy behavior"

Fix login API for users with @ in their usernames

An @ in the username caused the password to be treated as a bot password,
but apparently some real usernames still contain it. Try both logins
instead. Security considerations are the same as for the other bot
password syntax: the length check makes sure we do not provide any
information on a timing side channel about the password unless it is
extremely long.

Change-Id: I58f42544a08c3208c41f54cfae932632d9c5affa

Merge "Move PECL memcached class to /libs"

Partially reverting I8e684f06 to restore some legacy behavior

Apparently some calls to getDateCond() expect it to not set an offset.
Also, removing some of the new tests that no longer work since they
don't pass all the required parameters. (Before I8e684f06, the year
and month were both required parameters.)

Bug: T145597
Change-Id: I3a90b3da48e49ec9723b7100a7d92146154f74e3

Merge "Various dependency injection cleanups to LoadBalancer"

ParserTests: Make sure WikiPage::doEditContent() succeeds

If a parser test tries to create a page with invalid content (rejected
by Content::prepareSave() or Content::isValid()), throw an exception
instead of silently failing.

Change-Id: I89642604216699f27884120b3c297a0b277c47a6

Merge "Make sure emails from RecentChange::save() are for comitted rows"

Special:ChangeContentModel: Run EditFilterMergedContent hook

This will allow tools like AbuseFilter to filter changes made through
the special page, and matches the EditPage behavior that already runs
this hook.

Bug: T145489
Change-Id: I3204e3a228af2cdd4e2ab4e8c760cc126a8e1947

Various dependency injection cleanups to LoadBalancer

* Inject wfWikiID() and MWExceptionHandler into LoadBalancer.
* Factor out LBFactory duplication into baseLoadBalancerParams().
* Remove $wgDBtype hack. Presumably, sites with others DBs would
  not have multiple servers configured if does not work anyway.
* Make use of injected TransactionProfiler rather than calling
  Profiler::instance()->getTransactionProfiler().
* Avoid use of trivial wfSplitWikiID() function.
* Make DBConnRef enforce its arguments more strongly and
  optimize getWiki() to avoid causing a connection attempt.
* Avoid deprecated method call in LBFactory::destroyInstance().

Change-Id: If134b62d4f48cd68cb48ccbe149e72f12aa26819

API: Expose $wgEnableMagicLinks in meta=siteinfo

So Parsoid can use it.

Bug: T145590
Change-Id: Id792e07dc6fffec6d02b34168220b3a4e24b222b

Merge "Remove pointless double exception logging from JobRunner"

Merge "momentjs: Hack around bug in node/browser compat wrapper in locale files"

Avoid warnings when unable to parse the URL given to us

Change-Id: I56f1f36a3a54c6c23579f011d65dc75f30cb6dcf

Localisation updates from https://translatewiki.net.

Change-Id: I1c1a5601f4e102c7c8ac32908c91f5f3cff25eaa

Update OOjs UI to v0.17.9

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

Change-Id: Ibe5ba723888646f48ab4be9b88bc7a16ee0ae054

Merge "Fix some PhpStorm warnings in AuthManager"

Remove jquery.arrowSteps module

It was unused by any other extension
https://github.com/search?q=user%3Awikimedia+jquery.arrowSteps&ref=searchresults&type=Code&utf8=%E2%9C%93

and only used for UploadWizard so it was merged into there with
I054674c7025ad37a8592ca82ce7ce0efd41d393f

Depends-On: I054674c7025ad37a8592ca82ce7ce0efd41d393f

Bug: T144974
Change-Id: Id9b6af1f1774bc33ceb024126030eefdc43beba0

ChangeTags: Rename "extension" to "software"

Most end users don't care about whether a tag comes from MediaWiki core
or an extension. Since we now have tags that are added by core, let's
rename all of the "extension" terminology to the more generic
"software".

Not renamed in this patch is the i18n message "tags-source-extension",
which will be done in a separate patch to coordinate with TWN, and the
"source" output of the API, to avoid a breaking change.

Change-Id: Ic7b32b3e4bef0c0153a7cf302eef4aa07c52c9f2

Merge "Add a change tag for edits that change the content model of a page"

Merge "ChangeTags: Skip caching if some hooks aren't registered"

Add a change tag for edits that change the content model of a page

And adjust the 'tags-source-extension' message to indicate that tags are
not just defined by extensions, but the software in general. The
extension related functions will be renamed in a follow-up patchset.

Bug: T145344
Change-Id: Id4391bc100287b8e469636949eaf7845bbd1d67c

Fix typo in "editpage-invalidcontentmodel-text" message

Bug: T145491
Change-Id: I1814a84d49f6cf837f18a1e6e5b3db9602e1f55c

Merge "Add day to date filter for ReverseChronologicalPager"

ChangeTags: Skip caching if some hooks aren't registered

There is caching around the ChangeTagsListActive and ListDefinedTags
hooks that can be skipped if no callers are registered for them.

Change-Id: I0c8fccc358f80536fff7ad0bda1cab09181bca41

Merge "Avoid page_touched update for HTTP GET action=purge requests"

Merge "Make DeferredUpdates call setTransactionTicket() on all DataUpdate tasks"

Merge "Parser: Allow disabling magic link functionality"

Merge "SkinTemplate: Improve remote content handling in content navigation URLs"

Avoid page_touched update for HTTP GET action=purge requests

This will still clear the local DC parser cache and the CDN cache
in all DCs. Therefore, the next page views served by the local
DC will reflect the refreshed content, as will further GET/HEAD
requests by the client that issued the purge using GET/HEAD.

If the problem was imply a stale CDN cache entry, then all
DCs will be up-to-date. If the problem is stale parser cache,
then a proper POST purge request is required to refresh all DCs.

Bug: T92357
Change-Id: I9af12ca8cfff73298f404fd3e2dd4f546621c546

Merge "Add WikiPage::isLocal()"