Merge "SpecialVersion: Avoid deprecated wfMemcKey()"

Merge "page: Avoid deprecated wfMemcKey()"

Merge "SiteStatsUpdate: Avoid deprecated wfMemcKey()"

Merge "SiteStats: Avoid deprecated wfMemcKey()"

Merge "Block: Avoid deprecated wfMemcKey()"

Merge "objectcache: fix cache warmup bug in getMultiWithSetCallback()"

Merge "Fix TextPassDumperDatabaseTest::testPrefetchPlain postgres failures"

Fix TextPassDumperDatabaseTest::testPrefetchPlain postgres failures

The insertId() method was returning a string, which caused the
returnValueMap not to trigger due to int/string mismatches.

Also add sanity integer cast to WikiPage::insertOn().

Added a few more type docs.

Bug: T75174
Change-Id: Id1090f3e3d0481272a3d13c3af8f2588f06dc912

Merge "Revert "Localisation updates from https://translatewiki.net.""

Revert "Localisation updates from https://translatewiki.net."

Overwrites https://gerrit.wikimedia.org/r/#/c/353696/

This reverts commit d47331a707a00ec0b7ffab1f9fdcb02d70975898.

Change-Id: I2db66a32c3eb1baef84c0abe797ebb9bc052670d

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: Iabf3bace8c57875133115d1faab44be656807657

Merge "RCFilters: Style saved links option menu"

Merge "RCFilters: Style the Saved Links placeholder and add a title"

Merge "resourceloader: Ensure user.styles and site.styles having their own request"

WebStart.php: Update the file's doc comment

The doc comment did not reflect the removal of MW_NO_SETUP in
edc9edbc769b570c, or the fact that Setup.php no longer loads
AutoLoader.php or GlobalFunctions.php.

Change-Id: I19a12fd83e369828b9e08176ddf3bfca16aae465

resourceloader: Ensure user.styles and site.styles having their own request

Regardless of whether other modules exist with group=user or group=site,
these two modules in particular must always be in their own request for
legacy reasons.

