DumpFilter is autoloaded. No need to require in maintenance script

Change-Id: Ib5c5e9a5144eb5454d1e498bf6f18d9e5829a52f

Merge "Remove unused variables from closures"

Merge "Fix link prefix/suffixes around Category and Language links (take 2)."

Merge "Remove return bool from MysqlUpdater::doInterwikiUpdate"

Merge "Remove use of implicitGroupby() in WikiPage"

Merge "Remove contentSub from layout when empty"

Merge "Hooks: Introduce Hooks::runWithoutAbort() alongside Hooks::run()"

Remove use of implicitGroupby() in WikiPage

Change-Id: I17cc3ae5a6e90ba49978606e9ca6a1cdcdb57c03

Remove unused variables from closures

Change-Id: I5d2e0cd423a4080677f8517bcf3dcee10bf44799

WLFilters: Handle (un)seen (un)filled bullets on grouped RC

Bug: T176191
Change-Id: I9892f607eebe01d2b74f9eb7f20daf3bf6876398

SECURITY: Fix handling of CommentStore and insertSelect

CommentStore->insert() takes the raw comment, not quoted, and returns
fields appropriate for passing as $a to IDatabase->insert() or $values
to ->update(). Such fields need to be passed through
IDatabase->addQuotes() to be appropriate for passing in $varMap to
IDatabase->insertSelect().

Change-Id: Ie64b279ee7cf9c8c396af385e46c826e0597ab1e

Merge "mw.loader: Use Object.create() instead of $.extend() where possible"

Merge "Deprecate IDatabase::implicitGroupby()"

Merge "Remove use of implicitGroupBy() in ActiveUsersPager"

Merge "Remove implicitGroupby() use in ImageListPager"

Merge "Cleanup form for history"

Merge "Revert "Tweak UI for main filtering entry point""

Merge "WLFilters: always create a balanced fieldset"

Revert "Tweak UI for main filtering entry point"

This reverts commit feefb51581574f3144786d7f935f34ccc947efed.

Change-Id: I37dcaf5cd0c0fa63a53da06f06827731fb87de20

Merge "RCFilters: Actually check whether defaults are empty and update state"

Merge "Follow-up 4107d1d4: avoid JS errors when trying to call mw.track() from jquery.migrate"

Localisation updates from https://translatewiki.net.

Change-Id: Ia5f3e275d8bef120090b4c60e6aec4c204d55d52

RCFilters: Actually check whether defaults are empty and update state

Defaults come from either the backend or the saved queries, and that can
also change during the session. Allow the system to update itself properly
over whether defaults are empty at any given state, and hide/show the
proper button at the correct state.

Bug: T172387
Change-Id: I173c72a391886b41e899ac64621b504c5eb55b0c

Merge "Tweak UI for main filtering entry point"

Follow-up 4107d1d4: avoid JS errors when trying to call mw.track() from jquery.migrate

Some migrateWarn() calls (like the one for compatMode) happen before
mw is created. Don't crash on trying to call mw.track() in that case.

Change-Id: I9e33af2865bb10adc0473045f714fe4f9f699861

WLFilters: always create a balanced fieldset

Don't count on Xml::fieldset creating only the opening tag.

Bug: T176155
Change-Id: Idc4e450c9fa03fc78a012fb5eefa60174bb1245a

Merge "API: Pass revid when parsing text"

Merge "Move New Filters opt-out preference to own section"

API: Pass revid when parsing text

 * Provides consistency with expandtemplates
 * Necessary for properly rendering `text` with {{REVISIONID}}

Change-Id: I310d0778d74c46f15385cd411d2462b541ed339a

Cleanup form for history

Also fixing regression caused by Ie119c92aa4936e2f8982163d4b00d08949f49264

Change-Id: I59f623d2dc43f83cfca3ff31ef79fed0230f68af

Merge "Revert "Xml: Fix Xml::fieldset() when $content is not given""

Merge "Wrap fieldset in SpecialNewpages using HTMLForm"

