Fix 'expliciit' typo at resources/Resources.php

Bug: T201491
Change-Id: Ic6c60aad037df3599572f7248093d7dc1509819d

Localisation updates from https://translatewiki.net.

Change-Id: I2316460e88bc758daa875638518105524f97ea1d

Merge "Escape messages ipb-sitewide and ipb-partial."

Escape messages ipb-sitewide and ipb-partial.

Radio button htmlform treats options as being raw html

Change-Id: Ia25ed5b88e937414ea7993a1cf29fe44ed8e22d4

Avoid rawParams() in Special:Emailuser

Bug: T216348
Change-Id: I6b543c508260f5889f1236dd43022a6a0fe963a9

resources: Document why mw.user has two seemingly unused deps

The code in the module 'mediawiki.user' does not depend on the modules
'user.options' and 'user.tokens' so the ResourceLoader dependency is
not necessary.

Change-Id: I22b9175f7623dbdf1c08826904b76dee5d56ea40

Merge "widgets: Remove redundant equality check in SelectWithInputWidget"

Merge "objectcache: remove EventRelayer dependency from WANObjectCache"

Merge "Fix documentation for Revision::getComment and WikiPage::getComment"

objectcache: remove EventRelayer dependency from WANObjectCache

The use of that class for WAN cache by third parties is extremely unlikely
and the approach was abandoned at WMF in favor of mcrouter (dynomite was
also considered preferable).

Change-Id: I328eb74f21c23db80acc5178bc3c9cf9f823da13

ChangeTags: Deprecate ChangeTags::purgeTagUsageCache()

The cache key that it purges is no longer used anywhere.

Change-Id: I2c255936451cde09579594cc4524c8fe38cf3e7c

Merge "Improve documentation for different block types"

Merge "Remove double check of $oldRevision"

Improve documentation for different block types

Clarify the purposes of Block::type and Block::getType

Change-Id: I64a2695f430c7718c504edb1fa6531a529c115cf

Prevent PHP notice on SpecialDeletedContributions

Bug: T208544
Change-Id: Ie8d5c3d7257134857713853eec8e0eb42890366a

Refactor calls to deprecated function Content::getNativeData()

Function Content::getNativeData() was deprecated.  Replace with
calls to new function TextContent::getText() in most places.

Bug: T155582
Change-Id: I2bd508c72aac4faf474ba45ab1f92e2e8d2eb9be

Localisation updates from https://translatewiki.net.

Change-Id: Ic40eaf0cedbbf6a81585f4f0d686f4aa3ae7d160

Merge "objectcache: refactor and simplify some WANObjectCache code"

Merge "Update required PHP version to 7.0.13"

Merge "Use "try-catch" block instead of "if" block to prevent interruption of new installation"

Update required PHP version to 7.0.13

Bug: T209423
Change-Id: I66e563adb062bc132a1092d78bfd06e2210f382e

Merge "jquery.makeCollapsible.styles: Avoid FOUC for collapse button"

Merge "EnhancedChangesList: Load style module via addModuleStyles()"

Merge "Add missing dependency on module 'mediawiki.notify'"

Merge "mediawiki.special.changeslist.enhanced: Convert CSS to LESS"

Add missing dependency on module 'mediawiki.notify'

watchlist.js uses mw.notify.

Change-Id: Iff59e01d96f3d8ffddd9ddaf9d3b4463c8cac8d1

Merge "registration: Allow @-prefixed keys in manifest_version 2"

Merge "Update AtEase calls to use Wikimedia namespace"

objectcache: refactor and simplify some WANObjectCache code

* Move $value === false checks to isValid()
* Rename $isTombstone variable to avoid confusion with the new value
* Reorganize conditionals around set() calls in doGetWithSetCallback()
* Skip getInterimValue() check when there is no reason to assume anything
  might even be there (e.g. the key is not tombstoned)
* Fold the tombstone case for $useMutex into the ternary

Change-Id: I257110097cffe7fe87c6a9896e875f09d5c936d9

