Avoid fatal errors with uselang=s

languages is not a subclass of Language

Force valid built-in language codes to be at least two letters long.

Bug: 46710
Change-Id: Ife8885ed4bdb3a017a54e063f5dc58468ba231d5

Merge "Added "cluster" parameter to wfWaitForSlaves() for external DBs."

Merge changes Ic22a190c,Ia5b37f99

* changes:
  Removed duplicate class registration
  Removed pointless static declaration

Merge "mergeMessageFileList.php: abort on read error"

Removed useless clear() call in WikiPage::doPurge().

Change-Id: I3bc251ea987f089342c054afd5a02ccf321f4fe7

mergeMessageFileList.php: abort on read error

If there is a parse error or if one of the files in the extension list
doesn't exist, exit with an error. mw-update-l10n has set -e so it will
abort without syncing the new file.

Change-Id: Idaad65783127b075626c102a8dc02e22df1588b7

Added "cluster" parameter to wfWaitForSlaves() for external DBs.

Change-Id: I226a1793dfb901de8e1c0ac40c1dc3ad890de95a

Merge "Localisation updates from translatewiki.net."

Merge "Set lang in api createaccount regardless of $wgLoginLanguageSelector"

Localisation updates from translatewiki.net.

Change-Id: Id8c6772f24ba3a569be66e210a99b89c3091e9a3

Merge "Adding wfLogWarning for production warnings."

Remove ExternalUser authentication code

This was an experimental authentication system intoduced a couple
of years ago with a pretty narrow use-case. It's been pretty much
ignored since introduction, and makes login more complicated than
it needs to be.

I didn't drop the external_user table on the off-chance someone
out there actually has data in it, but they should use AuthPlugin
for their external authentication needs.

Change-Id: I794338dbb75961ee033d41fa44bb7aa22e54f447

Merge "Drop ClickTracking integration from mw.user.bucket"

Merge "Add to/tweak mediawiki and jqueryMsg JS tests, fix comment"

Improve mediawiki.js docs:

* Fix an error in the Map.set documentation.
* Use JSDuck formatting in more places.

Change-Id: I5e2d271b45c1dbb33fa67abe4a2cc79380831a23

HTMLForm button behaviour modifications

- Buttons can now be marked disabled and they will not be clickable.
- The default submit button may now be opted out of.

Change-Id: I8293a3c09da51152bfc7f6f6951416c71ce0bb38

Drop ClickTracking integration from mw.user.bucket

This change set drops the 'tracked' option from mw.user.bucket, which
previous triggered logging via the ClickTracking extension.

Change-Id: I1a57df2ba60dba6041ecad7dd5c87519eb5d143e

Most important documentation fix

Change-Id: I0482144112345ca76ed3d95e3ad3b846cce4d87b

Merge "(bug 46615) updateCollation.php: sanity check the collation before proceeding"

Merge "Remove non-functional handling of curl_error()"

Merge "ResourceLoader: Honor $wgAllow* settings in site/user modules"

Merge "Test: Assert that modules loaded correctly."

Merge "OutputPage: Set empty modules to state 'ready' instead of 'missing'."

Add to/tweak mediawiki and jqueryMsg JS tests, fix comment

* Remove getOuterHtml.
* Use htmlEqual where getOuterHtml was used, among other places.
* The comment reflected a change I considered making, but did not in the end.
* Add a few link tests
* Minor tweaks to other test code for consistency
* Replace wikiGetlink with explicit output

Change-Id: Id6a857c9686901f8db4944fcc081d4b9c4457fa8

Specify the utf-8 charset on error messages.

Although the contents of error messages are English text,
and thus it's irrelevant that it gets interpreted as
iso-8859-1 instead of utf-8, $wgSitename is printed by
DBConnectionError::searchForm(), and that can contain utf-8.

Cached pages may also be in utf-8.

Change-Id: I81c02493f0824cb010c80f4356f338787d78df6b

Merge "jquery.makeCollapsible: events for collapsing/expanding, tests"