mw.loader: Use Object.create() instead of $.extend() where possible

For cases where we want a variation of an object with some keys added
or changed, use native Object.create() to create a plain object that
inherits from the other object (without needing to loop and copy each
key/value separately).

This optimisation only works, of course, if the original object
by-reference does not change later.

Also remove hasOwn filter from sortQuery, because it must now consider
inherited keys as well. This filter was already obsolete given mw.loader
already has many unfiltered for-in loops in other places, and just in
general doesn't support environments that extend Object.prototype.

Change-Id: Ib405e32d4598dfb94629c2596ea0804d1c13e7c0

Merge "ResourceLoaderImageModule: Use CSSMin::buildUrlValue() where needed"

Merge "Improve encoding of embedded SVGs"

Merge "Run strval() over the File description"

Speed up populateIpChanges maintenance script.

Use BETWEEN in populateIpChanges maintenance script, which will make it
more efficient when copying revisions with a high rev_id. Also adding a
'max-rev-id' option to prevent the script from looping through IP
changes that have already been copied since the core patch was deployed.

Bug: T175962
Change-Id: I1df10c9b7237ad5002f76f9d354c36ce879d9d9f

Merge "Don't unconditionally run patch-editsummary-length.sql"

Merge "Split mediawiki.special.preferences into separate file for each function"

Merge "Return description fields for unprefixed image cache rows"

Merge "phpunit.php: omit --quiet in help"

Don't unconditionally run patch-editsummary-length.sql

Bug: T176041
Change-Id: I165ee4fa1c0cfadf5f8f400a0ea9db220ed7dbbb

Move New Filters opt-out preference to own section

- Add a new section to preference page with the following title:
"Opt out of improvements".
- Move the opt-out option and explanatory text to this new section.

Bug: T175765
Change-Id: Iaf77df2eec714777f54b95b58eb617b5e35bef75

Return description fields for unprefixed image cache rows

Bug: T175444
Change-Id: I5560187d3850253095b695dc7a3cfc954fba9318

Tweak UI for main filtering entry point

- Change the hamburger icon to plus sign.
- Change the search bar input placeholder text to "Main filter menu".
- Adjust the font style of main entry point placeholder.
- Adjust the font style of the advanced filters label.

Bug: T175217
Change-Id: I24742b68ef8bfeb9f3f71d5dc8818464412ab3f2

Merge "Fix findHooks.php"

Run strval() over the File description

Bug: T176090
Change-Id: I8488666c221a1bd4e4e063291e74819a07a4a20f

Localisation updates from https://translatewiki.net.

Change-Id: I8c413069268400a1a0f6ab9808c713666652271f

Revert "Xml: Fix Xml::fieldset() when $content is not given"

Apparently that is intentional and documented. Huh. Whoops.

This reverts commit 0f1cc3bb6e7c3be9b86d2c8736f7458cd40588f2.

Also add test cases for this bizarre function.

Change-Id: I0311f7c85cd575f2a66d50589ec364072be486fb

Merge "Do not run CollationFaTest if 'intl' is not loaded"

Wrap fieldset in SpecialNewpages using HTMLForm

That's cleaner and fixes Ie119c92aa4936e2f8982163d4b00d08949f49264

Change-Id: I8721ae5db79d287a49c2ab8070f8838174a8d687

Merge "mediawiki.action.edit.preview: Apply mw.language.bcp47 to lang"

Fix findHooks.php

Add the test entry point, because it contains documented hooks.
Filter out empty args, coming from trailing commas

Removed empty lines from hooks.txt, because a hook must be one block
without newlines between.

Change-Id: I71e0625da96cad03f5e38a000fa61d1d2bdac8ef

phpunit.php: omit --quiet in help

Bug: T176060
Change-Id: Iee7a0df6ba9fadf92d8e2d9f170cdfc66244d3d5

Remove return bool from MysqlUpdater::doInterwikiUpdate

All return values unequal to false are treated as "update done",
so return void is okay.

