doc: Modernise parameter names and documentation for 'replica' DBs

Non-breaking change. Remaining uses are public interfaces (a constant, two
globals, a config sub-parameter, SQL queries, storage function names), one i18n
message key, and a whole lot of maintenance scripts with calls to the deprecated
function wfWaitForSlaves().

Change-Id: I6ee5ca92ccf6a80c08f53d9efe38ebb4b05064d7

Follow-up d67121f6d: Blocks can apply to non-User objects too

Bug: T208398
Change-Id: I1d39f4ff709f37e7047f49964101e83c97cda0e9

mediawiki.special.preferences/tabs.legacy.js: Drop old `jsprefs` class, unused

Change-Id: I91c3069c7588fe9a77af169bab07ae296bccd2e2

Merge "Drop the LanguageGetSpecialPageAliases and LanguageGetMagic hooks"

Merge "resources: Drop jquery.localize, deprecated in 1.32"

Merge "Drop six authentication-related hooks, deprecated in 1.27"

Merge "Remove use of old API doc methods"

Merge "Use WikiMap methods for wiki ID logic in more places"

Merge "Set the SaveParser logger in RevisionRenderer"

Merge "rdbms: clean up return values of IDatabase write methods"

Merge "Use new WikiMap::getCurrentWikiDomain() for objectcache and profiler keyspace"

Use new WikiMap::getCurrentWikiDomain() for objectcache and profiler keyspace

Change-Id: I0e77c58a55b7ce908ec1d6399ec6629d00a48953

Merge "debug: Allow the DBQuery channel to be used"

Set the SaveParser logger in RevisionRenderer

Change-Id: Iacc28ce583fed8cf2a7f7b931754c64619cb4917

rdbms: clean up return values of IDatabase write methods

Also improved the atomicity and affected row count logic for
insert/replace with sqlite.

Also remove unused "fileHandle" code from insert().

Change-Id: If7b9148fd44f3a958899885753c7c86ba66bf193

Merge "Use WikiMap methods in ClassicInterwikiLookup instead of wfWikiId()"

Merge "Add isCurrentWikiId()/isCurrentWikiDomain()/getCurrentWikiDomain() to WikiMap"

Drop the LanguageGetSpecialPageAliases and LanguageGetMagic hooks

Deprecated since 1.16 and unused in known git.

Bug: T199291
Change-Id: Idcaf1b2c24c64653f700be4fe810ed124ba4ab7c

resources: Drop jquery.localize, deprecated in 1.32

Change-Id: Iaab8637cd336ab47c789ebcf1d0610695555095f

Drop six authentication-related hooks, deprecated in 1.27

AbortAutoAccount, AbortNewAccount, AbortLogin, LoginUserMigrated,
UserCreateForm, and UserLoginForm are all unused in Wikimedia
production and rare in other extensions.

This also scraps the FakeAuthTemplate and LoginForm classes and
the occasional remainig references thereto.

Bug: T193755
Change-Id: I24d6fa963f402d4311fa00fc11536a37ee3bd31e

Use WikiMap methods for wiki ID logic in more places

Change-Id: I25b53576a8fecb7cfb0e4d684f064bebf6c968fc

Use WikiMap methods in ClassicInterwikiLookup instead of wfWikiId()

Change-Id: Ia3d843c79fa31fec59a04b92f4619458ae06de62

Remove use of old API doc methods

The following deprecated methods, intended for overriding by extensions,
are no longer called and are hard deprecated.

* ApiBase::getDescription() (deprecated in 1.25)
* ApiBase::getParamDescription() (deprecated in 1.25)
* ApiBase::getExamples() (deprecated in 1.25)
* ApiBase::getDescriptionMessage() (deprecated in 1.30)

Also, the 'APIGetDescription' and 'APIGetParamDescription' hooks have
been removed, as their only use was to allow extensions to override
values returned by getDescription() and getParamDescription(),
respectively.

Change-Id: I486c4ccab4eca6a85cb17c30dbb2439876123ba1

Add isCurrentWikiId()/isCurrentWikiDomain()/getCurrentWikiDomain() to WikiMap

Use these in place of various wfWikiID() calls.

Also cleanup UserRightsProxy wiki ID variable names and removed unused
and poorly named getDBname() method.

Change-Id: Ib28889663989382d845511f8d34712b08317f60e

Merge "Parser: Remove markNoConversion for displaytitle error message"

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

Merge "Hard deprecate $wgTidyConfig['driver'] = 'disabled'"

