Merge "WebRequest::getRequestURL: Follow up Ibe00a6b8"

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

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

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

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"

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

Change-Id: Ie0b131c1a09bc75be39bbb7e9c87b8d3221a0fc5

[FileBackend] Cleanup behavior for coping/moving a file over itself.

* Previously, for doQuickOperations(), the default move function
  would just delete the file if it was moved over itself. In fact,
  the php-cloudfiles bindings have this same bug in its move function.
* For both copy and move, when the source and destination are the
  same path, make sure that the headers get updated as specified.
  This only applies for the 'overwrite' case (not 'overwriteSame').
* Fixed bad status for doQuickOperations() when copying a file over
  itself using FSFileBackend.
* Clarified the documentation for 'overwriteSame' option.
* Renamed destSameAsSource to overwriteSameCase in FileOp for clarity.

Change-Id: I3f609d9413795c654de27958b1e807b1fc785f31

Merge "(bug 1495) Include the site language in the fallback list"

(bug 1495) Include the site language in the fallback list

In Iaaf6ccebd8c40c9602748c58c3a5c73c29e7aa4d, the behavior of MessageCache
was changed so that it iterates over all possible on-wiki fallbacks including
the site language fallbacks. However, it omitted the site language itself.
Obviously, the problem appears only when the site language is not English,
because English is included always as the fallback language of last resort.
But in other cases, we need to add it explicitly.

Bug: 1495
Change-Id: Ib607a446d3499a3c042dce408db5cbaf12fa9e3d

Merge "Remove Special:ActiveUsers"

fix some spacing

* Removed spaces around array index
* Removed double spaces or added spaces to begin or end of function
  calls, method signature, conditions or foreachs
* Added braces to one-line ifs
* Changed multi line conditions to one line conditions
* Realigned some arrays

Change-Id: Ia04d2a99d663b07101013c2d53b3b2e872fd9cc3

Remove Special:ActiveUsers

This feature was poorly thought out from the start. It's a huge
performance drain when used, for a mimimally useful set of data

Change-Id: I138622e1c184f74e2a7ce13a9a544ab4e610d7a0

Localisation updates from translatewiki.net.

Change-Id: Ib8ac46b3ac8228020f8744978d141bd8c5a0301b

Merge "QUnit: Re-enable config.requireExpects and add missing numbers."

Merge "Update plural rules for Hebrew"

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

Taken from VisualEditor repo (authored by myself under MIT).
Hereby released under GPL as well.

Change-Id: I4dc1a0e5c618e0a20ae6f308b4ecf266d041e22c

Merge "TimeAdjustTest: Use dataProvider"

(bug 46529) Add gender aliases for Yiddish

Change-Id: I607016bddc6759998c36e73e44754663ae78be70

Merge "Add mobile target to jquery.getAttrs"

Add mobile target to jquery.getAttrs

mediawiki.tests.qunit.testrunner has a dependency on it, which we want
on mobile.

Change-Id: Iffa7166362fe568ce8ed95db427c432e111d65a1

Fixes to Special:FileDuplicateSearch form

- Use Html class instead of Xml where possible
- Add line breaks to the output for better readability

Change-Id: Ia58e0d017a2a54c2458da625bd27a775204908d9

Fixes to Special:WithoutInterwiki form

- Use Html class instead of Xml where possible
- Add line breaks to the output for better readability

Change-Id: I794da047f0664913cfd7ceb00247145f9b7d3a1b

Move RELEASE-NOTES entry to the correct section

Follow-up I719d84c5 (d240930d); that change is in the API,
so the RELEASE-NOTES entry belongs to the API section.

Change-Id: I48581bb59b3ac593443ff092c3fb7a42ac8958e1

Merge "release notes update due to Change-Id:  Idab489fc017dcf226ea39f2f1766e06064e4ec0a (bug 25325) fix hiding bot edits"

release notes update due to Change-Id:
 Idab489fc017dcf226ea39f2f1766e06064e4ec0a
(bug 25325) fix hiding bot edits

Change-Id: Ib07d4923dd6ac44520e459f0e9886bd1f8e0a4b7

Merge "(bug 25325) fix hiding bot edits"

(bug 25325) fix hiding bot edits

I added support for 'wlshow' parameter, which allows user
to decide if to include or exclude bot/minor/anon/patrolled
edits from feed watchlist.

Plus: I bound the params checkup and params' descriptions with
ApiQueryWatchlist (Yurik's suggestions).

Change-Id: Idab489fc017dcf226ea39f2f1766e06064e4ec0a

Merge "IcuCollation::$tailoringFirstLetters: letter removal rules for Finnish"

Merge "IcuCollation::$tailoringFirstLetters: 'sv', 'vi' verified"

Pass function name to insert() in replace() in Database class

Change-Id: I891370aa5d623ac9971c719da7996a93db641a2c

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

Change-Id: I79f51608d70ce70570ccabd7ecfba75e7852a897

Add to docs that input will be HTML escaped

Change-Id: Ie90a6bc29f54361ec74cbf54551266b94ce6061f

TimeAdjustTest: Use dataProvider

Also removes setting of wgContLang/wgLanguageCode, because test extends
MediaWiki*Lang*TestCase

Change-Id: I8a24fbe161c8f2ecae7591baf99845ff4434f983

Merge "DefaultSettings: Remove some empty lines"