Change-Id: I553f6acb450a5fe1708aec641181f9a07da1ac39

Do not run CollationFaTest if 'intl' is not loaded

Bug: T176040
Change-Id: I6b19bf1123d4dca5a1c8e002c0de65bab2138180

Localisation updates from https://translatewiki.net.

Change-Id: I40f96fd439726f95b0dc467cf2f2802f1a3144d7

Merge "Fix return in Preprocessor::cacheSetTree"

Merge "Fix return of Interwiki::invalidateCache"

Merge "Fix return in HttpStatus::header"

Merge "mediawiki.special.preferences: Allow linking to subsections"

Merge "Preferences: Do not set non-breaking space as a label"

Merge "Xml: Fix Xml::fieldset() when $content is not given"

Split mediawiki.special.preferences into separate file for each function

This JavaScript code does four things:
* Convert a simple form into one with tab navigation
* Convert a simple message box into pretty notifications
* Prevent browser window from being closed with unsaved changes to preferences
* Add some enhancements to the timezone field

I will be making some large changes here for T117781 and having
everything mixed in one file was making that difficult. I avoided
making any cleanup changes to make it easier to review this change.

Change-Id: I5e31a661177002e40708614011c1f7e08ec4f44d

mediawiki.special.preferences: Allow linking to subsections

The fix for T54469 allowed linking to individual fields, but not to
subsections.

The subsections have an id prefixed with 'mw-prefsection', like the
main sections, so the 'else' case was never triggered and they were
never handled. Fix the 'if' case condition to not match them.

Also, do not rely on the 'prefsection' class, which is (according to
the comment at the top of this file) only "for backwards compatibility
with older skins".

Change-Id: Ie7b4e3afbbe723a4e3b01aac95476b073a7213cd

Xml: Fix Xml::fieldset() when $content is not given

A function in the Xml class could generate unbalanced tags. How
ironic. Why do we even have this instead of just the Html class?

Change-Id: Ie119c92aa4936e2f8982163d4b00d08949f49264

Preferences: Do not set non-breaking space as a label

If there's no label, there's just no label. No need to pretend.

Change-Id: I6b7065a23ba6532641989682d802613e0545f7e8

mediawiki.action.edit.preview: Apply mw.language.bcp47 to lang

Apply mw.language.bcp47 to the language codes before using as values
for the HTML attributes lang and hreflang.

Change-Id: I4321d32f3b3a9729650060d59450701710b8c8f2

EditPage: Don't use $wgUser

Bug: T144366
Change-Id: Ia8b7ae5ec73241478d51c57aca376e5bb50ef874

Localisation updates from https://translatewiki.net.

Change-Id: I584dcdcb53c62280a20242101d8b3bf3248e3e4b

Merge "Fix a few minor mistakes in PHPDoc tags"

Merge "RCFilters: Only selectively scroll to popup when it's focused"

Merge "RCFilters: Allow empty saved query"

Merge "Fix $wgUpdateRowsPerJob default in docs/deferred.txt"

Merge "replacing "images" by "files" in listfiles-show-all for consistency"

Fix $wgUpdateRowsPerJob default in docs/deferred.txt

Change-Id: I0fdca6ac9d58286cf5f1ad9f55531e4643fa4fc5

Fix a few minor mistakes in PHPDoc tags

Change-Id: I2d5f876c9945ba21f8748562230170e3c7ea2ff2

RCFilters: Only selectively scroll to popup when it's focused

Bug: T175697
Change-Id: Id8f42332c661cf341953eb6af43e49a422181cc8

RCFilters: Don't cast days default to an integer

Values can be floats, like 1.5 days, etc. Cast instead to a float.

Bug: T175965
Change-Id: I14ba792f1cd435f89b2e09067b0a0e894a0a2557

RCFilters: Allow empty saved query

For this, we also need to check for saved query visibility on the
saved query model initialization, because since the model starts
its life all-empty, if the saved query is also all-empty, it will
not trigger any item updates on the main model.

