Update OOjs UI to v0.9.1

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.9.1/History.md

Change-Id: I936ea56e1c99f576f44115a9b2fa075ef1ab14bd

Merge "Mark DoEditSectionLink hook as deprecated"

Mark DoEditSectionLink hook as deprecated

Change-Id: I347fe68900c98080bf3070a1b0bf5a005048868a

Refactor out 'infinity' variants

Refactor out 'infinity' vartiant values which used in blocking and
protecting actions. This patchset adds GlobalFunction wfIsInfinity.

Bug: T68646
Change-Id: I60cc55a5bbd43c72916a1c2ea3807457d4e33765

Merge "Use underscores in help links"

Use underscores in help links

Replace spaces by underscore to build correct links to wiki pages. IE11
will show %20 for spaces. Also use urlencode to make the url safe.

Follow-Up: I2934b1708a0d207dcf3d940264f140613646f203
Change-Id: I5ef08441406e96aa9749476af0a81fc11fa4e4d6

resources: Move to library version of jquery.client

The jQuery Client library is no longer mastered in MediaWiki, and is
instead a proper, published library, which is now tagged as v1.0.0.

Change-Id: Idd19d738b392a5f742fef6f98c885e8c391a5cbf

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

Localisation updates from https://translatewiki.net.

Change-Id: I384c8214ba8453349d0b7cc90b954580bd571011

OutputPage: Condition-wrap the <script> for 'user.groups'

Follows-up 9d390a09cdfb. It already wraps the only=script requests
for 'site' and 'user', but forgot about 'user.groups' which is
not 'scripts' but 'combined' (as regular module requests).

That request responds with mw.loader.implement whih will be absent
if the environment is unsupported.

With normal module requests, this is naturally covered by those
requests not being fired from mw.loader in the first place but
with hardcoded requests like these the condition wrap with
document.write is unfortunately required in the current reality.

Change-Id: Ib3a7378d0c44e601760fbbc5174da09bd7b7f492

Mention metrics in IContextSource docs

Change-Id: I9b654a12578758471978ffd1e1ae20476aeeaa0d

Merge "mediawiki.js: clean-ups"

mediawiki.js: clean-ups

* Inline canExpandStylesheetWith().
* Make resolve() take an array of module names instead of being flexible
  (taking either a string or an array). It's a private method, so it doesn't
  need to have a cute API.
* Use simple string concatenation rather than [].join in pad()
* Make allReady() more efficient by having it bail early as soon as any
  non-ready module is encountered.
* Add anyFailed(), which is a complement to allReady().
* Remove filter(). In more than one place, it was used to check whether a
  module in some specified state existed in a list of modules, which is
  inefficient, since filter() continues iterating through the array of modules
  after finding a match and because it constructed an array with the matches,
  which was not needed.
* Replace some verbose array iteration with $.grep / $.map / $.each.

Change-Id: I22a2f824488ba39d03b546ac8560f551a3198d7f

Made JobQueueFederated no longer need "checkDelay" for delaying

This reverts commit b9fd0f311c292424fd49d7bff550f67e94a62bc4.

Change-Id: Ibff28591c268689d593cb56a58e0b33c22292eba

Merge "Revert "Made JobQueueFederated no longer need "checkDelay" for delaying""

Revert "Made JobQueueFederated no longer need "checkDelay" for delaying"

Breaks Cirrus and Wikibase

This reverts commit c8f96384aad312e52969b81dd8ce6b8d80c3857a.

Change-Id: I15278d52806bf30d6ea6c6b9eb8586160f65c42b

mw.loader: Replace log() calls with mw.track events

This allows other consumers (such as an error logging backend) to
subscribe to these errors.

Change-Id: I157e2fb5c6b4d77736fcaf0a87c624e76a833161

Merge "jsduck: Add 'mediawiki.startUp.js' to eg-iframe"

Merge "FeedItem: Use full URL with protocol for <id>/<guid> in Atom/RSS feeds"

jsduck: Add 'mediawiki.startUp.js' to eg-iframe

Follows-up 2ea420eed86. Without this, the startUp() is never called,
and mw.config remains undefined which caused some of the examples
to no longer work properly.

Also move onerror handler to first <script> (follows-up a64a07d).
Exceptions from other code (eg. startup or mediawiki modules),
were not being caught.

Also include 'error' in the line being logged from onerror.

Change-Id: I3956da40b0bd862acab6d4552932aa609c17d569

jsduck: Refactor example iframe to catch uncaught errors

