Merge "Do not output invalid links for deleted names on Special:Contributions"

EditPage: Fix passing the save/publish button label to other messages

We were passing the message key, instead of the message text.
Follow-up to 44f596231397eb9ea2a5fe345c79ae952d6a1247.

Bug: T166478
Change-Id: Iaf33b0e6fba71ebf1130fc067210ed49c1b06011

Use PHP DateInputWidget in Contribs and use for range filtering

* Add two DateInputWidgets to Special:Contributions, one for start and
  one for end
** If start input is empty but end input is not, display edits up to end
   input, and vice versa
** If both inputs are specified, display edits between the two dates
** If both inputs are empty, no date range is used
* Legacy options (year=/month=) are converted to use for the end
  timestamp, so URLs with them should still work.
* Unit tests!

Bug: T120733
Change-Id: Id15f2b2ce2954fe98dfbbb7b0e86c0e4e5713f5e

mw.widgets.DateInputWidget: Remove accidental 'user-select' on PHP widget's input

This is only meant to apply to the JS widget's handle.

Also moved another block of CSS only meant to apply to the JS widget.
This wasn't causing any actual issues though.

Bug: T166416
Change-Id: I6e58a38a1d5a8fb4b581734b02cc4f12640b4a75

Merge "Making the default toggle look clickable again"

Merge "ImagePage: Do not parse message 'imgmultipageprev'"

Merge "Return 400 on invalid CSP reports"

ImagePage: Do not parse message 'imgmultipageprev'

This message is used as alt text for an image thumbnail, and so it
can't contain HTML tags anyway. This matches the 'imgmultipagenext'
message now. (This inconsistency has been present ever since the
feature was introduced in 2006: 7f7e79022bfd421e49084b089e74c7a16b77bc03.)

Due to using OutputPage::parse() instead of Message::parse(), an
incorrect `<div class="mw-parser-output">` wrapper was also being added.

Bug: T166508
Change-Id: I092887cc91e2c800685d4e55177aed814bb3fce9

Merge "Make running Selenium tests using Sauce Labs easy"

RCFilters: Hide server-side 'empty results' message

RCFilters uses its own empty-results message that also relates to
the reason of why the results are empty (in case of conflicts, etc).
We want to make sure the server-side message is hidden and gone
while we load, and manage the empty-results message ourselves.

Bug: T163577
Change-Id: I1d0f07068653f369dea181f5c4f9964fd6d7cd58

RCFilters UI: Make monobook override of the 'checkmark' more specific

Bug: T165444
Bug: T165446
Change-Id: I6412fb84767332259955462e53bde8dffe17e905
Depends-On: Ibe4b4fc92de1a864059d9cd37168c7ef33b6d874

Merge "RCFilters: Dismiss Apex' insistence to add checkmark to optionWidgets"

Merge "Swap default page title for Special:BlankPage in Spanish"

RCFilters: Dismiss Apex' insistence to add checkmark to optionWidgets

Bug: T165446
Change-Id: I8188b34675e7e49f3603e7b7154e4e0800e94c37

Localisation updates from https://translatewiki.net.

Change-Id: Id10e621f71f0522f6dcb16369cca50d95abf33dc

RCFilters UI: Compare states instead of params when switching URL

For empty url (initial load) and for cases where the URL is
minimized (upcoming) the method should check filter states
to recognize whether there's a difference between current and
requested URL.

Bug: T166347
Change-Id: I9dcc82ce7dbc0ad7c6cf3169cee7269234c95298

Swap default page title for Special:BlankPage in Spanish

"Blanquear página" is a wrong translation in this context.
"Blanquear" is using "blank" as the verb and means "to blank a page",
but this page should translate "blank" as an adjective "this page is blank"

This was already in the list of aliases so I've swapped them so the correct
one is first and is picked as the default page name in wikis with Spanish
content language.

Change-Id: I29c2b02937878c09df09460eedbc5df697e51e34

RCFilters: Minimize url string

In order to minimize the URL query, we use a base representation of the
parameters as if they were all '0' or '' and internally expand on it.

- Only display parameters with a value that is not empty or '0' in the
  URI. Any parameter that is missing from the URI is presumed to have
  an empty value.
- Stop pushing defaultParameters everywhere. Default parameters should
  only be considered either on load (when/if needed) or when the user
  actively requests for them.
