Merge "[FileBackend] Added preloadCache() so callers can trigger cache getMulti()."

[FileBackend] Added preloadCache() so callers can trigger cache getMulti().

Change-Id: I2a30b4bfc16b4303abcb1f71206234dd2e0403af

Merge "Reduced some master queries via Revision::READ_NORMAL."

Merge "Added new mw message 'editpage-head-copy-warn' to EditPage header."

Merge "Work around preg_replace_callback() issue in CSSJanus"

Merge "Follow up to dirname( __FILE__ ) clean up"

Follow up to dirname( __FILE__ ) clean up

Remove a trailing whitespace added by me in dirname( __FILE__ )
clean up and add some comments to the PHP version checks (they
can't use __DIR__ as they have to run in older PHP versions.

Change-Id: I6a7e8cb7dbf384d91dda4d0ecbef7ce588bc3073

Merge "Adding ResourceLoader module "jquery.jStorage""

Merge "[FileJournal] Made getMasterDB() reuse the DB handle."

Added new mw message 'editpage-head-copy-warn' to EditPage header.

New copyright warning is to be just above the edit box.
This message is intended to be optional and locally customized.
Similar to editpage-tos-summary.

See Micro Improvements for more information.
http://www.mediawiki.org/wiki/Micro_Design_Improvements

Change-Id: Ifbd842ebef8775630d97491a3c735e046df5d0df

Merge "[FileBackend] Made Swift clear the auth cache on certain HTTP errors."

Merge "[FileBackend] Bumped Swift auth caching TTL."

Merge "[FileBackend] Reduced stat calls when using the multiwrite backend."

Merge "[FileBackend] Use strcmp() for listing comparisons."

Merge "(bug 39680) Convert valign to CSS vertical-align"

Work around preg_replace_callback() issue in CSSJanus

As reported in bug 38294, CSSJanus returns an empty string for some
input. This seems to be caused by preg_replace_callback() returning
null which, according to the PHP docs, happens "in case of error". Of
course there's no way to figure out what the error was :S

Work around this by checking for a null return value

Change-Id: I5db952bc32f73b94ac13e449d9aa1f8693602dbd

Localisation updates from translatewiki.net.

Change-Id: Ib471477326cecc9fdab5c0482bfaffb0fa2ce94f

Merge "Removed unmatched commits in Job class."

Merge "Moved warning stuff into MWDebug class."

Merge "Validate number input for {{PLURAL}}"

Moved warning stuff into MWDebug class.

- Group common code in one instead of doing checks one
  time in GlobalFunctions.php and another time in Debug.php
- Remove the code catching the fact that a warning is due
  to a deprecation warning, no longer needed
- Pass the caller offset from wfDeprecated() to
  MWDebug::deprecated(); this was breaking deprecation
  notices for global objects.
- Changed PHP error level for deprecation notices (when
  $wgDevelopmentWarnings is set to true) from E_USER_NOTICE
  to E_USER_DEPRECATED since we now require PHP 5.3+
- Added E_USER_DEPRECATED to wfSuppressWarnings() and
  removed the check for the E_DEPRECATED constant being set
  now that we require PHP 5.3+
- Fixed MWDebugTest by calling wfSuppressWarnings() and
  wfRestoreWarnings() in setUp() and tearDown()

Change-Id: I6810b57c90d384de55a2cf177047767cdb734f79

Fixed type hint docs

Change-Id: I637f11e9ac2a42865649c10ffcb96bdebf1b55eb

Merge "Add CLDRPluralRuleError, added in parent commit, to the AutoLoader"

Merge "Use the MWDebug class to display debug log back in the request."

Revert Uri class due to bug 39710

Revert 94f623363beb00a7a211e8f8f1f39bc12b4713ed. Apparently all
ReverseChronologicalPager subclasses were broken by the unexplained
interface change in wfArrayToCgi(): changing the interpretation of
null array values from "omit this key" to "include this key with no
value". The function has always been called with the former
interpretation, that's why the special case was in there.

Also reverted dependent changes 4b517fa and part of 3d97704.

Change-Id: I41dadbde5d0e0e4c54cc76bfc13c263c15c73828

Merge "Allow array condition for Database::conditional"

Merge "Use numParams() where possible."

Use numParams() where possible.

Change-Id: I2ba4fb7dad599ccd04fe69ab033ff72c1bacfe3e

Merge "(bug 39635) PostgreSQL has no LOCK IN SHARE MODE"

Reduced some master queries via Revision::READ_NORMAL.

* Also normalized 0 => false for the rev ID parameter in some places.
* Broke some long lines and shorted a variable name in Skin.php.

Change-Id: I6645315699ec7670ae22aa1dbf787d75d6e6b7ec

Merge "Use __DIR__ instead of dirname( __FILE__ )"

Use __DIR__ instead of dirname( __FILE__ )

Removed left over uses of dirname( __FILE__ ) to __DIR__, even in comments.
Running the PHP version test in maintenance/Maintenance.php earlier, so that
we no longer have to stay PHP < 5.3 compatible in there.

Change-Id: I5a00bd5c6af44b7f826c4e5576a7b3de7b5026d8

Use the MWDebug class to display debug log back in the request.

- This removes the duplication of MWDebug::$debug and
  OutputPage::$mDebugtext, so there's no need to store two times
  the same text
- Removed OutputPage::debug() since it's no longer used
- Had to keep OutputPage::$mDebugtext because it's still
  referenced by SemanticPageMaker extension
- Moved Skin::formatDebugHTML() to MWDebug::GetHTMLDebugLog()
  and replaced the call in Skin::generateDebugHTML() to use it
- Also check $logonly before sending entries to the debug
  toolbar in wfDebug(), for consistency
- Changed MWDebug::getDebugHTML() to also return the debug log
  in an HTML comment if $wgDebugComments is set to true and
  changed the location of this call to BaseTemplate::printTrail()
  so that its result is the latest possible. This also includes
  the debug toolbar.
- Removed MockOutputPage and related test cases since they are
  no longer accurate

Change-Id: Ie0f389f8566457b1c938c627ed930040741ac9d9

Added last missing @since tag to Message

Change-Id: I034ae0e5b1f19c1c28b1ecaecec69739434f699b

Merge "Added missing @since tags to all methods"

[FileJournal] Made getMasterDB() reuse the DB handle.

* Also made use of autocommit mode while at it.

Change-Id: I460e483d0103de70d505fe00dc9c67e33effde0c

(bug 39680) Convert valign to CSS vertical-align

Also convert align to CSS text-align/float

Change-Id: Ia7da2b51b7fdeee443def8cb06c63a422d89f6f1

Remove a couple of unused globals

Change-Id: I173334c43e1bc2ed5f404f82e770d3e20fa65914

Add CLDRPluralRuleError, added in parent commit, to the AutoLoader

Change-Id: Icac9fec9264b42202eddc61905f476693df57de2

Added missing @since tags to all methods

except for numParams, as Siebrand already did this in another commit

Change-Id: Ie3317af53c1df1eea99a77f402c6f7f1748dce1a

Merge "Use __DIR__ instead of dirname( __FILE__ )"

Use __DIR__ instead of dirname( __FILE__ )

We can now do this since we finally switched to PHP 5.3 for MW 1.20 and get rid of the silly dirname(__FILE__) stuff :)

