EditPage: Stop using globals for configuration in non-static functions

Bug: T144366
Change-Id: Ie884527b64f86b6a989117a45c6ffa6d1893d2b7

EditPage: Deprecate $mArticle and $mTitle for public usage

Change-Id: I2a931826ea142f2214c5f29944c3c3b18da19bad

EditPage: Deprecate public isCssJsSubpage related member variables

Just use the functions directly.

Change-Id: Ie374a5cd4c9255b595ff4a88025738720434a802

EditPage: Don't allow clients that mangle unicode to edit

Get rid of the hack that turns unicode into hexadecimal codes for
browsers that don't support unicode, and prevent their edits entirely.

And instead of relying on $wgBrowserBlacklist, use a hidden HTML form
field - if the contents are mangled and don't match the original, then
reject the edit.

Bug: T67297
Change-Id: I20c2e396d7dfd6a3b23b94b218f94a847522576b

Merge "Lazy creation for heavy menu"

Merge "Fix case of SpecialRecentChanges class"

Merge "Prepare for 1.31 branch"

Fix ip_changes insert counting

Change-Id: I02b457d9dcaadc97eead623e42a5a1ccf8335724

Merge "Do not depend on PHP_EOL in HTMLRestrictionsField"

Localisation updates from https://translatewiki.net.

Change-Id: I1058808c8a06869776b8c23e518ff260be9ab6d6

Fix case of SpecialRecentChanges class

Test fails with $wgAutoloadAttemptLowercase = false

Change-Id: Ib8cd202d36d35a36e7513f81cea4a7f8346661bc

Do not depend on PHP_EOL in HTMLRestrictionsField

HTMLRestrictionsFieldTest::provideValidate only provide test cases with
\n, which fails on windows machine.
I see no reason to use the system depending constant here

Change-Id: I7caf2c4d06c84cac69e20e03d00a93bcd8e7d405

Lazy creation for heavy menu

Assumptions:
* Generally if we have a model we don't need UI
* UI can be created lazy if it is not shown to user

Bug: T176250
Change-Id: Iba7f889d8610de2eb3056248cd1c664b0cd90940

Fix warning in DifferenceEngine

Make sure that the ar_namespace and ar_text fields are selected from the
database.

Change-Id: Ic2e8f99f43025510c14e0a8ff2b708b378cf7ef3

Merge "Set wgUseRCPatrol in ChangesListSpecialPageTest"

Set wgUseRCPatrol in ChangesListSpecialPageTest

wgUseRCPatrol = false makes the tests fail

Change-Id: I18965d33171137ff182800512c8c0e8e63de124e

Merge "Add classes to HTMLCheckMatrix items to identify forced ones"

Merge "RCFilters: Don't grey out results area when initializing, unless there's a default saved query"

Avoid per-title page_lang DB queries HTMLCacheUpdateJob

Change-Id: Ie2fa64400f05d90e615bd8ff26225e197605d244

Merge "EditPage: Try to avoid using $wgTitle"

Merge "Replace uses of each()"

Merge "EditPage: Don't use $wgRequest"

Merge "Improve populateIpChanges.php reporting"

Replace uses of each()

It's deprecated in PHP 7.2, may as well replace it now.

I note that, contrary to claims at
https://wiki.php.net/rfc/deprecations_php_7_2#each, none of our uses
were trivially replaceable with foreach.

* wfArrayDiff2_cmp() is processing two arrays by value in parallel.
* MagicWordArray::parseMatch() is doing something funky with the data
  structure returned by preg_match().
* HashRing was using it like "nextKey()", replaced with calls to key()
  and next().
* FormatMetadata and IndexPager were both using it as a shorter way to
  get both key() and current() for the first element in the array. I
  suppose a foreach(){ break; } would do the same, but that's confusing.

Bug: T174354
Change-Id: I36169a04c764fdf1bfd6603395111c6fe0aae5eb

