Supress email validation on cancellation in SpecialChangeEmail

Bug: 70676
Change-Id: Ic5246ea0ebfdff633092e2f99a179f31bd3eaa71

Merge "ApiQuerySiteinfo: Use proper localised skin display names if available"

Merge "API: Add timestamp of block to blockinfo"

ApiQuerySiteinfo: Use proper localised skin display names if available

Based on the code in Preferences::generateSkinOptions(). This should
probably be turned into a helper method one day…

Bug: 48953
Change-Id: Ib85a783bd19de8a59e7b81a4203a05b786e67772

CSS/JSON/JavaScript ContentHandler refactoring

* All content handlers that deal with code/data tend to have
English as their page language & pageview language, so moved common
code to the abstract CodeContentHandler class.

* Renamed JSONContent & JSONContentHandler into JsonContent*

Change-Id: I46819a0572ef5becc211d0d82471ff7102edaa3c

Merge "Prevent "Notice: Undefined index" in ApiQuerySiteinfo"

Merge "Don't list nonexistent groups as addable or removable"

Don't list nonexistent groups as addable or removable

You can't add or remove a group that doesn't exist, so don't claim that
it's possible on Special:ListGroupRights or the API.

Change-Id: I02d3f00142ca1cb0cdcbf30e79fecb3c96e96405

Prevent "Notice: Undefined index" in ApiQuerySiteinfo

$row['iw_trans'] wont be set if $wgInterwikiCache is used.

Change-Id: Iea1041b9c699667c251442ed7f4c95e3ba550e7c

Prevent "Notice: Undefined index" in wfDiff

Change-Id: I4cc7d10021a114a2035dd2d9274c2340242800cb

Merge "Mark ParserOutput of MessageContent as varying by userlanguage"

Merge "Make image/vnd.microsoft.icon be an alias for image/x-icon mime type."

Mark ParserOutput of MessageContent as varying by userlanguage

Hopefully this will stop cache pollution when a MessageContent
ends up in parser cache. MessageContent objects just do
wfMessage()->parse(), which uses user language.

Bug: 41065
Change-Id: I75e8256a61b828cb9fc93227eb517a34ac6b9d15

Update categorylinks.cl_type when moving a page

cl_type might need to be changed if the page was moved between
special-cased namespaces (Category ↔ File ↔ (other)).

Also update cl_collation, which also might have changed in the
meantime.

Bug: 69740
Change-Id: I46032a8ba04d77005731c02680f77a8808974087

Merge "Fix double output of mimesearch-summary"

Fix double output of mimesearch-summary

The execute method was calling a outputHeader(), and then calling
it's parent implementation which was also calling outputHeader()
causing double output.

Refactor class to use $this->getPageHeader() to output menus
before result list, as that appears to be what that method is
for.

Change-Id: Ic0a79cb73136e636f96e33859dadb384eca38255

Localisation updates from https://translatewiki.net.

Change-Id: I0d9118c2c9329361474b41519114484344cae73d

API: Add timestamp of block to blockinfo

This change affects list=allusers, list=users
and meta=userinfo.

Note: This change also add block expiry to meta=userinfo.
Unlike this field in other modules, it formats the timestamp
properly, instead of just dumping db contents.

Resurrecting from abandoned change Ifdeac5c5f547

Bug: 63326
Change-Id: I4b3e55fe2d07271e1ded89d36d0b98de0e643177

Merge "MediaWikiTestCase: Enforce children call parent::tearDown"

MediaWikiTestCase: Enforce children call parent::tearDown

Can't be implemented as a test case because that's not being
run before MediaWikiTestCase::tearDown.
Also there's no parent destructor, so no need to call one.

Also fixed BagOStuffTest.

Change-Id: Ifd8659b6c8a748c6716099f8822e2c50ab51bda7

Add MailAddress::newFromUser()

And tests!

Change-Id: I5214c50855f6bc756f6d748e435ae2124b2264c1

Add tests for MailAddress

Change-Id: Ia192919322743854096aceb24be0fcf9b1aa32b4

Split UserMailer.php into includes/mail/

Change-Id: Ic435bbdbb690028a5d34e8176522fcf4aa44fa92

Fix Special:Listusers i18n issues

