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

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"

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

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"

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

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"

Use setMwGlobals in ApiEditPageTest

Change-Id: If2b23d82816c87e454bfc2348e316bdb92d2809f

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