Add @var to class properties to clarify types

Change-Id: Ia6e9eb70c68b5922077c8e5940b436844fa36e25

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"

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

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

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

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

Change-Id: I53af241454872fd228869b45dc74175c56026d6e

Merge "Hard deprecate MWNamespace::canTalk()"

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

These improvements touch areas around cleaning up deprecated functions
such as intval(), strval() into casting with (int), (string), etc.

If such improvements is welcomed into the Title class, I can make more
of such but for now, I've just made a few, like 1/20 of these kind of
changes in the file so that, if it's merged and encouraged to continue,
I'll cleanup the file. Or as proposed on this patch, phpcs rules can be
used to handle such cases.

Change-Id: Ib90e880cb124e765379ddad531d4c89289d364de

Merge "user_group, the nonexistent table that keeps on giving"

Make LinksUpdate::getPropertyDeletions private

Change-Id: Iec1ae929566e25a278ef918f9f6b22343dce6cec

user_group, the nonexistent table that keeps on giving

Follow-up to 27c61fb1e94da9114314468fd00bcf129ec064b6.

Bug: T199474

Change-Id: Ie8e054f5898209c51538669149e966bee7754f1e

Revert "Convert MultiHttpClient to use Guzzle"

That breaks on Wikimedia beta cluster (T217733):

Warning: Invalid argument: option: 6 in
vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php on line 56

Warning: Invalid argument: option: 6 in
vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php on line 56

Warning: Invalid argument: function: not string, closure, or array in
vendor/guzzlehttp/guzzle/src/Handler/CurlMultiHandler.php on line 108

Fatal error: Uncaught exception 'ConfigException'
Failed to load configuration from etcd: cURL error 23: Failed writing
header

This reverts commit 1e048a08b565ae909e85465f8b09a27ed8480ce2.

Bug: T202352
Bug: T217733
Change-Id: I2384355043896128d3f191941e8da00fdc62361e

Add more documentation to DeferredUpdates::doUpdates

Change-Id: Id2f1fb624d03943e5eb4bed49d3435cf9c99fc25

Merge "resourceloader: Minor clean up in saveFileDependencies()"

resourceloader: Minor clean up in saveFileDependencies()

* Use early return instead of all-encapsulating conditional.

* Document why the try/catch is so big.

Change-Id: Ie19e18556e7ac0a12ad6b979367f8c6b786bbe31

Remove various references to cURL in code comments

Given we now use Guzzle as well, and don't require cURL for any
features at a basic level, our code should not be as assumptious
about the backend being curl. Such assumptions may be wrong
and lead to confusion and/or bugs.

Bug: T137926
Change-Id: I6ad7f76768348e1eb8c1fb46c8125cce9285dc22

Merge "Fix a rather fatal typo in rebuildrecentchanges.php"

Merge "API: Avoid duplicate logs to api-feature-usage"

Fix a rather fatal typo in rebuildrecentchanges.php

The JOIN condition was being ignored because there is no table called "user_group" in MediaWiki core.
Thus if and when using $wgSharedDB, the query would end up listing *all* registered users from the shared user table.
And even without $wgSharedDB, running rebuildrecentchanges.php would result in everyone's edits being marked as bot edits (recentchanges.rc_bot = 1) and thus hidden from the Special:RecentChanges page.

Thanks to Lcawte for reporting this bug.

Follow-up to 27c61fb1e94da9114314468fd00bcf129ec064b6

Change-Id: I18d658b67c50f2200341f732783c2e7524dd27f1

Merge "Block Special pages only if the user is sitewide blocked"

API: Avoid duplicate logs to api-feature-usage

It can wind up logging deprecations twice if extractRequestParams() is
called with different values for 'parseLimit', for example.

Change-Id: I921777089fb8cfb4339af6fd08ee3475ed31b7f6

Localisation updates from https://translatewiki.net.

Change-Id: I82b2f9fbde2a8c7524bd2de7ee207c829941a2cf

Merge "Clear cached HTML artifacts"

Merge "Convert MultiHttpClient to use Guzzle"

Merge "resources: Provide the moment gom-Latn i18n to gom as well"

Merge "Specify licence of mediawiki ontology.owl"

Convert MultiHttpClient to use Guzzle

Convert MultiHttpClient to use the Guzzle library.
Guzzle includes built-in support for concurrency, and automatic
fallback to php streams if curl is unavailable.

Bug: T202352
Change-Id: I703af901f9da33d20b5e0989941f3f7fd6609298

Merge "resourceloader: Remove addModuleScripts, and deprecate getModuleScripts."

