Hard deprecate Interwiki static methods

Change-Id: I803df3ef90bf6b602e85ae48f25745f4dc6aab3a

Merge "registration: Allow the mtime of extension.json files to be zero"

registration: Allow the mtime of extension.json files to be zero

When creating Docker images of MediaWiki using the Bazel build system, I
noticed that I'm not able to load any extensions. This is due to the
fact that Bazel always generates container layers with mtimes of files
set to 1970-01-01 for determinism/reproducibility.

Relax the check a bit to only fail when the mtime is false, which
happens when filemtime() fails.

Bug: T196672
Change-Id: Ieaeb3113a7d9c44f29cca2d062c5bb11ebeada0d

Restore original OpenSearch namespaces

Though that URL may no longer resolve, the correct XML namespace for
OpenSearch remains http://a9.com/-/spec/opensearch/1.1/ – see [1] and
[2] for some discussion clarifying this.

Similarly, the canonical namespace URL for MozSearch uses HTTP, not
HTTPS, as can be seen when visiting it (via either protocol) [3].

This reverts commit 3b86cf03c162298211da2f57333bcc645123a979.

[1]: https://github.com/dewitt/opensearch/issues/3
[2]: https://groups.google.com/d/msg/opensearch/8ecuX35-lEs/JR_cs0N6k68J
[3]: https://www.mozilla.org/2006/browser/search/

Bug: T196643
Change-Id: I8d14660550a5c67cefc421a334502963bfd6ed3c

Merge "installer: Fix display of UPGRADE by disabling InterwikiLookup"

Merge "Add code to write to change_tag_def table"

Merge "Add getLoginSecurityLevel() support to FormSpecialPage"

Merge "tests: Reset mwuser/pagecontent tables if necessary for postgres"

Merge "Introduce per-schema unit tests for revision storage."

Merge "Reset relevant DB tables before the first test of a suite."

Introduce per-schema unit tests for revision storage.

This introduces traits for testing different schema variations.
These are not very useful in this patch, but make it much easier
to add tests for MCR schema migration in subsequent patches.

The code in this patch was previously part of If259b1e1c49ceaa4.

Change-Id: I239572f75bebbc9c731a3e3860c4eff179dc15e4

Reset relevant DB tables before the first test of a suite.

MediaWikiTestCase truncates all the tables in the tablesUsed field
after each test. It should also do that before the first test of the
class.

Change-Id: I8c33be7b1bdd83559a9ea7803471a1f39e0eb870

Merge "Replace call_user_func_array(), part 1"

resourceloader: move Message methods from mediawiki.js to base module

Also set "use strict" in the base module.

Bug: T192623
Change-Id: I18a710e167df8be47a70da93a7aba0784fc62aef

Merge "Fix UploadBase::checkXMLEncodingMissmatch() on PHP 7.1+"

installer: Fix display of UPGRADE by disabling InterwikiLookup

Since 129067c907ea65f62, parsing section titles has looked up interwiki
prefixes with InterwikiLookup. In the web upgrader, this triggers
database access, and since that service is disabled, it throws
exceptions, causing parsing to fail.

Work around that by using a dummy InterwikiLookup service that knows
about no interwiki prefixes. Maybe one could be written to just read
from the stock `interwiki.list`, but that's a project for another time.

Bug: T196607
Change-Id: I13485a9af79297b552a1128240cb8597c2ef83d8

Merge "Revert "docs: Rename README to README.md""

Revert "docs: Rename README to README.md"

This file is parsed in the web installer, which doesn't
understand markdown - it uses the MediaWiki parser.

This reverts commit c0342da8e9a675ce6803d651a781addb801ef8f3.

Bug: T196607
Change-Id: If35f0212dafa80afe59c019ab25b875cd0977fc8

tests: Reset mwuser/pagecontent tables if necessary for postgres

As of 556c5cf464b9, postgres renames the `user` and `text` tables to
`mwuser` and `pagecontent` respectively. Have resetDB() explicitly
reset those tables when using postgres.

Bug: T195807
Change-Id: I5052dd663a4fb16389611cd2985b712fb9a15069

Fix UploadBase::checkXMLEncodingMissmatch() on PHP 7.1+

file_get_contents() started supporting a negative offset in 7.1+. But
we really just want to start with 0.

Also fix the order of arguments to assertSame() so that the expected
value is first.

Bug: T182366
Change-Id: I84c92652de5b51a43f6e2b58cd235d2889093453

Merge "Use the old linkshere-related messages without "-2""

Localisation updates from https://translatewiki.net.

Change-Id: I6ffcde700fa06cecb3d4bc4016ddc3c0ca2559e9

Merge "Add RELEASE-NOTES for I4678250331a48db4d50d1fc6c489c991a4dee920"