* Changed hard coded brackets to use message 'brackets'
* Changed hard coded space to use message 'word-separator'

Bug: 39845
Change-Id: If117e8f1b2436ed1d259add92fc8f6925403afc3

Merge "Move footer icons to assets/ directory"

Merge "API: Add rawcontinue parameter to action=query"

Merge "Use local var in Action::setHeaders"

Merge "Allow preloadparams for Special:MyPage"

ResourceLoaderWikiModule: Check content format instead of content model

For Gadgets 2.0, we have custom content models that represent
JavaScript and CSS pages. Comparing the exact content model here
is too strict, when we really want to know if the page supports
the JavaScript or CSS formats.

Also fix a debug log comment that didn't work due to improper quoting.

Change-Id: I64e86c74173b1c88cac71483327cacdd8b9967b9

Fix subclassing of CssContent and JavaScriptContent

We need to be able to set a different content model via the
constructor, so make it an optional parameter.

Change-Id: I8b31084e0cfbcca1b3785176eac492d837e0cb0f

API: Add rawcontinue parameter to action=query

Soon the simplified continuation will become the default for
action=query. This parameter will then serve select the return of the
raw continuation data. To allow for clients to transition without
"unrecognized parameter" warnings, I'd like to add the parameter now
before even adding deprecation warnings.

Change-Id: I5f6895b6a94c53a231a50707ee9463f91295da8f

Merge "Clean up mediawiki.legacy.protect a bit."

Merge "Move Config::set() to MutableConfig::set()"

Make image/vnd.microsoft.icon be an alias for image/x-icon mime type.

The mime type .ico files is detected as image/vnd.microsoft.icon on some PHP
installations. This fixes the issue by aliasing it to image/x-icon.

Bug: 69437
Change-Id: I1fdb4c7058546d6cdd47b1961706ef84a05dd360

Clean up mediawiki.legacy.protect a bit.

Also kills some inline JS stuff in ProtectionForm

Bug: 33871
Change-Id: Ie43c80bf5ebd6107458d5284cc68b7128f29359a

Merge "resourceloader: Condition-wrap the HTML tag instead of JS response"

Merge "Revert "Add support for svg in .list-style-image""

Revert "Add support for svg in .list-style-image"

The generated CSS is invalid, as you can't have multiple
list-style-images (unlike background-images). Browsers
either ignore the invalid properties with linear-gradient
and use non-embedded PNG fallback (Chrome, Firefox, modern
IE), or accept the properties but ignore everything after
the comma, using the fully transparent gradient (Opera).

This reverts commit 502fffe3bf2cb6a8fccb8059436a038e21153596.

Change-Id: I87f8142b5e718f96ca8ee74578224b4db7dedce9

Merge "Add margin below CAPTCHA box (and above submit) for signup and login"

Use local var in Action::setHeaders

Change-Id: I543d45835016fd6f4f2ec0e84f4e5453237bbf25

TablePager: Use wfScript() instead of $wgScript

Change-Id: Ib12bd3dfa102dc673789c27506becc2fd8138f56

Merge "SpecialEmailUser: Use Config instead of globals"

Merge "Typo: compatability -> compatibility"

SpecialEmailUser: Use Config instead of globals

Change-Id: I81f6fb942a9cb12332c4763ab4c26cfcc556ddcb

Typo: compatability -> compatibility

Change-Id: I533dfc9daff5e11c1732d5d81a54faa9d7b0c43a

Preferences: Use Config instead of globals

Change-Id: Iab8725ded4da532a3072fb68e3060bd8f8ddf30a

Merge "Make MySQLi work with non-standard socket"

Make MySQLi work with non-standard socket

MySQLi needs to specify the socket location as a parameter, and not as part
of the hostname.

Split the socket out of the hostname if it contains *one* colon (IPv6 safe)
after checking for the port number.

Bug: 65998
Change-Id: I66ee34e1029bc1630669bde4272d0eae9d3fd9f1

Use safe attribute accessor for RecentChange

The RecentChange does not guarantee that all attributes are populated.
Handily it provides a safe getter function in
RecentChange::getAttribute() that will return null with out triggering
an undefined index warning for missing attributes. Protects against log
spam like "Notice: Undefined index: rc_id".

