Merge "resourceloader: Mark clearCache() as private (only for tests)"

Merge "Update wikimedia/timestamp to v2.2.0"

Merge "Provide a way to restore an old revision with multiple slots."

Provide a way to restore an old revision with multiple slots.

Bug: T204732
Change-Id: I0ea2711e68c78465a5e5cfaa0181ad5ce983d35a

resources: Register 'sinonjs' as foreign resource

Change-Id: I9b1740f34c2415c27936a970979a49bf143771c7

Merge "resources: Make jquery.cookie a multi-file resource with changelog, licence"

Merge "resourceloader: Add filter cache version to module version hash"

Merge "resourceloader: Remove MW cacheEpoch from module version hash"

Merge "McrUndoAction: Fix preview"

Merge "Code to read from change_tag_def table instead of valid_tag"

resources: Make jquery.cookie a multi-file resource with changelog, licence

Change-Id: I47e25b1a7ac7d53932dfdfd37a47a38141f91be4

resourceloader: Add filter cache version to module version hash

We already had a $filterCacheVersion variable, but it was
only used for the internal cache for JS and CSS minification,
which is not enough. If there is a breaking change in either
of these processes, we also need to invalidate version hashes.

This commit renames ResourceLoader::$filterCacheVersion to
ResourceLoader::CACHE_VERSION and takes it into account in
getVersionHash(). Adding it to getDefinitionSummary() is not
sufficient, because content-hashed modules also need to be
invalidated when there's a breaking change in the minifiers.

This cache version can also be incremented when there's a
breaking change in image embedding or LESS compilation,
although content hashing deals with that already, so we
could also add a separate cache version for those that's
only added to getDefinitionSummary().

Bug: T176884
Change-Id: Ife6efa71f310c90b9951afa02212b2cb6766e76d

resourceloader: Remove MW cacheEpoch from module version hash

This seems quite nuclear. I'd actually like to deprecate the
wgCacheEpoch variable more generally in favour of a handful
of more specific version constants, but as starting point,
remove it from the hash used for load.php urls and localStorage
keys.

The latter is also controlled by wgResourceLoaderStorageVersion
already.

Also ref T32956 about a more standalone ResourceLoader.

Change-Id: I913f846090e82d3d822653b9b7ce22233cdb5e90

Merge "Remove unnecesary else blocks"

Merge "SpecialLog: Don't throw exceptions on invalid date from user input"

SpecialLog: Don't throw exceptions on invalid date from user input

If users provide invalid input to the date option on Special:Log (most likely
an intentional thing given the calendar input widget), don't let the
TimestampException bubble up - just discard the invalid date.

Integration test included, which fails without this patch.

Bug: T201411
Change-Id: Ie1a9a84343ae4e78e076586f759917e5fd5af33c

Merge "resources: Register jquery.cookie as foreign resource"

resources: Register jquery.cookie as foreign resource

Change-Id: Ia6b1aed25ad6d70aa2035340205406f68297e8bb

Merge "RevisionStoreDbTestBase, remove redundant needsDB override"

Remove unnecesary else blocks

Change-Id: Ic11e0e6b86557cbffcc5faa54de218b4676b4712

Localisation updates from https://translatewiki.net.

Change-Id: I679340214f984a89d207db7b865519010d4737e0

Merge "registration: Let extensions add PHP version requirements"

Update wikimedia/timestamp to v2.2.0

Depends-On: Ibe47c7e363d263be37c7ebe3891e46f04a3481bd
Change-Id: Id8901d0a0d30bf402220aca73f9c5f45043908ce

Merge "Pass title to Revision constructor in Title::getFirstRevision "

Merge "Fix caller name in Title::loadRestrictions"

Merge "resources: Update mustache.js from 0.8.2-d9aa703 to v1.0.0"

Merge "resources: Register mustache.js as foreign resource"

Merge "Remove MimeMagic"

Remove MimeMagic

Deprecated since 1.28

Change-Id: I793deb8669e353525464ccd840801e5015c545c8

