Merge "rdbms: Use correct value for 'sslmode' in DatabasePostgres"

Merge "rdbms: Fix incomplete doc for DatabaseMssql::tableName()"

rdbms: Use correct value for 'sslmode' in DatabasePostgres

Fix Postgres support by using ‘sslmode=require' instead of ‘sslmode=1'.

See https://www.postgresql.org/docs/current/static/libpq-ssl.html#LIBPQ-SSL-SSLMODE-STATEMENTS

Change-Id: I424b0e3e144bbe9f0a2bde9a3b4a674dde10c729

rdbms: Fix incomplete doc for DatabaseMssql::tableName()

I'm also replacing a loop with an array_pad() call. When the count given
to array_pad() is negative, the padding is done to the left instead of
the right. This is exactly what the loop did before.

I'm also adding the missing limit to the explode() call.

Change-Id: I6737fcf6b60b822269b855f1405607be7fc23754

Merge "Supress ChangeTags::addTags() exceptions for ManualLogEntry objects"

Merge "OOUI forms: Remove infusable = false"

Merge "HISTORY: Add MediaWiki 1.32.1 release change notes"

HISTORY: Add MediaWiki 1.32.1 release change notes

Add MediaWiki 1.32 release change notes, sourced from https://phabricator.wikimedia.org/source/mediawiki/browse/REL1_32/RELEASE-NOTES-1.32

Bug: T213714
Change-Id: I307ee0373021639835a6f3a3c4d3d6b7c6c3dbc8

Supress ChangeTags::addTags() exceptions for ManualLogEntry objects

Looks like some parts of the code try to publish log event when
$newId is 0 or null. This cause ChangeTags::addTags to throw
an exception as at least one of rc_id, rev_id, or log_id must be
specified.

When ChangeTags::addTags() fails (because both $rev_id and $log_id
are not present), just ignore the exception and continue with
the execution.

Also, if one of those is set to 0, we need to pass null instead
(do not insert 0's in to DB as both log_id and rev_id are foreign
keys).

Additionally log all places where ManualLogEntry::publish() is
called with incorrect arguments so later we can fix all occurencie
and remove that try{}catch around ChangeTags::addTags() call.

Bug: T218940
Change-Id: I495f79f2b7a7ef1503d229a689babdc12deb353c

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

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

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

Merge "Fix invalid HTMLForm::setValidationErrorMessage doc"

HISTORY: Add MediaWiki 1.13 post-release change notes

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

Bug: T213714
Change-Id: Id1030ccb01e15538ba563cabbc283a90cf1e6992

Merge "Stop gap to shut up log spam due to T212428."

Use finally instead of try-catch-throw

When this exception handler was introduced in change Ifbc1db4b82 (commit
9636d44448), MediaWiki still supported PHP 5.3.2, where finally was not
available. It was added in PHP 5.5, though, which we’ve required for a
while now, so we can simplify this a bit.

Change-Id: I7c967fb713de7495739e18ff6c6058a630129fda

HISTORY: Add MediaWiki 1.9 post-release change notes

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

Bug:T213714
Change-Id: Idbb105e332afeec1daafaf275da6ae432a698371

HISTORY: Add MediaWiki 1.10 post-release change notes

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

Bug:T213714
Change-Id: Ib313692a359d249ede38c51e823042a9f6f2b0e6

Merge "OutputPage: Fix typo in escaped"

OutputPage: Fix typo in escaped

Spotted by phan.

Change-Id: Iba372755869c602990ea063f3bfde36e19561adb

Localisation updates from https://translatewiki.net.

Change-Id: I7090f89a59d6688c2e196d972f016785871cd89a

Merge "db: Use variable functions/callbacks instead of call_user_func()"

Merge "block: Use correct function wfGetDB() as defined"

Merge "specialpage: Remove unused QueryPage::doFeed()"

Merge "Update some minor type hints"

Stop gap to shut up log spam due to T212428.

Bug: T212428
Change-Id: I442bbe9837e73167cfdaabb0451bf974dc5039f3

History page is now using pseudo elements for presentation

|, ( and ) separators are now added via CSS rather than being
made part of the HTML.

Bug: T205581
Change-Id: I272770418ee544b22f8f94312daf07010b7a2273

dictionary: Remove "mobilelanding"

Bug: T187716
Change-Id: Ia3a77245fe0daacc39c42fd83e18bf9e59f7c74b

Merge "Simplify Block::getBy and Block::getByName"

Merge "Use Block::setTimestamp to set the timestamp on a block"

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"

OOUI forms: Remove infusable = false

False is already the default value for infusable.

Change-Id: Ie0e9b7467b7f3f13c45372b38b7b80ba78852e55

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"

Use Block::setTimestamp to set the timestamp on a block

Follow-up to I767ed44ce4c2e, where the setter was added.

Change-Id: I888e758aef44a101068bd52e89636e2656899f62

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"

specialpage: Remove unused QueryPage::doFeed()

This code was written in 2004, and pretty much untouched since then.

The last non-trivial update to this code was done in 2014 via I1de6301.
But as far as I can tell this was just maintenance, not motivated by
somebody actually using this code.

I suspect this is unused for a long time already, long before 2014.
Possibly even unused in 2004. Really: I tried to dig into the revision
history of the core codebase to find when and where this was used, and
when it became unused. I could not find anything. I also used
https://codesearch.wmflabs.org to make sure there are no known callers
or subclasses hidden anywhere. I could not find anything either.

In conclusion I suggest to skip any deprecation phase.

Change-Id: I5f52c44760b903bcb348bebf8c6a0f6faf8180b7

rdbms: Drop unused constructor property and default method arg

Change-Id: Id83efd8f68b241ca0d7c5340cc2fd93379578fac

Update some minor type hints

Change-Id: If5a6372e79038d3750d30a931badd0c110da2fe5

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

db: Use variable functions/callbacks instead of call_user_func()

Performance-wise, this gives a little boost in performance and also
it's more readable I think. But I could be wrong so please someone
let me know.

See: https://www.php.net/manual/en/function.call-user-func.php

Change-Id: I2f0a00408c29549a786b56792411f88998a5f003

block: Use correct function wfGetDB() as defined

Change-Id: I92ff76a9b62dec0f1e3253230676d4b5c0a877f0

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