Merge "Selenium tests: Use assert.strictEqual instead of assert.equal"

Add RELEASE-NOTES for I4678250331a48db4d50d1fc6c489c991a4dee920

Change-Id: Ic880a54427bf8bd9adf96863da25c485cb3c5662

Merge "maintenance/purgePage.php output page being purged"

Merge "mediawiki.js: Fix eslint valid-jsdoc"

Merge "Avoid "unittest_imagelinks" already exists error in tests"

Update OOUI to v0.27.2

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

Depends-on: I6755650daf6be2e76785fda992d41c9e7e75e09c
Change-Id: Ieea0c26b0fa9d99c7fc7ebd6cca213e8c33db2ce

mediawiki.js: Fix eslint valid-jsdoc

Remove eslint-disable-next-line valid-jsdoc

Change-Id: I23fc5d68dd25048ed090000946806a01e66e4469

resourceloader: remove "mediaWikiLoadStart" and move mwNow() to mediawiki.js

Bug: T160315
Change-Id: I2d4745e9c34526e5e86f25a3d47799004a5908fe

Merge "build: Enable qunit/no-assert-equal and enforce"

Selenium tests: Use assert.strictEqual instead of assert.equal

assert.equal is deprecated

Change-Id: I2f83d596db2ad475de5e005baa849782261d433e

build: Enable qunit/no-assert-equal and enforce

This is the equivalent of the '===' rule ("eqeqeq") in eslint core.

Change-Id: I1e6ea8bc1e86c6629ebe938f1b4a6bd288bcfcf7

Merge "objectcache: clean up language about expiration/invalidation"

objectcache: clean up language about expiration/invalidation

Change-Id: I593e7bf9bd4f7fbb615d3d164e77273bd4033dde

Update wikimedia/wrappedstring to 3.0.1

Bug: T196496
Depends-On: I13dbdba2b9df52f2d4214c1327e20292c6941331
Change-Id: I5a5d7de4702c23f069c27bcf31b9ace97d54fd82

Merge "Populate 1.31 rc.0 -> rc.2 release notes"

Merge "build: Enable qunit/no-early-return,no-negated-ok and enforce"

Merge "build: Introduce eslint-plugin-qunit and enforce"

Populate 1.31 rc.0 -> rc.2 release notes

Change-Id: I15fa23e881c0e177d12a6f6374bc08af3b197435

Merge "DateTimeInputWidget: Fix disabled `border-color`"

build: Enable qunit/no-early-return,no-negated-ok and enforce

Change-Id: Ief1911517ae06545f842cbcbfd0cf3e815ac2b5b

Add getLoginSecurityLevel() support to FormSpecialPage

The base SpecialPage will handle reauthentication automatically if you
just implement getLoginSecurityLevel() to return an appropriate string.

But it doesn't work with FormSpecialPage, and if you try calling
checkLoginSecurityLevel() manually it'll lose any post data if the
reauth happens when the form is posted.

So this patch has SpecialPage::checkLoginSecurityLevel() preserve post
data across reauth (using logic similar to that in AuthManagerSpecialPage),
and has FormSpecialPage call checkLoginSecurityLevel() in the same
way the base SpecialPage does.

It also fixes the SpecialPage logic to not call
checkLoginSecurityLevel() when the special page doesn't implement
getLoginSecurityLevel(), as was the originally-intended behavior.
Apparently almost nothing actually gets to SpecialPage::execute() or
this would probably have been noticed already.

Change-Id: Ic89dc1b6583aaecd2efe3f5109896148a188c271

build: Introduce eslint-plugin-qunit and enforce

Change-Id: Iefe06cc865c458000ca5350c2d9205f5b768bac8

Localisation updates from https://translatewiki.net.

Change-Id: I6dec7f28bdbcee3b2de48f7ce2c584db45aea730

Merge "Introduce ApiMaxLagInfo hook"

Introduce ApiMaxLagInfo hook

This will be useful when an extension does something expensive and wants to
make sure users don't put too much pressure on the system.

Bug: T194950
Change-Id: I8a396d07c2d5764a680d0e5536cd1f99edbe141e

Merge "Avoid recreating ar_revid index after it's replaced by ar_revid_uniq"

Merge "objectcache: update MemcachedPeclBagOStuff for pecl memcached 3.0.0"

Avoid recreating ar_revid index after it's replaced by ar_revid_uniq

Bug: T193180
Change-Id: I274e33de0a348c0ee42b08b349272db7e2151647

Merge "ExternalUserNames: Update partly incomplete documentation"

maintenance/purgePage.php output page being purged

This is useful when piping lists of pages into the script
to see progress.

Change-Id: I077249ad012d0c463306c0a1a1e38d00624c5170

ExternalUserNames: Update partly incomplete documentation

