Merge "Fixed use of __METHOD__ in a closure."

Merge "phpcs: Fix Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore violations"

Fixed use of __METHOD__ in a closure.

Change-Id: I41dc0ba4aff39d5a65cd1ccb3fba6c102974e6c2

Merge "Localisation updates from translatewiki.net."

Merge "Disable Special:PasswordReset when $wgEnableEmail is false"

Localisation updates from translatewiki.net.

Change-Id: I6c19864b175b2cbb84273326a981467af2db8610

Disable Special:PasswordReset when $wgEnableEmail is false

Previously one could still navigate to the page by entering its URL directly,
and from there be able to request a password reset e-mail for any user who has
an e-mail address configured. This patch adds a check for $wgEnableEmail in the
SpecialPage code, replacing the interface with an error message when e-mail is
not enabled.

Change-Id: I42be68c6a068aafdf45623eb35bb21aa684f54f6

Merge "Minor fix to redis queue docs."

Merge "Consistency tweak: Wrap variable names into <code></code>"

Minor fix to redis queue docs.

Change-Id: I27394ddc8187ab3e9c275e2d72911318cc496604

Merge "Added a quick test for getAllQueuedJobs()."

Merge "Remove externaledit/externaldiff preference"

Update documentation and styling

Remove some trailing whitespace, break long lines, make PHPDoc consistent
and update it in places.

Change-Id: I340b43a47e121a44ebd4aaa2a45dff12e945e645

Remove externaledit/externaldiff preference

Change-Id: Ic3791d188688f556753ab5556a4d9733bbf2e85e

given name correction in CREDITS

Change-Id: I1afdad0129980a988e5626351cff94c38d01688f

phpcs: Fix Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore violations

ERROR: Closing brace must be on a line by itself
Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore

- For non-empty scopes it means the closing brace must be on a separate
  line. This is already the case in most classes in some cases the "lazy
  closing" is still used.
  array(
   'x' ); // Moved } to next line
  function () { return 'x'; } // Moved } to next line
  case 1:
    stuff; break; // Moved break to next line
- For empty function it serves as a visual distinction between there not
  being a scope block / function body and there being an empty function
  body.
  function foo(); // No body
  function foo() {} // Empty body - violation
  function bar() { // Empty body corrected
  }

Change-Id: I0310ec379c6d41cc7d483671994d027a49f32164

Add @since line for User::groupHasPermission()

Change-Id: Ibcd20a3921058a010334e0e0983e2d9541aea734
Follows-up: I41edb091fa35c8c68b6f95cc5fd208ea99418cdb

Merge "Minor coding style whitespace cleanup"

Minor coding style whitespace cleanup

Change-Id: Ic066eda3a1a2db51b2a953a81829ce7f92568125

Localisation updates from translatewiki.net.

Change-Id: Id27f47305ed695264d422df9fa51d6492b838b88

Vector: Move 'editwarning' feature from Vector extension

* Created a new Resource Loader module
* Created an entry for the RL module in the Resources.php
* Added messages to languages/MessagesEn.php and languages/MessagesQqq.php
* Called the new RL module from the EditPage.php
* Added required entries in the messages.inc, DefaultSettings.php

Corresponding change in the Vector extension: I164f17255bc927

Bug: 46514
Change-Id: I7bdbd09f4083ccb316156307400ccfacdfec79e1

Merge "Use HTMLForm::setWrapperLegendMsg() in FormSpecialPage::getForm()"

Add release notes for Iea1f7842

Change Iea1f7842 (Add timezone support to Language::sprintfDate) did not
include an update for RELEASE-NOTES-1.22.

Change-Id: Icad1228b90b1e30130cc820faaba7c0c8f2254a2

Use HTMLForm::setWrapperLegendMsg() in FormSpecialPage::getForm()

There is no need to call $this->msg() directly for the fieldset legend
as there is already a method in HTMLForm to do it.

Change-Id: I771520ea230301b51cece685f47e6a606ad860be

Merge "Remove dependency on ORMTable from ORMRow"

make error message pretty

Change-Id: I45d04691aca8a1e2c7bb94537da8e04d2bbf0481

Add nosummary, minor and watchthis to Special:MyPage and friends