Merge "objectcache: cleanup tombstone/mutex logic in doGetWithSetCallback()"

Merge "resourceloader: Restore the comment about <script> vs XHR loading"

resourceloader: Restore the comment about <script> vs XHR loading

Removed in 42956b99b558294e5aa8c6c0e2a8f18513478022.

I think it's useful to know this rationale (even more so because
the arguments about caching seem no longer correct these days).

Also, add the line about loading progress not being known when
using this method.

(Slightly related to T216034)

Change-Id: I81fa90483c0c85fecc5cb620196521ce8bdd33a7

objectcache: cleanup tombstone/mutex logic in doGetWithSetCallback()

This largely follows-up 6b2f13b055c84d.

* Make the comments and use of temporary variables easier to follow.
* Simplified some conditionals by remove redundant checks.
* Bypass the final set() call if $isTombstone is true. It will almost
  always be rejected due to the tombstone still being there anyway, so
  there is no point in the roundtrips. Also, the most likely case where
  it would succeed is due to the callback taking a long time to run, in
  which case data from the replication lag uncertainty period that the
  tombstone represents would be getting saved to cache with the full
  nominal TTL, which is wrong.

Change-Id: Ic28e15b24f39e128bd72ad4d905edb852bc907aa

mediawiki.special.changeslist.enhanced: Convert CSS to LESS

Change-Id: Ic71a6bce99fd6417f90b5abdd52dcfa10e62029d

Localisation updates from https://translatewiki.net.

Change-Id: I377eb66bceaf7e946bbcdb51cf767d96be3e24c9

Merge "Add missing documentation to ProtectedTitlesPager"

Merge "rdbms: Use more narrow type hints in LBFactoryTest"

rdbms: Use more narrow type hints in LBFactoryTest

Originally I wanted to fix the broken type hint for the `Database`
class. But I ended limiting the scope as much as possible and only
type hint at the interfaces that contain the methods the code actually
needs.

Change-Id: I450a55e80612f6e676f7c803dc904b4b653426d8

mediawiki.special.changeslist.legend: Convert CSS to LESS

Change-Id: I543cb50958689af6b32d3cd462a8f77f31724a03

Merge "rdbms: avoid duplicate spammy logging in LoadBalancer::getRandomNonLagged"

Fix CommentStoreComment RawMessage construction

If a CommentStoreComment is constructed without a Message argument, then
the RawMessage it uses instead should specify the comment text as a
plain-text parameter, not as a regular parameter: we don’t want any
syntax in the text to be interpreted at the Message level.

Change-Id: If14debde2bceae695c8955604ee96bd5005d8b66

Merge "resourceloader: Ignore warnings in JSMinPlus parser"

registration: Allow @-prefixed keys in manifest_version 2

As specified in
<https://www.mediawiki.org/wiki/Manual:Extension_registration#Retaining_documentation>.

Bug: T215650
Change-Id: I2ff523828389fbec39cdf7c55046e7da901a2ecb

resourceloader: Ignore warnings in JSMinPlus parser

It's old and unmaintained. The only thing we care about is if it
was able to parse the script and if not, what its error is. Its
return value or broken inner workings are insignificant at this
point and only cause noise.

Bug: T77169
Change-Id: Ie357ccfcc6141f894b452eb3996e168c1526990f

Update AtEase calls to use Wikimedia namespace

Helps ease migration (stops vendor patch failing)

Change-Id: I9a985b341e1c3664c4ced6f793b19149067a580a

Localisation updates from https://translatewiki.net.

Change-Id: Ia27cd1dfe1e1fecdff6431841c9e26a2daf3a865

rdbms: avoid duplicate spammy logging in LoadBalancer::getRandomNonLagged

This is already logged in LoadMonitor in getServerStates() in a less spammy way
(due to APC caching of server states).

Bug: T215611
Change-Id: Id70fdfa62eff9cb6446deea5e197f4c0af4928aa

Merge "Prevent fatal PHP errors when PageRestriction::getTitle() returns null."

Update @deprecated comment for the DB_SLAVE constant

