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

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"

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

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"

Split ApiUsageException and UsageException to class per file

Change-Id: I54c7d5e6616ac1a23082cc72e60ed92cb7eef715

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

Split ApiMessage to class per file

Change-Id: If8f51e6535d0ec6e35336a6702c5e35be35bcf09

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

rdbms: make runOnTransactionIdleCallbacks() reset DBO_TRX on exceptions

Change-Id: Ibbb2a3ebf9dd970772ee704aa643a3843f20a3b5

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'"

resourceloader: Include global LESS variables in LESS cache key

This prevents cache churn when the wiki-global LESS variables vary
between wikis because the cache key is used as a "global" instead of
db-local. This is good for the common case, but should still explicitly
vary if the vars differ between wikis.

Bug: T191937
Change-Id: If12fd07a7062792205384150d6f5fd9a83f996cc