Merge "Drop wfRunHooks, deprecated since 1.25"

Merge "Drop UserGetImplictGroups, deprecated since 1.25"

Merge "Drop DoEditSectionLink, deprecated since 1.25"

Merge "tests: Add helper function for ini_set with automatic cleanup"

Merge "Load installer i18n when running update.php"

Merge "messagecache: avoid caching message pages that do not override"

Merge "Do not retry the ThumbnailRenderJob."

Load installer i18n when running update.php

Bug: T206765
Change-Id: Ie1c1a5dbf7c905eaca55777448e5aba00267ed31

Merge "Revert "Unwrap HTML loaded from parser cache""

Do not retry the ThumbnailRenderJob.

The job is a warmup for the thumbnail cache, so loosing it is not
an issue. Most of the times thumbnail rendering fails because of
non-renderable or broken images, or because the thumbnail was
already created by a request. These conditions will not be fixed
by retrying, will only create more load on Thumbor.

Bug: T203135
Change-Id: Ib4640eb13ca93f7350ff0671fc559a76ebbbb1e9

Only expand `{{...}}` in messages once

If we're going to call `OutputPage::addWikiText` to parse the message,
we don't need to pre-expand `{{...}}` markup before passing it to the
parser.  This makes these callsites consistent with how
`OutputPage::addWikiMsg()` inserts messages.

Bug: T206574
Change-Id: I2f68f73365df876e6f9d017e4e3ddea39c4a1284

Merge "Drop File / MediaHandler::getStreamHeaders(), deprecated since 1.30"

Merge "Drop DeferredUpdates::setImmediateMode(), deprecated since 1.29"

Merge "Drop ChangesListSpecialPageFilters, deprecated in 1.29 and uncalled"

Merge "install.php: Allow extensions and skins to be specified"

tests: Add helper function for ini_set with automatic cleanup

Some tests need to change the value of an ini setting, and typically implement
cleanup handling themselves, usually imperfectly.

Provide a helper function, $this->setIniSetting(), which will take care of
teardown in the same way that $this->setMwGlobals() does.

Change-Id: I7be4198592f0aaf73a28d3c60acb307a918b1a1f

Merge "Hard-deprecate hooks APIGetDescription and APIGetParamDescription"

Merge "WikiPage: Fix viewing of wiki redirects to NS_MEDIA"

WikiPage: Fix viewing of wiki redirects to NS_MEDIA

If a user creates a redirect to a Media namespace title, a fatal
error is thrown on viewing such rediect because we protect against
redirecting to virtual namespaces. This fix catches this kind of
redirect and modifies the namespace to be File before the Title object
is created.

Follow-up from 613e2699.

Bug: T203942
Change-Id: Ib211d98498f635862fea6bf3e7395f4f8718b3d8

ActorMigration: Remove possibility of read-both

When this was originally written, the plan was to read both the old and
new fields during the transition period, while stopping writes to them
midway through. It turns out that the WHERE conditions to do read-both
correctly are generally not handled well by the database and working
around that would require a lot of complicated code (see what's being
removed from ApiQueryUserContribs here, for example).

We can simplify things greatly by instead having it write both fields
during the transition period, reading from the old for the first part
and the new for the second part, as is being done for MCR.

Bug: T204669
Change-Id: I4764c1c7883dc1003cb12729455c8107319f70b1
Depends-On: I845f6ae462f2539ebd35cbb5f2ca8b5714e2c1fb
Depends-On: I88b31b977543fdbdf69f8c1158e77e448df94e11

HTMLInfoField: Undo breaking change, deprecate instead, add release notes

Follow-up to e6eb87ae2028d9d3f9317d75cced42511ed97b9e.

Bug: T203202
Bug: T205956
Change-Id: I2d19d376d218c59e4ea36e8635e883afac2adea6

Merge "Drop wfUsePHP, deprecated in 1.30 and unused"

Merge "Fix TitlePermissionTest failures due to test leakage"

Merge "rdbms: Database::selectDB() update the domain and handle failure better"

Fix TitlePermissionTest failures due to test leakage

Bug: T201776
Change-Id: I46d44c031ec45ac0be4c372891d2992c1e4f35ab

Merge "Minor OrderedStreamingForkController improvements"

