Throw exception on wrong type in Language::isValidCode( $code )

This prevents that Language::isValidCode( null ) returns true.

Depends-On: I38d8358469903afb15331212206635283cedcd46
Change-Id: I5dc5bd45d648a548dfd27aaa1b3c29a2e99f0e8b

Site: Explicit allow setLanguageCode( null )

$site->setLanguageCode( null ) is already used in
DBSiteStore::loadSites and this worked because
Language::isValidCode( null ) === true.

This change explicit check for null and allow to set null without
depending on Language::isValidCode( null ) === true.

Change-Id: I0c9fe03cbf1434b0340e61e96360c0aaeb0fe89d

Language: Use Assert::parameterType for type checks

Depends-On: Ib0cb388958cf720052ca6df73d5145b7ab3f9529
Change-Id: I8287118cf8ec01326ead9c90a38f7b4134c3eba9

Merge "Remove deprecated ParserOutput::legacyOptions"

Merge "Improve documentation of the PageContentLanguage hook"

Improve documentation of the PageContentLanguage hook

Clarify that the type of second parameter that is being passed
to the hooks can be anything because one hook can change it and
then it is passed to another hook without the hook caller having
possibility to check or modify the value.

Clarify that hooks should only return Language objects.

Rename $wgLang to $userLang in the hook parameter documentation to
avoid false posivite matches for the global.

Fix some typos, use Title::inNamespace and add a test assertion.

Also, the $content parameter is unused by all implementations of
this method, and on quick look never passed by any caller. I kept
it for now, however.

Bug: T214358
Change-Id: Iae49d2998c2b762565d232c0337d84d43a4a900c

Remove deprecated ParserOutput::legacyOptions

Change-Id: I5f32dd741f3ee795ec599aacb687d5cee2c52835

Merge "jobqueue: allow direct server configuration arrays to JobQueueDB"

Merge "Add sanity check to notifyUpdatesForRevision() for $oldRev"

Merge "resourceloader: Reword and expand doc comment about FileModule "packageFiles""

resourceloader: Reword and expand doc comment about FileModule "packageFiles"

Change-Id: Id9d68205504ecbddea6ca248e108f368fdcac2d2

Merge "logging: Start using LinkTarget & UserIdentity in ManualLogEntry"

Localisation updates from https://translatewiki.net.

Change-Id: I7ee423c084fbe33d4731e43ce5506f6a6f0c40be

Merge "Fix the order of the parameters for htmlform-title-badnamespace"

Merge "Bump minimum SQLite version to 3.8.0"

Merge "Fix small typos"

Merge "Linker: Use RequestContext::getMain()->getLanguage() instead of $wgLang"

OutputPage: Use $this->getUser()

Use $this->getUser() instead of $this->getContext()->getUser().
ContextSource::getUser is a shortcut for $this->getContext()->getUser().

Change-Id: I7d19116b634500cd1a435bebe235c7562889f7bb

Merge "Avoid using stale data for revision visibility and actor data"

Merge "Streamline code involving .= string concatenations"

Streamline code involving .= string concatenations

This was inspired by Idbbdb31. Originally, I did a regex search for
code that did string concatenations like `$str = $str . …` and replaced
them all with the .= operator. A duplicate patch was uploaded by another
author. I rebeased this patch on top of the other, which leaves all
the manual optimizations I did.

Change-Id: Iaeb73d9c63302c9409bd1051b91e0d2bd77788a7

Bump minimum SQLite version to 3.8.0

3.3.7 is 12 years old, no chance in hell you will find it somewhere
that supports modern MediaWiki. Since Ubuntu 14.04 LTS has 3.8.2, this
seems like a reasonable conservative version.

Change-Id: I5fbb35bd2d4c0993889b79f463a3c35ce6765d3b

Merge "ProtectionForm: Add type hint to ::showLogExtract() header"

Localisation updates from https://translatewiki.net.

Change-Id: Ibf7f5a6cc435b330df709d40f8ea0687c65832c5

Avoid using stale data for revision visibility and actor data

Created class RevisionStoreCacheRecord to avoid returning stale
cached revision visibility and actor data when changes were
made after the cache was populated for that revision.

Bug: T216159
Change-Id: Iabed80e06a08ff0793dfe64db881cbcd535cb13f

