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

Merge "Block: Explicit convert Message to string"

installer: Get WebRequest in mw-config via RequestContext

Change-Id: I535a91d09cc74ab08b782973a946dcf3fc3e6b46

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

Merge "SVGMetadataExtractorTest: remove method existence test"

Merge "Installer: Fix an obsolete comment"

Merge "Remove $wgMediaInTargetLanguage"

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"

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

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

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()"

build: Remove failing HHVM test from Travis CI config

This is the only version of HHVM that MediaWiki still supports,
and while it is passing on WMF's Jenkins, its redundant counterpart
on Travis has been failing for months.

It doesn't appear to be adding value in its current form,
and it's unlikely that after failing so long, that someone would
invest in making it pass before support for HHVM is dropped later
this year (in master, which is the only branch we test in Travis).

It's also delayed feedback by a third (~10min -> 15min), because
it seems to be much slower than the others (unsure why).

Change-Id: I89bd09cb04afde89c8e8ec9a06f69ecb4b094714

Merge "UserTest: correctly reset fake time"

Merge "Add @var to Pager classes"

Merge "Added Id to the input box"

rdbms: undeprecate DBReplicationWaitError to align with current use

Change-Id: I2a180558b1a8cfaccf72bec7a809158de25eebc4

Merge "Update OOUI to v0.30.4"

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

Given that the DB classes already check read-only mode and bad
transaction states are already handled by 3975e04cf4d, this
code was not useful.

Change-Id: I4b6e8909196448a12cf74598de0272fa43bdfc62

NamespaceMultiselectWidget: Allow filtering by namespace ID or label

If the user types in a namespace ID, the menu should display the
corresponding namespace value, and allow its addition.

Depends on I9bc411e844d16034d8 being available in the OOUI release.

Bug: T216078
Depends-On: I9bc411e844d16034d86e0f8ed28fa20356a78bdf
Change-Id: Ifa113241b391a32ce64c0c2b1a6d55d1ef64de97

UserTest: correctly reset fake time

Change-Id: I5fc64506781fbc40e81719aa03441643c38d4e54
Follows-Up: Ica3e6e7165496bdc9b8f12972cf93847ecfffa50
Bug: T216631

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

This makes it consistent with set() and merge(). Also, one subclass
was already using the field in this manner.

Clean up the code related to WRITE_SYNC in SqlBagOStuff.

Change-Id: I0fb84f4475311889507d3ef98afd4476fb81174f

Localisation updates from https://translatewiki.net.

Change-Id: I28265dca10ec71e558c1df9135430c9d9c498a4e

Merge "Use short assignment operator in PHP"

objectcache: fix comment typo for addBusyCallback()

Change-Id: I05a4d84d40b81ba25b89238b36cdb64c71e2de1e

Introducing User::getLatestEditTimestamp()

This will be used by the GrowthExperiments
Mentorship module to show when a user made their
latest contribution.

* Introduce a private function to reduce duplication with
  getFirstEditTimestamp()
* Add unit tests for both

Bug: T216631
Change-Id: Ica3e6e7165496bdc9b8f12972cf93847ecfffa50

Merge "Normalize "LEFT OUTER JOIN" to "LEFT JOIN" in SQL queries"

Add @var to Pager classes

Some are needed to make better results with phan in extensions

For example:
File CentralNoticeCampaignLogPager.php line 12:
Assigning array{0:20,1:50,2:100} to property but
\CentralNoticeCampaignLogPager->mLimitsShown is
array{0:20,1:50,2:100,3:250,4:500}
[PhanTypeMismatchProperty]

With "@var int[]" the type is correct

Change-Id: Ic68910bf17344852ad11fcc000a47891e4bf0179

Update PHPDoc types in several maintenance scripts and related

There is like a billion of these in our core codebase. I would love to
update them all. But I don't have the time, at least not to do them all
at once.

TL;DR: It's an improvement. :-)

Change-Id: I0a9c51bdcf0f261f971586bb8703f418324b0f98

Merge "Remove various references to cURL in code comments"

Update OOUI to v0.30.4

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

Bug: T92026
Bug: T145674
Bug: T206729
Bug: T213884
Bug: T216780
Bug: T217014
Bug: T217213
Bug: T217381
Bug: T217790
Depends-On: Id50f068908accd022c7847c40795286c32199b4e
Change-Id: Id50f068908accd022c7847c40795286c32199b4a

Use short assignment operator in PHP

Use
  $var .= $foo
instead of
  $var = $var . $foo

Change-Id: I5dcdd7278e618c14968e5ac1fb8ea43ac2200deb