* Make logging function not depend on 'mw' existing.
* Add window.onerror handler.
* Keep exporting to mw.log because that's what existing examples
  use and is what uses should copy for interoperability with the
  regular runtime.

Change-Id: I2925532035974343125e34084f9ac7c8dd1fa279

Allow edit summaries to be up to 767 bytes long

This is just the db part. The changes to UI validation code will
come in a dependent patch later.

This changes the max size of various *_comment, *_description
and *_reason fields to be 767 bytes. This size is chosen to
be consistent with the largest we can get away with while
still having covering indexes.

It also unifies the various data types of these fields to
varbinary. (Note, this was previously done and reverted in
r80547, but I don't think the reason for revert applies to
the edit summary fields)

MyISAM has a much smaller max index size, and this wouldn't
work with that storage engine. However, none of these fields
are used in default indexes (only in WMF indexes), so this
should not be an issue for anyone using MyISAM. Postgress
and sqlite already supported the larger edit summary size.

Note: This patch increases min mysql version to 5.0.3.

Bug: T6715
Change-Id: I8558e80a18e4591f07f2c3e80f792ea4435c4e71

API: Remove explicit profiling

The profileIn/profileOut pair should be covered by the Xhprof profiling
of the method calls it was wrapping.

The profileDBIn/profileDBOut pair are covered by profiling done by the
Database class.

Nothing in extensions in Gerrit is calling anything besides the
profileIn/profileOut pair (and likely those are only to avoid core
formerly throwing exceptions from internal profileDBIn/profileDBOut
calls), and nothing in core or extensions-in-Gerrit is using the methods
for fetching profiling data.

The methods are left as stubs for now to allow for backwards
compatibility in extensions.

Change-Id: I05ba4e2762dc86d5e2bafc183dce701239b43f5c

Made JobQueueFederated no longer need "checkDelay" for delaying

* Cleaned up the weird logic for checking if delays are supported

Change-Id: I3a1a78bb48b4950b76f445bf8c27b5868d3dc89c

Localisation updates from https://translatewiki.net.

Change-Id: I3be2b9a50b5695cd9c247bb9a0a49af82eed6ffd

Merge "mw.ui: button: Update focus state"

Merge "Introduce the Restbase Virtual REST Service class"

Merge "Update the Chinese conversion table for Chinese WikiProjects"

Minor doc fixes

Change-Id: I80678e64cafcfd6e2c779452f466d8a630089568

Merge "mediawiki.jqueryMsg: Fix examples in comments"

Merge "Add StatsD metric logging"

Merge "Specify max compatible kss version"

Add StatsD metric logging

This patch adds a metric data service object to the IContextSource interface,
with full support for StatsD meters, gauges, counters and timing metrics, via
the liuggio/statsd-php-client, which this patch also introduces.

Usage example:

    $stats = $context->getStats();
    $stats->increment( 'resourceloader.cache.hits' );
    $stats->timing( 'resourceloader.cache.rtt', $rtt );

The metrics are flushed to a StatsD server, which may be specified via the
'StatsdServer' configuration key. If no such configuration key exists, the
metrics are discarded.

The StatsD client supplants MediaWiki's StatCounter class. wfIncrStats()
will continue to work, but it will delegate to the StatsD data object.

Change-Id: Ie10db1c154d225971398e189737de7c560bf0f90

Introduce the Restbase Virtual REST Service class

Restbase, the REST content API service, is to be queried instead of
Parsoid by current Parsoid users (most importantly VE). This patch
introduces the Restbase virtual REST service class and transparently
maps Parsoid calls into Restbase ones if parsoidCompat is set when
creating the service object.

Additionally, $wgVirtualRestConfig is introduced in DefaultSettings.php. This
is a first step towards global service configuration and management.

Bug: T89066
Change-Id: I4d4043e5052327bbd789331f1c05b607c45fe7cb

mediawiki.jqueryMsg: Fix examples in comments

Change-Id: Ia6fcb718e9c0ef073ed472af29c8ca4507f45b5a

Localisation updates from https://translatewiki.net.

Change-Id: I361e11d5dbbc13010b24e6dd49ff8153f5163535

Merge "If no secret key is available, don't try to use cache"

Merge "Added clarifying comments to IContextSource"

Rename JSDuck config to standard 'jsduck'

* Move configuration to /jsduck.json per standard. This way
  it can be run as plain '$ jsduck' without needing a maintenance
  script or custom Jenkins job. Similar to JSHint, JSCS, Grunt,
  and Gem etc.