Drop wfRunHooks, deprecated since 1.25

Change-Id: I0350fba3f1a4a4ff500af746609bc9c6b46f0bdf

Drop UserGetImplictGroups, deprecated since 1.25

Change-Id: Iffe29131fcc85d3477fe93508902d8ea982289f5

Drop DoEditSectionLink, deprecated since 1.25

Change-Id: I78b468da34467863047cd01623997dc67cc4fc76

Drop File / MediaHandler::getStreamHeaders(), deprecated since 1.30

Change-Id: Ia4f6cee5674a492c098c343943095cb668172311

Drop DeferredUpdates::setImmediateMode(), deprecated since 1.29

Change-Id: I8ae7dedf956034981081d2bcec630d6724921269

Drop ChangesListSpecialPageFilters, deprecated in 1.29 and uncalled

Change-Id: I03a9bce0d1085dc30dbc5c3408b4cea154ade19a

Drop wfUsePHP, deprecated in 1.30 and unused

Change-Id: Ic247af4e9b2b2e687ee74e990e69401426be7c31

Merge "RCFilters: Improve circle mixin and change to new standard icon size"

Minor OrderedStreamingForkController improvements

Follows-up c94dea7029cae.

* Avoid strlen() where a strict check suffices.
* Use substr() for both comparison and stripping, avoid
  potentially expensive, but also fragile, because strlen()-1
  can  produce $str[-1], which would
  > PHP Notice:  Uninitialized string offset: -1
  Luckily, the implied null would work as expected, given that
  `null !== "\n"` and produces the same result.

Change-Id: If61e5e412aaa2dc7c00c4441b3b7bd5f04160ec8

Merge "Remove pear/mail_mime-decode"

Merge "Hard deprecate OutputPage::addWikiText*Tidy() methods"

Localisation updates from https://translatewiki.net.

Change-Id: I9682ae829e3659b2cf672847961c3be3d8578568

Remove pear/mail_mime-decode

Bug: T109121
Change-Id: I540b8a59dbe88c27f49b3c4cd8740872afd5d8bf

messagecache: avoid caching message pages that do not override

Clean up individual message cache handling when the master key is
volatile. The goal is not to treat the message as gone but to re-fetch
it when accessed rather than cache the value in the main/process cache.

Bug: T193271
Change-Id: I4bcaf10c1516e7c96c2d0963722affeaf80272e0

RCFilters: Improve circle mixin and change to new standard icon size

Improve circle mixin, variables and apply standard icon size:
- Apply new icon size of 20px,
- Divert `.mw-rcfilters-mixin-circle()` into two mixins, applying former
only on circle classes to not repeat output of shared properties for
each color,
- Add new parameter to apply `px` based `min-width`/`min-height` to
circumvent rendering issues,
- Rename variables for unification with WikimediaUI Base vars and easier
readability

Bug: T190980
Change-Id: I76fec57ff237ba02afefcea28916d8348bac9b0e

rdbms: Database::selectDB() update the domain and handle failure better

LoadBalancer uses Database::getDomainId() for deciding which keys to use
in the foreign connection handle arrays. This method should reflect any
changes made to the DB selection.

If the query fails, then do not change domain field. This is the sort of
approach that LoadBalancer is expects in openForeignConnection(). Also,
throw an exception when selectDB() fails.

The db/schema/prefix fields of Database no longer exist in favor of just
using the newer currentDomain field.

Also:
* Add IDatabase::selectDomain() method and made selectDB() wrap it.
* Extract the DB name from sqlite files if not explicitly provided.
* Fix inconsistent open() return values from Database subclasses.
* Make a relationSchemaQualifier() method to handle the concern of
  omitting schema names in queries. The means that getDomainId() can
  still return the right value, rather than confusingly omitt the schema.
* Make RevisionStore::checkDatabaseWikiId() account for the domain schema.
  Unlike d2a4d614fce09c, this does not incorrectly assume the storage is
  always for the current wiki domain. Also, LBFactorySingle sets the local
  domain so it is defined even in install.php.
* Make RevisionStoreDbTestBase actually set the LoadBalancer local domain.
* Make RevisionTest::testLoadFromTitle() account for the domain schema.