Merge "Bump LocalFile::VERSION to invalidate file page cache"

Detect query timeouts and throw a specific exception

Throw DBQueryTimeoutError if a database query error is detected
to be a timeout.

Only DatabaseMysqlBase has been updated here.

This is a subclass of DBQueryError, so existing catch'es will work.

Bug: T175775
Change-Id: I4749dc33ad530d9b22504f02106b1ca49e8eb167

Merge "Do not run tests that depend on curl if it is not loaded"

RCFilters: Don't grey out results area when initializing, unless there's a default saved query

Check whether there is a default saved query on the server side, and if there is,
add the mw-rcfilters-ui-loading class.

Also centralize the code related to the saved query preferences.

Bug: T173533
Change-Id: I4138fde22bdd8cc55c65846b91184c3ad3057244

Merge "EditPage::getCheckboxes: Stop respecting wgUseMediaWikiUIEverywhere"

EditPage: Try to avoid using $wgTitle

The most common code path is from EditAction, so make sure
EditPage::setContextTitle() is called in that case.

Log any uses that fallback to $wgTitle in the GlobalTitleFail log group.

Bug: T144366
Change-Id: Ie6c7dfbaa432239389d210051372427b8fa045b4

EditPage: Don't use $wgRequest

Bug: T144366
Change-Id: I392e165a6ba3c913ce6e415dbfcfb3cd51178a4e

Improve populateIpChanges.php reporting

Instead of just the rows we attempted to insert, also count
actually inserted ones.

Change-Id: Ie747cb41873640776281794a90dbe3b6b8e3fa84

Merge "RCFilters: Don't remove mw-changeslist-line-prefix in enhanced mode"

Merge "RCFilters: Redo the way spinners and ready/loading states are managed"

Do not run tests that depend on curl if it is not loaded

Bug: T176193
Change-Id: Ia7b9b0196f800eb14463acc2a24df5ac1e48f3ed

Merge "Update OOjs UI to v0.23.1"

Merge "Mark various skin/OutputPage hooks as unabortable"

Update OOjs UI to v0.23.1

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

Depends-on: If38f5a074d655542c5704a9b17617d299ee17921
Change-Id: Iac39c51049266b302d66658bf1ea6b67c1bb4c8c

RCFilters: Redo the way spinners and ready/loading states are managed

The mw-rcfilters-ui-ready class was being set on no fewer than 4 elements,
all with different meanings, and some with no relevant styling at all.
It also did not distinguish between "the UI is initializing" and
"we are loading new results".

Remove mw-rcfilters-ui-ready completely (and remove some code that added
or removed it pointlessly), and create two new CSS classes on the <body>.
mw-rcfilters-ui-initialized is added when the UI has initialized, and
is used to hide the old UI and display a spinner while the UI loads.
mw-rcfilters-ui-loading is added when we start loading new results
and removed when they arrive; it's used to grey out the results
area and add a spinner.

Bonus: make the spinner appear in a different place depending on why
it's being shown: when initializing, it's shown in the place where the
UI will soon appear; when loading, it's shown near the top of the
changes list.

Change-Id: Ib5c8a36654ba44880823fdade8cad52ffa59ed69

Yet another attempt to fix the populateIpChanges script

It would seem we'd need to update $blockStart if there are no results
for a given block, and more importantly, continue and not break!

Bug: T175962
Change-Id: Ice1bdae3d16cf365da14c6df0e8d91d2b954e064

RCFilters: Don't remove mw-changeslist-line-prefix in enhanced mode

It contains the 'x' buttons to unwatch pages when the relevant
preference is enabled, so we shouldn't remove it.

To make the table still work, we have to add an extra <td>
to the nested rows, and start the highlight one row later.

Bug: T176264
Change-Id: Ie8eb913d55165ad2c548230cd61cd9ee189d9504

Don't attempt to copy to ip_changes in PageArchive class.