Localisation updates from https://translatewiki.net.

Change-Id: Ibbc40ff8c5e87f4450088ff9b5055f4c6b1738d0

Merge "Replace deprecated calls to OutputPage::parseInline()"

Merge "Change ParserOptions `tidy` default to `true`"

Merge "Deprecate OutputPage::parse() and OutputPage::parseInline()"

Parser: Remove markNoConversion for displaytitle error message

bacd87e49 moved the displaytitle error message from the content to
outside of the content. Only the content is converted by the language
conversion. The error message outside of the content is not converted.
Therefor markNoConversion is not needed here anymore.

This change removes the -{R|...}- around the displaytitle in the error
message when the language converter is active.

Bug: T208249
Change-Id: Ieec43e9af045d19b0b7a82afb889e076b347eed1

Merge "Use PHP 7 '??' operator instead of if-then-else"

Change ParserOptions `tidy` default to `true`

We are deprecating the non-tidy modes of the parser.

ParserOptions::getCanonicalOverrides() has always set `tidy` to `true` at
any rate, so this isn't going to invalidate any parser cache entries.

Change-Id: Ib703a041edf8a8d57e94f136965f72d9bbfcf222

Replace deprecated calls to OutputPage::parseInline()

Every one of these seemed to previously use the wrong value for
`$interface` -- the interface messages in ProtectionForm and
SpecialVersion were being parsed as content language (which is the
default for `OutputPage::parseInline`), and the one place where we
have actual data in the content language (the user signature, which
gets parsed in the content language with the rest of the (talk) page
content, see Parser.php::pstPass2) was being parsed as an interface
message!

Forcing the caller to be very explicit about AsContent() or AsInterface()
will help mitigate this in the future, one hopes.

Bug: T198214
Change-Id: Ib9d5d8d733a47c967bdf7db3e23fa39f11687063

Deprecate OutputPage::parse() and OutputPage::parseInline()

The OutputPage::parse() and OutputPage::parseInline() methods behave
misleadingly different from the OutputPage::addWikitext*() methods:
they don't tidy their output, they have different defaults for
interface/content language selection, and they (sometimes) add
wrapper divs.  Deprecate these and add new methods with tidy output,
clear language selection, and consistent defaults:
OutputPage::parseAsContent(), OutputPage::parseAsInterface(),
and OutputPage::parseInlineAsInterface().

Unify the implementation of the parse* methods with the addWikiText*
methods, to reduce the likelihood that the behavior will diverge again
in the future.

Bug: T198214
Change-Id: Ica79c2acbc542ef37f971c0be2582ae771a23bd0

Merge "CalendarWidget: Fix left arrow position in header"

Merge "Fix OutputPage::parseInternal() by stripping <div> wrapper"

CalendarWidget: Fix left arrow position in header

Overwriting OOUI's position out of box.

Change-Id: I7b4f4d65f2046e431d7fa03ffda33fe8d05c3f52

Merge "resourceloader: ResourceLoaderGetConfigVars is passed skin"

Merge "Avoid using return value of IDatabase::insert()"

Merge "Create JobQueueEnqueueUpdate class to call JobQueueGroup::pushLazyJobs()"

Create JobQueueEnqueueUpdate class to call JobQueueGroup::pushLazyJobs()

This assures that MergeableUpdate tasks that lazy push job will actually
have those jobs run instead of being added after the lone callback update
to call JobQueueGroup::pushLazyJobs() already ran.

This also makes it more obvious that push will happen, since a mergeable
update is added each time lazyPush() is called and a job is buffered,
rather than rely on some magic callback enqueued into DeferredUpdates at
just the right point in multiple entry points.

Bug: T207809
Change-Id: I13382ef4a17a9ba0fd3f9964b8c62f564e47e42d

Avoid using return value of IDatabase::insert()

Change-Id: I5992e4f03cd522b25607947291795d1da60d0291

Merge "Avoid broken markup due to conversion table DB queries in Installer:parse()"

Localisation updates from https://translatewiki.net.

Change-Id: If2370cd24814a27fae6bbb7d33a2074c97740df6

Avoid broken markup due to conversion table DB queries in Installer:parse()

Bug: T207979
Change-Id: I6a196a64865533a073fadc4a07f5627b67924921

Use PHP 7 '??' operator instead of if-then-else

Change-Id: Ia86f8433f30a166d38ee63d0d1745b26740767b9

Merge "Deprecate hacky wfSplitWikiID() method"