I keep searching for the proper replacement for this deprecated
constant. It's much easier to find when the @deprecated comment
just mentions it.

Change-Id: I199b876ecfb6e146848b395bc7bfad80742f000d

Add missing documentation to ProtectedTitlesPager

This was discussed in I2b5c95d. The reason no IDE was able to find this
usage of the deprecated SpecialPage::getTitle method was the missing
type hinting.

TODO:
* A lot of properties this class uses are not declared.
* A lot of stuff in here must be private, but is public at the moment.
But this is for later patches.

Change-Id: I12fd6e517e5bd3507032ee3aafcb1de96733985b

Use "try-catch" block instead of "if" block to prevent interruption of new installation

New installation blocked when checking if the DB exists,
because when select DB, if it fails, it will throw an exception.

So I modify the checking logic to determine if there is an exception
thrown instead of detecting the return value.

Bug: T215566
Change-Id: I6817997434df7adc79fbc1b224b77c0daa8cc11d

widgets: Remove redundant equality check in SelectWithInputWidget

The check `if ( $config['disabled'] == true )` is the same as the
check `if ( $config['disabled'] )`. Was this intentional or was it
supposed to be a test for equality and type (===)? If not, then I
think this patch removes the irrelevancy.

Clearly, if the $config['disabled'] is set to false, the isset()
check will return true but the second check will fail even with
this patch as it does the same thing.

Change-Id: Ibbe5b4949590f8ac954f613236056dd2e6dd18ba

Update wikimedia/xmp-reader from 0.6.1 to 0.6.2

Change-Id: Ie396666aae2122dc552f6f1a7f6fe6ff96387fd1
Depends-On: Ia5dcf97548ed1c1f17cce9701df5cf13eea814dd

Update wikimedia/php-session-serializer from v1.0.6 to v1.0.7

Change-Id: Id30bb862bec42e508b3551d39f5980d900ade206
Depends-On: Ia19d237198fd2a0a15db467ec9dd97e98a857e33

jquery.makeCollapsible.styles: Avoid FOUC for collapse button

The expand button on a <div> element has already a pseudo-class with the
expand text as content. For the collapse button this pseudo-class is
missing.

This change adds the missing pseudo-class for the collapse button to
prevent a missing button while loading.

The missing collapse button while loading is visible on
Special:RecentChanges. The expand button is always there.

Change-Id: I7fa80238d54ac43a96bf1d70bd42a0cccf90a73a

Merge "Revert "RCFilters: Export config vars in the RL modules where possible""

Revert "RCFilters: Export config vars in the RL modules where possible"

This reverts commit bf4f9d0313495dd67a53b9925edf9a3cc6ef36fe.

Reason for revert: causes issues with OAuth extension

The OAuth extension uses a hook to override certain tag messages, and
that hook uses RequestContext::getMain()->msg(), which tries to load
the current user. That's not allowed in load.php requests, so we get a
"Sessions are disabled for this entry point" error.

Change-Id: If58b7edfca116cd7a11f38556f74bed82609d96a

EnhancedChangesList: Load style module via addModuleStyles()

This change avoids a flash of missing collapsible toggle icons while
loading.

Change-Id: I090b1622868f2e5a01ca783dfce3be16b7d6b525

Update wikimedia/ip-set 2.0.0 to 2.0.1

Change-Id: I154a677bac28a451be4f203ff05ea4620260b54e
Depends-On: I1090817a5ff1e406cf51596a8b1558c7bd3ca7d7

Merge "ChangesList: Remove spaces from <td> in EnhancedChangesListGroup.mustache"

Merge "ProtectionForm: Move JS config var to RL packageFiles"

ChangesList: Remove spaces from <td> in EnhancedChangesListGroup.mustache

The special page RecentChanges can included in wiki content via

  {{Special:RecentChanges}}

In this case the generated content is parsed by a wikitext parser. The
wikitext parser converts

  <td>
    <span>Foo</span>
  </td>

to the HTML

  <td><p><span>Foo</span></p></td>