This actually happens in Revision::insertOn, so the code is redundant,
but also shouldn't ever execute because it erroneously was checking
for ar_rev_id=0 and not ar_user=0.

This is sometimes causing errors in production, in the rare cases
where ar_rev_id is NULL, and hence (int)ar_rev_id evaluates to 0.

Change-Id: I35a7310f600a9db683d7dc0e730da8ff31e337a2

Merge "CSSMin: Mangle whitespace in embedded SVGs"

Merge "Attempt to fix populateIpChanges maintenance script"

Merge "ContribsPager: Check if target is an IP range outside foreach loop"

Attempt to fix populateIpChanges maintenance script

With the previous edition, some IP revisions failed to copy,
e.g. rev_id 1507693 on enwikivoyage. This change first fixes to go
by mBatchSize, and removes other redundant clauses in the query.

Bug: T175962
Change-Id: Ia5f3e275d8bef120090b4c60e6aec4c204d55d53

Localisation updates from https://translatewiki.net.

Change-Id: I7b3144ed162fda4b58dd1df58e7c0b7310f2a087

Mark various skin/OutputPage hooks as unabortable

Follows-up I94c7ab656bd1a04. An initial pass of marking various
hooks as unabortable.

* BeforePageDisplay (T173411)
* OutputPageParserOutput
* AfterFinalPageOutput
* SkinTemplateNavigation (T107980)
* SkinTemplateNavigation::SpecialPage (T107980)
* SkinTemplateNavigation::Universal
* PersonalUrls

Bug: T173615
Change-Id: I0cb333dd5ff74f7dca725ff210200a0779a9a360

Merge "CSSMin: Improve encoding of quotes in embedded SVGs"

CSSMin: Mangle whitespace in embedded SVGs

Convert newlines and tabs to spaces (which can be unencoded),
and consolidate runs of multiple spaces into a single space.
Also remove any leading and trailing spaces that might result
(most files end in a newline, for example).

Bug: T175318
Change-Id: Ic66c6acb37079cae84dd80ab2d5f2c829cf2df96

Merge "Update signature of UserGroupsChanged to match documentation"

Prepare for 1.31 branch

Change-Id: I2a1ce7b7af7ef9991a054332593b1f63a0213469

Merge "SpecialRecentchangeslinked: Unconditionally join on the page table"

CSSMin: Improve encoding of quotes in embedded SVGs

When a URL (data: or otherwise) contains quotes, don't wrap it in
double quotes (") but in single quotes ('). This then allows
us to unencode double quotes (") in the data URI embedding of SVGs.

Bug: T175318
Change-Id: I3e7eab64e1c3e82066014fb594f82d786983ce90

DumpFilter is autoloaded. No need to require in maintenance script

Change-Id: Ib5c5e9a5144eb5454d1e498bf6f18d9e5829a52f

Update signature of UserGroupsChanged to match documentation

UserGroupsChanged is called in Special:UserRights with seven args
The caller in User.php only has five

Change-Id: I221da323c79bb8da0113533d01b429249d15e7de

Merge "Remove unused variables from closures"

SpecialRecentchangeslinked: Unconditionally join on the page table

As we do on SpecialRecentchanges and SpecialWatchlist already,
because the last revision filter needs it.

Bug: T176228
Change-Id: I65f0f971df24853999ca445f968dd49fb0640066

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

Add classes to HTMLCheckMatrix items to identify forced ones

Some items in an HTMLCheckMatrix form field can be forced to be
on or off, in which case they're disabled. This adds three new
class names, so that (for example) Javascript can identify these
checkboxes. There are not currently any classes assigned to them.

Bug: T172585
Change-Id: I984020ce2437effb3ff5f186470105fd80d4a00f

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"

Bump LocalFile::VERSION to invalidate file page cache

Bug: T175444
Change-Id: Ic0665d5554ed43c48d4fcdef087fbd33b597788c

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