These are options usable with action=edit and allows better options
for the inital edit view from urls or the InputBox extension.

Change-Id: I696c47ddf9f29764bc8cc6136c618c56dbc2d726

Remove dependency on ORMTable from ORMRow

IORMRow implementing objects take a IORMTable object in their constructor.
The later is a hard to construct service object while the former ideally
should just be a simple wrapper around a database row. This means IORMRow
objects are tightly coupled with IORMTable objects, which makes them
inflexible and makes various things such as testing logic contained in
them needlessly difficult.

This commit gets rid of this nonsense by allowing for construction of
ORMRow objects without providing an ORMTable. All methods dependent on
the table field have been deprecated. Most of these methods have a new
alternative in ORMTable. For instance, saving an ORMRow can now be done
by passing an instance of ORMRow to the updateRow method of an ORMTable
instance, rather then calling save on the ORMRow instance.

Backwards compatibility has been retained except for the fields passed
in the constructor no longer undergoing magical unserialization if it
looks like this is needed. I do not expect this will affect any existing
code though.

Change-Id: I86368821fc2cd0729df5342b8572eb470c0f77a0

startup: Lower blacklist for Firefox from 4 to 3.

Though Firefox is 3.6 is both unsupported by Mozilla, by jQuery
and by MediaWiki JS, it is not known to have fatal errors in
javascript run time execution so moving from Grade B (blacklist)
to Grade X (unsupported but given Grade A source code at own risk).

This is done following a thread in English Wikipedia's Village
pump requesting to get the Grade A resources again.

Follows-up 02fe025e6364ebf6f.

Change-Id: Ia54dd738b3ce0995fab6aae7a2729cc7b1a0dcb1

Fix message cache expiry semantics

* Use the stale message cache while the new one is being generated
* Revert I811755d4 (make message cache load failure fatal). This
  escalated several very plausible temporary site issues from barely
  noticeable to complete downtime -- for example, memcached being down
  on a site with only one memcached server.
* Remove $wgLocalMessageCacheSerialized, it's always been pointless
* Clarify a couple of comments.
* Increased lock wait timeout to 30s
* Make lock() fail immediately on memcached connection refused

Tests done:
* With local cache enabled: normal cold refill; refill local from
  global cache; use stale local cache during remote refill; use stale
  global cache during remote refill; cold cache wait for remote refill;
  saveToCaches() failure; memcached connection refused.
* With local cache disabled: saveToCaches() failure; cache disabled due
  to "error" status key; memcached connection refused.

Setting a 1-day expiry in memcached, with a ~10s CPU cost to replace, is
not the best idea since it inevitably leads to a cache stampede. Dealing
with the stampede by waiting for a lock is not ideal, even if it were
implemented properly, since it's not necessary to deliver perfectly
fresh message cache data to all clients.

This is especially obvious when you note that barring bugs, expiry and
regeneration always gives you back the exact same data, because we have
incremental updates (MessageCache::replace()). Keeping all clients
waiting for 10s just to give them the data they have already is pretty
pointless.

So, continue to serve the site from the stale message cache while the
new one is being generated.

One caveat: if local caching enabled, when the message cache becomes
stale, a sudden spike in network bandwidth may result due to the full
array (also typically stale) being fetched from the shared cache.

Bug: 43516
Change-Id: Ia145fd90da33956d8aac127634606aaecfaa176b

ApiFormatXml: remove broken xmldoublequote param

r55641 was a refactoring of ApiFormatXml::recXmlPrint() that added
a $doublequote parameter, yet the parameter was not actually passed
along in recursive calls, rendering the formatter's xmldoublequote
parameter useless.