Make UserEditCountUpdate faster by using auto-commit mode

Bug: T202715
Change-Id: I92c08694cb5e1c367809439cff42e33a56ff9878

Localisation updates from https://translatewiki.net.

Change-Id: Ib268ee41778efe523367bc2c6aaeecafd56efc85

Merge "API: Set robots noindex,nofollow on HTML output"

Deprecate hacky wfSplitWikiID() method

Change-Id: I59095ee7cc39f634d124fa711ec01a5a5d188d23

Merge "resoureloader: Restore support for plain callbacks in mediawiki.base's RLQ"

Increment edit count for all page moves

The user_editcount field should be incremented for all page moves, not just those done using Special:MovePage. Also, if talk pages and/or subpages are moved along with the base page, they will all be included in the edit count.

Bug: T195550
Change-Id: If75a45e4404ca7162b59bf85fa62a996383658b9

resoureloader: Restore support for plain callbacks in mediawiki.base's RLQ

Follows-up dec800968e, which moved the processing of callbacks that require
'modules' from startup.js to mediawiki.base.js.

In doing so, it made an incorrect assumption. It assumed that the simple
signature of RLQ.push(Function) is not needed after 'mediawiki.base' loads.

It is true that RLQ.push() is mostly an internal interface, and we only
use it from within the HTML output, and that once the async pipeline has
finished and startup.js has processed the simple callbacks, only calls with
secondary signatures remain in the queue.

But, while it is true that we don't use RLQ.push() outside the HTML, it is
not true that the HTML will fully load and execute inline scripts before
any of the async scripts execute. As such, the call to RLQ.push() in the
HTML footer was sometimes being ignored because 'mediawiki.base' had already
loaded by now.

Bug: T208093
Change-Id: I25012a2c6f41968b1b4f85614a3bc0416512d530

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

Merge "Move user_editcount updates to a mergeable deferred update"

Merge "rdbms: add transaction comment to IDatabase::masterPosWait()"

Merge "rdbms: fix connection reuse logic in LoadBalancer for postgres"

Localisation updates from https://translatewiki.net.

Change-Id: If9e3c1bf47afc4ee0f0c0afacebf3e32df62fb8c

Fix OutputPage::parseInternal() by stripping <div> wrapper

We should probably strip the <div> wrapper in OutputPage::parse() as
well: this behavior was added in 1.30, but it only applies when
$interface is false.  However, that requires a more careful audit
(a lot more places call parse() than parseInline()) and so I'll defer
that for now.

Change-Id: Iad5412f03af29c04deb653969dd71f6c86f0ae50

.gitignore: Fix .vscode

.vscode is directory

Change-Id: I82d1da1b51b4bd32423ad0bb70d30df109fae1ae

Merge "RCFilters: Apply same distance to FilterItemHighlightButtons as other elements"

Merge "RCFilters: Cleanup 'Namespaces' dropdown borders appearance"

Merge "Remove unnecessary, unoptimized, unused overview SVG"

Hard deprecate $wgTidyConfig['driver'] = 'disabled'

This was already deprecated in the release notes, and is not used in
production, but I'd overlooked adding an appropriate hard deprecation
notice in MWTidy::factory() to notify downstream users.

Change-Id: I8f4d8154a1d8a233017f54f0fb4bcfdf4a0373e1

Fix use of non-existent variable Parser::$config

Fix bug from Ib4394f370cb561ccf195338a1c2e9e465dcb3dc3

Add test.

Bug: T208000
Change-Id: Ia81cca1b64afef2af3cb8dff19719a7f0de9d306

RCFilters: Apply same distance to FilterItemHighlightButtons as other elements

FilterItemHighlightButtons have lost vertical alignment with
“Highlight results”. This patch reintroduces it.

Change-Id: If5578564efced5f0ba4d3ca7ed3d6787b5b0ce3a

Move user_editcount updates to a mergeable deferred update

This should reduce excess contention and lock timeouts.
Previously, it used a pre-commit hook which ran just before the
end of the DB transaction round.

Also removed unused User::incEditCountImmediate() method.

Bug: T202715
Depends-on: I6d239a5ea286afb10d9e317b2ee1436de60f7e4f
Depends-on: I0ad3d17107efc7b0e59f1dd54d5733cd1572a2b7
Change-Id: I0d6d7ddd91bbb21995142808248d162e05696d47

rdbms: add transaction comment to IDatabase::masterPosWait()