Merge changes Icdd8f392,Ied13e5ab

* changes:
  jquery.makeCollapsible: basic test suite
  jquery.makeCollapsible: use 'mw-collapsible' event namespace

Merge "(bug 36400) API: Fix sorting for iwlinks, langlinks"

ResourceLoader: Honor $wgAllow* settings in site/user modules

Bug: 46858
Change-Id: I17b3a2c1df5d52458f92c715242003b34111432c

Test: Assert that modules loaded correctly.

Since mw.loader catches any uncaught exceptions that modules
throw at run time, it is hard for QUnit and/or the browser
(e.g. PhantomJS) to know about a failure because there is
no assert failure and no uncaught exception (window.onerror).

VisualEditor's init module, for example, has a problem (bug 45175)
that sometimes causes an error, causing none of the tests to run,
yet the QUnit run would finish successful with simply none of the
VE tests included in the results.

Bug: 44299
Change-Id: Ib6e2b8d1be3e38fd9f1b948407c62da550fce0b4

Added Job::hasRootJobParams() convenience function.

* Added a missing @since tag.

Change-Id: I887d55074007551040eee486dbb9e8e44106e5a9

OutputPage: Set empty modules to state 'ready' instead of 'missing'.

Some cases were already covered by I595b3af190, but in the case
where the module is disabled entirely it was setting it to missing.

This is inconsistent with the behaviour of load.php (which will implement
an empty module resulting in state=ready) and confusing as it
implies there is a problem (a missing module) whereas in fact it
isn't a problem. Modules that depend on it to allow customization
wouldn't load as it will fail on missing dependencies.

Also makes asserting problems (bug 44299) hard as it would have
to whitelist state 'missing' for user/site/user.groups.

Bug: 46857
Change-Id: I46b5bc6f714abe4b66186d7b7a14c69195f00ee1

Added some missing @since tags.

Change-Id: I31c264aebf63f4bcaa3042f624e91dc27f5687e8

Merge "Localisation updates from translatewiki.net."

Merge "[FileBackend] Clarified the noAccess/noListing docs a bit."

Cleaned up return types for getAllQueuedJobs() and getAllDelayedJobs().

Change-Id: I305941c817dd13427d291b9265a71f9df9130f1c

Localisation updates from translatewiki.net.

Change-Id: I3cdedadb10249e5ea330e89e4d7f2ab187f804cb

(bug 36400) API: Fix sorting for iwlinks, langlinks

The iwlinks and langlinks modules continue parameters imply ordering by
page then prefix then title. But in certain modes, the actual queries
use a different ordering, which may result in skipped or repeated
results.

This changeset fixes that. To do so, it needs to re-add an index
iwl_prefix_from_title which was mistakenly removed in 2010 (r69721). And
while it's doing that, it cleans up errors in the sqlite and postgresql
handling of the iwlinks indexes too.

Also, per Asher, make the iwl_prefix_from_title and
iwl_prefix_title_from indexes non-UNIQUE.

Change-Id: I607e8bf9183a2d8152a6127a81c83a0b5bba0c61

Improve API 'origin' parameter description

When making CORS pre-flight requests, the origin parameter must still be
included. This means that it cannot be in the POST body, as that is not
sent for pre-flight.

Document that.

Change-Id: I3fc809cf2e4ca9c2566f62d32f96d24377c72863

jquery.makeCollapsible: events for collapsing/expanding, tests

Trigger a custom event to allow callers to hook to the
collapsing/expanding, allowing the module to be testable and making
it possible to e.g. implement persistence via cookies.

Added tests for the triggers themselves and for some basic
asynchronous collapsing functionality.

The following four events are fired (the namespace used is the same
one as the one used for firing the click event for initial toggle):
* beforeExpand.mw-collapsible
* beforeCollapse.mw-collapsible
* afterExpand.mw-collapsible
* afterCollapse.mw-collapsible

Change-Id: I580365770d061418a6734fcb54b2d86f2a821748

jquery.makeCollapsible: basic test suite