Change-Id: Id9b2c9cd2e678197aa81c78adced5d1d31ff57b1

[FileBackend] Made Swift clear the auth cache on certain HTTP errors.

* This can handle when the auth token is expired sooner than expected.

Change-Id: I2e5ecc0adbde032b987af37dfe05a4aadb12b319

Localisation updates from translatewiki.net.

Change-Id: I180c36b58f8f046c572798970d2cb4815a6a2cb0

Fix ZipDirectoryReader under Hiphop

ZipDirectoryReaderError subclasses Exception, and sets the code property as a
string. Php's documentation says this should be an int, so this is already a
little bit weird.

Under Hiphop, calling the parent constructor without the second parameter will
set the code property to 0, because the parameter defaults to 0. This leads to
uploads breaking, because calls to getErrorCode return 0 rather than the code
that was passed in.

The change renames the code property to errorCode, so it can't be confused with
exception's code property, which should be an int.

https://bugzilla.wikimedia.org/show_bug.cgi?id=39346

Patch by Chris Keeline <ckeeline@fb.com>

Change-Id: I9b07a5f7f8ba7c980c0cb0da7b65816dc3b97c4c

[FileBackend] Bumped Swift auth caching TTL.

Change-Id: I5f119ff640cc7af604191450f4577a4564307499

Adding ResourceLoader module "jquery.jStorage"

Adding jStorage from http://www.jstorage.info/ to MediaWiki
It's MIT-style licensed and useful for local caching of data.

Adding to .jshintignore since it is a third party library.

Change-Id: I2343744304191d5846cf346e4ac6ca083a6414b3