Change-Id: I95aa3b678b76cd06439d1feaba7f09a9c13f8931

RCFilters: Cleanup 'Namespaces' dropdown borders appearance

Cleaning-up by
- adding border only between namespace and talk options
- adding namespace border on all but the first occurrence

Change-Id: Icfc87757deda9b7655c4ea5c919b5c2f2e2ae09e

Merge "Clear the title cache in MutableRevisionRecordTest"

Merge "Make testFormatValueRestrictions() account for $wgArticlePath to avoid failures"

Merge "Move timing code out of the try/catch in doPostOutputShutdown()"

Merge "Silence transaction profiler master queries notices for rollback"

Merge "rdbms: remove duplicate @params in IDatabase::select()"

Merge "filerepo: Inject the WAN cache info FileRepo instances"

Merge "Remove unused fields in ForeignDBViaLBRepo"

Merge "rdbms: Use "mixed" type for $profiler field in LoadBalancer"

Clear the title cache in MutableRevisionRecordTest

Also do so in various other test classes.

Follows-up 170c49d61cb8. Fixes Travis CI regression:

> 1) MediaWiki\Tests\Revision\MutableRevisionRecordTest::testSetGetPageId
> Failed asserting that 2 is identical to 0.
> tests/phpunit/includes/Revision/MutableRevisionRecordTest.php:129

Change-Id: I41c8bda8e620ebe7608a393d81f3b0f13af68ba7

Make testFormatValueRestrictions() account for $wgArticlePath to avoid failures

Follows-up 170c49d61cb84411.

Change-Id: I37a57b5ece939c5743b7d08d02ce41c71daee5eb

filerepo: Inject the WAN cache info FileRepo instances

Also fixed some intellij IDEA warnings.

Change-Id: I667ed7ef9b49c1da1c8c7c9914daadb7c74b3c79

rdbms: Use "mixed" type for $profiler field in LoadBalancer

Change-Id: I8d110d0420a155bb62f4212933d9de2562dc7661

Localisation updates from https://translatewiki.net.

Change-Id: I67923401760c77676ed4e674170335333ac15d15

Merge "Fix code style in EmailNotification.php"

Fix code style in EmailNotification.php

Code style fix for namespace "use" statement.

Change-Id: I0755481f640aae535c8328824812e25db467e528

Merge "MailAddress->toString(): Reduce complexity by inverting ifs"

MailAddress->toString(): Reduce complexity by inverting ifs

Change-Id: I5185c0231a0944f36d960211e78cf36273c94d52

Move timing code out of the try/catch in doPostOutputShutdown()

Change-Id: Idc24fdaff5bbcb98445cc4dee8c94cc75cd00573

rdbms: avoid transaction status errors from ping() in rollback()

Change-Id: I38658cbf90518d9818824674da371637db15e156

API: Set robots noindex,nofollow on HTML output

There's probably not much benefit in search engines indexing the API
auto-generated help or various random "format=jsonfm" outputs.

On Wikimedia wikis we already have a robots.txt entry denying indexing
of everything under /w/, which includes api.php.

Bug: T207818
Change-Id: I196d401eeb5fcf49844265df41ad3cce83ffdcdf

Merge "WikiPage: Truncate redirect fragments before inserting them into the DB"

Merge "RCFilters: Fix centering of "no results" message"

RCFilters: Fix centering of "no results" message

Was caused by a stray margin-right rule that applied to the wrong divs,
and wasn't needed anyway.

Bug: T207808
Change-Id: Ib6cf518653fbadb0c7d67d6a175b4af92c8f8d87
Follows-Up: I76fec57ff237ba02afefcea28916d8348bac9b0e

WikiPage: Truncate redirect fragments before inserting them into the DB

The rd_fragment field is 255 bytes wide, but there is no limit on how
long title fragments can be. We don't want to let the database silently
truncate the fragment for us, because that can result in invalid UTF-8.
Instead, truncate it before insertion in a UTF-8-aware way.

Bug: T207876
Change-Id: I12745f3f4c174eaced56d80f3661a71d0e5637e6

Merge "composer.json, require ext-fileinfo"

Disable $wgEnableImageWhitelist by default

This opens up a hole for administrators to load images from external resources,
potentially leaking user's private information to external servers (IP address,
User-Agent, etc.).

Change-Id: Ie780637b292493e664e4d54671a5bb81046106f4

Merge "Use a pre-commit hook for change_tag_def count updates"

Merge "Hard deprecate functionality replaced with random_bytes()"