Merge "mediawiki.cookie: Export config via packageFiles"

ProtectionForm: Add type hint to ::showLogExtract() header

https://codesearch.wmflabs.org/core/?q=showLogExtract%5C(&i=nope&files=&repos=
according to codesearch shows that this method takes as first argument an output
page object and also, objects are passed by reference by default so removed &.

Change-Id: I167ae6300b37d4e9292bd8ad14351f1ca35184d2

Remove a few method_exists() checks

These functions are present in every PHP version we support now.

Change-Id: I11add0fc9f6c3c3caff859db9c0475c4d2818577

Merge "installer: Code augmentation in wfInstallerMain function"

Merge "Add @var to class properties to clarify types"

Merge "Move class HistoryPager to own file"

Merge "Move class RangeDifference to own file"

Merge "Remove suppress of PhanRedefineFunctionInternal"

mediawiki.cookie: Export config via packageFiles

* Add a private setDefaults() method to allow mocking from
  unit tests.

* Use matching keys between the data export and the API.

* Reduce duplication in the code.

* Access the Sinon stub explicitly instead of via the public path.

* Remove use of the now-redundant QUnit.newMwEnvironment().

Change-Id: I600332cdb738f0b443549948b9f070f3ccfa12aa

Merge "maintenance: Replace deprecated global $wgParser with getParser() service"

maintenance: Replace deprecated global $wgParser with getParser() service

$wgParser is deprecated and using globals should also be avoid. So, replacing
the use of global $wgParser by a mediawiki service.

Change-Id: I34b61338e500567773249c71da0a7125cfd50d02

Merge "installer: Remove redundant $key variable"

installer: Remove redundant $key variable

Change-Id: Ie6d59a1fd94ccfb2a3b5248f1f98ae640a4d5c2c

Merge "NamespaceMultiselectWidget: Allow filtering by namespace ID or label"

Add sanity check to notifyUpdatesForRevision() for $oldRev

Change-Id: Ibd8975efb33d061ab16dc1a59899b6f78976ec5d

Merge "Block: Explicit convert Message to string"

installer: Code augmentation in wfInstallerMain function

Cleanup null checks and string literals in wfInstallerMain function.

Change-Id: Ib117e05faf899b0c46820d337f1bd3ebd4272419

Fix the order of the parameters for htmlform-title-badnamespace

The name of the page is the first parameter, and the namespace is
the second parameter.

Change-Id: I738cdd920d0ce641837c5dc1da391602a15df794

installer: Get WebRequest in mw-config via RequestContext

Change-Id: I535a91d09cc74ab08b782973a946dcf3fc3e6b46

Remove suppress of PhanRedefineFunctionInternal

There is no longer an override

Change-Id: Ie7c99228135689d20ab6ffb287c978642851f3ee

Merge "mediawiki.cookie: Move file to its own directory"

Merge "SVGMetadataExtractorTest: remove method existence test"

Merge "Installer: Fix an obsolete comment"

Merge "Remove $wgMediaInTargetLanguage"

jobqueue: allow direct server configuration arrays to JobQueueDB

This is useful for using a separate sqlite DB.

Change-Id: I6a7bb1d4bddedca34d68b0d8460c96c6ef5bbcf6

Installer: Fix an obsolete comment

Change-Id: I0ae459b016f491ff3bc2c702e55be29fdcfec5eb

mediawiki.cookie: Move file to its own directory

In preparation for using RL packageFiles functionality, and
to make sure git can track the file, which it seems to be unable
to otherwise.

Change-Id: I966718db2028a45c3dce1c2c10218e5137e23c7d

Merge "Make PurgeJobUtils::invalidatePages avoid waiting on replication for no reason"

Make HTMLCacheUpdateJob avoid waiting on replication for no reason

Change-Id: Ica381dad0bdb0555add1c7925e0e48991e7e0964

Make PurgeJobUtils::invalidatePages avoid waiting on replication for no reason

Change-Id: I9ea4e4034ffc30a14b138279515447f990807e07

SVGMetadataExtractorTest: remove method existence test

It should be present in everything we support, as the earliest libxml
version would be 2.9.1 while >2.6.20 is required.

See https://3v4l.org/LdTIr