resourceloader: Remove addModuleScripts, and deprecate getModuleScripts.

The addModuleScripts() methods were deprecated in 1.31 and 1.32,
these are now removed.

The getModuleScripts() are now deprecated as well, always returning
an empty array. To be removed in 1.34.

Depends on commits for bundled/wmf-deployed extensions that
remove the last few remaining callers to the deprecated functions
in: 3D, Collection, Flow, GlobalUserPage, and Wikibase.

Bug: T188689
Depends-On: If9f0bc6aef85117587fa1929f34f8861c8d80314
Depends-On: Ia8d41b97fbf6822f5f8f7ac889408acce1ac9a3a
Depends-On: I503b919739ea474ff33726815b0da55e2f7e2724
Depends-On: I236ef637fd03b810a46eb361e25067a037e9d183
Depends-On: I62e17779753b977a452cc0c9694947941e999cc3
Change-Id: I5a19b8f164ccf666485d2971202194b747f882df

resources: Provide the moment gom-Latn i18n to gom as well

Bug: T191339
Change-Id: Icf7556a1fa71e12dd1029765f81914a36a3c1c22

Merge "objectcache: optimize WAN cache key updates during HOLDOFF_TTL"

Merge "Remove references to obsolete special page and messages"

Merge "Various small code and comment cleanups to IndexPager"

resources: Remove deprecated 'jquery.hidpi' module

Deprecated in 1.32 and has no further purpose in its current
form as a jQuery plugin for <img srcset>, which we now use
natively without fallback.

The remaining logic for bracketed window.devicePixelRatio is
simple enough to inline as needed without the cruft and overhead
that comes with a centralised approach.

Bug: T202154
Change-Id: I729dfabcbb40a0a794d6b166a584f45a64ac0338

Merge "logging: Remove 'null' return from ManualLogEntry::getTags()"

Various small code and comment cleanups to IndexPager

Change-Id: I351c12d51468bd428207017929fec1e796097206

Drop AuthPlugin and related code, deprecated in 1.27

Bug: T215843
Depends-On: Ie49709faa6f67b0a31bd546823d29dbfea0a70eb
Depends-On: If8109dbfdeeb88a6265f4cd79d1ae81bff48d934
Depends-On: I36b195fa2d6c23a76631ebaf869c787e44427d60
Depends-On: Iaa171b085fde331eab7d85c7de74523e27fc625a
Change-Id: I330b30d6582034a233fed204c0680b1ce84eec6e

Merge "Localisation updates from https://translatewiki.net."

Merge "Recentchanges: Hide instead of disable namespace selector checkboxes"

Merge "Set api-request log http.request_headers properly"

Set api-request log http.request_headers properly

$request->getHeader( 'Api-user-agent' ) was returning false,
and we can only accept string header values.

Bug: T214080
Change-Id: I3de8a455ad7dab9e3a67e24439e95f4968c93243

Localisation updates from https://translatewiki.net.

Change-Id: If706f62773b164f4c26a0eedb1e6709971c77b17

Clear cached HTML artifacts

Follow up to I6aad563e48f41c783df8b176a4f437e60a1255cc
Can be merged when that patch has been in production for a week.

Change-Id: I9f6b523f3ece055f8e6e45dfbfeb9852bebe4772

Remove outdated reference to design.txt

docs/design.txt was removed half a year ago in change I85be15a68f
(commit 6760216e10), and was long outdated even then, so let’s stop
pointing people at it.

Change-Id: I165ff1e12fb79c480930289d25833992be61b891

Merge "resourceloader: Remove warning for handled localStorage error"

Block Special pages only if the user is sitewide blocked

Update the default implementation of FormSpecialPage::checkExecutePermissions()
so that a Special page is only blocked if the user has a sitewide block.

This change allows the user to continue performing critical functions (like
resetting their password) even if they are partially blocked.

Bug: T209097
Change-Id: I5190297b7b235b6ebbdfa522323ce9bbd46b6729

Allows to add subsections to special pages groups on special page "Specialpages"

Adding all pages into one section creates some inconsistencies where users
cannot clearly distinguish pages. Thus extensions and extension suites like,
e.g. semantic extensions, translation extensions or social profile extensions
can improve the situation.

Bug: T216467
Change-Id: Iddc28e040fe975f8f823c84919fa476d2a13af0a

Merge "Document the 'sitewide' option for the Block class"

Specify licence of mediawiki ontology.owl

Bug: T216842
Change-Id: Ide976016bc5d421458cfe4ad661d88bb32da61a6

Merge "Fix typo in Priority Hints static variable name"