Deprecate Parser::fetchFile() since it's unused

Change-Id: Ic2bc3dd0479a373159a22da5f0a6961e212352ff

Complete coverage of Parser::getTemplateDom()

This test covers the branch of code when the $mTplRedirCache is already
populated, by using the same template that redirects twice.

Change-Id: Ie0ce277c75366b7b060e0da6873175976621aff9

Merge "Remove deprecated utfnormal back-compat"

Merge "Add 'Special:MyLanguage' in 'All skins' category link"

Remove deprecated utfnormal back-compat

Change-Id: Id8f497d0bbaab8282057c9d271a7c9b1e77d1a3f

resourceloader: Add coverage for non-js error case in makeModuleResponse

Adds coverage for line 1170-1175.

Change-Id: I2208264d3fca306b2740b243d7cdc209b224ebca

Merge "Improve Parser::braceSubstitution() coverage"

Merge "Remove eslint-disable no-use-before-define"

Merge "Hooks: remove some unnecessary code"

Merge "Complete test coverage of Hooks class"

Remove eslint-disable no-use-before-define

mediawiki.js: Reorder setGlobalMapValue() to avoid a forward reference.
mediawiki.action.edit.stash.js: Reorder two functions without to avoid
  forward references.

In all other cases add eslint-disable-next-line no-use-before-define.

Change-Id: Ic8b2f8cf61dcfb1a215ad3d3dddfd40282e3e836

Localisation updates from https://translatewiki.net.

Change-Id: I5c895f4b61a73570671acf36c817cc405f54ec0b

Merge "Improve AutoLoader test coverage"

Hooks: remove some unnecessary code

PHP automatically creates the array when you try to append an item to
it. Really.

Change-Id: I73b1efe749a9311feeb0d14e1c76b5bbbc0265bd

Complete test coverage of Hooks class

Change-Id: I9e720c44e6d6c9c5d726a8a147e07ee9b638913f

Improve Parser::braceSubstitution() coverage

Change-Id: I3d9426143fe486c6aed0494b68773a36e24c02d9

Improve AutoLoader test coverage

Change-Id: Iddb24cad41979a4798392ab08d104dac41ed736d

Parser: Don't catch exception just to rethrow it

This is left over from 4ff813680.

Change-Id: I624c2c22b7736af249647997565fe06f52d40fe2

Merge "AutoLoaderStructureTest: Allow PSR-4 directories to have files with 0 classes"

Merge "Rename ApiQueryContributions to ApiQueryUserContribs to match api module name"

AutoLoaderStructureTest: Allow PSR-4 directories to have files with 0 classes

Files like ServiceWiring.php can be safely located in a PSR-4 autoloaded
directory, because they have no classes.

Change-Id: I359b305df9071d6bc5afe4b5f29e762041f4aaef

Split AutoloaderTest into a structure and class test

AutoloaderTest covers the AutoLoader class, and AutoLoaderStructureTest
covers the structure part of the test.

Change-Id: Ic4e7bfd670e1c3413631bda31260cc1cc825b1a2

Merge "Increase cleanupTitles.php batch size"

Rename ApiQueryContributions to ApiQueryUserContribs to match api module name

Change-Id: I865628b87eda7be349522fcfaf94f5563142026e

Merge "Hard deprecate UtfNormal methods"

Merge "Switch includes/{sparql,Storage,tidy} to use the PSR-4 autoloader"

Merge "Switch includes/{preferences,services,session,shell} to use the PSR-4 autoloader"

Merge "Add structure test to ensure PSR-4 autoloader covers everything"

Hard deprecate UtfNormal methods

Change-Id: Iafbf89aa85ae77cd790aeac6911ccc2d014a6c65

Localisation updates from https://translatewiki.net.

Change-Id: I4500053ecbb780a723d5f2398a3d1693a9d12d31

Add 'Special:MyLanguage' in 'All skins' category link

Change-Id: I5f9f6c3c9151ea1570d0ec3f361af40fba8abd20

Avoid PHP 7.2 warnings in DBConRefTest about count() on non-Countable