Change-Id: Ibc7152e9224ef8d5803d0e0ff8314cdf9eb8eac4

Merge "Make addIdentifierQuotes part of IDatabase"

Make addIdentifierQuotes part of IDatabase

This is so it can be used in places that typehint an IDatabase,
ultimately so that I can make phan-taint-check happy.

Bug: T216348
Change-Id: Iabf3e9a6add380322a12cdbb6a337bad690f1370

Merge "objectcache: Use constant for $ttl param default in WANObjectCache::set()"

objectcache: Use constant for $ttl param default in WANObjectCache::set()

Change-Id: I3f936088cd63af4d535d7f46fe5619e8bb866999

Merge "rdbms: optimize Database::selectDomain() to avoid extra queries"

Merge "Remove `list-style-type` declaration from elements.css"

Remove $wgMediaInTargetLanguage

It's a temporary feature flag not included in any release, just
removing it outright. The functonality will now be always enabled.

Bug: T205040
Change-Id: Ia9da82e6f6b2d270f1790a99fc8c35ad5e6aee5e

Merge "resourceloader: Omit default 'debug=false' from load.php urls"

Merge "resourceloader: Separate context from extraQuery in ClientHtmlTest"

Merge "ApiQueryLogEvents: Add IGNORE INDEX to avoid MariaDB optimizer bug"

Merge "resourceloader: Replace ResourceLoaderDebug config use with context"

Move class RangeDifference to own file

Change-Id: I5262c7484017a0d920cc24297515d5946f614ee6

Remove `list-style-type` declaration from elements.css

This doesn't make any sense here, as
a) we should apply `disc` in alignment to Design Style Guide
b) it's overwritten anyways in both, Vector and MonoBook, with
   different styles

Bug: T151100
Change-Id: Ied6810a78d8fbe5371838921cda468e6873d2e57

rdbms: optimize Database::selectDomain() to avoid extra queries

Also clean up and align the mysql subclass versions of the method.
Enforce that raw "USE" queries are not passed in, since they would
break the tracking (even before this change).

Change-Id: I11c9145c6c0525f27a4ec8d94c500a22a712b320

Merge "jquery.tablesorter: Optimise getElementSortKey() code"

Merge "Move CacheDependency classes to own files"

Merge "objectcache: add metrics for WAN cache deletes and check key touches/resets"

Merge "objectcache: move ERR_* constants to IExpiringStore"

resourceloader: Omit default 'debug=false' from load.php urls

Change-Id: I2a2f92b0f0438420105e6a4e4d97eb5f8c480917

resourceloader: Separate context from extraQuery in ClientHtmlTest

The test was dual-purposing the same array both to create
a ResourceLoaderContext object, and to pass further down as
$extraQuery parameter.

This meant that ResourceLoaderClientHtml::makeLoad, and indirectly
ResourceLoader::makeLoaderQuery, did not have any control over
the formatting of the query string, due to it forcing the key/value
pairs regardless (in addition to being redundant).

This is needed in order to omit debug=false by default, which is
the next commit.

Change-Id: I3c098a9a5055ec8f2b64319c6cadd13c030ba76a

Move CacheDependency classes to own files

Change-Id: Icacf36674f10fa19a09499d68d596d5e2cc235f0

objectcache: remove BagOStuff::modifySimpleRelayEvent()

This was only there as part of EventRelayer use by WAN cache, which
has since been removed. No other usage really makes sense.

Change-Id: I201425bb88739dd8febbfbc3d24a15e6d3d88d9c

objectcache: move ERR_* constants to IExpiringStore

Also removed unused ERR_* constant from WANObjectCache and fixed
comment in the constructor.

Change-Id: I4377fc3f533113a459b648fad084ba097ff51762

jquery.tablesorter: Optimise getElementSortKey() code

* Document why this uses $().data() instead of $.data(),
  as the rest of this file does.

* Move single-use $() to where it is needed.

* Use native Image.alt directly instead of going via jQuery
  and attributes. This also obviates the need for a default,
  as the DOM handles that already.

* Remove needless $.makeArray(), use $.map() directly which
  supports the same array-like objects as $.makeArray().

* Use native elem.textContent directly for text nodes,
  instead of $(elem).text().