resources: Update mustache.js from 0.8.2-d9aa703 to v1.0.0

Upstream commit 'd9aa703' is the commit right before the tagging
of v1.0.0. The only difference is the version string.

Change-Id: Icf7b7abacac583984977183da6f93f5fd02404fa

resources: Register mustache.js as foreign resource

The original commit that added it (7c52c9265b) did not specify
where it came from or which version. The source refers to
<https://github.com/janl/mustache.js> and contains a version string
stating "0.8.2". However, this file does not match the source
code of released version "0.8.2".

Instead, our copy seems to match the source ode of release
v1.0.0 (except for the version string).

The only commit in the uptream repository that matches our
copy is the commit right before the "Tag v1.0.0" commit,
at which point it had all changes of v1.0.0, but still had
the version string of the last published version. I assume then
at this file was copied from the GitHub master branch at the
time.

I'll update it to v1.0.0 proper in a separate commit.

Change-Id: I37c69e71fd4b005532cb0ac80964c72d3921eae6

Localisation updates from https://translatewiki.net.

Change-Id: I8256de37811bb514aeca3153f98f8aefdf8cc489

resourceloader: Mark clearCache() as private (only for tests)

Exclude from coverage report as well.

Change-Id: I5281b42dab802d5a5a58ee33a3b12a77229503c8

Pass title to Revision constructor in Title::getFirstRevision

The information for a title already in the select result.
Give it to Revision class to avoid extra Title::newFromId in
Revision::ensureTitle for each row

Change-Id: I6f1a936ce2ed1834feb29850e65a497dafe35238

Fix caller name in Title::loadRestrictions

Seeing {closure} in the logs as caller is not helpful

Change-Id: I34aead30c55aefb7591cbc6031522d0f0e5bd17a

Localisation updates from https://translatewiki.net.

Change-Id: I693c5ed47ebe4326ec6c7e61f692cae48b71d8b6

registration: Let extensions add PHP version requirements

While MediaWiki Core already sets requirements for PHP versions, it should be
possible for extensions to tighten these requirements. This mirrors the PHP
parameter of extension infoboxes as well.

