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

The fast majority in core uses @return, a handful were using
@returns though (which works because JSDuck supports it as alias
for @return).

Also cleaned up minor things in touched areas:
* Hoist second description for jQuery.matchSrcSet (there can only
  be one, it was currently interpreted as continuation of the
  @return caption).
* Remove redundant @method.

Change-Id: I15d1d023f0424e38c58a252139e6bb2f0b36d2b4

Merge "Add test to validate special page aliases"

Merge "Added and updated Doxygen comments in content handler and diff namespaces"

Merge "jquery.makeCollapsible.test: Minor clean up"

Merge "Remove tests for jquery.delayedBind (deprecated)"

jquery.makeCollapsible.test: Minor clean up

* Combine plain var statements.
* Remove odd new line before `QUnit.start();` (the other tests
  didn't have it).

Change-Id: Ibd3bf643797358f9bb027a1b3b1264fb67f4f7f4

Remove tests for jquery.delayedBind (deprecated)

It's slated to be removed and deprecation warnings
are polluting test output.

Change-Id: I880d75874926f0945f9aba72cc176fb6eda6195b

Added and updated Doxygen comments in content handler and diff namespaces

Reasons for touching this are:
* "@param type $var" were mixed in a lot of places. Both works but the
  MediaWiki coding conventions suggest that specific order.
* Things like String and Bool aren't objects and shouldn't be uppercase.
* Tried to fill missing types in "@param $var".
* Tried to fill missing descriptions in "@return type" when I could.
* Removed duplicate descriptions if a @see is sufficend.
* Removed useless descriptions ("isUsefull returns true if usefull").
* Removed useless @return void.
* Replaces mixed[] with array (does have the exact same meaning).
* Tried to find better replacements for "varargs", phpDocumentor
  suggest $var,...
* Order should be @since, @param, @throws, @return, @see. This is the
  order Doxygen renders this.

There is always more to do but I think this is already much better
than before. Please feel free to put more change sets on top of mine
or request more changes by adding comments.

Change-Id: I05262ce06caabf79f68772302524ac91bbead1c6

Merge "API: Fix logic for list=recentchanges prop=userid"

API: Fix logic for list=recentchanges prop=userid

When prop=userid is requested, we need the rc_user field; when prop=user
is requested, we need both rc_user and rc_user_text.

But for some reason when only prop=userid is requested the code is
requesting only rc_user_text and not rc_user. Fix that.

Bug: 62301
Change-Id: I86f1f35ebd18e45335cdf43ab3ba0a3e9b0e06b4

Update SinonJS to v1.9.0

Source:
* http://sinonjs.org/releases/sinon-1.9.0.js
* http://sinonjs.org/releases/sinon-ie-1.9.0.js

Changelog:
* http://sinonjs.org/Changelog.txt

Change-Id: I754ec906c2fc61b0e55bb61876fc7aafe2b3a67c

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

New changes:
e1b2362 Rename OO.ui.MenuSectonItemWidget.less to the right name

Change-Id: I2a147ae2a86e8c893829798f133c249e4c9759af

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

New changes:
b19e296 Localisation updates from https://translatewiki.net.
4813d30 WindowSet: Add support for overriding the creation a window
218d29f Localisation updates from https://translatewiki.net.
50fd396 Add more dialog sizes and make them switchable
22f405e Make it possible to toggle the outline in a booklet layout
e4f6ce0 Hack for preventing opacity-related rendering glitches

Change-Id: If55568c69333d4749b285408d27fa6e9e42a7f37

Merge "Fix jQuery compatibility issues in jquery.color.js"

Fix jQuery compatibility issues in jquery.color.js

Fix two issues which caused jquery.color to be incompatible with
the currently used jQuery version:
* The $.fx.step hook in jquery.color.js relied on fx.state which is
  not provided by current jQuery. This commit replaces it with
  a manual flag (the same logic is used in the official jquery.color
  plugin).
* It used $.curCSS which has been replaced by $.css.

Change-Id: I3aa0d912b21da40167124c646fb1b35beb277a33

Merge "Warn on account creation when username is adjusted"

Merge "qunit: Use sinon sandbox for mediawiki.api.parse.test"

Merge "mw.loader.store: Wrap script eval in try/catch"

Warn on account creation when username is adjusted

The user name can be adjusted due to various technical restrictions:
the first letter is capitalized, underscores are changed to spaces,
numerous other less visible changes happen. Some of these tweaks can
be unwanted by some users.

Generate a warning if that happens. If the user has JavaScript
enabled, the check happens entirely client-side – a little warning box
is shown, the user doesn't have to do anything. Otherwise the check
happens server-side and the user has to resubmit the form.

The way this is done makes it trivial to also check if the username
is invalid or already taken, so let's do that. It also means that we
can't check for all error conditions, e.g. these enforced by
extensions – that is still handled server-side. (Client-side we
intentionally never say that whatever the user typed in is valid – we
only warn when we know it's not.)

API behavior is unchanged.

Co-Authored-By: umherirrender <umherirrender_de.wp@web.de>
Co-Authored-By: Bartosz Dziewoński <matma.rex@gmail.com>
Bug: 34447
Bug: 61416
Change-Id: Ic461a5e597ad71b854dc65bbf8a395c0f55d1fc3

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

Localisation updates from https://translatewiki.net.

Change-Id: Idcbfe9491adcb996ea88dc2bc9040e9640b6d4fc

Merge "HTMLForm: Clean up 0 handling"

HTMLForm: Clean up 0 handling

PHP, particularly with in_array, really has problems with integer 0
versus non-numeric strings. Let's clean that up by converting values to
strings more agressively and using the $strict option to in_array.

Oddly enough, the one place where strict in_array was being used already
broke when If4175332 stringified the values in one array but not the
other.

Bug: 62268
Change-Id: Id34e654eb8d0e70d093b11445273e542e491e265

Merge "Add mediawiki.ui.button to all pages so wiki content can use it"

Merge "Sync up with Parsoid parserTests."

Merge "Catch Serialization Exception in MediaWikiTestCase"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 5a274b028d774784d514439f4539507e52c8e05e

Change-Id: Ie2699166ac378f758145d41d727b18f8fa6072b4

Catch Serialization Exception in MediaWikiTestCase

In the case that something is not serializable then
we should fall back and just set the value.

This is the case for Closures..
See
http://wikidata-jenkins.wmflabs.org/ci/job/wikidata-client-tests/748/console

Change-Id: Ib5f76932d60840a2f97671c134e18adc619974cd

Merge "Add @deprecated hint to $j"

Merge "mediawiki.api: Emit warning when deprecated callback parameters are used"

Merge "Add tests exceptions"

Merge "Make sure we don't use objects by ref in setMwGlobals"

Make sure we don't use objects by ref in setMwGlobals

If we copy am object by ref into our temporary array
then this doesn't really make this method very useful
as the same object would be restored, with any changes
after the test has run.

Thus objects should be cloned when being dumped in
here, we use unserialize() and serialize() instead as
we would also want to clone the objects within the
object to avoid changes in there!

These is no point in adding an if is_object call so
we just serialize and deserialize everything!

Change-Id: I5e4ef114405888932014edae6b44afc6ee2c4863

Add @deprecated hint to $j

Follow-Up: I871a0906730a1038311256e3e6e8ce3a293bd9ca
Change-Id: I0d9204ea354316ca11f568c81ad38f4089c91e80

qunit: Use sinon sandbox for mediawiki.api.parse.test

This is the last of the mediawiki.api tests that was still making
real requests.

This test had become a parser test for no reason. It is supposed
to test the logic in mediawiki.api.parse which is only:
* making the correct request url
* extract data from response

Change-Id: I9b6eeabe1baa1de7d77623902ef71ba031eb40ac

mw.loader.store: Wrap script eval in try/catch

All sorts of things can happen to our localStorage entry, from
clueless users through malicious scripts and browser bugs to hard disk
failures. Let's not explode irrecoverably when the code from store
refuses to execute.

Bug: 57567
Change-Id: I1b40a82595fe37dfeeb3648f7ecfbcac01bf9079

Merge "Re-sync OOjs and OOjs UI update scripts"

Deprecate window.$j

Bug: 62198
Change-Id: I871a0906730a1038311256e3e6e8ce3a293bd9ca

mediawiki.api: Emit warning when deprecated callback parameters are used

The "ok" and "err" parameters have been deprecated since 031910f7fe.

Bug: 61372
Change-Id: I38faa01128a66a7a4442602b29a6492c7e073e20

Merge "code-quality: Special:PrefixIndex is transcludable"

code-quality: Special:PrefixIndex is transcludable

Replacing 'id' attribute with 'class' to prevent pages' HTML from
becoming invalid due to using the same id.

Change-Id: Ibf561318a5005424e8920e693a62ccdb05bbd765

Merge "mediawiki.page.ajax.watch: Clean up and simplify code"

Merge "mediawiki.debug: Migrate CSS to LESS"

Add mediawiki.ui.button to all pages so wiki content can use it

Change-Id: I835f3eb9370982e2696a2af5aad451eda7cd786d

Merge "Add site by navigation id feature to SiteList"

mediawiki.debug: Migrate CSS to LESS

(first step to make some minor design changes to the debug toolbar)

Change-Id: I5232ebec65f5ba10a68fc531ce3647ebc4768ccb

Hide message for DBUnexpectedError exceptions

DBUnexpectedErrors are now treated like most other exceptions; we now
hide the error messages (which could contain sensitive information
such as IP addresses) unless $wgShowExceptionDetails is true.

In that case, the messages (and redacted backtraces) would be shown
even if $wgShowDBErrorBacktrace is false, though hiding them would
add complexity to the exception handler for little benefit.

Bug: 26811
Change-Id: I063c241975ce5b12a04abc21821ac67c716b3d5e

Merge "Remove another undefined $env and TODO comment for it too"

mediawiki.page.ajax.watch: Clean up and simplify code

Change-Id: Ie66d08b84a39ad260866de6e0ff821e356133ad0

Merge "Remove unused $terms parameter"

Remove unused $terms parameter

None of the following extensions use these parameters:
- CirrusSearch
- MWSearch
- SolrSearch
- SphinxSearch

Change-Id: I9686ca0849f35a3a09bd9996e4c6eac4334a342a

Localisation updates from https://translatewiki.net.

Change-Id: I6301ccc1c875aea9851b5d134ba3061a094ebaf4

Add site by navigation id feature to SiteList

There are some places especially in Wikibase where we need to get a site
based on the local navigation id. This feature really should go into core.

Change-Id: I02eb1673bea58f79d02db2e1ecd21cbc5b8cb0d1

Merge "mediawiki.loader: Better document IE stylesheet limit bug"

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

New changes:
0eef4f9 Make ButtonOptionWidget background transparent
ba28817 Remove /* OO.ui.… */ comments from LESS theme files
8dad3cd Actually add new ButtonOptionWidget theme to build

Change-Id: I20a0f5163dc23b8f2ff9067cb971ab0db7207f8e

mediawiki.loader: Better document IE stylesheet limit bug

Also leave a comment on the line where the relevant exception
will be thrown from if it does end up hitting the limit still
(which wouldn't be due to mw.loader appending more than 31
stylesheets but due to there being more than 30 on the page
already).

Change-Id: I3a46affc5498fbaa1d442ca65cb4844dd8f8169c

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

New changes:
f0e179c Break up stylesheets into separate files to make a 1:1 ratio of JS and LESS
3290d40 [BREAKING CHANGE] Split styles up into base styles and a theme

Incidental changes:
* Add oojs-ui-apex.css to the commit build script
* Add oojs-ui-apex.css to the resource manifest as the default skinStyle

Change-Id: Ib5ef9125528fdff9e71864d9016c5f6a12aa8ef3

Merge "Only generate date header when rendering is guaranteed"

Merge "i18n: Use int: in message "resetpass-expired-soft""

Merge "Split date and time in message 'rclistfrom'"

Split date and time in message 'rclistfrom'

Some languages need a split of date and time to make better translation.

Added a new parameters for 'rclistfrom', converted the whole message
to a link, updated Qqq entries.

Bug: 19104
Change-Id: Id9127cf022be2394b2a4843572088e0ba8a888a8

Deprecates $wgRateLimitLog in favor of debug log

$wgRateLimitLog is meant to log rate limiting occuring in MediaWiki. The
feature was introduced by e46d19e72d3 / r9202 in 2005.  We might not
have wfDebugLog() at that time and we used error_log() with the global
pointing to a file.  A later patch converted to file_put_contents().

On Wikimedia production the variable is set to a udp:// URL, I am not
sure file_put_contents() can write to it.  Additionally the warning are
suppressed which I tend to hate.

Instead use the log bucket 'ratelimit'.

For backcompabitilibity purposes, if $wgRateLmitLog is still set:
attempt to set $wgDebugLogGroup['ratelimit'] with its value.

Change-Id: I86131c4a8053800afd9f41fa1906c998a88736c6

Merge "Add title to list item of language link"

Added missing space to error message

Change-Id: I9c2feda19b6c959e125af272c0e64b646584769d

Merge "Fix grammar in 'deleting-backlinks-warning' message"

i18n: Use int: in message "resetpass-expired-soft"

Spotted by Siddhartha Ghai
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Resetpass-expired-soft/en

Also replaced "to login" with "to log in" in message "resetpass-expired".

Follows-up 01c8c421f8bc.

Change-Id: I8fc039ab1867a5bc5a0ade94daad49d9d85edd98

Re-sync OOjs and OOjs UI update scripts

Import the changes in 674b2458a25 in the VisualEditor repository where these scripts
were first created (and are still run for non-MediaWiki users).

Commit message in that repo justifies.

Change-Id: I711f3c88cbc1916f3b782771095304b186f7405f

Use a secondary db connection for Revsion::getParentLength in api

Getting api connection before the connection is set to the named
connection and store it in a local var to use it later.
Same comment as in Special:Contributions

Bug: 61982
Change-Id: Ia66c4a65a24876e2a528034e8e003aa535e6b689

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

New changes:
3372b3d Localisation updates from https://translatewiki.net.
d02bccd Localisation updates from https://translatewiki.net.
e01b627 Convert remaining browser-prefixed rules to LESS mixins
0e1cbd9 Allow focusing of inputs in booklet layouts without continuous pages
e87a54e Add maxrows option to OO.ui.TextInputWidget
f3bc5c6 Adjust the size when a textarea is inserted into the DOM

Change-Id: I166c49e69bf49bf1d76dee85355a406f557e6004

Merge "Add Updater for Oracle"

Merge "Remove useless "src" param when wrapping doGetFileStatMulti()"

Only generate date header when rendering is guaranteed

The OldChangesListRecentChangesLine hook can skip rendering of
a particular rc line.  If that line was the one that would have
added a new date header then no date header is output.  The
pushes checking for a new date header until after we know a
line will be output so none of the headers get lost.

Change-Id: I64ddd99c6af0b562802504b803563cf77fc2eb28

Add Updater for Oracle

Bug: 61015
Change-Id: I748d7531be90155d8b99472d3459e46ce4855f07

Merge "Allow configuration of periodic task interval"

Localisation updates from https://translatewiki.net.

Change-Id: Iac95418170688f90f53dcc28f6c429f0e79b0ac4

Merge "Make it possible for extensions to specify which version of MediaWiki they support via Composer."

Merge "Add getMessage tests with Short and Long Contexts"

Add title to list item of language link

The key itemtitle can be used to set the title of the actual list item
element. This is useful for article badges as it is already done by the js
hack to add a title for the icon.

Bug: 60717
Change-Id: Ic05c7535d7a35d9ee523373e1e3620e72887459d

Remove useless "src" param when wrapping doGetFileStatMulti()

Change-Id: I6008384ea872d71dcd2d183186a4aee5048b4dd4

Special:Version: Treat AUTHORS.txt and CREDITS.txt as text, not wikitext

A review of existing extensions in git reveals that 3 of the 5
extensions that have a non-wikitext authors/credits file have this file
named AUTHORS.txt, while none of the 8 that have wikitext
authors/credits have a .txt extension. So let's fix the 2 that have
non-wikitext as "AUTHORS" and do this for the benefit of the other three
that presumably don't want to use wikitext in authors/credits.

Bug: 62153
Change-Id: Id95852ab9836fb284becd7e251374f672b69515f

Split the rest of Action.php into /actions/

Change-Id: I071ac5778af63a5cffffd59d804c99b2c799d4e5

Make it possible for extensions to specify which version of MediaWiki
they support via Composer.

This change allows extensions to specify they depend on a specific
version or version range of MediaWiki. This is done by adding the
package mediawiki/mediawiki in their composer.json require section.

As MediaWiki itself is not a Composer package and is quite far away
from becoming one, a workaround was needed, which is provided by
this commit.

It works as follows. When "composer install" or "composer update"
is run, a Composer hook is invoked. This hook programmatically
indicates the root package provides MediaWiki, as it indeed does
when extensions are installed into MediaWiki. The package link
of type "provides" includes the MediaWiki version, which is read
from DefaultSettings.php.

This functionality has been tested and confirmed to work. One needs
a recent Composer version for it to have an effect. The upcoming
Composer alpha8 release will suffice. See
https://github.com/composer/composer/issues/2520

Tests are included. Composer independent tests will run always,
while the Composer specific ones are skipped when Composer is
not installed.

People that already have a composer.json file in their MediaWiki
root directory will need to make the same additions there as this
commit makes to composer-json.example. If this is not done, the
new behaviour will not work for them (though no existing behaviour
will break). The change to the json file has been made in such a
way to minimize the likelihood that any future modifications there
will be needed.

Thanks go to @beausimensen (Sculpin) and @seldaek (Composer) for
their support.

Change-Id: I8df66a92971146ab79cd4fcbd181e559115ca240

i18n: use double-quotation for consistency

Change-Id: I6f5dd76db9db4467e6535254021d54b5d5f58f78

Remove another undefined $env and TODO comment for it too

Follow-Up: Ie69a68c8ff429ac79796890d1843cf2a2b9c32e8
Change-Id: I767cb09215f8b51a9e88c770d3b0f87c37452d00

Localisation updates from https://translatewiki.net.

Change-Id: I2dd97b0dda0af44be6ad3979d5934554a4d7e31c

Merge "SpecialUserlogin: Normalize username before passing to User::newFromName"

Update messages.inc and rebuild MessagesEn.php. Again.

Follow-up Id5c0225431 (7d923a6).

Change-Id: I663c28aa05a42917ba3988a02c2609b6c4e9a438

Merge "TableSorter: Fix column order when collecting headers"

Merge "Provide direction hinting in the personal toolbar"

mediawiki.user: Deprecate #bucket

Rationale:
- On the Wikimedia cluster, it is only used by AFT, which is moribund.
- It's too complicated for what it is.
- Its usage leads to a proliferation of large, persistent cookies that
  degrade site performance.
- If it were rewritten to be simpler, more generic, and cookie-free,
  it would still be an odd fit for the user module.

Change-Id: I5a66e1a4fb60f6c67d1fb5312ccb629bb7964053

Avoid API error/header leakage from jobs API

* Also cleaned up some HTTP headers while at it

bug: 62092
Change-Id: Ia2c1643e47aae53916c107c901cca654dc438a16

Merge "Fix for CLDR plural rule parser for old browsers without String.trim"

Fix __toString method of DatabaseMysqli

The superclass was casting mConn to string. For Mysql class this
is just the resource number. However for mysqli objects, that is
a fatal. thread_id seemed like the most convinenent id-ish number
to return instead.

Only really noticeable if you have $wgDebugTransactions = true;

Change-Id: I014bb7ab81d18c5bd07a267939b66a0a6161eb8d

Merge "SpecialRecentchanges: Don't use nonexistent messages for filter selector"

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

Update OOjs UI to v0.1.0-pre (51f922ba17)

New changes:
1ba6137 Make fieldsets and stack panels more compact
584ed14 Convert styles to LESS and implement grunt task
ad906c5 Add documentation for -1 tabIndex option, which disables tab indexing
3f962db Manage active state of BookletLayout pages
5f3ee83 Disable vertical scrolling in continuous layout mode
96ae071 Make text inputs gray when disabled
51f922b Refactor OutlineControlsWidget, add remove button

Note that 51f922b requires the addition of another message to resources/Resources.php
(eww): ooui-outline-control-remove.

Change-Id: If55f8a78f410f35e38f40a9884d7057c50a4610f

Merge "Expand MWException tests"

Merge "Removed refreshLinks2 comment"

Remove AdminSettings.php hint from Installer message

Message 'config-localsettings-connection-error' contains still a hint to
AdminSettings.php, but last parts was removed with
I522ef138d291cf7567250fef1de34bb41673bc6c

Change-Id: Ic34524cbf7b62a37ee6386322f51e4c4df8e425a

Fix for CLDR plural rule parser for old browsers without String.trim

This fixes MobileFrontend's "last modified" message on Kindle 3.4

Submitted upstream as https://github.com/santhoshtr/CLDRPluralRuleParser/pull/10

Bug: 62072
Change-Id: I350264847417f521c111e3cb81da46805608425a