Change-Id: I92da50e1c01f11585d2bac4f01dba1f671444a63

Localisation updates from https://translatewiki.net.

Change-Id: If38e8f3671ef5a53006836c5ac14bbadc6930098

Move class HistoryPager to own file

Change-Id: Ie12e79e6f32225442b5bd794826f8c760b2c02d6

Merge "jquery.tablesorter: Remove dead node check in getElementSortKey()"

Merge "Convert a few call_user_func*() calls to native PHP syntax"

jquery.tablesorter: Remove dead node check in getElementSortKey()

In 2012, commit 39d1f3967be9d added special handling image elements,
and for Cite references. This inadvertently broke handling of sorting
for tables with colspanned cells, which have node = undefined in the
cache.

In 2013, commit e203fdfc74f18 restored handling of node = undefined.

In 2015, commit 7a5f4e46d10ea made colspan expanded in the cache
just as rowspan was already. Thus making this code dead.

It's odd to see $(node) before this check, and even a call within
this conditional. This worked, as long way to return ''.

Change-Id: Ia823a3ec29371be39fab3d626ed8647ee469951b

resourceloader: Replace ResourceLoaderDebug config use with context

Reduce our reliance on static state and configuration, and
propagate more state in explicit ways, through context, and
request parameters.

OutputPage creates ResourceLoaderContext and ResourceLoaderClientHtml
based on the configuration (via ResourceLoader::inDebugMode).

Everything within those classes should not need to check it
again.

* ResourceLoaderClientHtml:
  Already doesn't check MW config, but it's test was still
  mocking it. Removed now, and confirmed that it passes both
  with true and false. The individual test cases set
  debug=true/false as needed already.

  It's sets were previously relying on the accidental behaviour
  that within a unit test, we don't serialise over HTTP, which
  meant that a pure PHP boolean would survive. With the new
  raw `=== 'true'` check, this no longer works. Set it as a
  string explicitly instead, which is the only thing we support
  outside unit tests as well.

* ResourceLoaderContext:
  Remove fallback to MW config when 'debug' is unset.
  This is never unset in practice given that all load.php
  urls have it set by OutputPage based on ResourceLoader::inDebugMode.

  This change means that manually constructed ad-hoc load.php
  urls that are missing 'debug=' parameter, will now always be
  read as debug=false. This was the default already, but could
  previously be changed through wgResourceLoaderDebug.

When changing wgResourceLoaderDebug, everything will still have
debug=true as before. The only change is when constructing load.php
urls manually and explicitly not set it.

Bug: T32956
Change-Id: Ie3424be46e2b8311968f3068ca08ba6a1139224a

Move class JobQueueAggregatorNull to own file

Change-Id: Icb76f25eb9c2a26c91ba564a78edd4c07b49b944

Add @var to class properties to clarify types

Change-Id: Ia6e9eb70c68b5922077c8e5940b436844fa36e25

Block: Explicit convert Message to string

Without a converting method like ->parse(), ->escaped() or ->text() the
method __toString() is used which is a equivalent to ->parse().

This change switches from ->parse() to ->text() and let Html::element do
the HTML escaping, because the message 'autoblockid' is not needed to
parse.

Change-Id: I653906bdca0864989ffed5960505b96053456066

Remove deprecated Language::markNoConversion

Language::markNoConversion was hard-deprecated in 7de2c566d included in
1.32.

Change-Id: I2cc200d7a814e73d9c6138860190f44232a6bd3c

Merge "Check normalization rules of usernames during signup"

Convert a few call_user_func*() calls to native PHP syntax

Change-Id: I54d94f4369eb4fa0b0ebe892a1d6cc57b2bdb1f9

Merge "build: Remove failing HHVM test from Travis CI config"

Merge "objectcache: add $flags argument to BagOStuff::delete()"

Merge "rdbms: undeprecate DBReplicationWaitError to align with current use"

Merge "Do not bother with WAN cache set options for immutable text blob keys"

Merge "Make IndexPager query direction code more readable"

Merge "objectcache: remove obsolete code from SqlBagOStuff::handleWriteError()"

Merge "Make ApiStashEdit use a separate key for the parser output due to size"

Merge "objectcache: fix comment typo for addBusyCallback()"