But the wikitext

  <td><span>Foo</span></td>

is converted to the HTML

  <td><span>Foo</span></td>

The module 'jquery.makeCollapsible' supports custom collapsible toggle
buttons but only when they are a direct child of the <td> element. When
the module does not find a custom collapsible toggle button it adds the
default button [Expand].

Because of the additional <p> element when including the recent changes
the module 'jquery.makeCollapsible' adds a default collapsible toggle
button.

This change removes the spaces between <td> and <span> to prevent the
<p> elements when including the recent changes.

Bug: T215737
Change-Id: Ida46956fc7f9ab923c542751ac13411380c02ab0

Merge "Job::factory should throw an InvalidArgumentException, not MWException"

Job::factory should throw an InvalidArgumentException, not MWException

Change-Id: I9a85562b205db604d2746ec5db0e72dacdb93c6d

Merge "mw.widgets: Use CSS grid for the calendar in supporting browsers"

mw.widgets: Use CSS grid for the calendar in supporting browsers

Bug: T214997
Change-Id: I05dca01585f3de86580e7ae94ceeae1716f56ba3

Merge "parser: Add new pcache metrics, split by page content model"

Merge "registration: Define packageFiles for v1 extension schema"

Localisation updates from https://translatewiki.net.

Change-Id: Id809e80a46547f7bf042eb6073efd7a519666ce7

parser: Add new pcache metrics, split by page content model

Change-Id: I31c3c5b863309ffcc4424c43891b577b3fb7a753

registration: Define packageFiles for v1 extension schema

Follows 56fa42d4332c.

Bug: T215756
Change-Id: Icbf433b6b147f53cf99533a7272fc83392a8999d

Merge "Add ability to define packageFiles in extension.json"

Merge "Change `.forEach()` to `for ()` loop with break"

Merge "mw.widgets: Make TitleSearchWidget read-only-able"

Change `.forEach()` to `for ()` loop with break

Follow-up to change 1edba8029, which replaced:
  $.each( array, function ( index, value ) { ... } )
by:
  array.forEach( function ( value, index ) { ... } )

`jQuery.each` makes a break of the loop on a `return false`.
`Array.prototype.forEach` ignores the return value and there
is no way to stop or break a `forEach()` loop other than by
throwing an exception. [1]

A `$.each` with a `return false` could be replaced by
`Array.prototype.find`, but `find()` is not part of ES5. [2]

  return this.getItems().find( function ( filterItem ) {
    return filterItem.isSelected() && filterItem.isConflicted();
  } );

So a simple `for ()` loop with a `break` or a `return` is
used in these cases.

Also explicitly specify `undefined` as possible return value
in FiltersViewModel.prototype.getFirstConflictedItem.

