Allow more-descriptive section names for parserTests

This patch allows `!!wikitext` (for `!!input`) and `!!html` (for
`!!result`). This is more in line with what those sections actually
contain and closer to the semantics of how they are used.

The old names are accepted as aliases to accomodate parser tests to
provide a migration path for extensions and other users of the
parser tests framework.

In addition to `!!html`, this patch also accepts `!!html/*` and the
more-specific `!!html/php`.  This allows tests to include a number
of different "outputs" for a given wikitext input, for example
`!!html/parsoid` and `!!html/php`.

Co-authored-by: C. Scott Ananian <cscott@cscott.net>
Co-authored-by: Arlo Breault <abreault@wikimedia.org>
Change-Id: Ie4e68960ca7c352af495ebb59ba83488935a44c4

Merge "mediawiki.api.watch: Don't use deprecated 'title' parameter"

mediawiki.api.watch: Don't use deprecated 'title' parameter

* Instead handle it client side, and not as a deprecated feature.
* Rename 'page' parameter to 'pages'.
* Add unit tests.

Change-Id: I81ec4282aedd91e3f0bb2ed0d6dd34e437b54f5d

Merge "Fix CodeSniffer errors and warnings"

Add $wgLocalInterwikis to handle multiple local prefixes

This is akin to $wgSkipSkin/$wgSkipSkins. It is quite plausible for a wiki
to have more than one self prefix (e.g. enwiki has w: en: wikipedia: and
maybe others).

Some recent changes code seems to use $wgLocalInterwiki for quite unclear
purposes:
- I removed the line using $wgLocalInterwiki from the RecentChange
  class, as the 'lang' field of $mExtra is not used anywhere in core code.
  Extensions may use it, but it would seem more appropriate for them to
  use something like $wgDBname (or indeed to consult $wgLocalInterwikis
  directly) if they need to identify a particular wiki.
- In the IRC formatter, the first prefix in the array is used (if set).
  Appropriate documentation is added to DefaultSettings.php.

Related to bug 954 comment 3.

Bug: 954
Bug: 955
Change-Id: I9dbb566385b464402c5e78510b95dd2ffb4d9489

Merge "Fix some omitted colons in Spanish magic word l10n"

Fix CodeSniffer errors and warnings

For the files touched in I06ee0f01d8 as logged at
https://integration.wikimedia.org/ci/job/mediawiki-core-phpcs-strict-HEAD/7798/console

Change-Id: I811884f7a5ada4d0551552d0a2c2775d6b17b59f

ChangesListSpecialPage: Separate all functionality for generating feeds

I should have done it at the beginning instead of trying to
extract it from recent changes.

Same for SpecialRecentChanges and SpecialRecentChangesLinked
(subclasses).

Created a new API module for it: ApiFeedRecentChanges.
It's somewhat un-API-like and hackish, but all feed modules are.

Old URLs redirect to new ones, so this should be fully
backwards-compatible assuming sane feed reader clients.

Change-Id: I06ee0f01d896bc66545a1800b24693ce7524e433

Merge "Partly revert 28b4dd5f for zh and zh-cn/hk/mo/sg/tw messages"

Merge "tests: Add unit tests for ResourceLoaderStartupModule"

Merge "qunit: Disable sinonjs-ie as it breaks the clock sandbox"

Merge "Check validity and availability of usernames during signup via AJAX"

Merge "Include all of /resources/mediawiki/* in jsduck index"

Merge "resourceloader: Pass jQuery twice to 'implements' module wrapper"

resourceloader: Pass jQuery twice to 'implements' module wrapper

This must be done in two parts. A follow-up change will update the wrapper
signature in ResourceLoader::makeLoaderImplementScript to bind the
arguments to '$' and 'jQuery'.

Because almost all event-handling and DOM interaction is mediated by
jQuery, ResourceLoader could provide a per-module breakdown of the work
performed on each page view by passing each module a copy of the jQuery
object that wraps certain functions (like jQuery.fn.on) in module-aware
profiling code. I expect that profiling would only be enabled on a small
fraction of requests. The effect on ordinary page-views would be a
possible small speed-up as a result of jQuery look-ups resolving in
module (rather than global) scope, but I expect the difference to be
neglible and it is not a part of my motivation.