This has already always been the case because even in the few cases where
an extension uses this group (eg. MobileFrontend's custom site module) it
would load it instead of another module in that group, never at the same
time. There is one notable exception, which is GlobalCssJs. However the
ext.globalCssJs.user.styles module is usually served from another wiki
which is why that went unnoticed as well. This commit fixes that so that
even if you're viewing a page on the central wiki, the modules are still
in separate requests.

Aside from this one existing edge case, there is also need to add
group=site to gadgets by default so that they load after the DynamicStyles
marker instead of before, which is currently causing problems with the
cascading order (gadget apply before core and skin styles due to being
in the same request group and alphabetically sorting before them).

Semantically, the appropiate solution is group=site, but this wasn't
possible due to core putting "all" group=site modules in the same request
(under the assumption there is only one such module). This commit removes
that fragile assumption.

Bug: T147667
Change-Id: I9eb725c083124d22a9af3bf3d075ade6f3b970a3

objectcache: fix cache warmup bug in getMultiWithSetCallback()

The warmup cache was not properly prefixed and was also using the entity
IDs instead of the cache keys. Thus, it effectively just wasted a
getMulti() query and resulted in the usual separate GETs anyway.

Added some unit tests for this.

Change-Id: I75b7a31214b515511856f9d95db32e8881d80ccc

Merge "jquery.makeCollapsible: Restore considering empty <a> as part of toggle"

Merge "Add missing ampersand to getMultiWithSetCallback() callback"

jquery.makeCollapsible: Restore considering empty <a> as part of toggle

Before 2d95d36a8e, clicks on links inside toggles with non-empty
targets that are not '#', were ignored ("pass through") since they
are not intended for the toggle.

In 2d95d36a8e, this was simplified to ignoring clicks from all <a>
elements inside toggles. However this ignored too much as links
without 'href' attribute are also sometimes used inside toggles
to look like links and have no href-target, which means clicking
them does nothing and is in fact meant to toggle the element.

Restore previous behaviour and restore previous test + add a new
test for this specific case.

Bug: T166298
Change-Id: Ia3a0648f809f94be0977a83b469fbd184aa72aff

Merge "Revert "Creating new WMF 1.30.0-wmf.2 branch""

Merge "Add kbp language code"

Revert "Creating new WMF 1.30.0-wmf.2 branch"

Also revert the automatic "Update git submodules" commits since then.

This reverts commit 264fc98588a4851324d3e5f2e39ece01e993127e.

Change-Id: Ica6ad77c46cc8a7f6b5576da2580ea9e07ac8637

Merge "Fix version of DonationInterface deployed to donatewiki"

Fix version of DonationInterface deployed to donatewiki

We had deleted some localization strings that donatewiki needs.

Bug: T166302
Change-Id: I27a03a64de80bdf58f7d7f2f69173933aa4b23ea

Localisation updates from https://translatewiki.net.

Change-Id: I0772f5f25bb805291ffee1b8170d22a49c3aa317

Merge "MessageCache: Avoid deprecated wfMemcKey()"

Merge "rebuildrecentchanges.php: Make purgeFeeds() use the correct cache"

rebuildrecentchanges.php: Make purgeFeeds() use the correct cache

ChangesFeed was updated in 3517be1cf0b7, to use the WAN cache,
so this change updates the purgeFeeds() function to use that cache
too.

Avoid the deprected wfMemcKey() too.

Bug: T166195
Change-Id: I28a8d729737f03ccaf069495a59e80401c59e778

SpecialVersion: Avoid deprecated wfMemcKey()

Change-Id: I7cfaf049a4e193bdb12b5233f0460e71cbe876af

page: Avoid deprecated wfMemcKey()

And the deprecated ObjectCache::getMainWANInstance(),
getMainStashInstance() while we're at it.

Change-Id: I951c356f1c394a9658216dea54c92824dca83cbc

SiteStatsUpdate: Avoid deprecated wfMemcKey()

And the deprecated ObjectCache::getMainStashInstance() while we're at
it.

Change-Id: Iebd4e029c3ab6443403087d991bb263ef83b1ed8

SiteStats: Avoid deprecated wfMemcKey()

And the deprecated ObjectCache::getMainWANInstance() while we're at it.

Change-Id: If9d9ab9c661c003a3aa6c2d956700b1870a52f41

Block: Avoid deprecated wfMemcKey()

Change-Id: Ie23261b7b6dd9d03d314cf77d290bffb8df593cd

Merge "ApiParse: Use existing OutputPage for prop=categorieshtml"

Merge "Move some more classes to comply with class per file"

ApiParse: Use existing OutputPage for prop=categorieshtml

Follows up Iaa85ac49.

This also ensures that the Skin correctly refers back to the constructed
OutputPage even when 'useskin' isn't specified.

Change-Id: I983e907c05acd753733e88904953f355e26f393d

Add kbp language code

Bug: T160915
Change-Id: I1de0c2eb0228697544f2e3bfedf76bc37ac3f679

Add missing ampersand to getMultiWithSetCallback() callback

Also fixed a php example block comment

Change-Id: Ib6967180ad0d45aec7b7d4b6d383723e2c95fefa

Merge "mediawiki.page.watch: Deprecate mw.page object"

Merge "Avoid treating mcrouter set()s as failing due to AllAsyncRoute"

Merge "Revert "Revert "Fix the web updater"""

Update git submodules

* Update extensions/wikihiero from branch 'wmf/1.30.0-wmf.2'
  - Fix styles queue violation for "ext.wikihiero.Special"

    The same module is explicitly being loaded twice, once just the
    styles, and again with the JS/messages/dependencies as well.

    This is wasteful and also unsupported.

    Move the two style rules to the main stylesheet and load the
    proper styles module it belongs to instead.

    Bug: T92459
    Change-Id: I012b4d9bff5bd0379f8b9eb663d5b5dbf71f1885

Revert "Revert "Fix the web updater""

This reverts commit a7b9cc4946a2f7fb081f64d9230bf944736a185b.

Change-Id: I91d4b821df126703f83377dd8aff3a79b629f720

Merge "Revert "Fix the web updater""

Revert "Fix the web updater"

May be causing beta to break, if that's true then we are going to revert 6519c42d248a78d2d42edee1beb21f926d227044

This reverts commit 5d56bcb925af653fed427e90a1cfb982de386720.

Change-Id: I5e9df08dc1583266bb11b52d5d68bf393e4c0fff

Update git submodules

* Update extensions/ProofreadPage from branch 'wmf/1.30.0-wmf.2'
  - Properly handle batches of pages to update

    Using LIMIT on UPDATE/DELETE queries is unsafe when using statement-based
    replication as you can't ensure the same sets of rows are updated.

    Instead, select a batch of problematic page_ids, update by them, then
    continue until we don't find any more to fix

    Bug: T166261
    Change-Id: I8a5d2989285c72450d6a48ccc12d88fdeeb3fb6f

Localisation updates from https://translatewiki.net.

Change-Id: I2e981505c8fd591998178998508e923f9e9fbb41

MessageCache: Avoid deprecated wfMemcKey()

Change-Id: I31e3cde0a94e355de9f32a17636aaf54aa2d40c4

Merge "Batch/pipeline backend operations in refreshFileHeaders" into wmf/1.30.0-wmf.2

Update git submodules

* Update extensions/TimedMediaHandler from branch 'wmf/1.30.0-wmf.2'
  - Make getContentHeaders rely on fallback width/height

    Bug: T150741
    Change-Id: I89cc083aac39f71460b10bfc0ce341c115bd6b64
    Depends-On: I2c0f39b2b01f364c3fab997ccc2f874b7f101d8a

Update git submodules

* Update extensions/PagedTiffHandler from branch 'wmf/1.30.0-wmf.2'
  - Update getContentHeaders signature

    Bug: T150741
    Change-Id: I714bd9a340c64e5b8536c031923ad0b486434632
    Depends-On: I2c0f39b2b01f364c3fab997ccc2f874b7f101d8a

Merge "Use file width/height instead of metadata for getContentHeaders" into wmf/1.30.0-wmf.2

Update git submodules

* Update extensions/PdfHandler from branch 'wmf/1.30.0-wmf.2'
  - Update getContentHeaders signature

    Bug: T150741
    Change-Id: I3a62185243d5a0d561da63a5e5ccafed093d41bb

Avoid treating mcrouter set()s as failing due to AllAsyncRoute

Per https://github.com/facebook/mcrouter/wiki/List-of-Route-Handles
a NullRoute response is always given for DELETE and SET. The former
already is already handled by MediaWiki treating NOT_FOUND as success.

Change-Id: I79c26bcd6b8ffe7eea73e0d45badcc4ed63f05e6

Merge "Fix up deprecation of wfMemcKey()/wfGlobalCacheKey()"

Merge "rdbms: Add compat alias for renamed LBFactory class"

Restore dependency on module 'mediawiki.page.startup'

Module 'mediawiki.page.watch.ajax' requires a dependency to module
'mediawiki.page.startup' because 'mediawiki.page.startup' creates the object
mw.page and 'mediawiki.page.watch.ajax' requires this object.

This dependency got lost in 06a0dec05723eb6a6604421e134345a5abe115a5.

Bug: T114288
Change-Id: Iebfda85c77eaf6bb19d2ae755813b1c36a95dac8
(cherry picked from commit ef456a78051bd01fa29ebeb1b89bdbfb6703b6c4)

Merge "installer: Replace bad calls to getText() on DBQueryError exceptions"

rdbms: Add compat alias for renamed LBFactory class

This was renamed to be namespaced, but there are still some uses
that justify having an alias, similar to how we did with most
other classes in rdbms that were renamed in 1.29.

Bug: T164896
Change-Id: I36426144491b976322c3c96c08a821300d3b4a46

installer: Replace bad calls to getText() on DBQueryError exceptions

The called method does not exist. Replaced with getMessage(),
similar to PostgresInstaller.php.

Bug: T165319
Change-Id: I900ec8c18307ce6f9e8c9f4cda79b1e33ed66dff

Merge "Update psysh to 0.8.5"

Batch/pipeline backend operations in refreshFileHeaders

Bug: T150741
Change-Id: I44efd47e7eb3eba71ae39808ee8e6969e141c5cc

Use file width/height instead of metadata for getContentHeaders

This allows us to populate X-Content-Dimensions without touching the
existing metadata format. Which makes the migration of existing content a lot faster by
only having to run refreshFileHeaders.

Bug: T150741
Change-Id: I2c0f39b2b01f364c3fab997ccc2f874b7f101d8a

Merge "Use file width/height instead of metadata for getContentHeaders"

session: Avoid deprecated wfMemcKey()

Change-Id: I4d77c2c52ef43cbc54878ce920595befd270a28e

ChangeTags: Avoid deprecated wfMemcKey()

And ObjectCache::getMainWANInstance() while we're at it too.

Change-Id: I2678f5f940ed29654600fec39ff5daee7f59a223

skins: Avoid deprecated wfMemcKey()

Change-Id: Id31998b365f486a651341563c0b16a358ce2b31d

media: Avoid deprecated wfMemcKey()

And ObjectCache::getMainWANInstance() while we're at it.

Change-Id: Ib22bd134c3faa56f8d8f111bb9ed99d826cbed40

registration: Avoid deprecated wfMemcKey()

Change-Id: I8f4382d55ba2d006f1573e945c83c3fd541a2cb1

InfoAction: Avoid deprecated wfMemcKey()

And the deprecated ObjectCache::getMainWANInstance() while we're at it.

Change-Id: Id7bb6a8c3e8269345e054d2c2bf202304b74c3cf

auth: Avoid deprecated wfGlobalCacheKey()

Change-Id: I2d6fa73fe857a0495df13aa25071e69b1ec2ba70

FileCacheBase: Avoid deprecated wfMemcKey()

Change-Id: I5476bf45deb55ef4ff158970102b1ce17d3fee29

AuthManager: Avoid deprecated wfMemcKey()

Change-Id: I254f9c057b8a151c593054dd4f2137586abf262a

Fix up deprecation of wfMemcKey()/wfGlobalCacheKey()

* Move @deprecated tags out of the header so the documentation comment
  still has a summary
* makeKey() and co. are part of BagOStuff, not ObjectCache

Follows up 1e5961dbf6a.

Change-Id: I168ce97597f288f6345a404354d68371e57522d3

Localisation updates from https://translatewiki.net.

Change-Id: I005052043dc4c3025e18f6dd2e29b435e3afab3d

mediawiki.page.watch: Deprecate mw.page object

Let's deprecate the mw.page object in the same way we did with mw.special.

mw.page.watch is the only property that exists inside mw.page, and it's mostly
internal. Let's expose that via module.exports instead (for use by Wikibase
and VisualEditor).

