backport release notes from Ia6efdf931b0b6be7556fc16c040aa7493845cdfc

Change-Id: I274a7429130db05c2dc8bba9bd32d28b933747d3

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"

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

Merge "[JobQueue] Ignore job release timestamps for de-duplication."

[JobQueue] Ignore job release timestamps for de-duplication.

Change-Id: I5d2a0b325eee7230450d7623e1dbcef1f8e681b9

Merge "Localisation updates from translatewiki.net."

Merge "Clarified FileBackend::doOperations() docs a bit more."

Localisation updates from translatewiki.net.

Change-Id: Ibfa55e01c4db12c5f0f2db0e862780d96462afdc

Made sure MediaTransformOutput::getLocalCopyPath handles storage paths.

* Storage paths are passed in for several cases in File::transform().

Change-Id: I61a4058b80a37f36b78e2dfe62ffdf6f73e6f41e

Merge "mw.loader: Guard against odd setTimeout behaviour in old Firefox"

Merge "Handle protocol-relative URLs in WebRequest::getRequestURL."

Handle protocol-relative URLs in WebRequest::getRequestURL.

Multiple slashes at the start of the request url will look like a
protocol-relative url, so strip the extra slashes.

Bug: 46607
Change-Id: I34c3fc9d8f9467e83cd201cb099c2d26f3a39b36

Merge "CoreParserFunctions::anchorencode should return a string"

CoreParserFunctions::anchorencode should return a string

CoreParserFunctions::anchorencode incorrectly returns false rather than
the empty string when passed an empty string.

A simple cast fixes it; this likely wasn't noticed before since PHP was
automatically doing the cast anyway when the return value was merged
into wikitext.

Bug: 46608
Change-Id: I97556dbc4dcc1f102f6fed499d43dada388cdc5d

Add RL targets support to OutputPage

The target URI parameter is already used by ResourceLoaderStartUpModule
to filter modules, this change introduces the ability to produce
mobile-aware load.php links in the core and filter out modules not suitable
for current target.

Change-Id: Ie03990e11d9a0ac8cd81dd0aae5c5a95f1d23e81

Fix case of some Title methods

Change-Id: I37ce7fe392f4941c500fa0a88007664501d7e338

Update code formatting

Also update some previous inconsistencies pointed out by Krinkle in change IDs:
* Ide20743a2e84ff68549286120e6cff9d9f396f54
* I811ca957b6588085d67606ebc0cd4033a1e53839

Change-Id: Ife33b931870d0d7e04fcb40974997436d27f528f

Update code formatting

Used PHPStorm's code formatting feature, with manual checks and updates.

Change-Id: I0496eba007056bc9c4b09163387f11d6cea2a55d

Removed deprecated usage of 2nd param in Title::getLocalUrl in SpecialAllpages

Before this change-set, a deprecated warning has been thrown when SpecialAllpges::showline would
be called with the last parameter ($hideredirect) set to false.

Change-Id: I4072a4a19a8f6403f3979c731974baba88ab574a

Handle invalid titles on ProtectedPages and ProtectedTitles

bug: 46493
Change-Id: Iaa6e3378c185c53b77a4ede0ddfdfed8c6a2268a

showSiteStats now shows ss_active_users statistic

When we implemented tracking the number of active users, we forgot to
update the stat dumper.  Simply show up the value of ss_active_users.

Change-Id: I7324d1aadfaabcd023abb93360f1f7e6d809ff3d

Merge "Added UserCache::getUserName() convenience function."

Check return value of dir() in Skin.php

This avoids a fatal, when something is wrong with $wgStyleDirectory or
the directory referred there.

PHP Fatal error: Call to a member function read() on a non-object in
Skin.php

bug: 42167
Change-Id: I3d63cb984bc2aa0f85ade04607862c639ba51fdb

[JobQueue] Added a function to purge the aggregator cache.

Change-Id: If9eabf202c372d5c594f5faeb75265ff78104f7f

Merge "JSDuck: Cover more of external.js and implement @source"

Merge "(bug 46059) giving #p-personal a higher z-index than #content in monobook"

(bug 46059) giving #p-personal a higher z-index than #content in monobook

... so that its children (like Echo's flyout) will float above
the content area.

Change-Id: Iccb0cc1a96942682007d6570d15a30dd80d7ebd0

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I6bf83c7b927a391e90515feb4eff40026c8a628e

Updated PostgreSQL insertId() docs to actually make sense.

Change-Id: Ia6c7240094fc431924b650a327fa19e93ed18261

Merge "(bug 46547) use content language for rotation comment"

Merge "JSHint: Ignore docs/html and docs/js"

JSHint: Ignore docs/html and docs/js

The MediaWiki doxygen documentation is generated by default under
/docs/html and includes some JavaScript files.  Since we do not maintain
that output, simply ask JSHint to kindly ignore them.

Same for the static site generated by JSDuck.

Change-Id: Ief4661128992b65aa85dfaa6f48d31745fe6ff19

(bug 46547) use content language for rotation comment

Bug: 46547
Change-Id: I93519557e15857176f50ea34cb59832c9575f20e

mw.loader: Guard against odd setTimeout behaviour in old Firefox

Bug: 46575
Change-Id: I80af730daa815f0c273fe942c570d1f0144bbbb1

test: ignore code coverage for CleanUpTest

The tests under tests/phpunit/includes/normal/CleanUpTest.php do not use
data provider. When being run for code coverage (bug 31518) they take so
long that they ends up timing out.

The whole class need to be refactored to use data providers (bug 46561).

Change-Id: I78c270e71d23ffbab00d8812ee91d380aef342c6

Fix case of some Title methods

Change-Id: I1a37ffb0e8cde4deac027721b0e463585b58d634

Merge "Move subclasses of LogFormatter to their own file"

Merge "Fixes to Special:WithoutInterwiki form"

Merge "Fixes to Special:FileDuplicateSearch form"