Merge "Fix 'Tags' padding to keep it farther from the edge and document the source of the new right padding value"

Merge "RELEASE-NOTES: Add note for breaking CategoryMultiselectWidget"

RELEASE-NOTES: Add note for breaking CategoryMultiselectWidget

Follows-up on I2146d8ce6505dae55750b4b12bd806b83796c1c5.

Bug: T183299
Change-Id: I799afa68048dfb7b2b30cdae09c9bf6a625f867f

Merge "Update OOUI to v0.27.4"

Allow overloading of getLabel() with return ' '

This is a follow-up to 125cbd8c017b872c78f047c08f494a458883db23.

Bug: T198338
Change-Id: I21626326dde368d70fcc33052e43ff90db5ea9c0

Merge "resourceloader: Embed 'mediawiki' directly in startup response"

RC Filters: Don't apply/clear highlighting while still initializing

this.controller.isInitialized() is still false on page load, we don't want to
clear/apply highlights at this stage.

Remove Controller's this.initializing in favor of this.initialized,
expose it through isInitialized(), and remove ChangesListWrapperWidget's
this.filterModelInitialized since it's now unused.

Bug: T198359
Change-Id: I284ded2c6dd891dfb3efab5c6800c26a806ce306

Merge "filerepo: clean up remote description cache keys"

Merge "RCFilters: Move aggregation of highlight classes to the backend"

ResourceLoaderImage module definitions can define a defaultColor

Bug: T197909
Change-Id: I0745e112d11026ed59d8daca990b313305cd094a

Merge "Fix broken label in Special:Import due to duplicate ID"

Localisation updates from https://translatewiki.net.

Change-Id: Ia319ca22827e6f4f8302c03342db181e3e350596

Merge "[MCR] populateContentTables maintenance script"

Update OOUI to v0.27.4

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

Depends-on: I340d724fe7ede73cd7d20fa98184f33ad19b9148
Change-Id: I52f35f231d67f6fb6f0abfc268f25c5677e4b7ba

resourceloader: Embed 'mediawiki' directly in startup response

Embed the essential files to define mw.loader directly as part of
the startup module.

* This means the internal 'mediawiki' module no longer exists.
  This is safe to remove because:
  1) While registered server-side for loading from startup.js, a PHPUnit
     structure test disallowed being specified as a dependency.
  2) Anything that attempted to load it client-side failed because the
     module was marked in the registry as 'raw', thereby excluding it
     from the data sent to the client-side. As such, it was seen as an
     unknown module that the client refused to fetch from the server.

* Deprecate getStartupModules() and getLegacyModules().
  These are no longer needed. There are no known callers anywhere in
  Wikimedia Git or elsewhere indexed by Codesearch, but easy enough
  to leave as no-op for one release.

* Remove ResourceLoaderRawFileModule class.
  No longer needed. Was created as a hack specifically for the 'mediawiki'
  module so that it would not leak global variables in debug mode.
  It has no usage anywhere in Wikimedia Git, nor elsewhere in Codesearch.
  Remove without deprecation given this was meant to be a 'private' class.

* Introduce (private) getBaseModules(). Previously, this list only existed
  locally in getStartupModulesUrl() by merging getStartupModules() and
  getLegacyModules(). This value was factored out into its own method.

* Make getStartupModulesUrl() private and rename to getBaseModulesUrl().
  It is only used internally to export the 'baseModulesUri' value.
  Its name was already confusing before, but it would've been even more
  confusing now given it doesn't even call getStartupModules() any more.

Bug: T192623
Change-Id: I14ba282d7b65e99ca54b7c2f77ba6e1adaddd11c

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

Fix 'Tags' padding to keep it farther from the edge and document the source of the new right padding value

Bug: T192275
Change-Id: Ib63a09b3b0edbca3bd2f2cd42a9485538738256f

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"

filerepo: clean up remote description cache keys

Hash the file name portion and make the string constant portions
more relevant to what the keys are actually used for (e.g. there
is no URL parameter in the key)

Bug: T198279
Change-Id: Idf6f97db26f5be291cdd3a50a91346677fe9c3e6

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"

RCFilters: Move aggregation of highlight classes to the backend

Construction of the highlight containers was moved to the backend, but
setupHighlightContainers() also aggregates the CSS classes used for
highlights for grouped entries in enhanced mode.

Move that to the backend too, and get rid of setupHighlightContainers().

Also move the namespace classes to getHTMLClassesForFilters() so that
they get picked up too, and pick up classes for tags separately because
the way they're handled is weird.

Bug: T197168
Change-Id: I4c374f82e7d128025f4e2b2f39b0adba14b76ef3

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

Fix broken label in Special:Import due to duplicate ID

Change-Id: I176b811e12719320432af01a22812fea872794f6

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

[MCR] populateContentTables maintenance script

This introduces a maintenance script for populating the tables
introduced by the MCR storage schema, namely:

  slots
  slot_roles
  content
  content_models

Per default, both the revision and archive tables are processed.

This script is part of the MCR schema migration: after instructing
RevisionStroe to write both the old and the new schema by setting
$wgMultiContentRevisionSchemaMigrationStage = MIGRATION_WRITE_BOTH,
this script can be used to back-fill the new schema for existing
revisions.

Doing this is a precondition to later setting
$wgMultiContentRevisionSchemaMigrationStage = MIGRATION_NEW to
complete the schema migration.

Bug: T182682
Change-Id: Iecc67c1b8c082be1a1039eeb52e76ad16b965226

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."