For now it just verifies that the very basic synchronous functionality
of the module works; not much more is possible without triggering some
events to hook to, and currently jquery.makeCollapsible doesn't
trigger any.

Change-Id: Icdd8f392e4921007a6bdfb3ae934949e4c875232

Remove spaces around ::

Change-Id: I24fe5ad907bcf5235aa91b650785c571502456a1

Merge "phpcs: Pass includes/resourceloader/"

phpcs: Pass includes/resourceloader/

- A cast statement must not be followed by a space.
- The method parameter $context is never used.
- Avoid function calls in a FOR loop test part.
- Opening brace should be on the same line as closing parenthesis.

Change-Id: I0eba7fcc9ceab372003d1134857346690c525e87

jquery.makeCollapsible: use 'mw-collapsible' event namespace

For consistency with the module name.

Change-Id: Ied13e5abdec107771e80418feb4bb3b2f0d54567

Deferred page_touched update via onTransactionIdle.

* This should reduce deadlocks and lock wait timeouts.

Change-Id: I595bc33d7643e7964d796b1d3da31f7cfab55024

Merge "In SqlBagOStuff::getMulti(), catch connect errors"

In SqlBagOStuff::getMulti(), catch connect errors

This case was somehow neglected in Ifba0d276. The other methods work
correctly.

Change-Id: If29f0212f07c13bd72469c580bb096a2a41a96b5

Merge "[LockManager] Added timeout to the lock() function."

Merge "Avoid duplicated effort in JobQueueAggregatorRedis::doGetAllReadyWikiQueues()."

[LockManager] Added timeout to the lock() function.

Change-Id: I32f97b813fc6036b2774d67c14c574420e7a1437

Merge "Add wfProfileOut to the new returns added by 19ecb69f"

Avoid duplicated effort in JobQueueAggregatorRedis::doGetAllReadyWikiQueues().

Change-Id: I76bfc23ecf94c974a9eefc7d430b26d70187409e

Merge "Use canonical class name."

Merge changes I8590352b,I164d60e0

* changes:
  ResourceLoader: Change $module by ref instead of resetting the key
  QUnit: Ensure test modules are top loading

Use canonical class name.

Change-Id: I7dc9980d334bc5ad6421135d713d890a8588281c

Add wfProfileOut to the new returns added by 19ecb69f

Change-Id: I97def12da054db7204955b95c3d590a3ab24ba55

ResourceLoader: Change $module by ref instead of resetting the key

Change-Id: I8590352b52ecf51def21a9c52af85aba0f050ff8

QUnit: Ensure test modules are top loading

Jenkins tests for extensions were exiting with a false positive.
They were marked "success" but in fact were wrong because
it only ran the core tests, the extensions' test suite was
loaded after QUnit already stopped listening.

Change-Id: I164d60e06118b64a1c6a17665a0fb1faeff744e5

Merge "Localisation updates from translatewiki.net."

Merge "Monobook: Solve padding issues with #content and #firstheading"

Monobook: Solve padding issues with #content and #firstheading

Makes it consistent with the Vector skin.

Bug: 26849
Change-Id: Iab04baea4131208373962e4ddf7c8ac83c7f2ca9

Merge "add index to image table to query by type"

Localisation updates from translatewiki.net.

Change-Id: I6a69fcf18f4a18174882fe6024ff9ae6d4b8b2c2

Merge "Update PHPDoc comments"

Remove non-functional handling of curl_error()

Per the docs, curl_error() returns a full error string, not an int
or string of an int. Just remove that and pass the error back up.

Change-Id: I7434bb2abda056fdee4d670c192b304c0cdf24f7

Merge "(bug 46787) API: Fix rccontinue handling"

Merge "API param validation: Set strict mode for in_array"

Removed duplicate class registration

Change-Id: Ic22a190c5d44e37e99067decfe8bec74422efcb1

Removed pointless static declaration

Change-Id: Ia5b37f99456614e8dbb1c44f3ebb4ad9398ce18e

(bug 46787) API: Fix rccontinue handling

