Merge "GlobalFunctions: Document the usage of wfUrlencode( null )"

Merge "HISTORY: Add MediaWiki 1.23 post-release change notes"

Merge "HISTORY: Add MediaWiki 1.19 post-release change notes"

Merge "HISTORY: Add MediaWiki 1.18 post-release change notes"

Merge "HISTORY: Add MediaWiki 1.12 post-release change notes"

Merge "fix up new text address handling in page content dumps"

Merge "HISTORY: Add MediaWiki 1.11 post-release change notes"

Merge "resources: Add caching for faster runs and offline use"

Merge "resources: Improve foreign-resources docs and error messages"

Merge "resources: Make manageForeignResources a regular maint script"

HISTORY: Add MediaWiki 1.23 post-release change notes

Add MediaWiki 1.23 post-release change notes, sourced from https://www.mediawiki.org/wiki/Release_notes/1.23

Bug:T213714
Change-Id: I20ad7615a2d9b0d3eb99abb41b125b30fc5d1ece

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

Merge "mw.feedback: Remove unused custom spinner"

GlobalFunctions: Document the usage of wfUrlencode( null )

This was introduced for testing in 53b0052df.

Change-Id: I318d6d6c04334ea75fbc9ac1771fab9a9db8cb83

Merge "registration: Follow-up on I54e8065c63f81016425c7dd40ce87e"

fix up new text address handling in page content dumps

Broken in I979b6c8f0a72bc1f5ecce1d499d3fdfa0f671588

Bug: T174031
Change-Id: Ia31fc7b559f59d53bb503a59abf9110f0e65178c

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

HISTORY: Add MediaWiki 1.19 post-release change notes

Add MediaWiki 1.19 post-release change notes, sourced from https://www.mediawiki.org/wiki/Release_notes/1.19

Bug:T213714
Change-Id: Ia3295c12938750337cfd42371a0ae72abfceff2c

HISTORY: Add MediaWiki 1.18 post-release change notes

Add MediaWiki 1.18 post-release change notes, sourced from https://www.mediawiki.org/wiki/Release_notes/1.18

Bug:T213714
Change-Id: I2bea0ca7cb02d32fd24392fa167351851062ccb1

HISTORY: Add MediaWiki 1.12 post-release change notes

Add MediaWiki 1.12 post-release change notes, sourced from https://www.mediawiki.org/wiki/Release_notes/1.12

Bug:T213714
Change-Id: Ib2d34bbb31525e4157d062bc4d89072098e106ce

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"

HISTORY: Add MediaWiki 1.11 post-release change notes

Add MediaWiki 1.11 post-release change notes, sourced from https://www.mediawiki.org/wiki/Release_notes/1.11

Bug:T213714
Change-Id: I3d0894a3f04b4d03b1a354c28025dc78a1b2e27b

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: Add caching for faster runs and offline use

Also honour any XDG variables if set (such as Wikimedia CI does)
so that gate jobs can automatically persist the cache.

Bug: T203694
Change-Id: I156af1c5097b8a87be228345fc0a0b80bb3ae338

resources: Improve foreign-resources docs and error messages

Previously, the help message was only shown when invoking the script
without parameters. Now, if invoking it with incorrect or mispelled
parameters, it shows what you can do instead (and no more exception trace,
which it did before).

Also document clearly but briefly how to use it, and with examples.

Change-Id: Ib5ba3dc02c54039495a33d6e0cc4d026ca708ba1

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"