Merge "resources: Make manageForeignResources a regular maint script"

Merge "Provide direct deprecation message for jquery.ui.spinner"

Merge "mw.feedback: Remove unused custom spinner"

Merge "registration: Follow-up on I54e8065c63f81016425c7dd40ce87e"

registration: Follow-up on I54e8065c63f81016425c7dd40ce87e

Change-Id: I610dfafcd180f89ee0e786f4516f9eb9c63a42b4

Merge "objectcache: Remove unused variable $cache which is overriden"

Merge "Fix MessagecacheTest::testLoadFromDB_fetchLatestRevision"

Merge "changetags: Remove unnecessary semicolon after catch block"

mw.feedback: Remove unused custom spinner

Change-Id: I6d34006b943366045a69a89aa574c77be609440f

Merge "Rename PoolCounter_Stub to PoolCounterNull"

Merge "rdbms: Drop unused constructor property and default method arg"

Merge "DefaultSettings: Remove deprecated $wgDBmysql5 from core"

changetags: Remove unnecessary semicolon after catch block

Change-Id: I89cc05dbbb8c0cf2382e251a3c144353af0906e5

Provide direct deprecation message for jquery.ui.spinner

The whole of jquery.ui is deprecated, but this provides
a more direct replacement suggestion for this module.

Change-Id: I1c09ca48765f3c82f2318fd53d679e4168e5d565

objectcache: Remove unused variable $cache which is overriden

This variable was defined to false and immediately overriden in the
if condition below it which makes the assignment above useless.

Change-Id: Ie72ec7a8860d2a38f59f38b3d861217d39da3e29

Merge "rcfeed: Avoid extending/inheriting from a deprecated class"

rcfeed: Avoid extending/inheriting from a deprecated class

Also, fix if...else syntax with return statement in the if block.

Change-Id: If14f0835731f264622cf90fcfd5c7cc850f63050

registration: Remove wrong `@return` PHPDoc comments

$this->attributes is an array class property that gets set by the
method extractInfo(). This method really doesn't return an array
though in a sense $this->attributes gets set which can be used but
it's not returned.

Change-Id: I54e8065c63f81016425c7dd40ce87e1eb8ba0830

Merge "Add filterFunction to visible{Byte,CodePoint}Limit"

Fix MessagecacheTest::testLoadFromDB_fetchLatestRevision

This re-enables the "flaky" testLoadFromDB_fetchLatestRevision which
was disabled because it failed on random patches for no apparent
reason, as documented in T219042.

The reason was an invalid timestamp in the test data, which caused
the timestamp to default to "now", which was "sometimes" newer than
the timestamp of the revision immediately created before.

Bug: T219042
Bug: T218918
Change-Id: I38f4816b2dc735a7698626c7e3fe4a93ec95a9d1

Merge "db: Drop child methods as they're same as parent methods"

rdbms: Drop unused constructor property and default method arg

Change-Id: Id83efd8f68b241ca0d7c5340cc2fd93379578fac

Merge "docs: Update outdated PageContentSave hook documentation"

Merge "upload: Simplify file extension extraction in UploadBase"

Merge "Use the ?? feature instead of isset() where it makes sense"

Merge "objectcache: remove BagOStuff::mergeViaLock() and update RESTBagOStuff"

db: Drop child methods as they're same as parent methods

Change-Id: I8b3827a57782f02497da2ab8191d1dad11e92bca

Rename PoolCounter_Stub to PoolCounterNull

Also move to own file
Remove "Stub", because it refers a php concept,
which is not used here

Change-Id: Ife952901b5ad8e4a408d229ee72de953a9acc220

Use the ?? feature instead of isset() where it makes sense

Change-Id: I3a54f36b33d99ef3ff4c63e32e7dfcbcfc296135

resources: Make manageForeignResources a regular maint script

The maintenance script was previously in a subdirectory which
is a bit unusual. It's now a regular maintenance script,
with the YAML file in a more discoverable place as well.