Bug: T193565
Change-Id: I6e51cd54c6da78830b38906b8c46789c79498ab5

Hard deprecate OutputPage::addWikiText*Tidy() methods

These have been replaced with OutputPage::addWikiTextAsContent().
There is only one apparent use outside core, in the ProofreadPage
extension.

Bug: T198214
Depends-On: I826e463c1623481c11156a55a5b7e1da0aa27b8b
Change-Id: I4b1a9429029bfbbf70b2d5da1b290319ecbbdf3a

Hard deprecate unused OutputPage::addWikiText* methods

Codesearch shows no users of `OutputPage::addWikiTextWithTitle()`,
and no users of `OutputPage::addWikiTextTitle()` (other than the
implicit uses by the other `OutputPage::addWikiText*()` methods).

These methods produce untidy output, which future parsers won't
support.

Bug: T198214
Change-Id: Id5ee3bdfa6c464e3a92af82af7bc7317ca9d07a9

Update users of deprecated OutputPage::addWikiText*Tidy() methods

This is a follow-up to Ia58910164baaca608cea3b24333b7d13ed773339 which
added new & improved names for the addWikiText*Tidy() methods.

Bug: T198214
Depends-On: I42d198e6d03afd5b976da741f235c175bfe9f767
Change-Id: I1993c2009e861fdb45c9b1bd42eab1e28a776d9a

Deprecate and rename OutputPage::addWikiText* methods

Tidy will always be enabled with our future parsers, and it is fast
and pure PHP now with the Remex implementation, so deprecate all the
untidy variants of 'OutputPage::addWikiText*()' and add new methods
which tidy by default.  Clarify the content language/interface
language distinction while we're at it by adding 'AsInterface' to the
name of methods which use the "interface language" by default,
and renaming the 'addWikiText*Tidy' methods to
'addWikiTextAsContent'.

The 'OutputPage::addWikiTextTitle' method has been deprecated, but it
is still used internally as the implementation for the newly-added
methods.  It is expected that the shared implementation will move in
the future to a new private method.  Setting the `$tidy` parameter of
`OutputPage::addWikiTextTitle` to false is independently deprecated;
for backwards-compatibility with old MW releases you may wish to
continue to invoke OutputPage::addWikiTextTitle() but set $tidy=true;
this will result in the same tidied output that the newly added
methods would produce.

Bug: T198214
Change-Id: Ia58910164baaca608cea3b24333b7d13ed773339

Hard-deprecate hooks APIGetDescription and APIGetParamDescription

Deprecated since MW 1.25.

Change-Id: Idf376b64490eaf8cf5a1b5c3a807fe74b3c2852c
Depends-On: I0bc394f11fda5f0118904756b99f1681897cd6fd

Add pear/Net_SMTP 1.8.0 to composer dependencies

Bug: T200254
Change-Id: I231759a1e0bd5b8750939acaad85ed327d052b79

Merge "ChangeTagsTest: Mark tables as "used" to avoid ID reuse"

Merge "Use non-deprecated login in ApiLoginTest"

Merge "Improve ApiLogin test coverage"

Merge "resources: Update CLDRPluralRuleParser to v1.3.2-pre"

Use non-deprecated login in ApiLoginTest

Change-Id: I62c7bfce88aeda7ec1429493e733f868887976a9

Improve ApiLogin test coverage

Coverage is 100% except for one session-related bit that seems a bit
involved to test right now.  It looks like it will be easier once
SessionManager becomes a service.

I removed the third parameter from the return value of
canonicalizeLoginData, since af37a4c7 made it always return true.

I also removed three lines of dead code from ApiLogin.php.

Change-Id: Ia0073eddd27c82827518e0031e3c313f83cfd7cc

Merge "Hard-deprecate authentication-related hooks deprecated by AuthManager"

Merge "Re-namespace RevisionStore and RevisionRecord classes"

Merge "Use job queue for deletion of pages with many revisions"

Merge "Tests: Simplify badaccess group check for patrol action"

Merge "Revert "Replace Media namespace redirects with File namespace""

Tests: Simplify badaccess group check for patrol action

The important part of the assertion is badaccess-groups, so we can drop the
other components and simplify the test code, plus also work around database
name issues that caused Travis CI to fail.

