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

Update/fix PHPDoc

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

Change-Id: I24d4f13fa03caf7861646837f185df4f1e077c6e

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

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"

Merge "Enable $wgVectorUseIconWatch by default."

Merge "Use MessageCache::getMsgFromNamespace() to fetch conversion tables."

Enable $wgVectorUseIconWatch by default.

The usability testing for this feature is over. We've had it enabled on the largest
family of wiki in existence for ages. People are generally expecting to see this
enabled when they install a new MediaWiki wiki.

Change-Id: I6e7f4cf6c874fe9e34cc141c4305a6808c1fe576

OpenSearch: Error for unsupported formats and adding format=jsonfm

Previously it would just completely ignore the format parameter.
Making it always serve JSON no matter the value of the format
parameter.

Now it properly errors when doing (for example) format=txt or format=xml.

Also adding support for jsonfm.

Change-Id: Ia98f54f41f39006312fb49ecd718f0f161f27c37

Renamed JobQueue::QoS_Atomic to JobQueue::QOS_ATOMIC per convention.

Change-Id: I45e633ff40a805df9dfeda88ac738cfe6ac04739

Merge "Start outputting deprecation warnings when legacy linker methods are called on skins."

Merge "Revert changes in fallback behavior"

Start outputting deprecation warnings when legacy linker methods are called on skins.

Change-Id: I97b046739428176540ddd80e72a80d5feefe3fc9

Merge "Prevent PHP notice by adding isset() check"

Merge "Use get_class( $this ) instead of __CLASS__ in RedirectSpecialPage::execute()"

Revert changes in fallback behavior

This patch set reverts the following:
* Iaaf6ccebd8c40c9602748c58c3a5c73c29e7aa4d
* Ib607a446d3499a3c042dce408db5cbaf12fa9e3d
* Ic59fd20856eb0489d70f3469a56ebce0efb3db13 (partially)

Bug 46579 comment 17 describes a desired solution. In
If88923119179924a5ec091394ccab000ade16b3e we are working on a fix, but it is
taking longer than we anticipated. There was a deployment window planned
about now, but we didn't make it. It makes sense to revert for now, and commit
a proper solution somewhere next week.

Bug: 46579
Bug: 1495
Change-Id: Iac7ac4357dd80e8cdb238a6a207393f0712b3fe5

Fix case of some Title methods (final round)

Change-Id: I41afed9c1d19aaca62685a51f881cf04a10998d8

Fix errors reported by phpcs in includes/HTMLForm.php

I set property "$typeMappings" to private (no clients outside this class
in our codebase).

I removed property $jsAdded, it is unused in this class and throughout
the codebase and seems a leftover from 2008.
Special:Preferences still works with this change.

Change-Id: Id5668141835e22e3e1b043109e50d73ccd6b1229

Prevent PHP notice by adding isset() check

Issue was caused by change
I34c3fc9d8f9467e83cd201cb099c2d26f3a39b36

Bug: 46627
Change-Id: Ida87efc622e9e90b835473f069559817565eafc1

Use get_class( $this ) instead of __CLASS__ in RedirectSpecialPage::execute()

So that it returns the subclass name and not always "RedirectSpecialPage".

Change-Id: Ibee18bc75e423e8874f9254da7cec59839ba15af

Refactored use of $wgMemc in JobQueueDB to use a field.

* Also made sure it avoids SqlBagOStuff for sanity.

Change-Id: If39784567392a8a2c8a0787447267d73a0aeb0f2

Use MessageCache::getMsgFromNamespace() to fetch conversion tables.

After Iaaf6cceb, MessageCache::get() goes through the fallback chain,
which is unwanted for conversion tables (for example, we don't want
zh-hans table for zh, where zh means "no conversion"). Since the only
needed feature is to fetch text from MediaWiki namespace (conversion
tables in PHP are stored somewhere else), it is now changed to use
MessageCache::getMsgFromNamespace() instead to avoid fallbacks.

Change-Id: I46e0be31c9c0fe0a6e4923fc1aff0fbbadbf1d67

Merge "mediawiki.api: Add xhr.abort method to promise object"

Merge "jquery.textSelection.test: Fix regression in Opera"

mediawiki.api: Add xhr.abort method to promise object

jQuery.ajax does the same internally to extend the promise.

Change-Id: I3a8ae7a4c92dda5038cf7722ebd1d41dab0a7ef9

jquery.textSelection.test: Fix regression in Opera

Removing a window.opera-guarded part of code that was supposed to
"[c]ompensate for Opera's craziness converting \n to \r\n and counting
that as two chars". That craziness was apparently fixed some time ago
and the tests were now failing.

This makes the QUnit tests pass on Opera again.

Bug: 41931
Change-Id: Ide3a95e8e86ee65c8e1ea357aefa44dd2bbd4ed4

(bug 44111) ErrorPageError log messages should be in English

We want log messages to be in English. ErrorPageError, however, will log
the messages using the language of the current user. This isn't often
noticed, because the normal page view code path doesn't log
ErrorPageError and the API doesn't usually have ErrorPageError thrown in
the first place.

The fix is simple enough: in the constructor, create a new Message
object and call ->inLanguage( 'en' ) on it. And, for good measure,
->useDatabase( false ) so customizations on the local wiki won't show up
in the log files either.

Since ErrorPageError already overrides report(), that will take care of
still showing the expected language to the end user.

Bug: 44111
Change-Id: I9a6ab43d63e76fa9708b62e32ddc3262aff282f7

Merge "Combine JavaScript and JSON encoding logic"

Combine JavaScript and JSON encoding logic

This will help with improving human readability of JS and JSON
objects encoded by both ResourceLoader and the API. This patch
also adds new "utf8" parameter to the JSON formatter of the API.

Changes to FormatJson class:

* Added escaping of '<', '>', and '&' by default to protect against XSS.
* Removed unnecessary escaping of '/' and added an additional option to
  unescape non-ASCII characters (those above U+007F) as well.
* Added PHP 5.3 pretty printing code (to replace Services_JSON) that
  uses a four-space indent as PHP 5.4 does.

Changes to Xml class:

* Defined Xml::encodeJsVar() in terms of FormatJson::encode()
  and added a pretty printing option. Also added a pretty printing
  option to Xml::encodeJsCall() as well.
* Deprecated Xml::escapeJsString() and QuickTemplate::jstext();
  callers have to add quotes themselves, hence the escaping of
  both double quotes and apostrophes.

Bug: 26818
Change-Id: I1987190f1ba5bf41738e7bd611209706c1f6bb5c