- Minimize parameters to the URL, and expand when reading into the model.

Similar to using base filters, we can use a representation of base
parameters to make the URL small but the representation all-encompassing.

Bug: T165445
Change-Id: I1d21c38137fde51fcd561e2de24592722bf532c6

Merge "Add fr (French) as fallback for kbp (Kabiyè)"

Localisation updates from https://translatewiki.net.

Change-Id: I62626aff6dbf017f7fb0fa3b6d33d3cc9ae2c249

Merge "Fix incomplete Language::getDatePreferences() documentation"

Making the default toggle look clickable again

The toggle should now look same same prior to the premade toggle changes. The brackets are not part of the link, but the text inbetween. The link has no href property, so the toggle action is still triggered.

Bug: T166359
Change-Id: I38698cd66d380626728eae9b8a9da1e465b207a6

Merge "wfIsBadImage: Avoid deprecated wfMemcKey()"

Merge "User: Avoid deprecated wfMemcKey()"

Merge "parser: Avoid deprecated wfMemcKey()"

Merge "LanguageConverter: Avoid deprecated wfMemcKey()"

Merge "CachingSiteStore: Avoid deprecated wfMemcKey()"

Merge "User: Avoid deprecated wfMemcKey()"

Merge "UploadBase: Avoid deprecated wfMemcKey()"

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()"

objectcache: respect process cache in getMultiWithSetCallback()

Backend cache queries are now avoided if they are fresh in memory.

Also factor out some code into private methods for clarity.

Change-Id: Ib343fde3dbf63f39e8cb09eca6278811a7d9738b

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

Return 400 on invalid CSP reports

Not really a server error since there's nothing we can do about invalid
user-provided data.

Bug: T166229
Change-Id: I87a7be32ae7e80c112be556bc13db19f11e614ca

Add fr (French) as fallback for kbp (Kabiyè)

Follow-up to 03e306dd3bc72fd134375a60ecd8b88541ea2973

Change-Id: I06f9788eb16cb1b40faac33aaa75d56e8075bc49

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"

wfIsBadImage: Avoid deprecated wfMemcKey()

Change-Id: I92133ed6a94902bdac2e9723f19a653a9962b757

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

User: Avoid deprecated wfMemcKey()

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

Change-Id: Id0280aec72202f3b3098c9c3b563541c2c0f2900

parser: Avoid deprecated wfMemcKey()

Tested that parser cache keys stay the same, before and after this
change.

Also use the more obvious ObjectCache::getLocalClusterInstance() instead
of looking up the main cache type in config and using
ObjectCache::getInstance().

Change-Id: Icef646b3c05e732ef4079d6900e6bce111debf2b

LanguageConverter: Avoid deprecated wfMemcKey()

Change-Id: I7fe8e3ad6de2eb0a156b046805fa0eca928d0892

CachingSiteStore: Avoid deprecated wfMemcKey()

Change-Id: I123b29dd41c6e0f020e0fcc661813addbec92b27

User: Avoid deprecated wfMemcKey()

Change-Id: Iffb4d85dfed740b01fcef5a25762dc751b292c41

UploadBase: Avoid deprecated wfMemcKey()

Change-Id: I717948d6550ed2d98c3a89b3e48e161c3af15d48

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"

Make running Selenium tests using Sauce Labs easy

If SAUCE_USERNAME and SAUCE_ACCESS_KEY environment variables are not
defined, tests are run with `npm run selenium`, as usual.

https://www.mediawiki.org/wiki/Selenium/Node.js/Inside_MediaWiki-Vagrant
https://www.mediawiki.org/wiki/Selenium/Node.js/Target_MediaWiki-Vagrant

If the environment variables are defined, to run tests using Sauce Labs
and targeting Beta Cluster, run this:

```
export SAUCE_USERNAME=***
export SAUCE_ACCESS_KEY=***
export MW_SERVER=https://en.wikipedia.beta.wmflabs.org
./node_modules/.bin/wdio tests/selenium/wdio.conf.js
```

https://www.mediawiki.org/wiki/Selenium/Node.js/Target_beta_cluster_using_Sauce_Labs

Bug: T139740
Change-Id: I622d9c55ae595ab772e733388d209623e3a9f81e

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"