* Move --processes=0 from maintenace script into config file.
  This should've been in the config file all along and serves as
  workaround for https://github.com/senchalabs/jsduck/issues/525.

* Use grunt-contrib-copy instead of a symlink for resources.
  For local development a symlink works fine, but for publishing
  from Jenkins to doc.wikimedia.org the /docs/js/ directory needs
  to be standalone. This was previously done with a manual post-build
  step that added an additional rsync, but this logic should be
  in the repository so that the doc entry point can be simplified
  and standardised to 'npm run-script doc' for all projects.

Change-Id: Iaaaac50ee78dd9ff8f24f1ef3a3685ad51cf33b2

Make PHP unit parser tests pass again after translation update

- sr-ec.json updated with 17607cf8965bf332f8c80d7a8e25a8c4e1107696

Change-Id: I7dfc9bcea2be85dd42d344858ce02246432f7fe1

Localisation updates from https://translatewiki.net.

Change-Id: Ifc570f5a883ef4269e9d85b9e634f09e95716581

FeedItem: Use full URL with protocol for <id>/<guid> in Atom/RSS feeds

Bug: T89975
Change-Id: Ib84932b70aa8194b7bdabe7c90d7fdcc66054a1c

Added clarifying comments to IContextSource

* Also removed some duplicated comments

Change-Id: I2c967d0f3219f3533378ecd5f9fccc40c8f9b219

Don't require JavaScript for addHelpLink styles

And avoid wfMessage.
Followup to 376c0a13df05f30367b2a89c2cac7bdab22600ed

Bug: T45591
Change-Id: I2f02c2a4db609356ff8be30f260490d19f3226d1

Merge "mediawiki.special.upload: One event handling function is enough"

Push enqueue-on-view jobs to the faster "local" queue

* This only matters for multi-DC setups, otherwise everything is local anyway

Bug: T91819
Change-Id: I0756b59ae1ba27a5d24cf7baef6d068b748e6e30

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

Localisation updates from https://translatewiki.net.

Change-Id: I5e91846766f3558874fd179fd7a1a3702901bda4

Merge "Remove isVolatile/markVolatile logic from LocalFile"

Remove isVolatile/markVolatile logic from LocalFile

* This is obsoleted by the "latest" flag
* Follows-up 5085a4b

Bug: T88506
Change-Id: I808c3a6f0edb3466cb600c06c5e742ab030baf14

Merge "Added EnqueueJob class that handles routing jobs to queues"

Update the Chinese conversion table for Chinese WikiProjects

The Chinese conversion table is substantially updated to fix a lot of
bugs reported in recent years, and the script generating conversion
table (LanguageZh.php) is also modified to facilitate the maintenance.

Zh-sg and zh-my is set to fallback to zh-cn to improve reading
experience, since there is only trivial difference among them, just like
zh-hk and zh-mo. Further optimization for zh-sg and zh-my will be
performed in local conversion table of Chinese WikiProjects.

Bug: T91620
Change-Id: I1bb0315d6d7a2c9653905654d933942e362bcc42

Merge "Made wfFindFile/wfLocalFile callers use explicit "latest" flags"

Merge "inputs.less: Change focus state"

Set up RuboCop configuration according to coding conventions

Updated RuboCop to the latest version.

Renenerated .rubocop_todo.yml file with “bundle exec rubocop
--auto-gen-config”.

See also:

https://www.mediawiki.org/wiki/Manual:Coding_conventions/Ruby#Base_confi
guration

Bug: T91485
Change-Id: I926d6ee114ccc01076e8df6ec3f95eb2af78c212

QA: Refactor for mediawiki_selenium 1.0

Refactored by following steps outlined in UPGRADE.md under the
mediawiki_selenium project.

Change-Id: I52717292df0541eb489b34ec083ec4a957f7546d

Made wfFindFile/wfLocalFile callers use explicit "latest" flags

* Callers that should not use caches won't
* Aliased the old "bypassCache" param to "latest"

bug: T89184
Change-Id: I9f79e5942ced4ae13ba4de0b4c62908cc746e777

Merge "resourceloader: Call setName() in test suite before calling getStyles()"

Merge "maintenance: Defer initilization of 'main' Config until after Setup.php runs"

resourceloader: Call setName() in test suite before calling getStyles()

This caused a database error due to NULL being inserted as name, which is illegal.

