parser: Validate $length in padleft/padright parser functions

$length is user input, so cast it to an int before passing it to min().
If there is nothing to add at that point, return immediately.

In PHP 7.1+ this raised a warning of "A non-numeric value encountered"
because min() will return the junk value, returning a string. Then we
try and subtract an int from it (return value of mb_strlen()),
triggering the warning.

Added a parser test to verify the behavior, and confirmed that it
triggers warnings without the patch.

Bug: T180403
Change-Id: I614750962104f6251a864519035366ac9798fc0f

Merge "WebInstaller::docLink: Use Html::element to generate the link"

Merge "WebInstaller::getDocUrl: Use getUrl to generate the URL"

Merge "Localize some special pages into Korean (ko)"

Merge "resourceloader: Re-enable testGetBlobCached() test case"

Merge "FormatJson: cleanup after PHP 5.5 support removal"

Localize some special pages into Korean (ko)

* (Manually) synced from MessagesEn.php
* Changed Listfiles so it is less ambigious.

Change-Id: I59da5461efcc34511f99872cfc26d2ae63a52531
Signed-off-by: Yongmin Hong <revi@pobox.com>

Merge "Escape text in Linker::specialLink()"

Merge "Get rid of call_user_func(_array)(), part 3"

Get rid of call_user_func(_array)(), part 3

Also cleaned up nearby code in a couple places.

Change-Id: Ibf44ee7c0ceb739d7e79406e4ff39303c316e285

Clean up CSPRNG support for PHP7

Replace it all with random_bytes(), leave
only MWCryptRand::generateHex() as a convenience helper.

Change-Id: Ic30376a90e66d8f00dab86e7e6466fb3a750b87d

Merge "Remove PhanParamReqAfterOpt from blacklist"

Merge "LoginSignupSpecialPage: Use $this->msg instead of wfMessage"

Merge "Replace call_user_func_array(), part 2"

LoginSignupSpecialPage: Use $this->msg instead of wfMessage

Change-Id: If1313112f39a449857028da6a4a22d6950c500a3

FormatJson: cleanup after PHP 5.5 support removal

* Use PHP 5.6 constant expression support in definition of ALL_OK.
* Remove one level of nesting in encode(). Follows up I801eaffc.
* Update HTML5 section number in doc comment for XMLMETA_OK.
* Made other minor doc comment fixes, such as capitalizing "JSON".
* Not done: changing $badChars and $badCharsEscaped to constants.
  This will have to wait until HHVM 3.18 support is dropped.

Change-Id: I06413dfe0fedddfd20d3e375eadd9daad6d6230e

Merge "Hard deprecate Interwiki static methods"

Remove PhanParamReqAfterOpt from blacklist

Change-Id: I569656068c9034f7325312afcce837e2712b4f29

Merge "Resolve required parameter after optional in language"

Merge "Resolve required parameter after optional in specials"

Merge "Resolve required parameter after optional in Xml"

Merge "Resolve required parameter after optional in Message"

Merge "Resolve required parameter after optional parameter in Linker"

Escape text in Linker::specialLink()

Linker::linkKnown -> Linker::link -> $text is wrapped by HtmlArmor

Bug: T85864
Change-Id: I8755948131ffca4a0b5949b2f8cabf19f59c1c9e

Merge "Rewrite IEUrlExtensionTest using a data provider"

Localisation updates from https://translatewiki.net.

Change-Id: I5bd33ca666aaa4ad76bbae4e0cc9d3e7cb44c345

Rewrite IEUrlExtensionTest using a data provider

Change-Id: I67568f997d682b88ad99676a57b85395b1b20f85

Resolve required parameter after optional in specials

Change-Id: If2c85d97d68b24fc5cb2bec19804034b994b348d

Add more logging to ChronologyProtector::initPositions()

Bug: T194403
Change-Id: I8f1ccb3c6e257ae48ae6bbecd8f2a8f51cd2ed41

Resolve required parameter after optional parameter in Linker

Change-Id: I2569e30922e93404a59fa282df8ae8d3d19912fd

Resolve required parameter after optional in Message

It looks like this parameter has had a default since it was first
created, and somehow kept it even after a new parameter was added
to the end. codesearch.wmflabs.org suggests, and I could not find,
and calls to this function from any other code that would need
the defaulted param (probably because they fail due to the later
required parameter).

Change-Id: Iae6e736c5c957d9fccb6ab5813a3b5b02eeb3f6f

Resolve required parameter after optional in Xml

Change-Id: If04d3b347ec2839b986d8fe85d3f2ed392fc83ed

Resolve required parameter after optional in language

Change-Id: I6e0c3b14f1bcea112be7ab694efff5a67622d637

Merge "build: Allow HHVM 3.21+ failures in Travis"

Merge "(y)etsin fixes, test refactoring, and misc fixes"

Follow-up Ieaeb3113a: Add RELEASE-NOTES

Change-Id: I79f1d28b54532a7495fb8e205c9b6636016587d7

Merge "CurlHttpRequest: remove open_basedir check"

Fix phpdoc for wfMessage $params for phan

phan and phpDoc expect that the type of variadic params is the type of
each individual item.

Change-Id: I8229d52b3e4b70a9de6c5bb813bab776c5547e5d

CurlHttpRequest: remove open_basedir check

This only applied to PHP 5.5 and older, which we no longer support,
and was apparently overlooked in I740f32ac.

Change-Id: I1d56ac0ef7ac2cc14da16e1397e9614564c7ee23

Release notes for  I57e05c5ae

Cherry picked from commit d5e4bf920c7.

Change-Id: Ie7c9393c5a77279b9342152b9fdc187f22acf5c0

