SpecialChangeEmail: error if old email was entered in new email field

Current behavior is to redirect to Special:Preferences without showing
any message.  Instead of that, showing an error message is user-friendlier.

Bug: T38561
Change-Id: I6173cd653f682d5a1be61104db213b577c4adfcf

Merge "Don't check namespace in SpecialWantedtemplates"

Merge "Database: Remove unused errorCount() method"

Merge "Use User::equals() where applicable in the class"

Merge "Update test documentation, fix broken links"

Merge "Fix use of preg_match_all in MagicWord.php"

Use User::equals() where applicable in the class

It is a minor cleanup operation. I replaced a comparison of user 'mId'
with a corresponding 'User::equals()' call.

Change-Id: I89dea55789f343e794429c38934c263168ef608d

Merge "Remove dead local var in MssqlInstaller"

Database: Remove unused errorCount() method

Also removed the protected $mErrorCount property used by this method.

Also removed DatabaseOracle::reportQueryError(). After removing its use
of $mErrorCount and applying relevant changes from 419221e4a71b ("Fixed
bug where catching DB errors left ignoreErrors() on"), the only effect
would be that, like for the other DB types, query errors would be
logged using wfLogDBError().

Change-Id: Ie26a5f5ef7dc5619d9db38d3eb6b7af3e6592aff

Merge "HttpError: Convert line breaks in text message to <br>"

Merge "API: Finish killing "raw mode""

Merge "Small cleanups to WANObjectCache"

Merge "Extend Title check in Skin for titles without associated titles"

Localisation updates from https://translatewiki.net.

Change-Id: I7269d03a73a9afa677d08dc517da608328932c18

Load the module mediawiki.page.watch.ajax only when needed

Load the module mediawiki.page.watch.ajax only when there is
a watch button on the page.

This change uses the same checks as buildContentNavigationUrls() in SkinTemplate.php.

Also combine two if checks.

Bug: T112523
Change-Id: I78895ffea1290f8c1d90630851d0d3d443800d66

Merge "ResourcesTest: Detect missing files in url(...) expressions"

Avoid unneeded memcached queries in showPatrolFooter()

Change-Id: I395019e85cbc80df7d77183ccb75b3691949efc6

Merge "Guard "NotPatrollablePage" negative caching against slave lag"

Guard "NotPatrollablePage" negative caching against slave lag

After some poking I'm fairly sure this is why pages get
"unpatrollable".

Bug: T100986
Change-Id: I137b25ec0538dcc7ad8b6f1b32d5103fd256462c

Small cleanups to WANObjectCache

* Added a few comments
* Renamed $locked => $lockAcquired for clarity

Change-Id: I45710974971731205d072a1f4b0f9cb37e2cb2a2

registration: Fix merging of array_plus

We want the local configuration ($GLOBALS[$key]) to override the default
values ($val). This matches what `array_merge` does.

Bug: T112868
Change-Id: I9c333a1fa67d3f24e09ffed3072b2897389f6139

Merge "mw.widgets.CalendarWidget: Simplify the logic for deciding animations"

Merge "mw.widgets.DateInputWidget: Ensure newly opened calendar shows current selection"

Merge "Make feedlink icon visible by setting a size"

API: Finish killing "raw mode"

It was kept around in the ApiResult rewrite because Wikibase was
(mis)using it as an "XML mode" flag.

Bug: T96596
Change-Id: Ic8259649c8cb0cce0444c907607c36d96fb2eb7e

Browser test: avoid loading elements over network

Page object let us search elements for which attributes match a given
regex.  Unfortunatly it loads the list of elements first, then foreach
of them trigger a call to Selenium to retrieve the attribute to match.

Attempting to regex match a link href, would cause hundred of roundtrip
to the Selenium host.

Instead, use a CSS selector to get the element we are interested in
which skips the bad page object code.

Bug: T92613
Bug: T112291
Change-Id: I12949a4d0ef61d38adbaaf622722c89033949cad

Merge "Add line breaks between label and value in "Username" and "Password""

Merge "Drop zh-tw message "saveprefs""

Drop zh-tw message "saveprefs"

Change-Id: I4b0da9f17e77a9a0d7af7522d7cb59d8891246f7

Localisation updates from https://translatewiki.net.

Change-Id: Id35e5c6fd7a706cc76e88d9181edb8f39a3ac201

Make feedlink icon visible by setting a size

Also avoid repeat and adding the necessary padding to give place for the
icon before the label.

Follows I1e47b626fa937066823f056aec6fbc0205fdb9b6,
Iabaebe9bf1e11a06bb951769693b1be4404e5b00 and
Ife5d84fb95944cb24939936785f3d8c4d9c2c187

Change-Id: Iaf8882142b58f2c793d9f8692a9d4051460c7144

Merge "Move query rewriting into search backend"

HttpError: Convert line breaks in text message to <br>

Used by MediaWiki::tryNormaliseRedirect() for example with plain text
containing line breaks.

Change-Id: I5949046af4065c5536080212b96374466849b91a

Purge remaining oversight references from code

i18n change coming in a separate commit

Change-Id: I46b9311cd01e95e524587ce40dcbae62f58dc5a8

Add line breaks between label and value in "Username" and "Password"

This is supposed to separate the label from the value,
which is needed to avoid messing it up in RTL languages.

Bug: T108321
Change-Id: Idde5841c1099e171cd911cc25fb1771eda885723

Remove dead local var in MssqlInstaller

Change-Id: I209166eab10b870de3d31ac1e551fd759459243a

mw.widgets.CalendarWidget: Simplify the logic for deciding animations

It's all in one place now.

Change-Id: Ie8831775c1110bbcbb0ee4b84211c3df1c6b3e72

mw.widgets.DateInputWidget: Ensure newly opened calendar shows current selection

CalendarWidget's #resetUI method was previously unused, it was clearly
implemented for this purpose.

Change-Id: I865a289b331d6163fb56f9e0ec3918cfd2b91343

Merge "Add missing argument to dieDebug call so it does not die"

Merge "Remove dead local vars in ApiRevisionDelete"

Merge "Moved some WANObjectCache number literals to constants"

Moved some WANObjectCache number literals to constants

Change-Id: I848440408b54c1dfd4961c15cc83aad9a1a19425

Remove dead argument in EncryptedPassword.php

Change-Id: Ib0bb2733284262c233e6de4c7645f0b215690156

Merge "FauxRequest: Avoid header leak"

FauxRequest: Avoid header leak

Actually override initHeaders to avoid init of the header, if no header
was set.

The tests was taken from I4c06369968e.
Follows-up 578a0c082f.

Change-Id: I86b2a9bae2f5938374fd9e5e46997c3fdfe87e39

Remove dead argument in SpecialEditTags

Change-Id: Idba1fd203e5616774a46e36b42fc124062261bee

Remove commented out code in SpecialExport

The removed code was about exporting/importing image information,
which is not possible at the moment, but also nobody seems to work on this.

Change-Id: Ie10d3bc23454ca797f41bd12c4e05360233978a2

Merge "Add '@since 1.25' annotation to wfLoad*() functions"

Extend Title check in Skin for titles without associated titles

Like talk or the subject title (for a talk title). This prevents Skin.php
from trying to get the talk title of a title in a namespace that can't hold
any.

Follow up:
https://www.mediawiki.org/wiki/Special:Code/MediaWiki/80409

Bug: T110878
Change-Id: I3c803d784f931a21b47065590e567f8bb111aa2a

resourceloader: Add missing argument to makeLoaderImplementScript

Change-Id: I661ec86ab53639be6999e15ac20c726477fae8a5

Remove dead local vars in ApiRevisionDelete

Change-Id: I9c5ab515e9c95402e73423d4aa8632d57660bdff

Add missing argument to dieDebug call so it does not die

Change-Id: I672f4f1a407b35bf88216508156035ef96a7289f

Add '@since 1.25' annotation to wfLoad*() functions

Change-Id: I2cf9c2eddaa8b3449bd2852c496515b5bc133981

mediawiki.Title: Correct documentation

Follow-up to a074db332cc103319adacabcc52b1f615217e222, which removed a
config option but not its documentation entry.

Change-Id: I3a0596b27da9e36eb2bb718a635130341545abab

Make MWExceptionHandler work with PHP7's exception changes

See https://wiki.php.net/rfc/throwable-interface

Bug: T107933
Change-Id: I703c8710c8adeff14c1cbaab89f5688f6cb96c24

Merge "When aborting EnhancedRC block line, block should reflect that"

When aborting EnhancedRC block line, block should reflect that

It was possible to abort the rendering of all block lines, but
the block would still be rendered (with nothing inside). It
would also render a "x changes" link, even though that "x" is
no longer correct.

This had been reverted in adba11dfe332c4c10dbbf0f817982d4a225de6b1,
but has now been fixed up.

Change-Id: Ic5d15e56bc2f46fa6aa8c9375f3cafeb13e1ea9c

Merge "Revert "When aborting EnhancedRC block line, block should reflect that""

Revert "When aborting EnhancedRC block line, block should reflect that"

Caused errors when there were two new topics created on the
same Flow board on the same day.

BadMethodCallException from line 496 of
/srv/mediawiki/php-1.26wmf23/includes/changes/EnhancedChangesList.php:
Call to a member function getTitle() on a non-object (NULL)
{"exception_id":"58b04b8c"}

This reverts commit b30417048bb4b6b785ecae4a8b90aa1f452c4412.

Bug: T112738
Change-Id: Ib404d78eaf1aa9ac7ea34516183bcc9956efc515

Merge "mediawiki.Title: Remove fileExtension option from newFromFileName"

mediawiki.Title: Remove fileExtension option from newFromFileName

Follows-up cc8b00ed5b, 55a4946533.

Has confusing behaviour and is not used anywhere in Wikimedia Git,
nor in on-wiki scripts.

Change-Id: Ibc1a821df93fab67856d025cf6671f41efdaebb1

resourceloader: Don't localise hidden exception

Mechanical stacktraces are mostly program identifiers (which are English) with some
glue text.

When stacktraces are disabled by the site admin, the replacement message doesn't
seem valuable to localise. And one could argue it should be in the user language
instead of the site language in that case, which complicates the output buffer
and caching layers for this.

It also reduces coupling with MediaWiki (Ref T32956).

Change-Id: If446f1b815add2be4baf1c333f18a4bab0cc9c37

ResourcesTest: Detect missing files in url(...) expressions

The way this is implemented is really dirty... but it found us
a few pre-existing bugs already (T111518, T111519, T111771).
I think it might be worth it.

* CSSMin: Add new method getAllLocalFileReferences() which skips the
  file_exists() check.
* ResourceLoaderModule: Make use of it to track missing files too.
* ResourcesTest: Verify that the missing files are missing.

Change-Id: I5a3cdeb7d53485f161ccf8133e76850cdf5b4579

resourceloader: Support absolute-path references in mw.loader.load()

Follows-up b5e69c4ef621 which made loader sources no longer expanded urls.

Change-Id: Ia2fcd13f46d548a98620c3e78b8f59b81d976eb4

Merge "resourceloader: Don't call wfExpandUrl() on load.php urls"

Merge "SpecialMovepage: Convert form to use OOUI controls"

resourceloader: Don't call wfExpandUrl() on load.php urls

wfExpandUrl() should only be used if the url needs to be absolute
and/or needs to point to the canonical, internal or default wgServer
for the wiki.

For meta-data or API responses this can make sense, but for resources
loaded by the browser from the current page, there is no added value
in expanding these to the default domain.

This is causing problems in custom setups such as at Wikimedia,
with "zero" and "m" sub domains for which $wgServer points to the
canonical domain.

Bug: T106966
Bug: T112646
Change-Id: Ie25d3a8f69564319a1b29b5c5904f3d352924a31

Localisation updates from https://translatewiki.net.

Change-Id: Ia4020fe3ad9f37dc5f8a5f1b27974582edfdd2f4

Remove unused private methods in FauxRequest

Change-Id: I49a0a8ced0819eeef3ff404fc14b0b90c36b0922

Remove WMF deployment hack for .feedlink from mediawiki.legacy.shared

To be merged 30 days after 579ec5dee1cdb08fc6c94121ff5570a501f98e0c is
deployed, and we know for sure that the new mediawiki.feedlink module
is included on all pages that need it.

Bug: T89981
Change-Id: I1e47b626fa937066823f056aec6fbc0205fdb9b6

Merge "Allow Linker::formatComment to link to another wiki per default."

Merge "mediawiki.Upload.Dialog: Remove size config from example docs"

mediawiki.Upload.Dialog: Remove size config from example docs

The dialog has a default size (inherited static.size value) and
we shouldn't be encouraging the user to change it, especially
as we add more things to the layout.

Change-Id: I684dfba14e8d872733a7ff2573b67578bce25762

Allow Linker::formatComment to link to another wiki per default.

This also adds tests against injection to LinkerTest.

Bug: T111676
Change-Id: I757f2b91f3b50d789d04e530049c34b4628253e3

Ignore exceptions from document.querySelector()

In Firefox and Safari, passing selectors with pseudo-elements
into querySelector() can cause it to throw an exception.
Ignore these exceptions rather than aborting the selector audit.

Bug: T112650
Change-Id: I9a1bd5eb6ff5ea11eb70ff5bda994b4e51d2431e

Merge "resourceloader: Jobs created in request() should wait for executing modules"

Merge "Use XmlSelect in SpecialEditTags"

Merge "Enable multiple default values for XmlSelect"

Merge "Removed clearSharedCache() hack in User.php"

resourceloader: Proxy 'window.performance' to support Safari bug

Follows-up 5b0749d, which first introduced a call to performance.mark
outside of startup.js – thus triggering this excotic bug in Safari.

In Safari the native thread MAY (emphasis on MAY) decide at any
point to recreate window.performance and blow away any previously
succesfully attached properties.

Bug: T112287
Change-Id: I653e580566da5335d34f1cf1ef52f51ceee2cf5a

Merge "build: Upgrade grunt-banana-checker to v0.3.0"

Merge "Provide VRS objects with a name for more informative debugging/logging"

Localisation updates from https://translatewiki.net.

Change-Id: I4dc82c7424593f75b6247775104503b7eec82fff

Provide VRS objects with a name for more informative debugging/logging

Bug: T112330
Change-Id: Iaa3af55ebc426d54e8bef94dba5415bc535e096b

Remove WikiReference::getHostname.

getHostname() was broken and seems to be unused.

Change-Id: I8d44a6907ad395ea12deebf404831c06e07ed401

Merge "Add support for fragments to WikiMap"

Merge "Add tests for WikiMap and WikiReference"

Use XmlSelect in SpecialEditTags

Bug: T93234
Change-Id: I758691f230cff04540b358e9881fe2bfac2d1b9e

Enable multiple default values for XmlSelect

Useful for when a <select> tag with the 'multiple' attribute is
needed.

Bug: T93234
Change-Id: Ib83aed3392cf15cce4637fac56979a0f6b3d6b2f

Localisation updates from https://translatewiki.net.

Change-Id: I4893e65a430118dbe35f90144afa492899f90150

Removing MediaWiki:Yourgender/de-formal

Bug: T111795
Change-Id: Iced432dda98b88c0c4fecfe30a2f6d3c67219273

SpecialPagesWithProp: convert implicit join into join_conds

Bug: T71083
Change-Id: Ic36ecd33a9b4b2ad8b3ca24fabbe496c5207b28f

Merge "Add missing images used in installer's CSS"

Add missing images used in installer's CSS

Copied from resources/src/mediawiki.legacy/images.
Follow-up to 1e06a9f02f9a3ca645f5abe5cd38943fcfa3aea2.

Bug: T111519
Change-Id: Idb7f02ee62eb535af3c2dbbaed53dabe04ae4e9e

Localisation updates from https://translatewiki.net.

Change-Id: Ib11cc9121522a71ea2a8ea13192e445f44ee4cb3

Rephrase api-help-param-list for clarity

What is currently shown on the ApiHelp is something like this:
One value: title, text, nearmatch

It would be clearer to say
One of the following values: title, text, nearmatch

The example is from
https://en.wikipedia.org/w/api.php?action=help&modules=query%2Bsearch

Change-Id: I92f17f90ae77c6c151b817bae8b2f08d858a0f64

Merge "Remove unused $wgAPIGeneratorModules from DefaultSettings.php"

SpecialUpload: Improve display of error messages

Before: https://phabricator.wikimedia.org/F2569195
After:  https://phabricator.wikimedia.org/F2569202

Change-Id: I212ea1fa4074868c3d2a18d51fb43da786f47d1d

Localisation updates from https://translatewiki.net.

Change-Id: I71bc38489ea62625740f6dd9d73ca046bcf8e400

Merge "mediawiki.Title: Add normalizeExtension method"