Change-Id: I0e6a95bd4fbaba06eae5137ac7af84f62bebf6f0

Merge "Batch/pipeline backend operations in refreshFileHeaders"

Batch/pipeline backend operations in refreshFileHeaders

Bug: T150741
Change-Id: I44efd47e7eb3eba71ae39808ee8e6969e141c5cc

Use file width/height instead of metadata for getContentHeaders

This allows us to populate X-Content-Dimensions without touching the
existing metadata format. Which makes the migration of existing content a lot faster by
only having to run refreshFileHeaders.

Bug: T150741
Change-Id: I2c0f39b2b01f364c3fab997ccc2f874b7f101d8a

Create object mw.page in watch.js

The object mw.page is only used in watch.js and it is not intended to extend
this object by other modules because with module.exports exists a more
flexible way.

This change avoids the dependency on module 'mediawiki.page.startup'.

Change-Id: Ib717261c20856d048c67e2196e9dc619b72ab1db

Merge "Restore dependency on module 'mediawiki.page.startup'"

Merge "Sync up with Parsoid parserTests.txt"

Merge "Move loading of mediawiki.toc from Parser to Skin"

Merge "ApiParse: Add Skin::getDefaultModules() in useskin mode"

Merge "OutputPage: Move hardcoded default modules to Skin::getDefaultModules"

