Add missing use statements

Swap some ResultWrapper/Database to interface IResultWrapper/IDatabase

Change-Id: I312cb3cca5742f72efdb0965ba35362f2eb1129d

Merge "objectcache: define makeKey()/makeGlobalKey() for ReplicatedBagOStuff"

objectcache: define makeKey()/makeGlobalKey() for ReplicatedBagOStuff

Proxy to the "master"/"write" cache object method. This is similar to the
approach taken in MultiWriteBagOStuff

Bug: T198279
Change-Id: If0933246b7ef4fc07ebeec4c3c9625b1137dbe05

Merge "mw.widgets.CategoryMultiselectWidget use TagMultiselectWidget"

mw.widgets.CategoryMultiselectWidget use TagMultiselectWidget

…also rename mw.widgets.CategoryCapsuleItemWidget to
mw.widgets.CategoryTagItemWidget.

Bug: T183299
Depends-on: Ia0732faff2d98f1ca07da5fbe1f2abc27f71a617
Change-Id: I2146d8ce6505dae55750b4b12bd806b83796c1c5

Merge "resourceloader: Add @covers for FileModuleTest"

Merge "resourceloader: Add test for getVersionHash parent-definition requirement"

Merge "mediawiki.inspect: Fix exception when calling mw.inspect() a second time"

Merge "mediawiki.inspect: Fix doc index for inspect.grep()"

Merge "mediawiki.inspect: Use property assignments for clarity"

Merge "Refactor ApiOptions for reusability"

Merge "Move construction of highlight divs to backend"

Merge "RCFilters: Share one color picker popup between all highlight dropdowns"

Move construction of highlight divs to backend

Create the highlight container div in the backend for
both regular and enhanced result view, so we spare
the load in the frontend.

Bug: T197168
Change-Id: I36bd7b7c4c124d305ac7b07e824dc2a58e152be4

RCFilters: Only attempt to remove tag if we can find an item for it

As noted in T198140 and T198231, removeTagByData leads us eventually to
OO.ui.mixin.GroupElement.prototype.removeItems(), which does not check if
any of the items are null values, and a change event is emitted.

This commit ensures that we can find an item for the data before attempting
to remove the tag.

Bug: T198140
Change-Id: I79a923a7b4e5f6c4d14fcce3c5855b4c56796384

Refactor ApiOptions for reusability

Because GlobalPreferences needs an API and that API needs to behave like
this one, it makes sense to make them share code. Move some functionality
into separate functions to make this possible.

Bug: T62856
Change-Id: Ic64934e7f7226a309d71c22626119ea233e2caeb

RCFilters: Share one color picker popup between all highlight dropdowns

Add a PopupWidget subclass that wraps the color picker, and pass that
down to the buttons in the menu items.

Bug: T198142
Change-Id: I84dabce988f4c99835f503bb8c8eb492f7fbfde1

Localisation updates from https://translatewiki.net.

Change-Id: I6c8bffce7566484cc46e852b20c0b29d655e8a36

mediawiki.inspect: Fix exception when calling mw.inspect() a second time

In mediawiki.js, the mw.inspect() method is defined to lazy-load the
'mediawiki.inspect' module, and then call mw.inspect.runReports().

The problem is that, mediawiki.inspect.js, re-creates mw.inspect as a plain
object, which blows away the mw.inspect() function that was there.

Calling it a second time threw "TypeError: inspect is not a function".

Fix this by making mediawiki.inspect.js extend the function object,
instead of re-defining it.

Bug: T197810
Change-Id: I61aa965f3e1fd0a1c9f9d98310632b4a8d5e1683

mediawiki.inspect: Fix doc index for inspect.grep()

Because this was positioned linearly after the sub-object for
`mw.inspect.reports.*`, JSDuck was indexing this as part of that
object instead of the outer `mw.inspect.*` scope.

This could be fixed by adding `@member mw.inspect` on this method,
or by re-starting the `@class mw.inspect` scope, but really
this should've been declared closer to the rest of that class,
which naturally avoids the index issue.

Change-Id: Ib2c9dc861f7f062c05bc059643a1a676906d338f

mediawiki.inspect: Use property assignments for clarity

This changes the assignment of these methods from being declared
at once in an object literal, to assigning properties in separate
statements.

This is functionally a no-op and produces the exact same values,
but was factored out of the next commit to keep the diff cleaner
when fixing T197810 (which changes the host object from a plain
empty object, to a function object, which cannot be declared
as a literal).