(bug 39635) PostgreSQL has no LOCK IN SHARE MODE

includes/Category.php tells us to use "SELECT ...
LOCK IN SHARE MODE" before "UPDATE".

This is MySQL-only construct:

http://dev.mysql.com/doc/refman/5.0/en/innodb-locking-reads.html

Change-Id: I80709da9e15c891f1605900e7c527d5042a88f73

Merge "Clean up transactions after test."

Merge "optimize ../images/spinner.gif, ajax-loader.gif"

Merge "pngcrushed skins/common/images/add.png"

Merge "Fix documentation."

Merge "wfWarn() should cause phpunit tests to fail."

Merge "use InvalidArgumentException rather then Exception"

Removed unmatched commits in Job class.

Job had two calls to commit() that did not correspond to any
call to begin(). This may prematurely about any "outer" transaction,
may cause database corruption, and trigger warnings. There did not
seem to be any good reason for these commits to be there.

Sorting out unmatched begin/commit calls is important to allow the
addition of support for nested transactions, and it also helps
with finding "interesting" database problems.

Change-Id: Iff394f97fbad6e9304d75e6ad69155ada80b9c33

Clean up transactions after test.

MediaWikiTestCase::teardown() will now rollback any transactions
left open by a test case. This is intended to make sure tests do
not provide "interesting" results casued by transaction state
leaking from other tests.

Change-Id: Ia225251efd5aafbaa6674e2732ab1ba7761bfadc

wfWarn() should cause phpunit tests to fail.

This forces $wgDevelopmentWarnings to true for phpunit tests.

Note that wfWarn uses the E_USER_NOTICE level per default, which may or may not
actually cause test cases to fail, depending on the phpunit configuration.

Change-Id: I36583fb063436cc8474873b468fc983d28377cbd

very basic test for ApiEditPage

Based on the generic Api logging system in APITestCase.

Change-Id: I28c0bd41f585cca199fdef9a78e60ef66625d37f

use InvalidArgumentException rather then Exception

Change-Id: Ie744c773b316278e4a92af28354a8fe4eb83b78f

(bug 39672) Fixing quote.

Change-Id: I2a8d968075acede726ba670f5ae000996e388288

Fix documentation.

Change-Id: I43f4ae1248bc4a24cf127fadb6e616da8d4516e2

Validate number input for {{PLURAL}}

The new plural parser is pickier about input.

Change-Id: I1d033d30fdad61db2f661f603d9866923245137a

Merge "Fix double braceSubstitution of some preview messages"

Fix exception on parse due wfMsgExt() -> wfMessage()

Replacing wfMsgExt() with wfMessage() in 4e1ccf0 causes an exception on
parse when the defaults are used for $current and $max. I don't know if
there are other similar fatal errors caused by that set of commits.

Change-Id: I84cfdede844bb2dd3c106721b972ed1cd8bfe480

Merge "Check fallbacks correctly in getHeaderVariant()."

Localisation updates from translatewiki.net.

Change-Id: Idac5f9de0b58872797168de1f7faafcdec35dee0

Merge "change raw database quoting to use addQuotes"

change raw database quoting to use addQuotes

Change-Id: Ia734f958b1458224f998c5b59df2ff49e597f868

Truncate rollback summary for whole multibyte characters

This avoids broken characters for long comment in history

Change-Id: I3104d53c1d3d9b950e23dccecf5cea36561b8c65

optimize ../images/spinner.gif, ajax-loader.gif

Optimized spinner.gif and ajax-loader.gif using the GIMP. Also lowered the
color count in spinner.gif to 16 (from 128). The changes reduced filesize
by over a half. Also removed GIF comments.

Change-Id: Id83b5a762a90c788e2e0fe50c0936a721a87fe70

Merge "Localisation updates from translatewiki.net."

Merge "Prevent Call to a member function msg() on a non-object"

Localisation updates from translatewiki.net.

Updates for namespaces.

Change-Id: Idd38014a8d421b3afd6b0fc8678f9a4bdf896689

Localisation updates from translatewiki.net.

Updates for magic words.

Change-Id: I7479b0d525385cca4eddafc57e260a90af65a522

Localisation updates from translatewiki.net.

Updates for special page names.

Change-Id: I19700829922f3111756719db3d606db9495f9f95

[FileBackend] Reduced stat calls when using the multiwrite backend.

Change-Id: I46805637a9dcaaea597bf6eeba204a6889a36a51

[FileBackend] Use strcmp() for listing comparisons.