Change-Id: Ida81bf998b462f2f6bb2b708df1f15bbc1933db1

Increase cleanupTitles.php batch size

Change-Id: I086892e44936fdaf9d5d1f3667325daaaaa7260e

Merge "languages: Use static array files for normalizer data"

Merge "MediaWiki\Auth\Throttler: Change "ip" key name"

MediaWiki\Auth\Throttler: Change "ip" key name

Apparently 'ip' is overwritten by logstash at some level, so use 'ipKey'
instead.

Change-Id: I0547acdbb9f3fc7a990bd4baa2d748e89bac8c3b

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

Localisation updates from https://translatewiki.net.

Change-Id: I1bb2563f7ebba3c66062b79bd523c3f226b54cb9

Merge "Enable merging of WrappedStringList between 'bottomscripts' and 'reportime'"

Remove non namespaced TimestampException

Change-Id: I6d5cc227cacc6a24e92002e589e439a3bbd033a7

Merge "Move includes/auth back to the standard classmap autoloader"

Switch includes/{sparql,Storage,tidy} to use the PSR-4 autoloader

Change-Id: I1ae3cb3833a28286eb4ae09b586785d156917e8e

Switch includes/{preferences,services,session,shell} to use the PSR-4 autoloader

Change-Id: Ic095ed65d05517cdb7b383cde8c664044f790f05

Add structure test to ensure PSR-4 autoloader covers everything

This ensures that there aren't any classes inside a PSR-4 autoloaded
directory that aren't being autoloaded properly.

Change-Id: I200a8535c2f47a6bf3287a7fe1182151493372f4

Move includes/auth back to the standard classmap autoloader

Partially reverts c587ab1375eaf61acbd3e066ff791d096e769189.

Change-Id: Ie8cfe356b07f08d481370b9f61b1d08ed6c9c799

Merge "Restore 'PreferencesForm' as alias for 'PreferencesFormLegacy'"

Follow-up If8dfdaf1: Hard-deprecate, drop two uses, other pre-5.3 back-compat code

Change-Id: I1c5eee3fe30d6687d88e07011a3d40b6770d0daf

Merge "EditPage: Fix duplicate classic toolbar"

EditPage: Fix duplicate classic toolbar

Caused by an incorrectly resolved rebase conflict
between 14cffee7c1ea7e6f411730a52aadb2458fcfb0fd
and 70941efd35562dcb7003229b56c91a98a67de7a9.

Bug: T195517
Change-Id: I294acff48fe9b1a1a9384d717f09d90e417204ab

Insert space before compatibility link in PHP version message

Change-Id: Ia056ba7f9eddbcf31082c7b2f7a64e7dbf02cba4

Merge "Add unicode mapping for ICU 60 and 61"

Add unicode mapping for ICU 60 and 61

Change-Id: Ifbbc8d7ecc788bc2c6b07a8ebba46a9648545786

IcuCollation: Deprecate getICUVersion(), no need for PHP53 back-compat

Change-Id: If8dfdaf187b32b7b9a2c09a240416b9f481593f1

Restore 'PreferencesForm' as alias for 'PreferencesFormLegacy'

In change 4633f4d46a8db46721474e9ffa6a130d8b7a2404 it was changed
to an interface implemented by both PreferencesFormLegacy and
PreferencesFormOOUI so that existing typehints for some functions
parameter would accept them both. Replace those typehints to use
HTMLForm instead. There was really no guarantee in the past that
they would only be given PreferencesForm or its subclasses, either.

Because the typehint change affects some hooks, note it as a
deprecation in MW 1.31 and a breaking change in MW 1.32.

Also add @since tags and correct some typos in code comments.

Follow-up to 4633f4d46a8db46721474e9ffa6a130d8b7a2404.

Change-Id: I61749f1d864cf68afe90cd9e15ba2d7a74252501

Localisation updates from https://translatewiki.net.

Change-Id: Ie807be662d077fe0e177e82834d70581210a5f3e

Merge "mw.page.gallery: Prevent jumpiness due to viewport height changes"

mw.page.gallery: Prevent jumpiness due to viewport height changes