Bug: T197810
Change-Id: I6ba1891acf8807cb9e01fa11a608e0c509e5f2bc

Skip MediaWikiTestCaseSchema1Test on Postgres

According to Anomie, the idea behind this test is semi-broken, and would
benefit from T191231 being implemented first.

Bug: T195807
Change-Id: I9561fd0a82c37cf044e292e00499ecb611fcded6

Merge "Declare HTMLSelectNamespace::mAllValue"

resourceloader: Add test for getVersionHash parent-definition requirement

Change-Id: I69cdfea96c1e64bd8a7495eb6e56d0aefbe37643

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit b068bb51d29e294a4f4a875ae829cca8cf314205

Change-Id: Ie8e8a7ed631894f56372e286ed01d1583f7a8979

Merge "Checkboxes should be styled on non-JS browsers"

Declare HTMLSelectNamespace::mAllValue

It is protected, because some extension extends this class

Change-Id: I576e9e7e843844433655b11b8847c9e3060ba8e9

Merge "Fix PHP7 warning "non well formed numeric value encountered""

Merge "Add some output for progress in populateChangeTagDef.php"

Merge "Expand $wgDBDefaultGroup documentation"

resourceloader: Add @covers for FileModuleTest

Add @covers for various helper methods used by public methods, where the helper
methods actually contain most of the logic being tested in FileModuleTest.

I've changed these methods from protected to private (confirmed no usage)
to further pin down that their contract doesn't matter beyond making the
public methods work.

Change-Id: I2aef0d322b38bc3595e7d2c2339112b16fc66b8d

Fix PHP7 warning "non well formed numeric value encountered"

PHP 7.1 warns when non-numeric string is implicitly cast to integer.

Change-Id: Ia46ea793e9495548c7d421b3372f6deaeda163f5

Merge "French space armoring: Use unicode regex instead of octal escapes"

Merge "Armor against French spaces detection in HTML attributes"

Merge "wdio-mediawiki: Tag v0.2.0"

Merge "Avoid bad method call to patchPatch() in DbTestRecorder"

Merge "Align loading animation for first load later reloads"

wdio-mediawiki: Tag v0.2.0

Change-Id: Ia975b2d35220a6ad8ca90e6198ca2ba74c256ea3

Expand $wgDBDefaultGroup documentation

Per Krinkle's comment:
https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/440022/2/includes/DefaultSettings.php

Change-Id: I94e6e7f5048de610d2ae0722cb3de216288ed182

Avoid bad method call to patchPatch() in DbTestRecorder

Bug: T193995
Change-Id: Ibc480b04463792b7cd720a6eb080e0960a30e440

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

Localisation updates from https://translatewiki.net.

Change-Id: Ie589a2973df5cc34bc5b622cd135608b7bb29a34

Merge "resourceloader: Update StartUpModule to use file hash instead of mtime"

Merge "resourceloader: Fix duplicate semi-colon in generated startup code"

Merge "skins: Remove redundant mediawiki.hidpi script"

resourceloader: Update StartUpModule to use file hash instead of mtime

This was overlooked as part of the whole shift from timestamps
to file hashes (T94074, f37cee996e, T104950, 28f6d7fbde).

The key name and method call here matches that of FileModule.

The impact of this still using mtimes was fairly low, given:

* The startup module is only cached for 5 minutes.

* The startup module's hash varies on "everything" which includes
  wgVersion, as such, when a new branch is cut and cloned and this file's
  timestamp is reset, there will still be other factors causing the
  overall hash to vary. As such, this change is unlikely to improve
  200-304 response ratio.

Change-Id: I6543fb75575e9a793a7fc93e15d2f3e0b5a04342

resourceloader: Fix duplicate semi-colon in generated startup code

The startup.js template used '$CODE.registrations();', which makes
sense syntatically (and for the linter), but the substitution logic
was only replacing the '$CODE.registrations()' portion.

This made the generated output contain two consecutive semi-colons.

Change-Id: Ibef1a0d932b19037987a7effa12aa57f578a142d

Merge "LCStoreStaticArray: create cache directory when it doesn't exist"

skins: Remove redundant mediawiki.hidpi script

This is an internal script automatically loaded by Skin.php to
activate the 'jquery.hidpi' polyfill for all images on the current
page in browsers that don't natively support the 'srcset' attribute
on the HTML img element.

This script is loaded via ResourceLoader for which Grade A currently
requires:
> IE11+/Edge, Chr 65+, Ff 52+, Saf 5+, Op 15+, iOS 6+, Android 4+.