XPath support was the sole reason this parameter was added. It was
argued that because in the XPath 1.0 grammar, a Literal is:

    '"' [^"]* '"' | "'" [^']* "'"

it is not possible to match attribute values that contain both an
apostrophe and a double quote.

However, conforming XPath implementations include a concat() function
that can be used to work around this limitation. Furthermore, XPath 2.0
allows escaping a double-quoted " as "" (and a single-quoted ' as '').

Because there was no Bugzilla report (other than mine) filed in the
last 3-1/2 years, and working around the XPath flaw is possible, I
think it's safe to remove the parameter. The worst that could happen
is that a not-very-robust API client chokes on the "Unrecognized
parameter" warning.

Bug: 46626
Change-Id: I9a4d8c13b78ffd2634d03c8c8b4bbeff69f4bc08

Merge "add htmlform-no and htmlform-yes messages for generic yes-no questions"

Merge "Added a script to copy jobs from one queue to another."

Merge "Honor transform limit for images with many old revisions"

Merge "(bug 33454) Add timezone support to Language::sprintfDate"

Added a script to copy jobs from one queue to another.

Change-Id: I9173f21563089e094d67deaf203797d90aa84990

Merge "ResourceLoader: Don't cache ob_get_level"

Honor transform limit for images with many old revisions

When preparing Id0ec6a0a, for some reason I thought the thumbnailing
only applied to the current revision. Fix that oversight.

Change-Id: I2e8aecc76a1190bac353a2b9855bc6a19ec06dec

(bug 33454) Add timezone support to Language::sprintfDate

Add an optional timezone parameter to Language::sprintfDate, add format
characters eIOPTZ, and correct crU.

While we're at it, remove backwards-compatability code for 'N' and then
merge the existing switch cases for cr and wNzWtLoU that are basically
identical, since all those cases need to be changed anyway.

Bug: 33454
Change-Id: Iea1f78428bc0d32d6395818311dbe4b94d776c42

Merge "Fixes to Special:PagesWithProp"

Merge "Add SVG version of user icon in Vector personal portlet"

ResourceLoader: Don't cache ob_get_level

Follows-up I0eba7fcc9ce. This was br

Bug: 46836
Change-Id: Ida0e48931781be77327073dc4ed3966949f4b082

Merge "mediawiki.jqueryMsg: Implement HTML support and improve HTML escaping."

Add SVG version of user icon in Vector personal portlet

An SVG version of the user icon is provided only for browsers supporting
SVG. To ensure browser compatibility, the SVG version is provided in a
two-layer background where the first layer is a transparent gradient.

The fact that browsers supporting CSS gradients are a strict subset of
those supporting SVG guarantees an appropriate fallback.
Embedding is used to avoid extra http requests.

Bug: 35341
Change-Id: I914da0649459744ccca9e1a78e9f48fe66e1a77f

Added a quick test for getAllQueuedJobs().

Change-Id: Ia39461bc1e2356da1a694bf1898b8ca8e4200ff8

Merge "Localisation updates from translatewiki.net."

Merge "Cleared ResourceLoader blob store after update.php finishes."

Localisation updates from translatewiki.net.

Change-Id: I83961608dc6bc3c0cea008592300972c027b5de1

Batch all StatCounter stats into one datagram in flush().

Change-Id: I50a2cd24764e0a284c4a1138f8424ebf3990792e

Merge "Make sure a job is logged as OK if the status is NULL for sanity."

Cleared ResourceLoader blob store after update.php finishes.

Update.php is supposed to clear all localization caches. This
makes sure to clear the ResourceLoader's own message cache
as well.

Bug: 44524
Change-Id: I15ceb60f5e2e4d3ed9ba818fe6cd619f9adce5b2

Merge "jsduck-gen: Add --version parameter"

Merge "startup: Blacklist incompatible mobile browsers"

jsduck-gen: Add --version parameter

Follows-up I1d5b626.

Change-Id: I7a48e549e6acd2bdb7be9f6f2c34ff0554bb791c

Merge "Revert "Remove is_numeric check from Title::checkUserBlock""

Revert "Remove is_numeric check from Title::checkUserBlock"

Breaks unit testing (see bug 47031). The commit managed to land in
the branch because of a bug in Jenkins (bug 46723).

This reverts commit 8cc0b601aa2db6db09ac0e4d70847293d75875aa

Change-Id: I4b3fadccaae9c35964a0c47d63b22c4f35148a24

Merge "Remove is_numeric check from Title::checkUserBlock"

Remove is_numeric check from Title::checkUserBlock

This should allow the usernames of administrators such as "7"
to show correctly on permissions error pages.

I extracted the working code from UserBlockedError::__construct
into a separate method Block::getPermissionsError, called from
both places with context provided as an argument.

Bug: 46768
Change-Id: Ic3fa926a5a4c109faff35fffbccb60fb06ea4a18

Get rid of preg_replace( '/.../e', ... )

This is deprecated as of PHP 5.5, and the remaining uses are quite
silly. Tim said I should remove his easter egg from Special:Version,
as it already was broken, and a new one can be added in a separate
commit.

Change-Id: I0f09f4efc7afe5933c8317462026a475530a5324

Allow search backends to return statuses instead of result sets

This will allow a backend to propogate more useful errors up
the stack.

Bug: 43544
Change-Id: I54ab520762381e8f46d5ca17c0bad8820eacb964

startup: Blacklist incompatible mobile browsers

These user agents have been blacklisted on the mobile site for some time.

They have been re-reviewed alongside the A grade phones supported by
jQuery mobile [1] and jQuery [2]

[1] http://jquerymobile.com/gbs/
[2] http://jquery.com/browser-support/

Change-Id: If7743c476e95002174763514428c17dfcbe7da4c

Merge "bug 42649 - PHP Fatal error: Call to a member function isLocal() on a non-object in Title.php"

Remove first letters that have an overlapping prefix.

First letters are supposed to be primary collation elements.
However, we do not want expansions to be considered
as firstletters (aka thorn "þ" -> "th" which isn't
the same as any other first letter (since "t" !== "th" )
however if þ was a first letter, the word "the" and
even worse the word "too" would be sorted under it, which
is wrong.

Looking for feedback if this all sounds sane. I have tested
it, it got rid of the contractions while at the same time
not removing any letter it wasn't supposed to.

Once this is merged, we could get rid of all the
-<langcode> entries. The other firstLetter array
entries for tailorings could be merged into
generateCollationData.php too, since incorrect
things would get pruned automatically, which
would probably make the logic in Collation.php
simpler.

Bug: 43740
Change-Id: I4bd3d39ec2938a53e2c6728adc48ee6cf9778d74

bug 42649 - PHP Fatal error: Call to a member function isLocal() on a non-object in Title.php

Change-Id: Ie6027c818785b12f6be9c47caab77ffa4a587a79

Merge "Fix doc and deprecated warning of Title::fixUrlQueryArgs"

Merge "mw.loader: Fix regression that caused CSS load after scripts."

Merge "Localisation updates from translatewiki.net."

mediawiki.jqueryMsg: Implement HTML support and improve HTML escaping.

* Whitelisted elements.
* Whitelisted attributes allowed on all elements, and additional ones
  per element.
* Currently, only b (bold) and i (italics) and the common attributes are
  implemented, but more can be added.
* Treat error message as text
* Implement appendWithoutParsing as an extra safeguard to ensure
  jQuery never implicitly parses HTML.  Reuse this where a similar
  case was handled one-off.
* Rename some of the link functions and variables to be clearer about
  relationships.

Bug: 44525
Change-Id: Id8902af9568092d0318d0ac9d1ca6c9c195d36fe

Fix doc and deprecated warning of Title::fixUrlQueryArgs

Break long line

Change-Id: Iaeef8ce87c3ffd4609651f7cc0e8340a0f4bebfc

mw.loader: Fix regression that caused CSS load after scripts.

Follows-up 705d50c which introduced cssText buffer (yielding
one tick of the event queue to reduce the number of forced
repaints).

However since that made CSS loading asynchronoous (be it only for
a split second) it caused some nasty side-effects.

This was also reflected in the unit tests that had to resort
to doing setTimeout from within the mw.loader implemented script
to assert the styles. This is now sane again: Scripts execute
after styles are inserted.

Bug: 46401
Change-Id: Ib54a3e78710b7f798c6730d3f540d3284001e2de

Revert "mw.loader: Fix regression that caused CSS load after scripts."

Broke master tests since it causes a JSDuck issue. See bug 47018.

This reverts commit be5f73b595b8fef0697bf83e9ebd13e85756c099

Change-Id: I2d945b7ffb7cfbcc5024a1c3bdcf0c9506b0cf76

Localisation updates from translatewiki.net.

Change-Id: I3f39cc877ae087d2833fbdce1778ac4487f7090f

Add missing wfProfileOut before throwing an exception

Change-Id: I1d830da0597f19efd0b2ae48642389975e736e23

Consistency tweak: Wrap variable names into <code></code>

Followup to Ia6efdf931b0b6be7556fc16c040aa7493845cdfc

Change-Id: I51eea3de9adb0be6695ef6ff18059d536dce801b

Add missing wfProfileOut to LocalisationCache

Follow up Ib94a8c18c4e270a7bd46faa17eb27a22ff950f75

Moved some empty lines to match the places of other profile calls in the
file.

Change-Id: I6616b18027cdb9e70b43948aaff999a2e25b07e0

Merge "LocalisationCache: add profiling, fix doxygen warnings"

Merge "FormatJson: microoptimizations for UTF8_OK mode"

Merge "mw.loader: Fix regression that caused CSS load after scripts."

Merge "Call ApiPageSet::getFinalParamDescription"

Merge "Normalize as and or language names"

Merge "Avoid fatal errors with uselang=s"

FormatJson: microoptimizations for UTF8_OK mode

* Replace strtr with str_replace where faster.
* Use addcslashes to escape json_encode's output. Because no control
  characters are included, the only characters that have to be
  escaped are \ and ". (irrelevant for PHP 5.4+ installations)

Re-encoding a ~1.5 MB API response from the Chinese Wikipedia:
* PHP 5.3: 32% faster (from 347 ms to 239 ms)
* PHP 5.4: 70% faster (from 51 ms to 15 ms)
* HHVM: 42% faster (from 326 ms to 191 ms)

Change-Id: I7c9342682986d40a2f2436ac978390b6018a3521

Fix release notes for I5e6f7d48d6904.

Change-Id: Id442e6fcbdc410798579bc0bf1088ccede08e21b

Localisation updates from translatewiki.net.

Change-Id: I182906ee394da739d6b8fb64bc7743c05b3b6ac6

Merge "Update formatting and docs"

Update formatting and docs

Also remove some unneeded else{} statements and reverse if/else if that
reduces indentation.

Change-Id: I1f7bd26119850fc944fbe84ae78bdd95cafc7bbb

Merge "startup: Add unit tests and add Firefox < 4 to blacklist."

Remove the "Quickbar" feature and all things related to it

"Quickbar" was a feature of the Standard and CologneBlue skins that
allowed the sidebar to be displayed on left or right side of the page,
floated or fixed, and hidden on diff pages.

Standard was removed (Ia6d73c2d), and CologneBlue doesn't support this
anymore (bug 41246), so all things quickbar can now be safely removed.

* Removed user prefs option + interface
* Removed related messages
* Removed code for this in SkinLegacy
* Removed dead code in DifferenceEngine and Language

Change-Id: I5e6f7d48d6904a052a3a11547d3ebe6161463018

Localisation updates from translatewiki.net.

Change-Id: Icab7dc96cc3e81918512ea4f4b55fea2bd6cf7d5

Merge "remove references to removed skins from mediawiki.util.js"

remove references to removed skins from mediawiki.util.js

Dead code removal. Followup to Ia6d73c2d.

Change-Id: Ic43462d6775af682626e43a1bdfab3f8077f0101

Merge "Remove link to Special:ActiveUsers from Special:Statistics"

Merge "fixed php doc in ApiBase"

Remove link to Special:ActiveUsers from Special:Statistics

Besides being broken, causes PHP warnings.

Bug: 46719
Change-Id: I23d9c9bdc9d38eb4256d0f7061e225cf86b576f8

Localisation updates from translatewiki.net.

Change-Id: I222f66b12c0d6bdb03c7105b3d598304f9ffd61c

LocalisationCache: add profiling, fix doxygen warnings

Change-Id: Ib94a8c18c4e270a7bd46faa17eb27a22ff950f75

Merge "Protected function UploadBase->validateName changed to public"

Protected function UploadBase->validateName changed to public

*Any class that is not part of the UploadBase class
(so it can't call protected functions or members on an UploadBase object).
If you, for example,build an extension that uses an UploadBase instance,
there is no way to get the title validation error.

Bug: 38221

Change-Id: Ie6c3f95a09ef84777313c3ec2edd31c50c397e28

Merge "Fixed fatal when using fake load balancers."

startup: Add unit tests and add Firefox < 4 to blacklist.

Bug: 35906
Change-Id: Id8214c6a54f1bed2f9e1abb668248c46dcf7317e