Merge "Ensure $user is passed by reference in TitleMoveComplete hook on HHVM"

build: Allow HHVM 3.21+ failures in Travis

Keep them in the post-merge matrix, but don't make its failure
affect the commit status indicator (e.g. "non-voting").

Wikimedia only runs HHVM 3.18 in production at the moment, and
it seems these newer versions don't currently pass the test and
fixing them is probably not a priority at the moment.

Follows-up 8fde6115a8368e.

Change-Id: I8f6e637882e89663257cafd8c832f71131a9c488

Merge "HashRing: use bisection instead of guess and refine"

Merge "Enable InterwikiTest::testDatabaseStorage()"

HashRing: use bisection instead of guess and refine

Use bisection to find the position within the ring, as in ketama.
According to my benchmarking this is faster than the guess/refine
algorithm for rings with 100 locations, i.e. 16000 ring nodes requiring
at most 14 bisection iterations.

Change-Id: I6042f382de093081971b5ec210eda8dfa74988f2

Fix wfMessage() annotation

$params is a variadic parameter now, so it must be treated as an
array. Same thing for wfMessageFallback().

Change-Id: I606deb7ea0fb9be25ac79aadfbab0cc44bdc36a0

Merge "Rewrite HashRing to use consistent hashing"

Replace call_user_func_array(), part 2

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.

In one occurrence, a simple conditional instead of trickery was much more readable.

This patch finishes all the easy stuf in the core, the remainder is either unobvious
or would result in smaller readability gains. It will be carefully dealt with in
further commits.

Change-Id: I79a16c48bfb98b75e5b99f2f6f4fa07b3ae02c5b

mediawiki.skinning: Remove styles for `#jump-to-nav` from core

There are no longer any skins bundled with MediaWiki releases that
use it. Styling for this should be the skins' responsibility.

Vector and MonoBook no longer use this element, and core's
providing of these styles actually creates conflicts.

For skins using 'mediawiki.legacy.oldshared' (predating MonoBook),
no changes are required! Their jump-to-nav styles remain preserved.

For skins that use 'mediawiki.skinning.interface' and have
MonoBook-inspired accessibility links within a `<div id="#jump-to-nav">`,
need to make one of two changes:

1. Adopt the new CSS-only approach documented in T195256,
   as used by current MonoBook and Vector.

2. Or; Add a copy of the old CSS (included below) to your own skins'
   stylesheet and ensure the `jquery.mw-jump` module is loaded (previously
   by default), either by adding `$out->addModules( 'jquery.mw-jump' )`, or
   by adding it from the skin's Skin::getDefaultModules() override.

    @media screen {
        #jump-to-nav {
            /* Negate #contentSub margin */
            margin-top: -1.4em;
            margin-bottom: 1.4em;
            -moz-user-select: none;
            -webkit-user-select: none;
            -ms-user-select: none;
            user-select: none;
        }
        .mw-jump, #jump-to-nav {
            overflow: hidden;
            height: 0;
            zoom: 1;
        }
    }

    @media print {
        .mw-jump, #jump-to-nav {
            display: none;
        }
    }

This migration guide will be added to the T195256 task description,
to which the release notes refer.

Bug: T195256
Change-Id: I84bcd23180b3d1fa541728989f44a376189df95d

Update OOUI to v0.27.3

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

Depends-On: Ifc0bfbbffb66dd45ebf442a227ae734de9b41964
Change-Id: Ifc0bfbbffb66dd45ebf442a227ae734de9b41961

Localisation updates from https://translatewiki.net.

Change-Id: I76a6f7a48e28f135120f73121eac582dd96b3411

Ensure $user is passed by reference in TitleMoveComplete hook on HHVM

Suggested by Anomie in T118683#3688320.

Bug: T118683
Change-Id: I57e05c5ae42c8b85d0a8f9631a459c5686b27893

Enable InterwikiTest::testDatabaseStorage()

The patch mentioned in the skip message was never merged, but the test
passes now.

Change-Id: Ifa28fbf1d8967fc3dac0f81032ef3f2439efb86c

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"

Rewrite HashRing to use consistent hashing

This scheme avoids disruptive re-hashing when a node is
ejected; only the adjectent clockwise sector is affected,
the later taking responsibility for the former. Addition
of new nodes is likewise less disruptive.

When used with MD5, this maps keys the same as libketama.

Also:
* Make HashRing::getLocations() faster by guessing the
  sector list index and moving clockwise/counter-clockwise.
* Made HashRing Serializable for more robust storage.
* Allow for different hash functions to be specified.

Note:
* This makes PoolCounterRedis use the new hash scheme.
* Likewise for JobQueueFederated (for job de-duplication).
  Switching from one mode to the other will cause some of
  duplicated effort from old jobs only temporarily.

Change-Id: Iff432a4e5dc3fb40b2dda70fb3dbe9dda0b74933

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

Also set "use strict" in the base module.

Bug: T192623
Change-Id: I18a710e167df8be47a70da93a7aba0784fc62aef

WebInstaller::docLink: Use Html::element to generate the link

Change-Id: I1e3163d9814345e6b108072886b9756cc84d168a

WebInstaller::getDocUrl: Use getUrl to generate the URL

This changes ensures to generate the same URLs on doclinks like in the
navigation bar.

mw-config/?page=Welcome has as "Restart installation" the URL
  /.../mw-config/?page=Restart&lastPage=Language
and as "a copy of the GNU General Public License" the URL
before this change
  /.../mw-config/index.php?page=Copying&lastPage=Welcome
after this change
  /.../mw-config/?page=Copying&lastPage=Welcome

Change-Id: Ide5cb52d71363c3a54e1b6ac48c272e1de823322

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