Merge "Report logs for each individual test failure"

Merge "Title: Minor clean up for stricter checks, casting style, and doc types"

Report logs for each individual test failure

The initial implementation of reporting logs with test failures was
incorrect, it always reported the logs of the most recent test run.
Attach logs to the Test when a failure is reported and pull them
back out in the result printer.

Bug: T217489
Change-Id: I5aa55d6fa7a7ec03a2e71636b6b0366ea40605cb

Merge "Use the .= operator to concatenate a var with another"

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

Doing so can only result in occasional needless set() call rejections
or automatically lowered TTLs.

Change-Id: I2f007ba88e8f9c9f1ac3cd3d2daf3cd2289e315a

Make IndexPager query direction code more readable

Rename $descending variable in doQuery() to $order, along with the
$descending argument to reallyDoQuery() and buildQueryInfo(). Use
new IndexPager::QUERY_* constants for checking and inverting such
values. Fix the comments so that they do not imply the opposite of
what is true anymore.

For compatibility, the constants are boolean values such that any
subclass defining reallyDoQuery()/buildQueryInfo() can use that
argument the same way as before.

Change-Id: I912d3678c755c25463a2fadbec6888f3a87d4215

Localisation updates from https://translatewiki.net.

Change-Id: I337a2609e45ccd9b3bb71c7d60e3847e320b5db6

Use the .= operator to concatenate a var with another

Rather than using $var = $var . $foo, to avoid the redundancy, use
$var .= $foo which is a valid PHP syntax and popularly used.

Change-Id: Idbbdb31a7b5561ed97f9ba0f05f6ac78c9419f82

Merge "ActiveUsersPager: Fix ordering and return 0-action users"

Normalize "LEFT OUTER JOIN" to "LEFT JOIN" in SQL queries

Change-Id: I4de7f0527d9e16f9128ef3b24567511d4c74245d

Merge "Stash WatchedItem changes so that the jobs run from the queue"

Merge "resourceloader: Remove wgAvailableSkins from mw.config"

Merge "Normalize use of "INNER JOIN" to "JOIN" in database queries"

resourceloader: Remove wgAvailableSkins from mw.config

Added in 2011 for use in QUnit tests (r80790, r80792).

That was a terrible idea in hindsight, which I removed again in
2015 with 6b758fc982972e, and 0f9e4ca0fb7c.

If a need for it were to arise in the future, we'd export it
within the file module that needs it, or via addJsConfigVars()
on only the pages that need it – not globally.

Bug: T217772
Change-Id: I3885cacaa9b33e6947dbaf26f9b6839e19588603

ActiveUsersPager: Fix ordering and return 0-action users

Ia9d2ff00 introduced two incompatibilities with what the base classes
were trying to do:

* The $descending flag passed to buildQueryInfo() does not actually mean
  DESC if $this->mIsBackwards is set. The old code just blindly assumed
  that was the case.
* For paging to work right, we must return all $limit rows, we can't
  filter out users with 0 actions. Thus we need to LEFT JOIN
  recentchanges (and move some conditions around to cope)..

Bug: T217525
Change-Id: Iba105a31ff61fbb14931bf8903538bfe7b0ba48b

Merge "resourceloader: Remove ResourceLoaderMaxQueryLength from mw.config"

Merge "resourceloader: Use local var for getConfig() calls in StartUpModule"

Merge "Fix HeaderCallback failing on headers without a colon"

Merge "Add help text to "partial" and "sitewide" options on Special:Block"

Normalize use of "INNER JOIN" to "JOIN" in database queries

The ANSI SQL default join type is INNER and this might save
some line breaks here and there.

Change-Id: Ibd39976f46ca3f9b71190d3b60b76ca085787a00

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

Bug: T204742
Change-Id: Ibab189c8e0dee5e840770bdb0336516fdfc75e4b

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

Change-Id: I3dc707af53e480b27b7349aca53292f3bb26c45a

Merge "Make LinksUpdate::getPropertyDeletions private"

Merge "Add more documentation to DeferredUpdates::doUpdates"

Add help text to "partial" and "sitewide" options on Special:Block

Describe the differences between sitewide and partial blocks under the
respective radio options.

Bug: T213101
Change-Id: Ibf098c6becaf4824f8da2cade22e23f4ce00283b

resourceloader: Remove ResourceLoaderMaxQueryLength from mw.config

This is only used internally. It doesn't need to be exported
as part of mw.config.

Change-Id: If4206d797b5d426eb999c34ad5dde3ba141c6f05