Change-Id: Idee844f0d40a2a084e17f201b5e1501d59a0464d

Add margin below CAPTCHA box (and above submit) for signup and login

Only applies if the CAPTCHA box is shown, which is if ConfirmEdit is
installed.

Bug: 68788
Change-Id: I8fa5f9e0b0394594bb59b1c3f62077e5e7428525

Add support for svg in .list-style-image

* You can now use .list-style-image-svg for support for svg.

* Please see I5196952088e584d3121600e9e829647e8c195bbb for it being used on vector.

Note: This has been tested by me.

Change-Id: Iffff2bc52e1572e661930242ad15937bc83abace

Merge "Group E-mail settings stuff in Setup.php"

Localisation updates from https://translatewiki.net.

Change-Id: I4c78d0591a5cf4fff95dd076bf08741eec4ca050

Merge "RawAction: private variable removing m prefix + doc better"

Merge "Sort modules in alphabetical order and leave note about this"

Merge "SearchResult: clean up construction"

Merge "Fix for Ia9baaf0b: Make previously public variables public again"

SearchResult: clean up construction

- Remove old pre-1.17 constructor format. No core or extension callers
- Remove only usage of newFromRow(), needless abstraction

Change-Id: Ifad161aa36837bb4c8b5c9aecac1c46e2e6cc84c

Merge "Clear the DBO_TRX flag for sanity in ExternalStore"

Clear the DBO_TRX flag for sanity in ExternalStore

* This avoids an explicit commit of an implicit transaction when the site
  config for the ES cluster does not clear that flag by mistake.

Change-Id: I57d8b04cbf3249849db1643fc82673e9de8ea15b

Merge "Commented and re-ordered the LoadBalancer fields"

Merge "Remove patch-restructure.sql"

RawAction: private variable removing m prefix + doc better

Change-Id: I526ebcbb1429af42fa9e33cdf59d12c4854247f7

Merge "Fix SpecialPageFactory::getTitleForAlias"

Merge "Add release notes for BagOStuff::replace() removal"

Commented and re-ordered the LoadBalancer fields

Change-Id: Ie05cfd9a4a60f07ea9710c6bf7657ca956a7dd5e

Remove patch-restructure.sql

Unused. Basically reimplemented as doSchemaRestructuring(). That
method could possibly be broken up and make more use of patch
files though.

Change-Id: Idb0a3740dd1401632f2e273ff46dd856a5d4d735

Fix SpecialPageFactory::getTitleForAlias

It takes the result of resolveAlias and calls SpecialPage::getTitleFor.
Unfortunately, resolveAlias returns an array of [name, subpage] while
getTitleFor expects two string arguments.

This commit makes SpecialPageFactory::getTitleForAlias call
SpecialPage::getTitleFor with two string arguments instead of an array.

Bug: 67045
Change-Id: Ia1a0abf753b03fc69d3d4de025c88312e3e45763

Merge "Removed MultiWriteBagOStuff::replace"

Add release notes for BagOStuff::replace() removal

Change-Id: If90237c54a56a3d6e14835654cb3a514a3ae6cb3

Removed MultiWriteBagOStuff::replace

Change-Id: Ifa6373cda099d1b87e90592598267987e9185a25

Make Special:Unblock link to contributions page for IP blockees

Currently, Special:Unblock links to the user page of anon editors,
which is inconsistent with other areas of the software,
and also less useful, as IP userpages are often redlinks.
This patch converts these links to Contributions links.

Originally reported on Wikia, with an (incomplete) patch here:
https://github.com/WikiaVolDev/app/compare/VOLDEV-31

Change-Id: I0fc73c7ee4fd7c81ed876717e8f60749c51685ae

Merge "kss: Add links upto depth 2 in ToC"

Merge "Removed some b/c code from file backend"

kss: Add links upto depth 2 in ToC

Change-Id: Ibdf130ee01a3c754276470200a4c61b01731199f

Merge "Removed deprecated BagOStuff::replace() method"

Merge "API: Fix list=allusers with multiple values for augroup"

Merge "Properly deprecate srprop=score|hasrelated"

Merge "Remove "related" searches"

