Upgrade wikimedia/scoped-callback to 2.0.0

Depends-On: Ieae6ba7239fd8beedfd9d3061c7dfaf97df301ed
Change-Id: Ibda68b7183cd1e4b222a7823b11a4e7e62e54953

Merge "Use makeLink instead of makeKnownLink on Special:WhatLinksHere"

Merge "Make linkshere-related messages backward compatible"

Merge "Remove renamed .inc files from phan config"

Use makeLink instead of makeKnownLink on Special:WhatLinksHere

Per I11e663cbce32b4199f16df6ed1e9b980630ece7a, makeKnownLink is used for
"linkshere" related messages. However, it always shows a blue link, even
if the page doesn't exist. Instead, use makeLink to get the old
behaviour back.

Bug: T195933
Change-Id: I92ad23dc7bd064a1a95a14f19cd3f134b4c98570

Localisation updates from https://translatewiki.net.

Change-Id: Idb5f51f8339fd3774d0090a5be5660356d5bdc84

Merge "Follow-up If8dfdaf1, I1c5eee3f: Don't assume INTL_ICU_VERSION exists on Special:Version"

Follow-up If8dfdaf1, I1c5eee3f: Don't assume INTL_ICU_VERSION exists on Special:Version

Change-Id: Ib0d3d33d74c70949769a0bda9e3901be3d197d1d

Merge "Use PHP7 constant expression instead of a magic number"

Merge "Exit maintenance script with non-zero status if execute() returns false"

Merge "Use PHP 7 '??' operator instead of '?:' with 'isset()' where convenient"

Merge "Use PHP 7 '<=>' operator in 'sort()' callbacks"

Revert "Strip soft hyphens (U+00AD) from title"

This reverts commit 6b8a5a137d3f449b4056e3de82fa6747b45f1f9a.

Change-Id: Ica5abe69c316792aa2f7eafad9b1d63183b282a8

Revert "Strip Unicode 6.3.0 directional formatting characters from title"

This reverts commit 7564624d1ca80a4d2f1cc2b4d3d32d5d2e0bca38.

Change-Id: I5d596f8f3c784920829de6ae50b270b0396369e0

Remove renamed .inc files from phan config

Follow-Up: Ic441087702376b1ca0e70554c71cdf7ecad908af (for userOptions.inc)
Follow-Up: I999f5842625c752f01c1eb4b012431c4a24ce7ce (for importImages.inc)
Change-Id: I3e1c694f61960e9e4b50a7855927d039e93928fb

Merge "Maintenance script to wrap the mysql binary"

Remove deprecated global $parserMemc and wfGetParserCacheStorage

Deprecated since 1.30, no longer used in core, bundled components,
and WMF-deployed extensions.

Change-Id: I09715f94750f083712719ef599a2ea4260297681

docs: Update for deprecated $parserMemc

Remove mention of this deprecated global in favour of documenting
the backend configuration instead.

Also update the outdated docs for where the sidebar is cached,
which was changed a while ago to use WANObjectCache instead.
See Skin::buildSidebar().

Change-Id: I3d34a760cc05cab6fe7ddd8684257d6ec6b49309

Make linkshere-related messages backward compatible

With I11e663cbce32b4199f16df6ed1e9b980630ece7a we changed message
parsing and message name, which broke some customized messages where $1
was used for various purposed as the page title. This patch shifts
message parameters so to still pass plain title as $1 while keeping HTML
link as $2. This won't change parsing method (still HTML since we use an
HTML link), nor message key (still with "-2") due to different parsing
method.

Bug: T189860
Change-Id: I68a8639a61237c67deb61aefd5360dea3cced6cd

Merge "Update OOUI to v0.27.1"

Merge "registration: Initialize PSR-4 namespaces at same stage as normal autoloader"

Exit maintenance script with non-zero status if execute() returns false

There's a whole lot of shutdown code in doMaintenance.php that is
skipped when you use exit() directly. In HHVM by default, destructors
are not even called.

There are a few cases where maintenance scripts want to do something after
doMaintenance.php returns, so returning null should continue to mean no exit.

Change-Id: I0891e2ee3af7ef2c64c03b70edcf9e281ce1e7ba

Maintenance script to wrap the mysql binary

This is more or less equivalent to the WMF shell script "sql", but it is
reasonably portable and written in a less hackish way.

Change-Id: I240d8d4e65ce46680cac257ee376a9b026c13f92

