Fix phpdoc of class vars in ResourceLoader

Change-Id: Iced0b17d0c964f3d137b6714946577c7d1a24e30

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"

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"

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

Merge "Give links to a user's suppressed edits on Special:Contribs"

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

Add visibility to DifferenceEngine methods

Change-Id: I415719442ca1778be36f173067d4b090fbe9265a

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

Fix Uzbek converter

Improve the Latin to Cyrillic conversion of "e" per https://meta.wikimedia.org/w/index.php?title=User_talk:SPQRobin&oldid=7720901#Request

The code of this change is inspired on LanguageKk.php

As far as I tested, it works as intended

Change-Id: I20f293c8d1f753af7b5344543ec2ce459edcba0c

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"