Bug: T192376
Change-Id: Iae9c537a82046e26cbe5d1703bfe428a38a91f10

Replace call_user_func_array(), part 1

Uses new PHP 5.6 syntax like ...parameter unpacking and
calling anything looking like a callback to make the code more readable.
There are much more occurrences but this commit is intentionally limited
to an easily reviewable size.

Change-Id: Idcec077ef3fdf029b632cceafd0150851ad723e3

Merge "resourceloader: Remove support for Module::getModifiedTime() and getModifiedHash()"

Merge "clientpool: refactor Redis authentication error handling"

Merge "RCFilters: Add an opt-out preference for filters on watchlist"

Sync RELEASE-NOTES-1.31 with REL1_31'

Follows-up 0e7353831e9, which accidentally removed the
pre-upgrade section.

Follows-up Ia4b29611ccee4acac which was landed
in master before the branch cut.

Change-Id: I2a38e79b8dbe173ff99797a87b7aa5193a03212e

resourceloader: Remove support for Module::getModifiedTime() and getModifiedHash()

Deprecated since 1.26. No subclasses in Wikimedia Git define these methods,
no calls to methods by this name in Wikimedia Git.

If a module subclass were to still define such a method, it is simply
not called anymore. The version hash system introduced in 1.26 will
still invalidate modules based on wgCacheEpoch.

Bug: T94074
Change-Id: I65b2a625a30f22c8a9d14a3505605546fa5bab83

Merge "Begin introducing PHP 5.6 variadic parameters where appropriate"

DateTimeInputWidget: Fix disabled `border-color`

Fixing disabled `border-color`, which was set to a too dark tone.

Change-Id: I767b79d51abab05958cf796878c7e10fdaf8ce82

Merge "rdbms: include client ID hash in ChronologyProtector cookies"

Merge "makeCollapsible: Reduce duplicate code for adding eventhandlers"

Deprecate $wgDBmysql5

Bug: T196185
Change-Id: Iea74b5dd1fc49c911dcc0070604f05100e56ce0d

Localisation updates from https://translatewiki.net.

Change-Id: Ie0a45d76ed84797881c62ffcea29ba698df924c9

makeCollapsible: Reduce duplicate code for adding eventhandlers

Change-Id: I3d15ef24f8a3fafb05f2d9ed614e29baa6555bde

Begin introducing PHP 5.6 variadic parameters where appropriate

Change-Id: I5670b8482e8d3bcb0b3a2b4d2ce9834cfc37e171

Merge "Use PHP 7 "\u{NNNN}" Unicode codepoint escapes in string literals (part 2)"

Merge "Use PHP 7 "\u{NNNN}" Unicode codepoint escapes in string literals"

Merge "build: Updating mediawiki/mediawiki-codesniffer to 20.0.0"

Restore section IDs on Special:EditWatchlist

Bug: T194503
Change-Id: Iaa14a882ab1ca0b56ff964d125962ab8a5d91cb5

Use PHP 7 "\u{NNNN}" Unicode codepoint escapes in string literals (part 2)

This is a follow-up to Idc3dee3a7fb5ebfaef395754d8859b18f1f8769a
containing some less trivial changes.

Change-Id: Ia7af2c1d000307d43278cde4a246df413d4ef263

Use PHP 7 "\u{NNNN}" Unicode codepoint escapes in string literals

In cases where we're operating on text data (and not binary data),
use e.g. "\u{00A0}" to refer directly to the Unicode character
'NO-BREAK SPACE' instead of "\xc2\xa0" to specify the bytes C2h A0h
(which correspond to the UTF-8 encoding of that character). This
makes it easier to look up those mysterious sequences, as not all
are as recognizable as the no-break space.

This is not enforced by PHP, but I think we should write those in
uppercase and zero-padded to at least four characters, like the
Unicode standard does.

Note that not all "\xNN" escapes can be automatically replaced:
* We can't use Unicode escapes for binary data that is not UTF-8
  (e.g. in code converting from legacy encodings or testing the
  handling of invalid UTF-8 byte sequences).
* '\xNN' escapes in regular expressions in single-quoted strings
  are actually handled by PCRE and have to be dealt with carefully
  (those regexps should probably be changed to use the /u modifier).
* "\xNN" referring to ASCII characters ("\x7F" and lower) should
  probably be left as-is.

The replacements in this commit were done semi-manually by piping
the existing "\xNN" escapes through the following terrible Ruby
script I devised:

  chars = eval('"' + ARGV[0] + '"').force_encoding('utf-8')
  puts chars.split('').map{|char|
    '\\u{' + char.ord.to_s(16).upcase.rjust(4, '0') + '}'
  }.join('')

Change-Id: Idc3dee3a7fb5ebfaef395754d8859b18f1f8769a