Because ResourceLoader must bind both '$' and 'jQuery' to module scope,
we pass jQuery twice. A separate commit will update the wrapper function's
signature in ResourceLoader::makeLoaderImplementScript. This has to happen
separately to ensure that no module binds jQuery to undefined due to
being loaded by an out-of-sync version of mediawiki.js.

Change-Id: I0c9edac359ceb16fc273de1e51ea70322f18353e

qunit: Disable sinonjs-ie as it breaks the clock sandbox

The ie fixes file assumes running in the global scope. Since it
runs locally in our environment, it ends up destroying its
references to e.g. the Date object.

Then when calling sandbox.useFakeTimers, the Date constructor
will yield an object with no methods and tests will fail at a
random point when e.g. trying to access date.getTime().

* Remove 'clock' from the sinon properties we inject into the
  test context. We use the sandbox instead (just like for server).

  This also fixes a shadow clash with the tests that stored the
  clock instance in `this.clock`.

Change-Id: I6085bccf5038e4751da48ee765fe81560f58c6e8

Merge "When checking whitelist of extensions, only count last extension."

Localisation updates from https://translatewiki.net.

Change-Id: I87ed70aa87a61057fa81f0eae63363af85fd71a0

Update oojs to v1.0.8

Code:
 http://oojs.github.io/releases/oojs-1.0.8.js

Release notes:
 http://git.wikimedia.org/blob/oojs%2Fcore.git/v1.0.8/History.md

Change-Id: I3ae46a0050d74c5c4fa9004455328e0661cab2dd

Partly revert 28b4dd5f for zh and zh-cn/hk/mo/sg/tw messages

For unknown reason (maybe some bug), L10n-bot exported messages in
these languages which had been disabled for translation. Let's remove
them for now and observe whether the bot exports them again.

See I0007fd00579971d4d5bce5e4545492f87ea52867.

Change-Id: I70d384c3a01aff5bbade01917c92e7be9fb936f4

Add release note for removal of preference "noconvertlink"

Follow up 333bf3ae5b412fae1e4f57a62a220c941ef50536.

Change-Id: I7a3e391f56d2e2839b3210d79a9f5b630f0f6fed

Merge "Remove user preference "noconvertlink""

Merge "Add missing DROP SEQUENCE to postgres' tables.sql"

Merge "Cleanup MediawikiTestCase"

Merge "Throw exception when trying to stash unset globals"

Merge "Introduce stashMwGlobals method to MediaWikiTestCase"

Merge "Don't load the apex theme for oojs-ui if we're using Minerva skin"

Don't load the apex theme for oojs-ui if we're using Minerva skin

Change-Id: I12c9f91276d34e56d0e9f5bb2e990a72756dd6b9
Dependency: Ib9d52327425f8a12e12d82f1421f5cab0774c510

Localisation updates from https://translatewiki.net.

Change-Id: Ia03f6dcf21ee8e589758e039566adb661893804f

Merge "Further optimize User::saveOptions() by doing deletes by PRIMARY KEY"

Further optimize User::saveOptions() by doing deletes by PRIMARY KEY

* We do not care about gap locks here, just as in the case
  with new users. Races should be rare and are simply FWW.

Change-Id: Ie483a2d5df501619acbd868bf894dfc6c911a753

CSSMin: Don't do file_exists on random data in getLocalFileReferences()

If it's a URL, it obviously can't be a local file; check that first to
avoid PHP warnings about malformed paths.

Bug: 60960
Change-Id: Id784c089c3de8af79af7524ef5ab5cc2f7b8af9e

Merge "Add a link to Special:WhatLinksHere in deleting-backlinks-warning"

mediawiki.api.watch: Fix promise return value format

Follow-up to Ic77b546c.