Bug: T206130
Change-Id: I137be892a611cd1f2d61baa77ad9528659587adf

install.php: Allow extensions and skins to be specified

Allow the extensions and skins installed by maintenance/install.php to
be customised using --skins= and --extensions=. If the argument is
am empty string then no extensions/skins are installed. For backwards
compatibility, the default is to install all skins, but to install all
extensions only if --with-extensions is given.

The new CLI options may be specified multiple times, but for
convenience, comma-separated lists can also be used.

Also:
* Rename $option to $options
* If an extension has a dependency error, propagate the very readable
  error message generated by ExtensionRegistry back to the user.
* Split getExtensionInfo() from the loop body of findExtensionsByType(),
  so that CliInstaller can use it to validate its parameters and get
  error messages.
* I didn't like the idea of removing the "s" from the directory name in
  order to construct the JSON file name, so I split
  findExtensionsByType() from findExtensions(), with the former not
  having this hack. In findExtensions(), make the previous assumption
  that the directory name is always "extensions" or "skins" explicit,
  throwing an exception if it is otherwise.

Change-Id: Id0fb63cd4e61a047ef3396ee1c38d6073dfc7fd1

Merge "Remove ugly function existence check"

Merge "Fix index name in comment to "PHP_INFO""

Hard-deprecate authentication-related hooks deprecated by AuthManager

Hard-deprecates AbortAutoAccount, AbortNewAccount, AbortLogin,
LoginUserMigrated, UserCreateForm, and UserLoginForm.

Change-Id: Ibb267def9c295997ef71d820ff482e1e6a816873

sinonjs: Update from 1.17.3 to 1.17.7

Upstream Sinon.JS no longer supports the pre-2.x v1.17 branch
anymore but there have been a few post-2.0 releases to address
bugs in v1.17 that we can still benefit from.

https://github.com/sinonjs/sinon/blob/v1.17.7/Changelog.txt

Notable changes include:

* Fix Blob feature test to support running tests on Safari 9.
* Avoid calls to object-local 'hasOwnProperty'.
* Improve error messages and stricter signature/type checks so
  that debugging code is easier when things go wrong.
* Fix various gaps in the XHR mock.
* Misc fixes for Node.js support.

Also remove outdated comment about ie-hacks from Sinon 1.15.

Change-Id: I66d1b461465b92798ad7eb2efcf4df2731cc78a4

Merge "RCFilters: Override frameless button opacity to show real highlight color"

Merge "Use <div> wrappers instead of <p> in ProtectionForm"

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

Localisation updates from https://translatewiki.net.

Change-Id: Ia87e35c10f78bea2f7107605bb1adcc00acccbd6

Merge "RCFilters: Apply `pointer` cursor on Hamburger menu"

ChangeTagsTest: Mark tables as "used" to avoid ID reuse

If ChangeTags::updateTags() isn't passed all three of $rc_id, $rev_id,
and $log_id, it attempts to look up proper values from the subset that
is passed.