build: Updating mediawiki/mediawiki-codesniffer to 20.0.0

Disable MediaWiki.Commenting.FunctionComment.DefaultNullTypeParam and
MediaWiki.Commenting.PhpunitAnnotations.NotClassTrait for now, and
whitelist all failing files MediaWiki.Usage.ForbiddenFunctions.*.

Change-Id: Ib55ba3a70a1f922662c94895ab59eb02830faf3e

Merge "Make archive.ar_rev_id unique"

Use the old linkshere-related messages without "-2"

As follow-up of I68a8639a61237c67deb61aefd5360dea3cced6cd, we can now go
back to the old messages, which have a clearer name and don't need
furher customization.

Bug: T189860
Change-Id: Ie8cbebe286bc18d016949ff05e7aef18e6aa8952

Merge "Make cleanupSpam.php query for http and https"

Make archive.ar_rev_id unique

To follow up I39b0825c, this change replaces the existing non-unique
index on the column with a unique index, to help avoid some of these
sort of bugs in the future.

Bug: T193180
Change-Id: I932478c9c6a13210bc9dff75286d0f08da56682c

Merge "Remove outdated comment"

Make cleanupSpam.php query for http and https

Bug: T186795
Change-Id: I9e350306f85d3d3ca4daba626e49a0792605acc7

Localisation updates from https://translatewiki.net.

Change-Id: I8b7bbd573c2cc2a9e6499fd372443b2b67f0cb8c

Remove outdated comment

The comment was added in 39273d2c (r38161) 10 years ago and is outdated
since 90f4db81.

Change-Id: I09069679a1aa60d3e54aec35a3961275ea253f34

Merge "Record a log entry on page creation"

objectcache: update MemcachedPeclBagOStuff for pecl memcached 3.0.0

The get() $cas_token parameter was changed into $flags, which can act
as a switch to make the return value an associative array of details.

pecl and PHP-based memcached BagOStuff class both pass all tests now.

Bug: T196125
Change-Id: I4678250331a48db4d50d1fc6c489c991a4dee920

Avoid "unittest_imagelinks" already exists error in tests

This occured in MediaWikiTestCase::setUpSchema if the original wiki prefix was not empty.

Change-Id: I6b0a21473986334bcfbb85ef416759aed44f79a3

Localisation updates from https://translatewiki.net.

Change-Id: Iccda6c6d7b123642f366c37440486b9132357286

Merge "Add language support for Batak Mandailing (btm)"

rdbms: include client ID hash in ChronologyProtector cookies

Previously, if an internal service forwarded the cookies for a
user (e.g. for permissions) but not the User-Agent header or not
the IP address (e.g. XFF), ChronologyProtector could timeout
waiting for a matching writeIndex to appear for the wrong key.

The cookie now tethers the client to the key that holds the
DB positions from their last state-changing request.

Bug: T194403
Bug: T190082
Change-Id: I84f2cbea82532d911cdfed14644008894498813a

Merge "objectcache: make RedisBagOStuff pass all tests"

Merge "installer: Don't allow setting $wgDBmysql5"

Merge "installer: Hide binary/utf-8 charset option from users"

Merge "rcfilters: Stop the "Live updates" button animation from affecting text rendering"

Localisation updates from https://translatewiki.net.

Change-Id: I31d331db53d5f1425541930dbff6f7b28ba470f7

installer: Don't allow setting $wgDBmysql5

It was possible to pass --dbmysql5 to the command line installer to
enable this experimental option. This removes that ability, and removes
setting of $wgDBmysql5 in the generated LocalSettings.php, so new
installs will fallback to the default of false.

Bug: T196185
Change-Id: Id5ff69f493d14cf7b16157f729fbe4f4b4e2e8c8

Merge "objectcache: add setMockTime() method to BagOStuff/WANObjectCache"

Add language support for Batak Mandailing (btm)

Bug: T195940
Change-Id: I488b5aea8d33c17ee801a59963ceced13c944ee4

Merge "Kill non namespaced ScopedCallback"

objectcache: add setMockTime() method to BagOStuff/WANObjectCache

Change-Id: I3e5760814fb7dbe628eb0d979d690c3275fc3c15

objectcache: make RedisBagOStuff pass all tests

* Provide a default lock-based BagOStuff::cas implementation
* Make RedisBagOStuff::merge() use mergeViaCas()
* Use the raw unserialized string as the redis CAS token to
  avoid any bad interaction with __wakeup() methods changing
  field values every time
* Make RedisBagOStuff::incr() return false when there is no
  such key, not null
* Rewrite merge() test to make the order of write/cas phase
  of the parent and child merge() calls well defined instead
  of arbitrary usleep() calls
* Avoid cache key reuse in test runs

Change-Id: I388ec173cf3858bb2fc7a8c8a00cda68703074ce