> Function: DatabaseSqlite::replace/single-row
> NOT NULL constraint failed: unittest_module_deps.md_module
> Stack trace:
> #3 includes/resourceloader/ResourceLoaderFileModule.php(420): DatabaseSqlite->replace()
> #4 tests/phpunit/includes/resourceloader/ResourceLoaderTest.php(88): ResourceLoaderFileModule->getStyles()
> #5 (): ResourceLoaderTest->testLessFileCompilation()

This test shouldn't be trigggering database updates, but that's for
a later change to stub out or refactor.

Bug: T91567
Change-Id: Ic451bd41e2ffc188d2efd6b7ce61b03b9de61296

Merge "mw.Feedback: Terms of use always appended and allows for link"

Specify max compatible kss version

core/docs/kss/styleguide-template/index.html uses {{whenDepth}} and
other functions deprecated in kss 2.0, so set an upper version bound in
package.json

Bug: T91725
Change-Id: I55669f3cc1c34911f717fe5fce3ea6ed00510177

mw.Feedback: Terms of use always appended and allows for link

Bring the terms of use link outside of the general checkbox message
and have it appear regardless of whether the checkbox is shown.
Also make sure it is parsed correctly if/when it has a link.

Change-Id: Ic1072690e33927e472f8ba529b16fa32280a6abc

mediawiki.test: Prevent "Error: expected" from going to the real console

Keeps the build logs cleaner and less confusing.

Change-Id: Ibbbf5dd367cab798885df34b9dd73f84229d3c5e

Remove rel="archives" from HTML output

Follows-up fab055d6a1.
Introduced in 4299d27956 (r45988)

That link type was removed from HTML5 in 2011. It is not recognized
by the W3C Markup Validator.

https://html5.org/r/5925
https://www.w3.org/Bugs/Public/show_bug.cgi?id=11486
https://html.spec.whatwg.org/multipage/semantics.html#linkTypes

Change-Id: I61ddd7ee3df41bfb62423d0498217d31e7952902

Unbreak parser tests

Caused by f988ef004cd4e6c21649fb30a7d421aadae8f495, which would
have failed tests if l10n-bot hadn't self-merged.

Change-Id: I4674178f51c9b67bc057a84b06b3e4153857860b

Localisation updates from https://translatewiki.net.

Change-Id: I8fd55473983eff489284bbc3ff46c1bcffdb1ce1

Merge "Fixed detection of unsigned mysql column in updater"

Fixed detection of unsigned mysql column in updater

MediaWiki running with an old mysql or php without mysqli cannot
use the newly added flags property of Database::fieldInfo.
For backward compatibility add the missing properties using the flags
property to the result of Database::fieldInfo.

Added missing unsigned, numeric, blob and zerofill properties to the
MysqlField using flags in mysqli.
The missing fields in mysqli are taken from
http://php.net/manual/en/function.mysql-fetch-field.php

Follow-Up: I641dc1fad0a5bb14815f55e0357243a562fd672b
Change-Id: I1da904df72f8af7725fc7ff94121ef85b18a36f8

Remove duplicate param escaping code

wfEscapeShellArg() can handle multiple params, escaping each.
This patch changes wfShellExec() to call wfEscapeShellArg() directly
instead of doing the gluing itself.

This patch also extends wfEscapeShellArg() to accept an array parameter
optionally instead of as separate args, which is often useful.

Added also unit test cases for single, multiple args, and single array args.

Change-Id: I7a0761cc2ba98c210a9eacadd12da407d933e42a

Merge "Browser should clear cache for API responses"

ParsoidVirtualRESTService: Don't use "else if"

https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP#C_borrowings

Change-Id: Ica51fd60e3da75d3b882cf3172ef7500afeab0db

Merge "Made the "bypassCache" flag for file locator methods use the master DB"

Browser should clear cache for API responses

By default we send "private, must-revalidate, max-age=0" for regular
logged-in wiki viewing.  This changes API responses to match.

Later, someone should update the Cache-Control header generation so
that it works the same for API responses as it does for OutputPage.
This is becoming more important since we're using the API instead of
OutputPage for editing with VisualEditor.

Bug: T74480
Change-Id: Ib309df8568de2c7137b6d13b9ca4004150a772dd

mw.ui: button: Update focus state

The focus and hover states were the same. Updating it according to the
design specification - https://phabricator.wikimedia.org/M31

Bug: T88449
Change-Id: Ibdd5e1553174bfde82af3a23b746a3918b8f0416

Made the "bypassCache" flag for file locator methods use the master DB

* Added a File::load() stub method
* Cleaned up the File loading flags bitfield a bit