Merge "mediawiki.action.edit.preview: Enable 'useskin' mode for action=parse API"

Merge "ApiParse: Add 'useskin' param for general OutputPage/Skin support"

Restore dependency on module 'mediawiki.page.startup'

Module 'mediawiki.page.watch.ajax' requires a dependency to module
'mediawiki.page.startup' because 'mediawiki.page.startup' creates the object
mw.page and 'mediawiki.page.watch.ajax' requires this object.

This dependency got lost in 06a0dec05723eb6a6604421e134345a5abe115a5.

Bug: T114288
Change-Id: Iebfda85c77eaf6bb19d2ae755813b1c36a95dac8

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit ebac189004d6edc4801719a5802766113bc84beb

Change-Id: I09bd3a72fd6210abc200bead2a16ad4106bcc6f9

Creating new WMF 1.30.0-wmf.2 branch

Merge "Avoid SearchEnginePrefixTest postgres failures"

Update psysh to 0.8.5

0.8.4 introduced sudo, which is like TestingAccessWrapper
but shorter to type. 0.8.5 fixes a bug in 0.8.4.

Vendor patch is in I375c9d681999e8ff46d8bbf2120194e7a4612011.

Change-Id: I07b174c6f25f1d289f0b233f7aec493cb51919cd

Protect language converter markup in the preprocessor (take 2).