Some of the tests in ChangeTagsTest blindly use certain IDs, expecting
them to either not exist in the relevant tables or at least to be sane
(e.g. the same log_id isn't used on multiple rc_id rows). Depending on
the exact behavior of previous tests, this expectation may not hold.

The simple fix is to just mark the relevant tables as "used" by
ChangeTagsTest so MediaWikiTestCase will truncate them.

Change-Id: Ie60092b8a6e50f6f8a5cfdaaf4cac8ddb4fc6f53

Merge "Clean up ApiLoginTest"

Revert "Unwrap HTML loaded from parser cache"

This reverts commit 1bb5b58eb1cd87eda898d5c7806987dab5a51b65.
A month has passed, the workaround for old parser cache entries
should not be needed anymore.

Bug: T203716
Change-Id: I446b47cc6b4c43aaae33675d62086d842b04ddcb

Merge "Improve exception message in DatabaseDomain"

Merge "Make unclosed transaction errors more useful"

Merge "maintenance: Detect "unknown module name" error in manageForeignResources"

Make unclosed transaction errors more useful

Move unclosed transaction error reporting to Database so it can
include information about the caller that started the transaction.

Change-Id: I834d957f172c03005de522f3029bb634b3c7220e

Improve exception message in DatabaseDomain

Change-Id: Ib45497388d07b0531cfe1c06349105e47b9fad08

Clean up ApiLoginTest

Change-Id: Icd77f8a5ee0c26cfbd3cc8f220412cfa1bf49d60

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit f6c64caaf1becc5b5780c871d495a3d9f5aac7a8

Change-Id: I0d2722c100b522c68a8e35a8701d5bf7ef80dccb

Re-namespace RevisionStore and RevisionRecord classes

During development a lot of classes were placed in MediaWiki\Storage\.
The precedent set would mean that every class relating to something
stored in a database table, plus all related value classes and such,
would go into that namespace.

Let's put them into MediaWiki\Revision\ instead. Then future classes
related to the 'page' table can go into MediaWiki\Page\, future classes
related to the 'user' table can go into MediaWiki\User\, and so on.

Note I didn't move DerivedPageDataUpdater, PageUpdateException,
PageUpdater, or RevisionSlotsUpdate in this patch. If these are kept
long-term, they probably belong in MediaWiki\Page\ or MediaWiki\Edit\
instead.

Bug: T204158
Change-Id: I16bea8927566a3c73c07e4f4afb3537e05aa04a5

RCFilters: Apply `pointer` cursor on Hamburger menu

Bug: T196900
Change-Id: I53310b54616622b214a0a30f3bf826f48d05c7b1

Merge "Update OOUI to v0.29.2"

Merge "Make RefreshLinksJob MCR compliant."

Fix index name in comment to "PHP_INFO"

Change-Id: If006e96f5bae20791c1d388faad685203a061f22

Merge "Revert "Re-enable tests from TitlePermissionTest""

Revert "Re-enable tests from TitlePermissionTest"

It's still broken, and 4 days passed with broken Travis CI builds.
Re-reverting for now.

This reverts commit 589741b5417adbf0e3b0d72640ff6ff297e50141.

Change-Id: I85a0b02d3f32303a90118e2705a7b2afc721cb57

Revert "Replace Media namespace redirects with File namespace"

Per T203942.

This reverts commit 613e269920a7d268b012d61fec24fc4d04b1cd7a.

Change-Id: I4febbe9c650894976a580e584d0a7c11de12ff37

Update OOUI to v0.29.2

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

Bug: T206344
Depends-On: I607825c7df3d843f80f8d151c5c79c39cd629c17
Change-Id: I6db8fbb28ae1c190b1d9e2b4d4defdb20579a169

Fix parser test failure due to changed translation

The 'sr-ec' translation of 'editsectionhint' message was changed
in dff3fa2d4bf2c68b2020ffbf965459fb2e71c50f.

Change-Id: Ibb6ad22e3926406a3e8267b87ffd39ab51663e63

Localisation updates from https://translatewiki.net.

Change-Id: I787568a8746dd03db3f5c2d1c75e248c5c4ce809

Merge "Make SpecialPageTestBase always call parent::tearDown"

Merge "Fix guarding of MySQL's numRows()"

Merge "Suppress "Headers already sent" in PHP 7.2 too"

Make RefreshLinksJob MCR compliant.

So far, RefreshLinksJob re-generated the ParserOutput ba calling
Content::getParserOutput. This only works for the main slot. It
needs to instead get the parser output for all slots combiend,
by using RevisionRenderer and RenderedRevision.

Bug: T174035
Change-Id: I253dda26bfa5aefa15f8b1dcc59e69fc7e9d0cb7

Suppress "Headers already sent" in PHP 7.2 too

The "h" is now capitalized, so we need to update the regex.

Change-Id: I1111e1228868ec66d930c7a3b0d7972e5c6356b9

Make SpecialPageTestBase always call parent::tearDown

Change-Id: Ifc244d5117e7da4bf46cdf273eca9fc4ae5c3a42

RCFilters: Override frameless button opacity to show real highlight color

Overriding OOUI's frameless button opacity, which is targeted in this
place for icons to show real highlight color.

Bug: T190980
Change-Id: Id3355ecad8ff01ef8159375eaa04f7d284fc598b

Merge "Improve ApiFormatJson test coverage"

Merge "Deprecate MediaWikiTestCase::stashMwGlobals"