Bug: 62422
Change-Id: I2ac9e0da0f1c825e1cefd98d21406fcb089c0827

Check validity and availability of usernames during signup via AJAX

This is done in addition to the server-side checks. Users who have
disabled JavaScript will not notice any difference.

The way this is done (via action=query&list=users API) means that we
can't check for all possible error conditions, both for the username
(e.g. these enforced by extensions like AntiSpoof) and other parts of
the form (e-mail, password…) – these are still handled server-side
only. Thus we intentionally never say that whatever the user typed in
is valid – we only warn when we know it's not.

Doing this "properly" would require some reworking of the internals of
the signup process and this way is already a huge improvement.

(Split off from the reverted a0c72523.)

Bug: 34447
Change-Id: I42c00b54651fd4e7217a862b3116c6113935f5ae

Merge "Let wildcard actions work in list=logevents&leaction="

Merge "Add final period to API module descriptions"

Add final period to API module descriptions

Change-Id: Icae68c1ab1fd0006e00a3a9a56ae8f831d3d0d45

Localisation updates from https://translatewiki.net.

Change-Id: I1d75011f4c5f78746fb1b05ef05858ac6a35eec4

Merge "Remove final period from api module descriptions"

Shorter lines in LanguageUz.php comments

Follows up gerrit change 116914

Change-Id: I75215416277eb8f5d9e16a1bf91637c673be0a48

Wrap long text

Change-Id: I37fe6fbdc8779ba86318e587c3d9e217bede400a

Remove API developer email addresses

They just end up being used for direct support requests that should
really be done on public mailing lists etc

Change-Id: If704ec8dff5fc4669165b580290cc29cf953a25b

Cleanup MediawikiTestCase

Cleans docs
Adds since tags
Fixes typos
Removes totally unused stuff
Adds scopes

Change-Id: I80d542196a0f2265aacdd8ae89f919773832c14c

Throw exception when trying to stash unset globals

If we are trying to stash an unset global then
throw an exception!

Also adds a test

Change-Id: I25f493a0a535201c08ca9624c2c650f61d9e256d

Introduce stashMwGlobals method to MediaWikiTestCase

This method is factored out from the existing
seMwGlobals method.

The Doc from the initial method is also split and
improved and since tags has been added

Also adds tests

Change-Id: I0637194d637abf485a245b00587743f0f6dd495a

Merge "Remove 2 false values returned in execute()"

When checking whitelist of extensions, only count last extension.

When we are doing blacklisted extensions, we count all extensions
as some programs (like apache sometimes) consider extensions that
aren't the final extension. However when doing whitelists we need
to only count the last extension, otherwise people can name files
foo.goodExt.BadExt. For example [[commons:File:Deamado ko.png.bmp]]

I do not believe this represents a security risk as bad files are
still filtered out. However it does allow unwanted files to be
uploaded.

Bug: 62451
Change-Id: Ie27c15f749812710571f432bc5915e498f8017e3

Deprecated useless BagOStuff::replace method

* Almost nothing actually uses this (one extension)
  as it doesn't really solve race conditions like merge().
* Removed subclassed versions too.

Change-Id: I4d4123afcc0576a90a3cbf1e4e6daff7ebce39c3

Merge "Cleanup SampleTest.php"