Use PHP7 constant expression instead of a magic number

Change-Id: I84e13dc6019c429359df3395f0731d17859be06c

Merge "Remove workaround for PHP bug 66021 (PHP < 5.5.12)"

Merge "PhpHttpRequest: Drop back-compat code for PHP 5.5 and before"

Merge "PHPSessionHandler: Drop now-pointless return methods for PHP5 compat."

Merge "Use PHP 5.6 '**' operator instead of 'pow()' function"

Merge "Use PHP 5.6 constant expressions for some bitfield constants"

Merge "Add a simple .editorconfig"

Remove workaround for PHP bug 66021 (PHP < 5.5.12)

The PHP bug 66021 <https://bugs.php.net/bug.php?id=66021> was fixed by
https://github.com/php/php-src/pull/518 and is included in PHP 5.4.28+
and PHP 5.5.12+.
This workaround is not necessary anymore because the minimum PHP
version for MediaWiki is 7.0.0+.

Change-Id: I801eaffc253fd88e0d3c87cfe97777837bd3902d

Use PHP 7 '??' operator instead of '?:' with 'isset()' where convenient

Find: /isset\(\s*([^()]+?)\s*\)\s*\?\s*\1\s*:\s*/
Replace with: '\1 ?? '

(Everywhere except includes/PHPVersionCheck.php)
(Then, manually fix some line length and indentation issues)

Then manually reviewed the replacements for cases where confusing
operator precedence would result in incorrect results
(fixing those in I478db046a1cc162c6767003ce45c9b56270f3372).

Change-Id: I33b421c8cb11cdd4ce896488c9ff5313f03a38cf

Use PHP 7 '<=>' operator in 'sort()' callbacks

`$a <=> $b` returns `-1` if `$a` is lesser, `1` if `$b` is lesser,
and `0` if they are equal, which are exactly the values 'sort()'
callbacks are supposed to return.

It also enables the neat idiom `$a[x] <=> $b[x] ?: $a[y] <=> $b[y]`
to sort arrays of objects first by 'x', and by 'y' if they are equal.

* Replace a common pattern like `return $a < $b ? -1 : 1` with the
  new operator (and similar patterns with the variables, the numbers
  or the comparison inverted). Some of the uses were previously not
  correctly handling the variables being equal; this is now
  automatically fixed.
* Also replace `return $a - $b`, which is equivalent to `return
  $a <=> $b` if both variables are integers but less intuitive.