Properly deprecate srprop=score|hasrelated

Logs stats on usage, output param descriptions for callers.

Change-Id: I000b91ff4d81dc1563231cd3ae0b56952b9574fd

Remove "related" searches

Poorly documented and never used anywhere properly.

- No core search backend supports it.
- MWSearch implements it but we've never turned it on.
- SolrStore copies MWSearch but it doesn't even work.
- CirrusSearch doesn't bother because it's silly.

Change-Id: I8b052e8e772030fba6c361fdb2775b10a7f7a15a

API: Fix list=allusers with multiple values for augroup

The existing query only works with a single value for augroup, or mostly
if it's combined with auprop=groups or auprop=rights (since most users
don't have every possible group).

When used with multiple values for augroup, it will raise an error if it
happens to encounter enough users who have more than one of the
specified groups. And further, it will lead to repeated groups for these
users if combined with auprop=groups or auprop=rights.

To avoid both these issues, let's use EXISTS instead of a JOIN to test
augroup. While auexcludegroup doesn't have this problem, we may as well
make the same change there, too. And doing that, there's no reason to
continue with an error when both augroup and auexcludegroup are used.

Bug: 70496
Change-Id: Ia7086ce87012c22651ac4c7a3f75558347276226

Removed deprecated BagOStuff::replace() method

Change-Id: Ife2f8e0f10ad7be76b79d5aee435491ea78d4655

Removed some b/c code from file backend

Change-Id: Ie8dbff588b067a202b24dfeaf792301c1077f08b

Update OOjs UI to v0.1.0-pre (bd008e8aed)

New changes:
845287a Update OOjs to v1.1.1
f7d11f7 Make popups position themselves correctly when dimensions change

Change-Id: I16ae22e6a921666de5b7ada7d7750621eb3ca5a7

Localisation updates from https://translatewiki.net.

Change-Id: Ief6252f5235622a08d6c0c8f41ce4c0a49e15d9e

SpecialListgrouprights: Use Config instead of globals

Change-Id: I4c8e48adab856de4e3b736b6ae0366588fb82bc5

Merge "API: Deprecate binary sortkey input in ApiQueryCategoryMembers"

Merge "HTMLFormFieldCloner: Make sure the delete and create buttons have a parent"

Merge "Remove term from search results info"

HTMLFormFieldCloner: Make sure the delete and create buttons have a parent

Also fix the type in documentation.

Change-Id: I0fee2dbb9dd1db157acfba2063a2e4e40152cddf

Move Config::set() to MutableConfig::set()

Introduces a new interface for Config types
that are mutable. Not all Config instances
should be mutable, like MultiConfig.

Change-Id: I56e193cbbf72b7afdf551f60ea635fc347e14b3e

API: Deprecate binary sortkey input in ApiQueryCategoryMembers

Depending on what method is being used to encode the binary sortkeys,
it's very possible that sending a binary value will be mangled by
WebRequest's UTF-8 normalization.

Rather than trying to fight it, let's just do what should have been done
in the first place when r86257 changed sortkey output to hex-encoded.
Too bad now we have to deprecate the old params and add
differently-named new ones instead of piggybacking on the original
breaking change.

Bug: 70690
Change-Id: I5a031e980967cbe574dddc261d56604cecc7147a

Clean Resources.php for phpcs

Change-Id: I138527fd235fc65a7ee10338681a6cf83abddfdd

Clarify "mustposttoken" API error message

Bug: 70656
Change-Id: Id30e54472299bc3998644914eca4e19e9429fd2b

Update OOjs to v1.1.1

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fcore.git/v1.1.1/History.md

Change-Id: I2a78dcd147def3b02314c176e51accb2cefe872e

Remove term from search results info

In surveying other search engines, nobody returns this extra information
and clutters their search results. The information is already available
in the <input> box as well as various context clues (like your actual
results).

Change-Id: Iea21a203731331ca31ced786fb7c0011bd4932eb

Merge "Disable JavaScript for MSIE7 users, update tests accordingly"

Localisation updates from https://translatewiki.net.

Change-Id: I9b3a5891bac18ec36541b13836a957fdcefec5d5

Merge "Don't cast SpecialPageFactory::$list to an object"