[1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/forEach
[2] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/find

Change-Id: I0af89d39ed789f49e06b25cac14ecd995ba58cbd

Merge "Exception is never thrown in the getConnection() method"

mw.widgets: Make TitleSearchWidget read-only-able

Change-Id: I3eed52438c1e416d4dcdedb564bb9df3745d0f77

Remove unused @throws anotation from PHPDoc comment

Change-Id: If60ad5c12edbf4ea96c5c337f0c05cf0a8d5e43d

ProtectionForm: Move JS config var to RL packageFiles

The check for type Array of the JavaScript config variable is obsolete.

Change-Id: I4a38cfc211360c831958ce8ca88584cdf56a727c

Add ability to define packageFiles in extension.json

Bug: T215756
Change-Id: I97c3ccc13c54b424952ce15509b98a217b2670b1

Merge "Add jakub-onderka/php-console-highlighter 0.3.2 explicitly"

Add jakub-onderka/php-console-highlighter 0.3.2 explicitly

We pin it in various libraries, might aswell in core for completenes

Change-Id: I659c880c9e16e4eab7050874f75c9eb5c9fa958c

Update cssjanus from 1.2.0 to 1.2.1

Change-Id: I99f11a24ff0a8b02b1c0103ad8cb8faeb165c84e
Depends-On: Iee4c24b6b5dd92493160caafa4764b6d6e427147

Merge "Updated pear/net_smtp from 1.8.0 to 1.8.1"

Merge "ParserTestPrinter: use Shell class"

Merge "JpegPixelFormatTest: use Shell class"

Merge "Linker: Remove some else statements, and unnecessary temporary variables"

Merge "Make attachLatest.php wait for DB replica lag"

Updated pear/net_smtp from 1.8.0 to 1.8.1

Depends-On: I6543ac3e6482da197ce2d644823861d1db4624d8
Change-Id: I91b3c2864536a53fcc829a1e10a3bc0d4638db2b

Linker: Remove some else statements, and unnecessary temporary variables

Change-Id: I9b9383459edf06280135d9709da889fc5a688c98

Update dev dependancies

* Updated psy/psysh from 0.9.6 to 0.9.9 (dev-only).
* Updated nikic/php-parser from 3.1.3 to 3.1.5 (dev-only).

Change-Id: I05e45327e93d3bc733db73c0efe573b3223cc902
Depends-On: I896ac6c5afa8221ceca726eb7895e05d87d7e6f3

Localisation updates from https://translatewiki.net.

Change-Id: I26dda2663489848ffc75fc54b766fb827eeff4de

Make attachLatest.php wait for DB replica lag

Change-Id: I4f402ba772d0b0dda8360eaacf11e318f659efc0

build: Use headless Chrome and Firefox for QUnit testing

Also:

* Remove "watch" task from Gruntfile.
  This wasn't exposed via npm-run and afaik rarely if ever used.
  It seems mostly obsoleted by modern editors which perform
  linting and style guidance as-you-type and on-save.

* Remove "chromium" target from karma config, the default
  target supports both Chromium and Chrome.

* Remove unused "default" alias in Gruntfile. Was only
  accessible if developers installed grunt-cli globally, which
  we don't require and shouldn't encourage. The new minify task
  is now exposed as "npm run build".

* Remove "test" alias in Gruntfile. Was only used by "npm run test",
  which now uses "grunt lint" directly, which makes it more clear
  that these are only linters, and not unit tests.

This references T211784 because ensuring no headed browsers are used
(which would need Xvfb to start), is a prerequisite for the new Node 10
images. We may still need to provide a slower Xvfb-wrapped variant for
some repos, but would be nice of mw-core wasn't one of them.

Bug: T211784
Change-Id: I270a16d2b4f799ac7c9f84e9e2ee23806bdbc57e

ParserTestPrinter: use Shell class

Change-Id: I9e2eb3b01a9907d72e3edaa6660a4057d513942e

JpegPixelFormatTest: use Shell class

Change-Id: I53099827dc2b2b26b7f3cbc9f0e12030868fdafd

Merge "build: Update karma to 3.1.4"

Merge "Hard deprecate Language::setCode"

build: Update karma to 3.1.4

This fixes the bug where HeadlessChrome was always shown as
version 0.0.0 in the console output, which makes it rather
difficult to debug which Chrome version is installed.

Upstream https://github.com/karma-runner/karma/issues/2762,
which updates the ua-parser library to a version that fixes
that bug.

Change-Id: I0b7fe7f8a90649592c4d31fb3055cdc9e8d34814

RCFilters: Export config vars in the RL modules where possible

Right now there are only a few settings that don't depend on the request
context, but we can at least export those. The tag data in
particular can get pretty big.

Downgrade the type hint for ChangeTags::tagDescription() and
tagLongDescriptionMessage() from IContextSource to MessageLocalizer, so
we can pass a ResourceLoaderContext to these functions. (Since
IContextSource extends MessageLocalizer, this won't break any callers.)

truncateTagDescription() can't be downgraded to MessageLocalizer because
it needs a Language object, so hack up a fake RequestContext when
calling it. It would be nice if this weren't necessary; perhaps we can
move getLanguage() from IContextSource up into MessageLocalizer?

Bug: T201574
Change-Id: I9b66e35de826a07aa9551ba285e64e4852293229