This change introduces a new 'platform' key (in addition to 'MediaWiki', 'skins'
and 'extensions', where non-MediaWiki software requirements will be listed
in the future, starting with a PHP version constraint. Further keys are
supposed to be added to allow setting constraints against php extensions
and other abilities of the platform.

Bug: T197535
Change-Id: I6744cc0be2363b603331af9dc860eb8603a1a89a

Fix some double escaping, and some incorrect escaping for convert()

convert() is weird and requires things to be escaped before it
is called.

This fixes a bunch of phan-taint-check warnings.

Change-Id: I422b313ca05ff61ae05e06856347cc0de832cc49

Fix some phan-taint-check false positives

Change-Id: Ic5ccbb3f97722476bee7188b83b80cdc652d2a64

Deprecate QuickTemplate::msgHtml & fix phan-taint-warning in includes/skins

QuickTemplate::msgHtml() (And the weird override that does the
same thing a different way - BaseTemplate::msgHtml()) are inherently
unsafe as they echo out a raw html message. This is strongly
discouraged in modern code. According to codeSearch tool, nothing
uses these methods, and there is a "@private" annotation on the
QuickTemplate::msgHtml() docblock. Thus hard deprecating it.

Change-Id: I4e9e157e922a36787adef4d0bf7608605c27f0c4

Merge "Remove global namespaced LBFactory"

Merge "Set consistent min value options for RC and Watchlist filters"

Set consistent min value options for RC and Watchlist filters

Bug: T204623
Bug: T199049
Change-Id: If73b7cf6734f4a6a8d8e47ce8051a827ae5452e1

Merge "CommentStore: Hard-deprecate newKey()"

Remove global namespaced LBFactory

Change-Id: Ia87aa5c9426af83627a1bbc9bd914546c89c1e47

Merge "Hard deprecate non-Remex tidy modes"

McrUndoAction: Fix preview

Apparently DifferenceEngine::renderNewRevision() never really worked
right, it only functions when diffing *saved* revisions. Replace the
preview code with code adapted from EditPage::getPreviewText(), using
RevisionRenderer for the actual rendering.

Bug: T205101
Change-Id: I416ede84c8d10fc28866e608c9cfc3a2a7410e48

Hard deprecate non-Remex tidy modes

Let's rip the band-aid off.  Remex is pure PHP so there's no reason to
be running any of the other tidy implementations any more, and we won't
be able to support them in the future.

Follow-up to 7b23382823f2ea7bab8d20ad68950ff34f50a005.

Bug: T198214
Change-Id: Id3d07d44f8434231826e86e623554cac3decfa96

Don't show unsaved revision in DifferenceEngine::renderNewRevision.

There was a conditional that was documented to make the method bail out
if the current revision was unsaved, but it was missing the actual return
statement.

Note that preview code in EditPage does not use
DifferenceEngine::renderNewRevision.

Change-Id: I86a25e9464435ac17c72f78f7ce1f354adf3d055

Merge "selenium: invoke jobs to enforce eventual consistency"

Localisation updates from https://translatewiki.net.

Change-Id: I3c35fe761a9e90b551fb1435a354e337808f9f00

Merge "tests: Stop using deprecated LBFactory, use namespaced version"

Merge "Stop MailAddress accepting a User object"

tests: Stop using deprecated LBFactory, use namespaced version

Change-Id: Ifbbbfe1bc58e57739b88588e3b6feb703f20534d

Merge "Collapse some unnecessary else conditions"

Stop MailAddress accepting a User object

Bug: T204863
Change-Id: I98f110e47ec7ba3b1cd22e8d76eea4bdf7d3d155

Merge "Drop support for array syntax for $wgSpecialPages"

Drop support for array syntax for $wgSpecialPages

Bug: T204863
Change-Id: I03fc19cf70a7feeb7d0003de7fbc7b254a3e9560

Collapse some unnecessary else conditions

Change-Id: I9fe505f2f4ed4d10e13813d9f05127fd6571c874

Merge "Ensure parserTests pass even if wiki has wgUsePigLatin enabled"

Merge "user: Allow "CAS update failed" exceptions to be normalised"

Update HISTORY for 1.27.5/1.29.3/1.30.1/1.31.1

Bug: T199025
Change-Id: Iaf6b8f32d1c4c21b20483817ad0e9d3dbf3e7e22

SECURITY: Don't exclude .htaccess files from `git archive`

Bug: T199029
Change-Id: I7ada3dddc4f5f7f2800882ccc9c73c8494f7fac9

SECURITY: Do not allow botpassword login if account locked.

Reported by Rxy

Bug: T194605
Change-Id: Ib41005e69ab4db6f849837de12f0d41398b58f9a

SECURITY: Make 'newbie' limit in $wgRateLimits really override 'user' limit

The order of operations was incorrect.

Bug: T169545
Change-Id: Ia910aa2a494914d3b0017daac9ab294ea9fa8705

Merge "Soft deprecate non-Remex tidy configurations"

Merge "Hard-deprecate the $wgUseTidy option"

Merge "Use wfIsHHVM() instead of a HipHop-specific environment variable"

CommentStore: Hard-deprecate newKey()

Not used by any extensions known to Wikimedia git.

Change-Id: I3c07ad6db9d88feddad1cf29d4794fef622298ea

Soft deprecate non-Remex tidy configurations

Future parsers will not be able to emit output compatible with these
configurations.

Bug: T198214
Change-Id: Id7921a166a62457f289e6c0c4bba6c8563be4760

Hard-deprecate the $wgUseTidy option

This has been soft-deprecated since MW 1.26; this hard-deprecation
sets the stage for future removal of this old cruft.

Bug: T198214
Depends-On: Idf246d05d116f63a73105b50a1929a7721fbe7b9
Change-Id: I2e7d990da1da378eb6e828d4b3c0f5a41791dd92

Use wfIsHHVM() instead of a HipHop-specific environment variable

Change-Id: I5bbf3e4f65d9b6a0d7419f67e3931e77e92b7e6c

Merge "Set break-word on figcaption"

selenium: invoke jobs to enforce eventual consistency

Many tests use the API during their setup phase, which does not
participate in clearing the job queue, but constantly adds new items.
Consequently the job count keeps mounting. In order to be able to assert
the state of the application once all jobs are processed we need a way
to invoke them.
This is a minimalistic alternative to Idd4924e16925cbf0cb9e42645670c3fd76b72353

Bug: T199446
Change-Id: Ifc49bc5c06eef71da8aff1b1837c32debc2c45dd

Merge "resourceloader: Simplify StringSet fallback"

Merge "resourceloader: Use null-objects for 'sources' and 'registry'"

Merge "resourceloader: Minor clean up in mw.loader and mw.html"

Localisation updates from https://translatewiki.net.

Change-Id: Ifab02635d866e244cff2ab4eecccdc70119e413a

Define which languages explicitly fallback to 'en'

In the message store, all messages fall through to English,
but only a few languages should actually explicitly fallback
to English (English variants and dialects).

These new explicit fallbacks are used by ResourceLoaderImageModule,
and this change doesn't affect the message fall through system.

Bug: T203350
Change-Id: I6b68a17f4d69341bccdae748727b5133a600d8bc

Merge "Add MessagesBi.php"

Merge "Add MessagesSco.php"

Merge "mw.jqueryMsg: Handle non-string parameters to functions that expect strings better"

Merge "mw.jqueryMsg: Add tests for non-string parameters to functions that expect strings"

Merge "Special:Preferences: Remove unnecessary OOUI styles override"

Merge "resourceloader: Add internal RLPAGEMODULES constant"

Merge "resourceloader: Make ResourceLoader::makeLoaderRegisterScript() internal"

Merge "resourceloader: Factor out encodeJsonForScript"

Special:Preferences: Remove unnecessary OOUI styles override

Since OOUI v0.24.4, menus/popups of infused PHP widgets are placed
into the default overlay, so this workaround is not needed.

Change-Id: I4f8a76cb258e44ed4f2fbba41425100d2e0e9b45

Merge "StructureTest::testUnitTestFileNamesEndWithTest() should not shell out"

Merge "Use new isTalkPage/isTalkNamespace methods"

Merge "DateInputWidget: Update OOUI variables to latest"

Merge "DateTimeInputWidget: Rename vars to reflect latest WikimediaUI theme vars"

Merge "Special:Preferences: Reduce PanelLayout border contrast slightly"

Use new isTalkPage/isTalkNamespace methods

Change-Id: I291738ff6ebb63df2e405e98131dd80f3ddf2db6

Special:Preferences: Reduce PanelLayout border contrast slightly

The border choice for `#a2a9b1` was intended to provide AA conforming
contrast for widgets. In case of framed PanelLayout it is not necessary
due to it's different use case and the additional high padding.

Change-Id: Ia425594c1795d1d8668cd62daff32ff9b97d67c2

Merge "resourceloader: Remove check for Function#bind and Object.defineProperty"

Merge "Avoid ResourceLoader LESS parsing bug"

resourceloader: Remove check for Function#bind and Object.defineProperty

* Function#bind is a given per the explicit feature test for Grade A.
* Object.defineProperty is part of ES5 which is also tested for.

This code is from before we required ES5 and is now redundant.

Also remove the unused eslint-disable rule for global 'console' use,
the code in question does not use console as global unconditionally.

Change-Id: I6c77b25856da8b7717aeba8298b17a9231540d58

Avoid ResourceLoader LESS parsing bug

Bug: T204816
Change-Id: I4558e2c05dd3377090de11aa79f20911fa3365eb

Localisation updates from https://translatewiki.net.

Change-Id: I383c3794e40e4269c1f8ddcc60ddf1bc202b5eff