According to MDN and CanIUse, the basic 'x' syntax of srcset is supported, and
enabled by default, in:
> Edge, Chr 34+, Ff 38+, Saf 7+, Op 21+, iOS 8+, Android 5+.

This means in the following browsers, MediaWiki will no longer attempt to
replace images in articles with their hidpi versions.

| Browser                | analytics.wikimedia.org (22 May - 22 June)
| ---------------------- | -----------------------
| IE 11 on Windows <= 7  | 3.4% (OS does not support HiDPI)
| IE 11 on Windows 8+    | 1.1%
| Safari 5 & 6 (desktop) | <0.1%
| Opera 15-20 (desktop)  | <0.1%
| iOS 6 & 7 (mobile)     | 0.1%
| Android 4 (mobile)     | 0.5%

While the total of 1.7% is higher than our usual point where we decide
to remove support, I think we should consider dropping the hidpi polyfill
still for several reasons:

* MobileFrontend no longer uses 'srcset' attributes. As such, these browsers
  don't actually change their behaviour based on the polyfill.

* For IE 11/Win8 in particular, most users don't have an HiDPI monitor,
  but we still download the polyfill. HiDPI on Win8 is primarily tablets.

* In all cases where the polyfill activates, we download the HiDPI images
  in addition to the standard resolution (which downloads and renders first).
  This is because client-side JavaScript is not able to replace it sooner.
  This could be considered a waste of bandwidth, as it can double or tripple
  the bandwidth cost for end users.
  This also means pages complete their loading much later because the browser
  first renders the page nearly to completion with standard resolution images,
  and only at the end our polyfill activates to restarts all image loading.

The experience gracefully falls back to normal web rendering, where the standard
resolution of the image is used. This would match what users of these devices
see on other websites, given client-side emulation of srcset is fairly rare.

== Modules

The 'mediawiki.hidpi' module was removed, and considered internal to Skin.php.
It contained no public methods. I confirmed there were no matches in Codesearch,
and no matches in mwgrep on Wikimedia wikis.

I did not remove 'jquery.hidpi', which is what contains the actual polyfill
and the jQuery.fn.hidpi() public method. (Codesearch shows 2 extenisons using
it, and mwgrep returned 1 unused gadget on Meta-Wiki referencing it).
It has been kept, but marked as deprecated. To be removed in a future release.

Bug: T127328
Change-Id: I42ce0feea1fbfe534f00e05a7cd8d81df0c33d8f

Merge "resourceloader: spin base module code out as a proper module"

LCStoreStaticArray: create cache directory when it doesn't exist

Array store (LCStoreStaticArray) should behave as standard LCStoreCDB,
creating the target directory when it doesn't exist.

This fixes unexpected "No such file or directory" errors when
replacing 'files' with 'array' in $wgLocalisationCacheConf['store'].

Change-Id: Ie49cf03ca93ac47fc98665de36e6207bef287193

Add some output for progress in populateChangeTagDef.php

I ran it on beta cluster and all was done without one single line of output

Bug: T193871
Change-Id: I05a7c7827203eb159b5845a874b94d74f5d5e299

resourceloader: spin base module code out as a proper module

Bug: T192623
Change-Id: I6f7dc40488a990d0f8a25e84ebc9eb25ad4c2975

Merge "Revert "Use pathinfo() in AutoLoaderStructureTest::testPSR4Completeness""

Revert "Use pathinfo() in AutoLoaderStructureTest::testPSR4Completeness"

This reverts commit 634c2ec2afd93827e5aaf571396eb76d35948188.

Reason for revert: that strips the PSR4 directories from the class.

Example:
$dir BlueSpiceFoundation/src/"
$file BlueSpiceFoundation/src/ConfigDefinition/IntSetting.php

$abbrFileName: IntSetting
$expectedClassName: BlueSpice\IntSetting

$abbrFileName should be relative to $dir and not just the filename.

Bug: T198077
Change-Id: Ie934e309fee0392439b4e26d86249f0650e5ea67

Merge "Use pathinfo() in AutoLoaderStructureTest::testPSR4Completeness"

Use pathinfo() in AutoLoaderStructureTest::testPSR4Completeness

When setting AutoloadNamespaces to './' in extension.json, the test
AutoLoaderStructureTest::testPSR4Completeness would fail. The directory
path is not made canonical while the file is, which causes the substr()
call being used to strip too many characters. For example:

  $dir : /mediawiki/extensions/Wikidata.org/./
  $file: /mediawiki/extensions/Wikidata.org/Hooks.php