Change-Id: I38690f23dbe89839bee0e1d43ecfd68adeb19c0f

Fix off by 1 error when displaying the number of a line with an error

Change-Id: I876ba04f6903127a9d5adedfe0a5e2598a90068c

Merge "Use foreach for readability"

Merge "make xml abstracts, stubs and page log dumps work again"

Use foreach for readability

Change-Id: Ia4701bc50d8c57b9dcafe2da3bb399d59f4c8d0e

Make use of the list() feature where it makes sense

This code is functionally identical, but less error prone (not so easy
to forget or mix these numerical indexes).

This patch happens to touch the Parser, which might be a bit scary. We
can remove this file from this patch if you prefer.

Change-Id: I8cbe3a9a6725d1c42b86e67678c1af15fbc5961a

make xml abstracts, stubs and page log dumps work again

Broken in I979b6c8f0a72bc1f5ecce1d499d3fdfa0f671588

Bug: T174031
Change-Id: I494fe7578f936a2316c27f9c419e981055c38ed4

objectcache: remove BagOStuff::mergeViaLock() and update RESTBagOStuff

All subclasses are now using the CAS variant, which respects $attempts.

Change-Id: Ia7ec6a7f3337cabe95c54c1142c3c5464c1794e7

Fix overzealous prefix check in MWLBFactory

Follow-up 9df277a4ba3

If the server array ommitted 'tablePrefix', Database will
use '' as the prefix, so if $wgDBprefix is also '', then
no error should happen.

Change-Id: Ic207c3fceae8fd45c87f7fa24678079217a60730

Merge "Fix PhanPluginDuplicateConditional* errors (#1)"

Merge "Upgrade to newer phan"

Merge "session: Fix missing return in SessionBackend::resetId()"

Merge "API: Fix missing return in ApiResult::addContentValue()"

Disable flapping MessageCacheTest::testLoadFromDB_fetchLatestRevision()

Bug: T219042
Bug: T218918
Change-Id: I93ed4d088691bfc50aeef1f9c07af5a6dc50de4b