bug: T89184
Change-Id: I1aa4b096c0cad5f5ca34321cc897019005c53a76

Fix three more invalid double underscore special page aliases

Cf. f8207aeb17f955a79a726df775d3643c550605fa

Change-Id: I6e72273524c54016296c1a77d978fd6a872a542e

Update OOjs UI to v0.9.0

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.9.0/History.md

Change-Id: I73d20f6556558bd5675c11addfcd27d417173693

Merge "Profile wfShellExec() calls by caller"

Profile wfShellExec() calls by caller

Change-Id: Ide437ec586b9e123ca73b5abd17278a72375dfb0

Added EnqueueJob class that handles routing jobs to queues

Bug: T89308
Change-Id: Iadb34f24d8bbe94c0f9f119e530c0bbe1060df0a

Add "ValidSkinNames" to extension.schema.json and ExtensionProcessor

skin.json goes through the same exact code as extension.json does, and needs
to be able to add skins.

Change-Id: Idd0acaf6735b4c308d3ab584c49da4e97448347f

Localisation updates from https://translatewiki.net.

Change-Id: I4d53140c8b780ef5d6118ac535ab2a70c7e0082b

Merge "Shorten long lines in PrefixSearch.php to make phpcs pass"

Merge "API: Properly handle limit and continuation for generator=prefixsearch"

If no secret key is available, don't try to use cache

In the unlikely event that no secret key is available, we shouldn't
rely on the cache at all in TemplateParser.

Adding new compileForEval() function and and moving eval() outside
of if statement to prevent code duplication.

Also, if the template fails integrity check, generate a notice
instead of throwing an exception in case we change the secret key.

Change-Id: Id44fdcc9533fc8a9c77e84fcebaa064f602477c6

Merge "Add top help link to MediaWiki.org in several pages via indicator"

mediawiki.special.upload: One event handling function is enough

- Common event handling function for all radios
- Only remove `mw-upload-source-error` once per radio change
  (there is no way to disable a radio button without enabling another one)

Change-Id: I1e5c7c403efb26c74df39669491bc49602a9df4f

API: Properly handle limit and continuation for generator=prefixsearch

The search engine is queried for limit+1 titles, so we know whether to
output a continuation value. Actually do this for generator mode,
instead of just returning the limit+1 titles.

Bug: T91503
Change-Id: Ib80c6d3fa9e8b52ae1f5c58fbe74c8e909329e59

Merge "Add anchor "mw-oldid" for beginning of page content in diff view"

Add anchor "mw-oldid" for beginning of page content in diff view

* #top, #mw-content-text and others exist, but all point to the
  beginning of the diff, not of the actual page content.
* Pick mw- prefix and technical but familiar related name, in
  the hope it's not confusing nor too hard to remember.
* Lowercase letters, similar to #top, #footer, #file etc. were also
  considered and discard, inspired to usual names:
  https://www.wikidata.org/wiki/Q10966628

Bug: T4313
Change-Id: Ia90f2541327480535e9943ac30bfa19791a19620

Add top help link to MediaWiki.org in several pages via indicator

All the chosen targets are translatable public domain help pages
on MediaWiki.org. Mostly special pages and actions for privileged
users for now.

Adapted from the Translate extension, credit to Niklas Laxström
(TranslateUtils::addSpecialHelpLink).

Depends on 6f5b29ff4e6fdf21b5a8cacaf10d6aceaee26a7d, whose commit
message has a typo addIndicator() instead of setIndicator().

Bug: T45591
Change-Id: I2934b1708a0d207dcf3d940264f140613646f203

maintenance: Defer initilization of 'main' Config until after Setup.php runs

Bug: T90680
Change-Id: I72d6306404bc1f6c7a605853130a026e9858b493

Shorten long lines in PrefixSearch.php to make phpcs pass

Change-Id: Ibf18835ff614e53887912b8e997a3230ed29dd68

Merge "Remove unused alias/layer of test abstraction wfShellMaintenanceCmd()"

Merge "Moved some JobQueueAggregator logic out of JobQueueGroup"

Remove unused alias/layer of test abstraction wfShellMaintenanceCmd()

Change-Id: I28d303f9b89ffad052d06eb2dbb9e6f8cfaeea24

Merge "Removed obsolete "containsOldMagic" code"

Moved some JobQueueAggregator logic out of JobQueueGroup

Change-Id: I28ba1a25db225d4cf5f503a6c0f4405f13118151

Merge "Delay language conversion cache construction until needed"