* Do not recalculate if only viewport height has changed, but not
  width. Viewport height doesn't affect this at all.
* Set a min-height on the gallery element before resetting image
  dimensions so that content following it only reflows once, when
  we calculate dimensions again.

Bug: T194193
Change-Id: I6362863cc0f3f7b76c9ec7437d37ff6762ecf153

Merge "Switch includes/{auth,edit,editpage} to use the PSR-4 autoloader"

Merge "Re-enable MediaWiki.Files.ClassMatchesFilename sniffs"

Merge "Have class match filename in some maintenance scripts"

Bump PHP version requirement to 7.0.0+

Bug: T172165
Change-Id: I740f32ac859d9bb3787fdf8414f82ae6f410492f

Drop $wgStyleVersion, deprecated in 1.31 and now unused

Bug: T181318
Change-Id: Ib988f3118786debda135d6ad3d27deda5a51fc8a

Merge "Skin: Drop appending of wgStyleVersion in getSkinStylePath()"

Enable merging of WrappedStringList between 'bottomscripts' and 'reportime'

* Fix ResourceLoaderClientHtml to return what it was documented to
  return, a WrappedStringList. It accidentally used the wrong join()
  method, causing it to create a plain string too early.

* Update method documentations from ClientHtml::getBodyHtml to
  BaseTemplate::getTrail for 'bottomscripts', and from
  ResourceLoader::makeInlineScript for 'reporttime'.

* Update BaseTemplate::getTrail to join by new line instead of native string
  concatenation.

This by itself would suffice for the most common case, but in order
to also account for the possibility of extensions using hooks for
'SkinAfterBottomScripts' that concatenate, update bottomScripts()
to pass a plain string to the hook and merge it later.

Change-Id: If0d227cb9db67229a27d489c082db790ea8e3840

Merge "resourceloader: Move style deprecation warnings to <body>"

Remove mentions of jquery.mw-jump in code comments

The idea of listening for an event and doing something, with on('focus'),
is rather trivial and not worth attributing to another module, in particular
when that module is being phased out :)

Bug: T195256
Change-Id: Ic6a65254cbcc5e9503fece1afb7861a706713d77

Merge "Add ability to filter based on rc_title in API"

Switch includes/{auth,edit,editpage} to use the PSR-4 autoloader

Change-Id: I9210d26c1031d69f6691fef038cc2db6f391b70e

Re-enable MediaWiki.Files.ClassMatchesFilename sniffs

Change-Id: I63f54b0336168a50aef72e38202e153677eafd3c

Have class match filename in some maintenance scripts

Each of these scripts had a class name that was not referenced outside
of the script file itself, and are safe to rename as a result.

Change-Id: Id605aca11db51ee433baeaa998a0e33184c930ca

Merge "Update wikimedia/wrappedstring to v3.0.0"

Merge "IP: Fix sanitization of IPv4 ranges"

Merge "Fix MediaWiki.Commenting.LicenseComment.InvalidLicenseTag errors"

Merge "RELEASE-NOTES-1.32: Line wrap to 80 chars"

Skin: Drop appending of wgStyleVersion in getSkinStylePath()

This setting is deprecated.

Bug: T181318
Change-Id: I9e56764e37b5f10ce1bddfb7826c18982fb1ebb2

Merge "Enable deleteAutoPatrolLogs to delete pre-2011 autopatrol logs as well"

RELEASE-NOTES-1.32: Line wrap to 80 chars

Change-Id: I673efe2777da17bd44788ee8a11707e040242176

Fix MediaWiki.Commenting.LicenseComment.InvalidLicenseTag errors

Change-Id: I936c3f5fca1a0061f215e80469f5d882cb32ee29

Enable deleteAutoPatrolLogs to delete pre-2011 autopatrol logs as well

Bug: T184485
Change-Id: I3f123ea0ae69f80beec7425be0f3e4c00a409c0d

Update wikimedia/wrappedstring to v3.0.0

Notable changes:
- Remove back compat layer for WrappedString\ -> Wikimedia\.
- Improve merging of nested lists.