There are several bugs in Icc43b62f:
* When handing the continuation, the code checks for dir=descending when
  the actual value is dir=older.
* When the above is fixed, the continuation code assumes "ORDER BY
  rc_timestamp DESC, rc_id ASC", which would filesort.
* rc_id is not added to the ORDER BY clause anyway.

Bug: 46787
Change-Id: Ia6ebd4ea0458b8013d4ecb71954dcfbdacff2c00

Update/fix PHPDoc

Also initilise two possibly undefined veriables ($cl/$dl) and
add a type hint.

Change-Id: I24d4f13fa03caf7861646837f185df4f1e077c6e

Update PHPDoc comments

Change-Id: Id20f632762d705ba5e5eb18088f6e9e551b3f42b

API param validation: Set strict mode for in_array

The in_array gives true when the allowed values contains a 0 as value,
for example by namespaces,

action=query&list=allpages&apnamespace=test|tset

was not rejected

Change-Id: I9220a955ffaf2bcb0d1d5b27c948af2f85714110

Merge "Remove removed message keys from maintenance files"

Deferred user_touched update via onTransactionIdle.

* This should reduce deadlocks and lock wait timeouts.

Change-Id: I7d028f9efbe6b2f73240aa653eb9775020f33e8c

[FileBackend] Clarified the noAccess/noListing docs a bit.

Change-Id: I3f008e3d1931423fc3c73af587d657040fdffe83

[JobQueue] Fixed use of ipairs() in JobQueueRedis.

Change-Id: Id507c0a3dfaf8b1fde92eb99f7204e31775b042c

add index to image table to query by type

this speeds up Special:MIMESearch and allows
queries for all files of one type.

Change-Id: I85715b0553a771ab88d9763ba921923551988bd0

Remove removed message keys from maintenance files

Follow-up to change-Id Ia6d73c2deb9428d214b7d69b29235094de75c52e

Change-Id: I17a9960265fa44c45468ec5cdc5b5839dce475cc

Merge "Remove several ancient skins"

Merge "Reduced indentation levels, broke long lines."

Remove several ancient skins

Standard, Simple, Chick and MySkin have all been removed.
Nostalgia has been moved to an extension for posterity.

Change-Id: Ia6d73c2deb9428d214b7d69b29235094de75c52e

Reduced indentation levels, broke long lines.

Also updated formatting, added docs for class variable $mNavigationBar, fixed
some hard coded parentheses and removed an unneeded else block.

Change-Id: Ic9a5f9f34199d89474cefbac763488cac1265094

[JobQueue] Added JobQueue::getAbandonedCount() and use it in showJobs.php.

Change-Id: I5d96c61165b2693589d5cf36309fdb6a8b5a137e

Merge "Localisation updates from translatewiki.net."

Merge "Several Parsoid-specific tests for serializer/parser edge cases."

Merge "Add docs for constructor"

Merge "Fix PHPDoc"

Merge "Remove unneeded else{} and return"

Merge "Unscrew something that doesn't make sense"

Localisation updates from translatewiki.net.

Change-Id: I11310d9ad7a01f38a5a4128c99f9d022a771f319

Merge "Removed "flush on destruct" from StatCounter."

Merge "Remove completely unused $wgRedirectScript/redirect.php"

Remove unneeded else{} and return

Else action was default. Returned the result of a void function.

Change-Id: I9ddf051ded0228dd3dfdfe9312f467d9c57f1716

Fix PHPDoc

Change-Id: Ic40017023fe21c8480883e21d27a44cf9854175b

Unscrew something that doesn't make sense

The probably makes the B/C work that never worked.

Found using PHPStorm's "Type compatibility" analyser.

Change-Id: I657c06aa7c8ef1ed5bc608a453f78adf0e04b375

Remove completely unused $wgRedirectScript/redirect.php

Change-Id: Ia624d65fbb1c787293054e12162b1444ab7c1edc

Merge "Remove unused local variables"

Remove unused local variables

Change-Id: I8eb774c2857dcc87404fd8a7e5fb66c5a4c9643e