Fix PhanPluginDuplicateConditional* errors (#1)

Change-Id: I61e4d4a60dda27f43debab71a9c02f14c1468311

Upgrade to newer phan

MediaWiki core now runs phan 1.2.6, bringing in nearly 2+ years of upstream
fixes.

Configuration was moved from `tests/phan` to `.phan/`. The legacy bash wrapper
script is still kept in the repository in its own location for any extensions
that are still using it. It should be removed before 1.33 is released.

Since there's a lot of new issues being flagged, all currently failing issues
are suppressed, and will be fixed in follow-up patches.

We're dropping the jetbrains/phpstorm-stubs repository in favor of just
the minimal stubs we need. Stubs for PHP extensions are kept in
the new `.phan/internal_stubs` directory, since they're in a slightly
different format than normal stubs.

Normal stubs are kept in `.phan/stubs`. wikidiff2 and excimer are kept with
these since we're also the upstream for them.

Change-Id: I3fe437befa17f4fbaf97aa6271f659b56021f396

session: Fix missing return in SessionBackend::resetId()

Spotted by phan.

Change-Id: Ic11864916d50e79031c30a0e74dbbcf548ce31a0

API: Fix missing return in ApiResult::addContentValue()

Spotted by phan.

Change-Id: Ifca5e15a1360008b5d91e1b6f483da8e0367819a

Merge "add lbzip2 output processor for exports"

add lbzip2 output processor for exports

Bug: T214293
Change-Id: I98e26b833df473bbeb3dc1b881f428174d776b64

Merge "Move class RevisionListBase and friends to own file"

Merge "Do not suppress php notices in SpecialPageFatalTest"

Merge "block: Avoid use of empty() which doesn't warn on var non-existence"

DefaultSettings: Remove deprecated $wgDBmysql5 from core

This was an experimental configuration setting and was supposed
to be removed in 1.33. Also, remove option from db.

Bug: T196185
Change-Id: I357348505589ff963d668acadce3b58fe2a1b0bb

Merge "Remove reliance on Block properties being public"

Merge "Avoid the use of IDatabase::insert() return value"

Merge "rdbms: halt on some common broken $wgDBServers configurations"

Merge "registration: Fix if or elseif constructs with return statements"

block: Avoid use of empty() which doesn't warn on var non-existence

As empty() is more tolerant and doesn't warn on variable non-existence
and will go ahead and treat it as if it has the value null, it's better
to use ! in this case to make sure we catch even things like typos in
the variable names within the code base.

Let's make sure intended checks are actually checked instead of being
tolerant. Follow up on: Ibaf555fac293da46fa397ccb9, thanks @Krinkle.

Change-Id: Ibc45dba373bbbb6735a9681ed07a2438d4073ee0

registration: Fix if or elseif constructs with return statements

Change-Id: I7abd09d8959daddaa161bd92a1b1b23a4abbab32

Remove reliance on Block properties being public

Use getters and setters for $mReason, $mTimestamp, $mExpiry and
$mHideName; use Block::getType to check if a block is an autoblock
instead of checking $mAuto; no change needed for $mParentBlockId,
which is not accessed externally.

Change-Id: I767ed44ce4c2e21f53962d75fb86891add2282f6

Avoid the use of IDatabase::insert() return value

Change-Id: Ibaf555fac293da46fa397ccb993cfb61bfcd67aa

Localisation updates from https://translatewiki.net.

Change-Id: I0b1430a5ae16b2b048e6b7467991c5e53aa18dd1

Merge "Update OOUI to v0.31.1"

Add filterFunction to visible{Byte,CodePoint}Limit

The underlying {byte,codePoint}Limit functions already supported this
parameter to apply to the string before limiting the length; add it to
the visible limiting functions as well, displaying the remaining
characters after the text is filtered.

Change-Id: Id8f039ef21ed64dd97c6711efa56d59bb0264669

Merge "User::(get|is)Block*(): We don't need to prefix variables with their type…"

Merge "Only load latest revision in MessageCache::loadFromDB"

Only load latest revision in MessageCache::loadFromDB

In Id044b8dcd7c, we lost a condition that ensured that the cache would
be populated with the latest revision. Now it was being populated with
all revisions, with a random one winning.

Bug: T218918
Change-Id: I1a47356ea35f0abf35bb1a3489d0d3442a3400a5

User::(get|is)Block*(): We don't need to prefix variables with their type…

Change-Id: I38e10453051dd300217542de68daab352993b2a6

Merge "Remove i18n parameters"

Remove i18n parameters

Since the confirmation message can no longer show the edit count, remove
the data attribute that contains it.

Also make the selector for the confirmation more specific.

This is a followup for change 498096

Bug: T218354
Change-Id: I08dd75d3b161d7869357e486b25693dde635eb14

Fix WANObjectCache typo in phpdoc

Change-Id: I6b9d68bc4832fe71217d5c0ef66bebdfa9b459ce

Merge "don't die producing xml files if rev text export conversion fails"

rdbms: halt on some common broken $wgDBServers configurations

Mention the changes, usually simple, needed to fix the problem.

Bug: T215850
Change-Id: I9911f5f5213682ffb52ac548e3082db548e37b3d

Merge "Use mediawiki/mediawiki-phan-config 0.3.0"

Use mediawiki/mediawiki-phan-config 0.3.0

Use the library instead of duplicating most of the config/defaults that it
provides. MediaWiki core is different of course, so we have to override a
bunch of file/directory lists, but there was a lot being duplicated.

This is the first step in migrating to a newer phan version.

Change-Id: Ib5987ebdf208138d97e1aba8ef54438064063fe9

Merge "rdbms: avoid connections on more lazy DBConnRef methods"

Merge "rdbms: update wasQueryTimeout() for DatabaseMysqlBase"

Merge "rdbms: add 1062 error code to DatabaseMysqlBase"

rdbms: avoid connections on more lazy DBConnRef methods

Also:
* Fixed LoadBalancer::getAnyOpenConnection for both
  DB_MASTER and DB_REPLICA, which are not real indexes.
* Lock down DBConnRef::close since it can only cause trouble.
* Relax DBConnRef restrictions on tablePrefix()/dbSchema()
  for the harmless "getter" mode case.
* Remove redundant DatabasePostgres::getServer definition.

Change-Id: Ia855d901cc3c28147e52284fdabb1645805d4466

Merge "Support more coupled DBs in AtomicSectionUpdate/AutoCommitUpdate"

rdbms: update wasQueryTimeout() for DatabaseMysqlBase

Change-Id: I533d68972c758a5c8bb251cd838b8802799f0318

Merge "rdbms: make LBFactory::setLocalDomainPrefix preserve the schema"

rdbms: add 1062 error code to DatabaseMysqlBase

See https://dev.mysql.com/doc/refman/5.7/en/server-error-reference.html

Change-Id: Id4960a5693332e6291ddcb5564bf6f8177a90c3d

Merge "Title: Test the ->equals() method more thoughroughly"

Merge "Fix fatal due to lock name mismatch in Maintenance::unlockSearchindex"

Merge "rdbms: restrict more unsafe IDatabase methods in DBConnRef"

Merge "objectcache: convert APC and hash BagOStuff to using mergeViaCas()"

Merge "objectcache: optimize merge()/incr() for WinCacheBagOStuff"

Title: Test the ->equals() method more thoughroughly

Move from TitleMethodTest to TitleTest for simplicity with duplicating
into TitleValueTest. We're not using the language- and namespace-specific
test setup with these tests anyway.

Change-Id: Ieec78c35f04faea5e01da8d39ed88f7c4876ac84

Merge "Fix phpstorm IDEA errors in SkinTemplate"

rdbms: restrict more unsafe IDatabase methods in DBConnRef

Change-Id: I1933da825e6b20b4ec742e0aa39d63e9a75f9a1c

Merge "Do not create new archive file names for old files"

Merge "RevDel: Avoid log_search rows with empty values for target_author_actor"

Localisation updates from https://translatewiki.net.

Change-Id: I0dd7b417499db34a6b99d3491c34e361e84a5b9e

Support more coupled DBs in AtomicSectionUpdate/AutoCommitUpdate

This is useful for updating a secondary database based on a
commit to the main one, especially for using auto-commit.

Change-Id: Iad9084d2fb0490ecdfd5fcc97db33bfbcbfa5d21

RevDel: Avoid log_search rows with empty values for target_author_actor

During migration, RevDel may wind up being used on items where an actor
has not been assigned yet. The code creating log_search rows for
target_author_actor needs to take this into account.

Also, to clean this up on Wikimedia wikis, I've added code to
MigrateActors to delete these rows before (re-)migrating log_search
and a --tables option so a re-run can skip trying to process all the
already-processed tables (cf. T188327#4892827).

Bug: T215525
Change-Id: Ica15e2e30445e23761e6d3d6405b3eb39a086161

WebRequest: Simplify getGPCVal

* Call $contLang->checkTitleEncoding() only with a string value as
  parameter. (A boolean value here caused T218883.)
* MediaWikiServices::getInstance()->getContentLanguage() always return a
  Language object. An exist check is not necessary.

Change-Id: Idd3e3a2baa5072e862d7502e30079a1b33d6a866

Merge "Use delegated events for rollback confirmable"

Merge "Rearrange code in User::getBlockedStatus to avoid isAllowed calls"

Use delegated events for rollback confirmable

This change is necessary to make the rollback confirmation work even
when the list of items which can be rolled back is updated via
JavaScript (this is possible on RecentChanges, for example).

Bug: T218354
Change-Id: Idafe1380b283cf0056ff1efdffa04aca92cf1217