Change-Id: I60da57bd6cdb77a21763bb34afb51a02d905eb3a

Localisation updates from translatewiki.net.

Change-Id: Ic2cec634659baf997488ef0b328185f4c3c0e591

Allow array condition for Database::conditional

Change the one use of that method to use the new syntax.
Add some tests.

Change-Id: I9ad4e5b5e97d1d13cfd858105167ca5d20ffaf83

Use __METHOD__ in SpecialUndelete.php

Change-Id: Ia223f91a26b35677818d3ed46331813da7dbf209

pngcrushed skins/common/images/add.png

(Because it was the only image in this folder that wasn't already crushed.
Also, why not?)

Change-Id: I69f56c4b282e2e06a159c73899de8f55517dc122

Merge "Fix some css vector/monobook issues:"

Merge "Fix type of exception no longer matching after someone changed it without running the tests..."

Localisation updates from translatewiki.net.

Change-Id: I90ad39edeeb93bfb65bd7ee1b36dded75ac2f0a2

Fix type of exception no longer matching after someone changed it without running the tests...

Change-Id: I5b1c3556f40d869f0f5d4a617f01f981d607d11f

(bug 39284) Improve the tablesorter for currency.

* Make the detector for currency not trigger on cells
  starting with "." or "?".
* Make it trigger on values ending with a currency symbol.
* Add the Yen sign.
* Add a basic unit test.

Change-Id: I3c1fdf41db04ea0726ba7613fa5e1365f8fb8493

Merge "Revert "Use the context language for the toc in SpecialEditWatchlist.""

Prevent Call to a member function msg() on a non-object

- Added HTMLFormField::msg() to check whether $this->mParent->msg()
  can be called and otherwise use wfMessage()
- Made Licenses::msg() inline to prevent conflict with HTMLFormField::msg()

Change-Id: I7fa95e1d61ac4128d1d4d5f091c39bcbc8ecc651

Merge "Split up doOperations() test to be more robust when tests fail."

Merge "(bug 8618) support for summary preview in live preview"

Split up doOperations() test to be more robust when tests fail.

Change-Id: If7c0f1476e694feeb791754608ea964899be0c38

(bug 8618) support for summary preview in live preview

We have to check for the existence of .mw-summary-preview and add it if
it's missing.

See change Idda2e6f7 for more details; this change is analogous.

Change-Id: I4f4f91ed3b51a54b645bca9697c2840b717bddf6

Merge "support for interwikis in experimental JS preview"

Merge "Enable dropping of tables without having to create a file for each if you want correct output in the console"

Enable dropping of tables without having to create a file for each if you want correct output in the console

Change-Id: I374885517928430d00446b9711ea5005f58dc2a1

Merge "Make the width/height in the SVG metadata box be in the SVG's units"

Merge "Fix separated login link so that create account and login are always next to each other."

Localisation updates from translatewiki.net.

Change-Id: I0a9cc229253bd8cca3a2c0d2e87ae33b519b9cd2

Check fallbacks correctly in getHeaderVariant().

Currently when users with Accept-Language: en accesses a zh site,
getVariantFallbacks() returns zh for it and zh is treated as their
preferred variant as header variant, making the configuration
setting $wgDefaultLanguageVariant ignored.

Per comments above getVariantFallbacks(), it returns the main
language code for an unknown variant (en in this case), so when
it's returning the main code, just ignore this fallback (when the
user is really sending Accept-Language: zh, it can be caught by
validateVariant() above and fallbacks are not consulted).

Change-Id: I187bdcd70b778883cc80ceeab005de3aca2e286a

Make the width/height in the SVG metadata box be in the SVG's units

This was suggested by a user at commons. The width/height in px is really
a MediaWiki thingy needed for rendering the image, from a metadata prespective
its the original units (aka the image should be 10cm, etc) that is
the information associated with the file.

The dimensions in pixels is still specified in the long description
of the image (the subtitle directly under the image).

Note: After gerrit change Ic58efbf2 is merged, this will cause no width or height
to be displayed for svg images with cached metadata, until they
get purged. I think that's perfectly ok as the width is
available elsewhere on the page.

p.s. I added one semi-unrelated code comment in SVGMetadataExtractor.

patchset 2,3: Fix test (hopefully, how come getting an up to date
version of PHPUnit is so difficult)
patchset 4: rebase
patchset 5: fix my accidental remove of a comment

Change-Id: I312fbd1c935a0095644c3b63c4929632c6f6e387