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"

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

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

(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

[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

Removed "flush on destruct" from StatCounter.

Change-Id: Ib6ff0680d858e12969259997da163d0d696d65eb

Merge "backport release notes from Ia6efdf931b0b6be7556fc16c040aa7493845cdfc"

Merge "Fix PHPDoc"

Add docs for constructor

Change-Id: I976dc8fc8a3736aaa7964cedb8e9470eacf5fda0

backport release notes from Ia6efdf931b0b6be7556fc16c040aa7493845cdfc

Change-Id: I274a7429130db05c2dc8bba9bd32d28b933747d3

Fix PHPDoc

"The" is an unknown class.

Change-Id: Ic8ce1f1252f4b435dbca789226056781d606d8ee

Fix PHPDoc

Was incorrectly documenting Article instead of Page. Also changed $article
to $page to avoid confusion.

Change-Id: I366769a201b5fda1c8d59119bdac908fd7727cbf

Use double quotes to use variable $oper properly

Introduced in Change-Id: I29461e5332f3fc6523cbd899e14394aa88865d93

Change-Id: Ic4e8d133fc9ab362f3f463309d799e80fff5d883

Reduced indentation levels, broke long lines.

Reversed login a two cases to get there.

Also updated PHPDoc, removed an unneeded comment and removed superfluous
newlines.

Change-Id: Ica5f7d24171e2eaeccc0743f8800e18cf2de8006

Reduced indentation levels, broke long lines

Also removed some unneeded else blocks to ensure more consistent return values.

Change-Id: Icf1d6fecfbd512fadad61442c968f0ef1ba30a88

Installer can now customize the logo in LocalSettings.php

Bug: 40617
Change-Id: Ia6efdf931b0b6be7556fc16c040aa7493845cdfc

Merge "Remove leading spaces in two messages"

Merge "Fix pretty JSON when strings end with backslashes"

Remove leading spaces in two messages

Also replace two hard coded space by a word separator.

Spotted by 아라 in
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Upload_source_url/en

Change-Id: If3fbe2ed9b2bd7e156cc31055e9c74da3d79240a

Merge "Fix errors reported by phpcs in includes/HTMLForm.php"

Localisation updates from translatewiki.net.

Change-Id: I2868edbb08e2acf3f26e36556382b39df38e597a

Fixed small doc typo.

Change-Id: Ia2a86e1c8126bde48c17f0c8f56905ed325d8be1

Fix pretty JSON when strings end with backslashes

If a string encoded as part of the output ends in a backslash
(e.g. an edit token), FormatJson::prettyPrint() may incorrectly
treat the unescaped double quote marking the end of the string as
a character that is part of the string.

This is a serious problem in that the "pretty" output may not
necessarily be valid JSON; a later string literal might contain
one or more of these tokens: :[{,]}

To fix the bug, I exploit strtr's behavior when it is given an
associative array having keys of the same length to skip over
escaped backslashes while replacing escaped double quotes with "\x01".

I also updated the corresponding unit test.

Change-Id: I159105b6493c14b82cd0a41a95e04bfed744931e

Localisation updates from translatewiki.net.

Change-Id: If3443de8eb3957afd1c2b4666a60aa8e8edd76f4

Apply IP blocks to X-Forwarded-For header

Adds a new configuration variable ($wgApplyIpBlocksToXff), which when
enabled will scan the XFF header for IP addresses and check if any of
them have been blocked. $wgApplyIpBlocksToXff is disabled by default.

Bug: 23343
Change-Id: I3faa9c3e8107c6e46cdf21f8c18adda1f42890d7

Merge "Make ResourceLoaderStartupModule work on mobile"

Merge "Add RL targets support to OutputPage"

Move QoS_Atomic -> QOS_ATOMIC.

Change-Id: I876272bd440affab0e934199895e12fe2a5c9449

Make ResourceLoaderStartupModule work on mobile

Change-Id: I795828ac05e2b71499db6a328004be4516849f7d

Merge "(Bug 23472) Removed undesirable space after external link url in printout"

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

Added several serializer tests
* For catching future serializer regressions.
* For fixing existing incorrect serialization.

Added couple parser tests
* For testing encap of partially-tpl-generated attribute key
* For testing lists interrupted by comment-lines

Change-Id: Ifbb7da59f65e790b66a621b7964179e4eb1f3fc4

Merge "Localisation updates from translatewiki.net."

Remove rel="next" that accompanied returnto.

Removed rel="next" link that was put on every
page that had a returnto because otherwise
Firefox (and possibly others) might prefetch
the page, which is not desired in returnto
situations.

Bug: 46680
Change-Id: Ifaf40c663dc25e51bffc317144d9bdc1dab21785

Localisation updates from translatewiki.net.

Change-Id: I54df77791c76bec5602b16c9af6132995458e9b6

(bug 46682) Drop doc mention of removed importUseModWiki.php script

The script was removed in commit f51b580f0c7ee63539e16d3017f8dd0de0ee2391
(October 2011) due to lack of maintenance.

Bug: 46682
Change-Id: Ida8369d4ad9326664b072adb0cb73f2f24465d1e

Merge "Revert "Apply IP blocks to X-Forwarded-For header""

Merge "Made sure MediaTransformOutput::getLocalCopyPath handles storage paths."

Revert "Apply IP blocks to X-Forwarded-For header"

Test are now starting to fail for everything.

This reverts commit a5d70e3ae6b43743b63f8d4e8efdfd6e26e35d40

Change-Id: I30c9eb9c00be12ff080e85452e17c2a310f03bd3

Avoid sending multiple UDP packets for the same key in wfIncrStats().

* This should help reduce collector data loss.

Change-Id: Ibe55648422d1b8aac86dd6fa83973d3c8715b0aa

Merge "Apply IP blocks to X-Forwarded-For header"

Merge "WebRequest::getRequestURL: Follow up Ibe00a6b8"

Apply IP blocks to X-Forwarded-For header

Adds a new configuration variable ($wgApplyIpBlocksToXff), which when
enabled will scan the XFF header for IP addresses and check if any of
them have been blocked. $wgApplyIpBlocksToXff is disabled by default.

Bug: 23343
Change-Id: I3e38b94d10600a60d2d4857de54307f34c4662c4

Merge "Fix link target and retina thumbnails for manual thumbnails. (Bug 45405.)"

Merge "[JobQueue] Added more isQueueDeprioritized() checks to avoid refreshLinks spam."

WebRequest::getRequestURL: Follow up Ibe00a6b8

* Only match consecutive slashes at the beginning of the URL, where
  they are actually a problem.
* Fix bug 46607 in cases where the server provides an absolute URL.

Change-Id: Ibe00a6b8722786170d09b846c1c4054b73da3d9e

Fix link target and retina thumbnails for manual thumbnails. (Bug 45405.)

The output for [[Image:Bad.jpg|thumb=Foobar.jpg|Title]] used to be:

<div class="thumb tright"><div class="thumbinner" style="width:1943px;"><a
href="/wiki/File:Foobar.jpg" class="image"><img alt=""
src="http://example.com/images/3/3a/Foobar.jpg" width="1941" height="220"
class="thumbimage" srcset="http://example.com/images/0/09/Bad.jpg 1.5x,
http://example.com/images/0/09/Bad.jpg 2x" /></a>  <div
class="thumbcaption"><div class="magnify"><a href="/wiki/File:Bad.jpg"
class="internal" title="Enlarge"><img
src="/skins/common/images/magnify-clip.png" width="15" height="11" alt=""
/></a></div>Title</div></div></div>

Note that the target of the <a> is the thumb, not the original image,
and that the srcset is loading the full resolution version of Bad.jpg.

The attached patches fix the link target and srcset issues
(suppressing the srcset when a manual thumb is used).  It also adds a
new "Thumb.png" pseudo-file to the parserTests so that we can write
new tests documenting how manual thumbnails are expected to work,
and adds the 'php' option to the thumbnail tests (since the Parsoid
parser generates different output).

Change-Id: I5be80bfce855b85f9debf3ef1776b877d1f84b9f

Merge "OpenSearch: Error for unsupported formats and adding format=jsonfm"

[JobQueue] Added more isQueueDeprioritized() checks to avoid refreshLinks spam.

Change-Id: I29cb22ed471f129efba2bb7b0743c700456f0f4d

Merge "Omit RC_EXTERNAL edits from UDP feed"

Omit RC_EXTERNAL edits from UDP feed

- bots and tools can get these edits from the feed of the external wiki

Change-Id: Id9fbbb2463516b47798f9120b039004c94c49e0b

Bump 1.22

Change-Id: Ib257f41b18764e7363d2e4aac661da76bed67413

Merge "If isConfirmedEmail (already confirmed), skip dbtouch/hook call"

If isConfirmedEmail (already confirmed), skip dbtouch/hook call

Bug: 46655
Change-Id: I5ea64611f68e01b1e82cd9dbe564123e81b17ed9

Merge "adding test cases for bug 42353, code will be submitted in a separate review for Parsoid"