* (Do not replace `return strcmp( $a, $b )`. It is also equivalent
  when both variables are strings, but if any of the variables is not,
  'strcmp()' converts it to a string before comparison, which could
  give different results than '<=>', so changing this would require
  careful review and isn't worth it.)
* Also replace `return $a > $b`, which presumably sort of works most
  of the time (returns `1` if `$b` is lesser, and `0` if they are
  equal or `$a` is lesser) but is erroneous.

Change-Id: I19a3d2fc8fcdb208c10330bd7a42c4e05d7f5cf3

Use PHP 5.6 '**' operator instead of 'pow()' function

Change-Id: Ieb22e1dbfcffaa4e7b3dcfabbcc999e5dd59a4bf

Use PHP 5.6 constant expressions for some bitfield constants

I searched the entire codebase for 'const' and looked for things
that looked suspiciously like manually calculated bitfield unions.
As of PHP 5.6, we can have them calculated automatically when
defining constants.

Change-Id: I7d971d1a63f8916db2f8f6c053c7dd0a13add92d

PhpHttpRequest: Drop back-compat code for PHP 5.5 and before

Change-Id: Ic2b144e333639dfeff0a86b14e4b4927e9bf6db0

PHPSessionHandler: Drop now-pointless return methods for PHP5 compat.

Change-Id: I8c3a4d12a34407f29764adb010f91dc870e715d7

Merge "rdbms: make runOnTransactionIdleCallbacks() reset DBO_TRX on exceptions"

Merge "Special:Preferences/reset: Always use OOUI not the deprecated vform"

Merge "jquery.lengthLimit: Fix 'cut'/'paste' event handling"

registration: Initialize PSR-4 namespaces at same stage as normal autoloader

readFromQueue() injects the content of AutoloadClasses to
$wgAutoloadClasses however it missed doing the same for
AutoloadNamespaces.

When using the installer with an extension having AutoloadNamespaces
set, its classes would not be found.

Make ExtensionRegistry append to AutoLoader::$psr4Namespaces, and add
a test to cover the new behavior.

Bug: T195783
Change-Id: Id61155867a4ca7d9bc4a347f8671da74b0fa490b

Localisation updates from https://translatewiki.net.

Change-Id: I74622ed8964fc45ad35223301346fa1533e3f14a

Merge "Call Database::timestamp in some tests"

jquery.lengthLimit: Fix 'cut'/'paste' event handling

For the 'cut' and 'paste' events, the input value is only updated after
the event handlers resolve.

Bug: T64319
Change-Id: Ia0b85ce986eb05aceb7096c56a235477970a4d6f

Merge "preferences: Preserve Status from saveFormData() in submitForm()"

Merge "rdbms: do not silently rollback empty transactions on error"

Merge "resourceloader: Create mediawiki.base.js and move mw.inspect to it"

Merge "resourceloader: Improve coverage of ResourceLoader::respond()"

Merge "OutputPage: Remove 'X-UA-Compatible' header (was for IE8-10 JS compat)"

Merge "MediaSearch widget: Bring in ApiResultsQueue tests from upstream"

resourceloader: Create mediawiki.base.js and move mw.inspect to it

As a first step to splitting the 'mediawiki' module, start by
splitting the file. The two files (like the other files in the
same directory) are still concatenated and run at the same time.

The only difference being that the private variables aren't available,
and it forces us to think about not being able to call the methods
during initialisation given the base file will execute after the
main one, making its methods not yet available during the split
time between the two.

From the perspective of regular modules, this changes nothing
as it will still load as one atomic unit defining the same public
API as before.

Bug: T192623
Change-Id: I552ae02e49c4d30070e166a504f454e334e8e75e

resourceloader: Improve coverage of ResourceLoader::respond()

- Cover case of simple module load.
  The bulk of this use case is already covered by a lower-level
  test for makeModuleResponse(). The added case here exists
  to cover the wrapper method, ResourceLoader::respond().

- Cover logic for catching and logging internal errors.

Change-Id: I4315bb00137ff80ee2b790c6b4d4b5fbd93f6bc1

Special:Preferences/reset: Always use OOUI not the deprecated vform

Though it's "nice" to show this as a big red button to users, sadly
(a) vform is deprecated and shouldn't be used, and (b) the format's
restriction on button width intereferes with the length of the sole
control on this form, even without the prolix override expansion of
for which wikis' communities are wont to do.

Bug: T195977
Change-Id: Icc294bde1d3ed72837e7152003a2fbd522c9067d

Update OOUI to v0.27.1

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

Depends-On: Idc156553fda9c5137d7e25a3d7a3c61837b4cc3a
Change-Id: I993a251172d1a238a5d2a52a20957afc659521e8

Merge "resourceloader: Remove use of $.params in mediawiki.js"

resourceloader: Remove use of $.params in mediawiki.js

Bug: T192623
Change-Id: I05ce4a05fda5ae13ad85c3d8da8362488b5d46af

Merge "Message: Only clear message cache when set language is different"

Message: Only clear message cache when set language is different

Change-Id: Ic0d810b9017b2cc693282807540ddf3e4d952b9d

Merge "ResourceLoaderLessVarFileModule: Add parameter 'lessMessages'"

rdbms: do not silently rollback empty transactions on error

Since there might be important view snapshots, temp tables, or effects
from SET statements or the like, go into TRX_ERROR state for "possible
transaction level errors" even if no recognized writes took place and
the transaction was not explicit.

Change-Id: I32c34bc28b845e343d0167a220412824838eaed8

Merge "Split ApiUsageException and UsageException to class per file"

Merge "Fix PreferencesForm alias"

Merge "Split ApiMessage to class per file"

Merge "Add @deprecated tags to DatabaseBase"

Merge "Don't autoload classes in AutoloadStructureTest"

Fix PreferencesForm alias

Change-Id: I6f24c6283b4165b9209e53f8fc3b8ecb5feb9ca4

Add @deprecated tags to DatabaseBase

It was there in REL1_27, but REL1_28 Database was the default...

Bug: T195576
Change-Id: I06d3a48c3112da95f38bb7f6a055fe7682e03096

Call Database::timestamp in some tests

Also add non-optional database fields to insert.
The fields are defined with a default in mysql,
but not in postgres or oracle.

Bug: T195807
Change-Id: I201638e3438cb0e5004f8da121e64a770e6a8fca

Add @deprecated tags to various class_alias calls

Bug: T195576
Change-Id: I10cd8415891bfe4a278eee06c9cfe905b3e036dc

Localisation updates from https://translatewiki.net.

Change-Id: Ia47406929fcf2e288dbc4621ae6cf64ad2c3d070

Merge "Xml.php: Do not escape double quotes in $contents of Xml::element()"

Merge "API: ApiBase::getParameter() shouldn't throw on other params' errors"

Don't autoload classes in AutoloadStructureTest

Autoloading classes is a 100% accurate way to ensure the autoloader
worked, but there are cases where if optional dependencies aren't
installed, then autoloading the class will fail. We can re-implement the
logic behind the PSR-4 autoloader, and ensure that classes will be
autoloadable by turning the filename into a class name, and making sure
that class name is the one we found in the file.

Bug: T195823
Change-Id: I5df378180e567c257386482383ef73812592f989

Merge "OOUIHTMLForm: Improve HTMLCheckMatrix for further clarity"

Merge "Remove deprecated wgResourceLoaderLESSImportPaths"

Merge "Improve ExtensionRegistry test coverage"

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

OOUIHTMLForm: Improve HTMLCheckMatrix for further clarity

Increase distance between rows slightly and use white to
emphasise row `:hover` color contrast.
Also
- amend `padding` to full pixel widths and
- center-align all but the checkbox labels for better orientation.

Bug: T194536
Change-Id: I03e193beaa8c6479a710056d095cb79098544389

Remove deprecated wgResourceLoaderLESSImportPaths

The last remaining users of this feature were MobileFrontend and Minerva,
which have been migrated to direct imports.

Bug: T140807
Change-Id: I1a66a2ad314bde332297798520e5ec3e0e3d4c9b

Merge "resourceloader: Add coverage for empty case in makeModuleResponse"

resourceloader: Add coverage for empty case in makeModuleResponse

Adds coverage for line 1049-1056.

Also, follow-up 6292d54dff by simpliying the regex by using /s modifier to
enable PCRE_DOTALL which includes matching of new lines.

Change-Id: Icec34dfe107d418951b3d155234295c79410ffaa

Merge "Fix help link on Special:PasswordPolicies"

Merge "Complete coverage of Parser::getTemplateDom()"

Fix help link on Special:PasswordPolicies

'passwordpolicies-helppage' shouldn't be configured in Git - it's
intended for wikis to set a local override. Instead, set the proper
destination in the addHelpLink() call itself.

As a sidenote, a MediaWiki configuration setting documentation page
isn't the best place for user-facing documentation, a proper help page
should be created in the future.

Bug: T195799
Change-Id: If47b86bbd14f05235fa1cb8c910e2362400d23ac

API: ApiBase::getParameter() shouldn't throw on other params' errors

This regression was introduced in Ia19a1617b7.

Bug: T195777
Change-Id: I1e1eb3861ced83f79e56d2325ab693ef4e393999

Merge "mediawiki.action.edit.styles: Fix alignment of "Cancel" link/button"

Localisation updates from https://translatewiki.net.

Change-Id: Ied787b7f07a91cabbe26a85d04e2ed730444637e

mediawiki.action.edit.styles: Fix alignment of "Cancel" link/button

Follow-up to f6afdf96d93713981b6bbe4ceeb8363cfc491803, which
accidentally caused it to have the margin applied twice.

Bug: T195673
Change-Id: I7c0ad98e77bf9156c623dcc5b592de52f8e87df8

Merge "Send a cookie with IP/IP-Range blocks when blocking logged-out users"

Merge "resourceloader: Include global LESS variables in LESS cache key"

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"

Improve ExtensionRegistry test coverage

Change-Id: Id1b87e020a220c96b42d86485da67d47b9d565f9

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

OutputPage: Remove 'X-UA-Compatible' header (was for IE8-10 JS compat)

Save 26 bytes each request.

We're not sending JS to those browsers any more, they're dying out, and
apparently just the `<!DOCTYPE html>` is enough to trigger Edge mode.

https://stackoverflow.com/questions/26346917/why-use-x-ua-compatible-ie-edge-anymore

Bug: T27378
Change-Id: I5418d4b043a8c8669886c39a93732e4a7d50337e

Merge "Improve Parser::braceSubstitution() coverage"

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