Bonus: Actually check whether defaults are empty; there was a
typo and problem with the original test.

Bug: T172387
Change-Id: Ibc536b577e4c3abcc04fa229f9978b651b2f9e61

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

Localisation updates from https://translatewiki.net.

Change-Id: I64d648923f684e73152ad4ea9fe8267cda9e5d39

ApiFeedWatchlist: Fix copy-paste error

Change-Id: Ibea5bd881c746d7c27c97fc470ee1e3ac9d4b699

Merge "Instruct stashed upload failures to reupload file"

Merge "Make chunkedUpload match upload behavior"

ResourceLoaderImageModule: Use CSSMin::buildUrlValue() where needed

We can't just assume that the URLs that we generate are safe to use
in url() without quotes. There's already a function that automatically
uses quotes where necessary, so use that.

Change-Id: I6663e49b82592eb45b8854fcff56ac966b39f8ce

Improve encoding of embedded SVGs

Unencode spaces, slashes, colons and equals signs.

Bug: T175318
Change-Id: Idebdfca8f93d5e090deba5bf5a256d7054e2d6c8

Merge "RCFilters: Live update: no data returns 204"

Merge "mediawiki.Title: Remove obsolete Object.create ES3 fallback"

mediawiki.Title: Remove obsolete Object.create ES3 fallback

We no longer support non-ES5 browsers.

Change-Id: If1e18dc3111899f3b0a5ae53909624186101a8ba

Merge "mw.loader: Remove Deferred overhead from execute() hot code path"

Merge "mw.loader: Add unit test to confirm implicit 'site' dependency for 'user'"

Merge "Fix saved query names cutoff"

Merge "De-duplicate HTMLCacheUpdate jobs with a page array of size 1"

mw.loader: Remove Deferred overhead from execute() hot code path

This was introduced by 0ac4f99804c for the concept of globally
implied legacy dependencies.

Commit ba257035b07 then re-purposed this for the site/user module
dependency.

The legacy wait was removed in 0ac6076b4c0, thus leaving only the
site/user code.

This commit now simplifies the code back to how it was before 0ac4f99804c
(but keeping the improved error handling from ba257035b07).

Change-Id: I2eae3b78fbe4f03a7a098d8a6233bdc2b79171b8

mw.loader: Add unit test to confirm implicit 'site' dependency for 'user'

The underlying logic is in mw.loader#execute and mw.loader#runScript,
but currently untested.

Adding tests in prep for refactoring this code.

Change-Id: I77ffb5f23443e24ca1d86f36ddf434673e9c1538

De-duplicate HTMLCacheUpdate jobs with a page array of size 1

BacklinkJobUtils consistently uses the "pages" field in leaf
jobs, even when there is only one page per leaf job.

RefreshLinksJob already has this logic for de-duplication.

Change-Id: Ia189bbc9df44f2161cfed4192c23b2ac3cfa65ce

Fix saved query names cutoff

Fix saved filters name misalignment and cutoff for
letters that have descenders on RC page.

Bug: T174111
Change-Id: I32e61fad98f38b48e0234467056a939d634eb15c

Prefer that enqueueable post-send updates are run immediately

If the user is not waiting at this point, so there is not much
reason to enqueue a job over just doing the work now. Running
the update now gives more immediate results however.

This has the effect of  making LinksUpdate run post-send for
forward link updates, since the addUpdate() call in WikiPage uses
the default POSTSEND mode. These updates used to be synchronous
in the past, before proper post-send update support. With post-send
updates, there is not much benefit to using the job queue here.

If post-send updates are not supported, this will continue to
use the job queue.

If a caller needs such updates to enqueue post-send to avoid DB
updates on HTTP GET or if the update is too big to run outside of
JobRunner, it can always just use JobQueueGroup::lazyPush() with
a direct job object or JobSpecification.

Change-Id: Ibc4b1e17538cc8b1fba7d13759e1ebb83abed869

Merge "Improve flaky SiteStatsTest"