Include all of /resources/mediawiki/* in jsduck index

Not all classes/methods in these files have documention yet (the
only effective change is adding of mw.Debug), but at least the
other files are now in conforming syntax.

mw.Debug:
* Made method descriptions more consistent (a few of the "build"
  method didn't start the description with "Build").

mw.Feedback:
* There was no @class anywhere. Worked because @class was implied
  for the method name starting with a capital. It was still
  ignoring the @param tags signature for the constructor though.
  Fixed now.

Co-Authored-By: Mark Holmquist <mtraceur@member.fsf.org>
Co-Authored-By: Timo Tijhof <krinklemail@gmail.com>
Change-Id: I74c8dcfeac9754e38f9dec23873081e8fb6d51d4

Split long line

Change-Id: I719e63a9b1d6e1f1fd2031e443e2f40adfbcb4ca

Merge "Use setMwGlobals in ApiEditPageTest"

Merge "Update lessphp to b7cd5c79e8"

Merge "Fix phpdoc of class vars in ResourceLoader"

Merge "Update OOjs UI to v0.1.0-pre (efc7297353)"

Update OOjs UI to v0.1.0-pre (efc7297353)

New changes:
3ab4d6d Balance margins around icon on iconed ButtonWidget
8d69f53 Add missing 75% transparency to add-item
2c0ae23 Add indicators to demo page
1700e98 Localisation updates from https://translatewiki.net.
262cb8d Localisation updates from https://translatewiki.net.

Change-Id: If265e4760b69957a1cc0138f9834ea984763f1f8

Update OOjs to v1.0.7-pre (22e610a5e9)

New changes:
544b0a8 doc: Improve overall documentation and fix minor issues
293f89f inheritClass: Use Class.super instead of this.constructor.super
59c78c7 Registry: Remove redundant type validation logic in #register
22e610a Factory: Use Class.super instead of hard coding parent class

Change-Id: Idae726f126b15e3c884e5bbda24eb2b7c11d801f

Merge "Clean up ExportDemoTest"

Merge "CSSMin: Clean up the logic in getLocalFileReferences()"

PathRouter: Fix documentation typo

Change-Id: Ib8021c721e7438163d65ddf6b83ecee8fc1639c6

Merge "Revert "Rollback: Make update of recentchanges table later""

Merge "Remove unused stuff from tests"

Merge "Add a deprecation notice of $wgRateLimitLog in RELEASE-NOTES"

Merge "Remove HTML back-compat styles from I5fe0543e"

Localisation updates from https://translatewiki.net.

Change-Id: I02cf02d1e9ce69f9b23eb7ddd23b1482e4ab7891

Fix phpdoc of class vars in ResourceLoader

Change-Id: Iced0b17d0c964f3d137b6714946577c7d1a24e30

Clean up ExportDemoTest

Change-Id: I28574b6c83cbd1128ad97b4868cc365336a7d4b8

Remove unused stuff from tests

Change-Id: Iddabfbc80378b02fa4a2c58f80d50241be8105d3

Let wildcard actions work in list=logevents&leaction=

The documentation of leaction= shows actions like "thanks/*", at the
moment this will result in a search for log_action='*', which is always
a empty list.

Changing the validation of param leaction to allow specific any string
for the asterisk in this wildcard action.

Change-Id: Ia77e499909ce6f25ce74617367fc5b622ea9a7c9

Cleanup SampleTest.php

Removes stuff which isnt true

Change-Id: I9696fa2e7f969e1d4bcf234a05031fc9ed784227

Merge "resourceloader: Consistently pass inDebugMode to encodeJsCall() in load.php"

Merge "tests: Add ResourceLoaderTestCase and abstract context creation"

Merge "LocalSettings.php must be readable by the server"

tests: Add unit tests for ResourceLoaderStartupModule

Change-Id: I7671813e1d7b4ea75265608c22d8efe8805560e4

Merge "doc: Clean up documentation in ResourceLoader.php"

resourceloader: Consistently pass inDebugMode to encodeJsCall() in load.php

Still quite a few were being minified unconditinally. This is in
preparation for adding unit tests for the startup module where
it'll make reading the unit tests (and looking at the diff in case
of a failure) a lot easier if the strings aren't minified.

Change-Id: Ia06787e0ce608fcafac4596c980606d06107f517

tests: Add ResourceLoaderTestCase and abstract context creation

Change-Id: Ib4b265256e60a2f2109da73dc7edba6a75587ce2

doc: Clean up documentation in ResourceLoader.php

Change-Id: I33e47e1bfb7dbff1304e56033c62fb80dfefdcf0

LocalSettings.php must be readable by the server

Change check in WebStart to is_readable and output a different
text on NoLocalSettings template, when the file exists.

Bug: 61094
Change-Id: I4dbdb33e468bc731fa1d6eef8e0f66e1d552d0f8

Have ApiRunJobs run periodic tasks

Without this apirunjobs can't handle delayed jobs.

Small inline comment updates as well.

Change-Id: I85ba0ac99eaebc16eaa7e2eb7019546241f3149c

Merge "Fix Uzbek converter"

Merge "i18n: use "int:" for consistency"

Update lessphp to b7cd5c79e8

Source:
* https://github.com/leafo/lessphp/blob/b7cd5c79e8/lessc.inc.php
* https://github.com/leafo/lessphp/compare/261f1bd2...b7cd5c79e8

Updates:
 * 1d62556008: CS fixes
 * d3fd5cec9e: use the built-in php magic database instead of the system one
 * 34fdf60f32: data-uri: add tests and use lessc::findImport to locate the file
 * 72b25725d9: implement data-uri([mimetype,] url) from less.js 1.4.0
 * 5cf7eb272e: Update lessc.inc.php
 * 7e64e41a59: Fix color contrast and threshold calculation
 * 69adc23a92: Use property_exist to avoid "Undefined property: stdClass::$parent"

Change-Id: Ie626b4f1117b44669bd2c6c43a0186d95b36435a

Merge "Add visibility in PoolCounter classes"

Merge "Fix undefined property in PoolCounterWork"

Add visibility in PoolCounter classes

Change-Id: Ieb8916235fa37247aa11a4d1fad59d929997d0e6

Fix undefined property in PoolCounterWork

Change-Id: I033288d31dcef2120eb9239d1bc690d5cb0edda6

Merge "Update OOjs UI to v0.1.0-pre (8b13d2b7f7)"

Merge "jsduck: Clean up odd use of @returns instead of @return"

Update OOjs UI to v0.1.0-pre (8b13d2b7f7)

New changes:
b983861 Fix OO.ui.ButtonOptionWidget documentation header
621a966 doc: Improve and fix minor issues in base classes
59665a2 Localisation updates from https://translatewiki.net.
d9a0a7d Add icon and indicator widgets

Change-Id: I5cd34605aa13b3cd24dbda5e4c9a5ac9e4584561

Revert "Rollback: Make update of recentchanges table later"

The automatic patrolling on a already rolled rollback should be kept,
see bug 62157.

Bug: 62157
This reverts commit e6980c32f11a9ed3ed1bf7068d5ff569cb43b457.

Change-Id: I3b626452a0bab11e8b2a12545976cbd1df92ae1f

Revert "Add getMessage tests with Short and Long Contexts"

Looking at the getMessage function the long and
short contexts should be strings not bools.

I was misslead by the defaults thus these tests
aren't actually testing much and should be removed
to avoid misslead more people!

This reverts commit cd9711ea350e9367829702eddc0adad8c9d88afb.

Change-Id: I889d42f720bf678a3543860893d0fedfca757dce

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

Merge "Add test for Message::params"

Merge "Split MessageTest::testMessageParamTypes"

Localisation updates from https://translatewiki.net.

Change-Id: I14a8aeb0af4aa632d89d7b6e27eca33bf855f46d

Split MessageTest::testMessageParamTypes

Change-Id: I2099fa8a27a88bceb77c25e1ee08fb71eca318dc

Add test for Message::params

Change-Id: I66098828d72ac6cb594bf886d5a7f61a8616a1a0

Revert "Warn on account creation when username is adjusted"

This reverts commit a0c72523c7a2c9ca16329f903ca6b269ccccb787.
which has terrible effects on mobile.

Most browser inputs on a mobile phone default to lowercase meaning
a username with a lowercase starting character will always trigger the
warning.

This means the user will have to re-enter their passwords, fill out the captcha
again and their email again (as all these are sensitive and not retained when
reloading the form with the warning)

If this is truly wanted on desktop, more thought it needed for the mobile use
case.

Bug: 61416
Change-Id: I8b93e5a4ca687f87305ced4343dcb28ca2909939