Depends-On: I89c015e638fe3e77ef12b88948d3cfa007afa7b0
Change-Id: I10cc2ed66e1d78a77fec7ffeafc75962f7d73090

build: Updating mediawiki/mediawiki-codesniffer to 18.0.0

Change-Id: Id39e8dcf22262d1b3c57628467163a995fe4053b

Merge "WatchAction::onSubmit return correct value, not always true"

Merge "Setup: Remove various Profile sections"

Merge "Update 'OpenSearchDescription' links"

Merge "Add 'Special:MyLanguage' in Short URL link"

Merge "OutputPage: Remove appending of wgStyleVersion to legacy resources"

Localisation updates from https://translatewiki.net.

Change-Id: I12af86ca297eae5af2e7c74fce0d198e6c473827

OutputPage: Remove appending of wgStyleVersion to legacy resources

For addScriptFile(), just remove the appending of wgStyleVersion.
Going forward, anyone still using this, should simply append a query
parameter themselves in a way that is specific to that one url
(instead of relying on a generic global variable). Alternatively, one
could use OutputPage::transformResourcePath if the file is in /w/.

For addStyles(), also remove the appending of wgStyleVersion. Since this
method takes paths relative to /w/skins, we can easily update this to
automatically use transformResourcePath(), so that file-hash based query
parameters are automatically added.

Test Plan:
* Add calls to top of OutputPage::output():
  `$this->addStyle( 'Vector/README.md' );`
  `$this->addScriptFile( "{$GLOBALS['wgScriptPath']}/composer.json" );`
* Before, they are both inserted as `<link>` (head) and `<script>` (body)
  with a query parameter based on wgStyleVersion.
* After, the `<script>` (end of body) has no query.
  After, the stylesheet (head) has a SHA1 content hash as query.

Bug: T181318
Change-Id: Ie5ab5066ef7d07279086bde838d7305e9e4eabaf

WatchAction::onSubmit return correct value, not always true

The onSubmit method documentation states, that the caller can expect either
true for a successfull run, false if not tried and an array of error messages
in case of an failure. WatchAction however always returned false, even though
a Status object is availble with all needed information.

The behaviour of WatchAction::onSubmit is now changed to return the appropriate
value taken from the returned Status object of WatchAction::doSelf.

Also:
 * Added WatchAction test class to higher test coverage, especially for the
   static methods
 * Marked getUnwatchToken as deprecated, it's not used and a caller can easily
   switch to getWatchToken with "unwatch" as the action parameter

Change-Id: I2c1b91e1884a0d5f27f5e7ab9eafd6173642c21c

Merge "Translate ApiHelp, ApiSandbox and Autoblocklist to Spanish"

Merge "resourceloader: Remove 'user.styles' dependency from 'user' module"

Merge "OutputPage: Remove support for non-existent /w/skins/common directory"

Merge "Remove the unused 'jquery.farbtastic' module"

Merge "Convert content.externallinks.css to less"

Remove the unused 'jquery.farbtastic' module

This was added in (r99923; 4d8c7e96ed1f) while reviewing the
'gadget-preferences' branch in SVN of the Gadgets extension during
development of MediaWiki 1.18, intended for use in a color picker
on Special:Gadgets as part of an experimental feature that was
never completed (r94051).

The module has no usage anywhere in Wikimedia Git, nor in any
third-party hosted repos indexed by MediaWiki Codesearch, nor
anywhere on-wiki at WMF in site scripts (per mwgrep).

Bug: T193826
Bug: T192623
Change-Id: I8ed6c09dc7efd750ad4688b895b2e3f808a0e52b

Merge "resourceloader: Remove mwLoadEnd code (moved to Navigation Timing)"

Merge "Translate special pages into Gorontalo"

Merge "Crimean Tatar/crh transliteration odds and ends"

Translate special pages into Gorontalo

Bug: T189128
Change-Id: Icb3685f4115c63fb5b50ce5429f564481d2cdfd6

Translate ApiHelp, ApiSandbox and Autoblocklist to Spanish

Change-Id: Ie8cab864112cde17a5f5ced7645fff25c558abdf