$abbrFileName = substr( substr( $file, strlen( $dir ) ), 0, -4 );
>>> oks

Use pathinfo() to parse the filename. Yields 'Hooks' as expected.

Bug: T198077
Change-Id: Ia8a11d87788b32ddb426a16a61b410b05ff5f15e

Merge "Make WikiPage::convertSelectType protected"

Browser tests: use Util.getTestString in specialwatchlist.js

Follow-up on I5c5e9b22af5406f347636b22e68657b2674db6c9 and
I0747c2d361e951f8b3765a3dca548e55edb72216

Change-Id: I35ad793e6d9fe4c3bf58df9b4fcf6471c0a81f70

Merge "Browser tests: extract commonly used getTestString function"

Merge "Add browser test for basic watchlist functionality"

Merge "mediawiki.hook: Move mw.hook tests to new mediawiki.base.test.js file"

Merge "Remove deprecated 'dom-level2-shim' module"

Merge "resourceloader: Add coverage for StartupModule::getDefinitionSummary"

GitInfo: Don't try shelling out if it's disabled

Bug: T198037
Change-Id: I364f9bc0e78439474101f4b2a171805c91f50a72

Localisation updates from https://translatewiki.net.

Change-Id: I9a6171181bfab383402ad99e1f59315d5aded560

Add the es-419 language code to support South American Spanish

This code is useful for targeting Spanish spoken in the Latin America
and the Caribbean region. There are no plans to make this available as
an interface language, hence I am not adding a language file with a
fallback to 'es'.

Bug: T112889
Change-Id: If7f0ed7a13f1cc86985ce5ce509dcf543cc1c0ff

Make WikiPage::convertSelectType protected

When override WikiPage this function is needed
to map the string to const

Change-Id: I4e4d597a77c859ad17bd6c29bed875fb4248b5fb

Merge "rdbms: make some LBFactory fields private"

Remove deprecated 'dom-level2-shim' module

This is an empty placeholder module that does nothing. It was kept
for backwards compatibility with older extensions that were still
supporting MediaWiki versions that in turn supported older browsers.

There is no longer any use of 'dom-level2-shim' anywhere in Wikimedia Git,
nor anywhere found by Codesearch,
nor anywhere on-wiki at WMF (including user scripts).

Bug: T127328
Change-Id: I416b216471c321d56d3b6d72bc9ef7dcc0f16093

Merge "MWExceptionRenderer: Fix db error outage page"

resourceloader: Add coverage for StartupModule::getDefinitionSummary

Change-Id: I913c8ecd5f51db851ef408c4281cdd75a0e0b239

Merge "Make MessageCache::getMsgFromNamespace() process cache hook-defined keys"

Merge "rdbms: rearrange LoadBalancer fields and make some private"

rdbms: rearrange LoadBalancer fields and make some private

Put the object/callback fields first, followed by config fields
and then state fields, similar to LBFactory.

Change-Id: I40ceaa48842b505f15efe5e41ca7a6de437b6ff1

Use \u{00A0} instead of &#160; or &nbsp;

Directly use the UTF-8 encoding of the 'NO-BREAK SPACE' (U+00A0) instead of
the HTML/XML entities &#160; or &#xa0; or &nbsp;.

With the UTF-8 character the generated HTML is shorter and better to read.

Also change the special value for the label in HTMLForm from &#160; to
U+00A0 but also support &#160; for backward compability.

Bug: T154300
Change-Id: I882599ac1120789bb4e524c4394870680caca4f4

resourceloader: move using() from mediawiki.js to the base module

Bug: T192623
Change-Id: I9095749a013cf97bff45d6d11afd9b2cee7e189b

resourceloader: remove internal use of using() in mediawiki.js

Bug: T192623
Change-Id: Id10ddf9f98661a8e328f40717b12cc08cbc04064

Make MessageCache::getMsgFromNamespace() process cache hook-defined keys

Change-Id: I6f8a0c002b54dc48bc3e6902ab1ba41b2f99412a

[MCR] Rollback for all slots

Bug: T194034
Change-Id: Ifd23bc1cd64ddc090e1c1c26aacda37e8ba7a18b

Merge "Make undo fail if more than just the main slot is affected."

mediawiki.hook: Move mw.hook tests to new mediawiki.base.test.js file

Follows-up 3801e54c29a.

Also:

* Split the test into smaller more dedicated tests.
* Make minor changes so as to only assert values in the tests'
  outer scope. Assertions within callbacks are an anti-pattern
  that is fragile and can easily miss or mask problems. Using a
  single state observer that is modified by the callbacks makes
  for strict assertions with no implied or untested behaviour.
  Callbacks running in a different order or a different number of
  times now cause assertion failures - instead of causing the
  assertion to not be run, or to be run multiple times, which
  would pass.

Bug: T192623
Change-Id: Ice1560b754f8df29ca583eea19f559020fafaf12

Merge "API: Check assert parameters earlier in the request"

French space armoring: Use unicode regex instead of octal escapes

Follow up to Id8cdb887182f346acab2d108836ce201626848af

Change-Id: I35f7b35746e63a98a115a0dbc9d5869b691c3a9c

Merge "RCFilters: Fix saved filter name truncation for Firefox"

RCFilters: Fix saved filter name truncation for Firefox

Firefox seem to have a bit of trouble with variable-width flexbox
div that need to be truncated with an ellipses.

This fix includes a couple of changes:
* Split the three elements of the title into three separate divs,
  and define the flex divs properly for a variable-width one in
  the middle.
* Taking advantage of #1, clean up the class names too (move to
  under '-top-xxx' classes for better organization in the LESS
  file too.
* Add a max-width: 100% rule to the label that gets
  truncated; this seems to force Firefox to recalculate its proper
  width while in flexbox and actually add the ellipses.

Bug: T197835
Change-Id: Ibe2002a5fe9006f262b2256cae222fed9e9a8fd0

Merge "Only use languages with localisation in the installer"

Merge "MCR: rename $baseRevId paramter to match actual semantics."

Make undo fail if more than just the main slot is affected.

Bug: T194412
Change-Id: Ifdf9bc9d884844f9ffeb8019d9b13d5737862063

MCR: rename $baseRevId paramter to match actual semantics.

The $baseRevId in WikiPage::doEditContent is used only to indicate what
revision an edit reverted to. It is not used to indicate the actual base
revision of an edit in any sense. Specifically, EditPage never sets it.

So, this change renames the parameter to $originalRevId to match $undidRevId.
It also renames PageUpdater::setBaseRevisionId to setOriginalRevisionId.
Further, this introduces a paramter to PageUpdater::hasEditConflict():

Before this change, PageUpdater::hasEditConflict() was based on the
revision set via PageUpdater::setBaseRevisionId(), assuming the semantics
of "base revision" used by EditPage. However, this is NOT how the $baseRevId
parameter in WikiPage works.

Bug: T197685
Change-Id: Ib78257d4d6ee7c4ec093d5706904c599b02c73e0

Merge "Fix Special:Userrights temp groups with no non-temp groups"

Localisation updates from https://translatewiki.net.

Change-Id: Ibd413089aba90336bcc5bd0be2fdccb78130781c

Fix Special:Userrights temp groups with no non-temp groups

Special:Userrights was only checking the count of non-temporary groups
when deciding whether to show the list of temporary and non-temporary
groups. It needs to check both.

Bug: T197860
Change-Id: Ie8a9caad01f1a610584fb4a1272ad3800e9c8e6a

API: Check assert parameters earlier in the request

Specifically, check the assert and assertuser parameters before setting
up the action module, so errors in parsing the module's parameters due
to being logged out don't override the client's intended "am I logged
in?" check.

Note this means that assertion failures will no longer use custom module
output formatters. This seems like an acceptable tradeoff: on Wikimedia
sites in May 2018 there were no requests that would have been affected
by this change.

Bug: T197672
Change-Id: I02a71395d5ed9f445e57162f2136292825f8dbb5

Align loading animation for first load later reloads

Bug: T192210
Change-Id: Ia058201697889f7d6d8575b9fb8fe5dbb383747a

Armor against French spaces detection in HTML attributes

This change also solves T13874 in a generic way.

Bug: T5158
Change-Id: Id8cdb887182f346acab2d108836ce201626848af

Set $linkTrail for Abkhazian.

Set $linkTrail for Abkhazian. I don't speak the language, just copied
the characters from the referenced Phabricator task.

Bug: T189923
Change-Id: I0db35e9ac46aea19d082c1e9fa35a9120034e4bd

Merge "RCFilters: Readjust welcome text so results don't jump after load"

Merge "LanguageConverter: Fix @return description"

RCFilters: Readjust welcome text so results don't jump after load

Bug: T197833
Change-Id: If3a433c84c53d80a4470fbc112b75c00cae20bac