This revises 28774022769d2273be16c6c6e1cca710a1fd97ef, which was
reverted in master due to unexpected issues with `-{{...}} ` markup
on translatewiki and enwiki.  Test cases are added to ensure that this
is parsed as a template, not as language converter markup.

https://www.mediawiki.org/wiki/Preprocessor_ABNF is the canonical
documentation for the preprocessor; this will be updated after this
patch is merged.  The basic principles described in that page are
maintained in this patch:

* Rightmost opening structure has precedence: `-{{` is parsed as a
dash followed by template opening.

* `{{{` has precedence over `{{` and `-{`: `-{{{{` is parsed as
`-{` `{{{` since we first grab the rightmost `{{{`.

A bunch of test cases were added to verify the "ideal precedence"
order described on that wiki page.

This patch introduced some minor incompatibilities in existing
markup, in particular with chemical formulae in templates.
Fixes for these are being tracked at
https://www.mediawiki.org/wiki/Parsoid/Language_conversion/Preprocessor_fixups

Bug: T146304
Bug: T153761
Change-Id: I2f0c186c75e392c95e1a3d89266cae2586349150

Merge "Initialize mw.util.$content in mediawiki.util"

Merge "Date range filtering in Special:NewFiles"

Merge "mediawiki.page.patrol.ajax: Remove dependency on mediawiki.page.startup"

Initialize mw.util.$content in mediawiki.util

This avoids the public method mw.util.init().

Add a noop function and a deprecation warning to mw.util.init().

The dependency to module 'mediawiki.page.startup' is not necessary anymore
for using mw.util.$content.

Change-Id: Ib8ca3f9afa43de0ff0bb87569dd2ddfddf2a69b8

Merge "Use test user helper methods in UserGroupMembershipTest/UserTest"