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

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"

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

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"

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

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"