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

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

Added last missing @since tag to Message

Change-Id: I034ae0e5b1f19c1c28b1ecaecec69739434f699b

Merge "Added missing @since tags to all methods"

Remove a couple of unused globals

Change-Id: I173334c43e1bc2ed5f404f82e770d3e20fa65914

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

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

(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"

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

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

Localisation updates from translatewiki.net.

Change-Id: Ic2cec634659baf997488ef0b328185f4c3c0e591

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

(sort of bug 32410) Fix EXIF GPSAltitude calculation when below sea level.

In EXIF, GPSAltitude is stored as a fraction string like "1/2".
For values below sea level we were negating this value, in order
to represent the sign and the magnitude in the same value. However,
I forgot to convert that to an integer before negating it. PHP was nice
enough to do a best effort conversion of the string to an integer.
This resulted in altitudes below sea level being taken as just the
numerator of the altitude, which gives results that can be significantly
off.

Also add unit tests for the GPS related image metadata stuff. Change the
existing GPS test to use a fractional altitude (Since this issue isn't
appearent if the denominator is 1). Add tests for XMP as well, since
XMP had same issue, and has to do same processing as EXIF stuff does.

In some future time, may want to consider just converting all exif rational values
to real numbers during the extraction process for generally better sanity.

Patchset 2: rebase

Change-Id: I49032b52a4c840b28e667a6a2b8ae23c508df247

(bug 27111) Make cascading foreign repo's fetch description properly

This changes action=render on file pages to include the shared image
description (if it exists). The reasons for this are two fold:

*Logically, most users would consider the shared description part of
  the content of the page, and not part of the site chrome, so it
  should be on action=render
*Foreign file repos use action=render to get image descriptions.
  if you use say en.wikipedia.org as your file repo, commons images
  get cascasively included, but all their descriptions end up
  being the noarticletext message, which is bad. This makes
  the shared image descriptions work.

Some notes on the implementation:
* on action=render the div containing the shared description
  doesn't have the id that the one on normal page view has.
  This is to prevent duplicate ids in the cascading repo scenario.
  I added a class so people can identify them if they want, and this
  allows people to easily identify the "outermost" shared description
  as it would be the only one with the id.
* This doesn't include the "shareddescriptionfollows" message
  (which is disabled by default) that would normally separate the
  local from the non-local description. I wasn't sure, but thought
  that would be considered part of the site chrome.

Patchset 2/3: trailing space/spelling mistake
Patchset 4: rebase

Change-Id: I18bdf29de62526d699740607b5015da4b01fd43d

Merge changes I1aa3b081,I65ee788c,I58a9cdfe

* changes:
  Adding JavaScript CLDR plural parser.
  CLDR plural parser in PHP
  CLDR Plural rules based plural form calculation

Merge "(bug 31817) Teach Tidy about the HTML5 inline tag bdi."

Merge "Added sanity check to StreamFile for mwstore paths."

Fix double braceSubstitution of some preview messages

They will fail if sysops write "{{((}}" stuff in it
(where [[Template:((]] contains "{{").

Change-Id: Ie1c3b4a128d9eff23da724b17a729fac9561668e

Merge "(bug 30390) upload.js: Decode url encoding in file name."

(bug 30390) upload.js: Decode url encoding in file name.

Change-Id: I3a48f8a8033ff6a8cb0b1cb322c2af1fceccbf39

Merge "[FileBackend] MultiWrite options to avoid pushing certain changes to all backends."

Fix double parsing of "previewnote" message

Change-Id: I2144f026d3b301edb521bf83e1b089de5f35ad64

Adding JavaScript CLDR plural parser.

The JavaScript code of the parser was written by Santhosh.
The original project is hosted at GitHub:
https://github.com/santhoshtr/CLDRPluralRuleParser

Introduces resourceloader modules: mediawiki.cldr and
mediawiki.libs.pluralruleparser.

hi.js and ar.js removed since it has only convertPlural method. More [lang].js
needs to remove convertPlural, but not done in this commit.

The actual rules will be taken straight from CLDR and they are not
integrated in this commit yet.

Change-Id: I1aa3b081f4dad68515fd6cd46e4ab2dbdb3291eb

Fixed streaming for thumbs that are already in storage.

Change-Id: Iafbd5c435a5d63738a43f1ac4e52e6c4148d6784

Added sanity check to StreamFile for mwstore paths.

Change-Id: Id402f077037f3e84c9158f7d48be0ed82ba4a1cf

Merge "Use Status::getWikiText() instead of getErrorsArray() in Article::doDelete()"

[FileBackend] MultiWrite options to avoid pushing certain changes to all backends.

* This can be used to no-op dir and file operations on the clone backends
  for the 'thumb' and 'temp' zones to reduce load or avoid hitting certain mounts.

Change-Id: I2c57904b8264b7479b17736333b0b9228b469bbc

Make message escaping conistent between skins.

Also changed one call from Linker::link() with
'known' and 'noclasses' options to Linker::linkKnown().

Change-Id: Ifa1fb1e1dfc28b255d8ece13e0a95aa536207df8

Merge "Replace deprecated wfMsg* calls with Message class calls."

Revert "jQuery 1.8"

This reverts commit 9023aa2d830029da5745e92212f03ddbc71da4c2.

Regression reported on Wikinews, see bug 39572

Change-Id: I135c9931e5d3c8b85e1e9c4cd58326c682d878f2

Revert "Update jQuery UI to 1.8.23"

This reverts commit 6c85cc71f83526126505814d54ab80d4a983a9d0.

Regression reported on Wikinews, see bug 39572

Change-Id: Ie6a3dbf59d0b292d7b72ee60af3fe2ec52b243fd

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

This reverts commit c85016e27d789bc5990dddb48d02bda3d85f11a1

This change is redundant, since the root cause was fixed in
Change I333021ee and I66fb2d11

Merge "Document Linker::tocList()'s $lang parameter correctly"

Merge "Localisation updates from translatewiki.net."

Document Linker::tocList()'s $lang parameter correctly

Change-Id: I6982dc30a0ceeedc8e549ee3bdaa5bc139d6364c

Localisation updates from translatewiki.net.

Change-Id: I18b1a8aa25c18e87b8e17696c830b0e0655c6dea

Follow up I66fb2d11: use wfGetLangObj() instead

Change-Id: I333021ee4edc7481b664986c27e12fbcac9d0d08

Merge "Fix tocList() breakage in 6fe1f050"

Merge "Fixed READ_LOCKING bitfield check"

Fix tocList() breakage in 6fe1f050

The $lang parameter defaults to false, but Message::inLanguage() doesn't
like false, so do an explicit check for false.

Change-Id: I66fb2d11269a988ae92b1594314807a4eed54f1b

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

Fixed READ_LOCKING bitfield check

Change-Id: Ia37761bc21a37cad4d8357b9cbfc623b59065a11

Use the context language for the toc in SpecialEditWatchlist.

Change-Id: I029372346e18eb75f81eab24c7398b53701ac8ed

Replace deprecated wfMsg* calls with Message class calls.

Doing this in steps of roughly 100 changes per commit, so that it remains
reviewable.

Change-Id: Ie349afa5c809c887c787c7c04c49c9dd3478ccac

(bug 39563) Fatal error:  Inaccessible function via __call(): msg

Change-Id: I706ce99400edb86740afc5ce8b205030be7f3c68

Use Status::getWikiText() instead of getErrorsArray() in Article::doDelete()

So that the usage of the Status class is more consistent with other places.

Change-Id: I8f7944905df7bc241b21f014d8544486ea012f6b

Merge "Replace deprecated wfMsg* calls with Message class calls."

Merge "Fix "return to" display on Special:UserLogin."

Merge "Bug 39524 - action=info uses untranslated (raw) protection permissions"

CLDR plural parser in PHP

Wrote a CLDR plural rule parser to replace the eval()-based one from
I58a9cdfe. It converts the infix notation of the XML files to a
sanitized RPN notation, referred to in external interfaces as the
"compiled" form. The RPN notation is cached and then executed by a
fast non-validating evaluator.

Timings for the largest rule in the XML file are ~1.2ms for
compilation and ~200us for execution.

Also:
* Lazy-load the plural rules when recache() requests them, instead of
  loading them for every request.
* Language::convertPlural() needs integer keys, and CLDR only gives
  string keys. The previous code was not mapping them so it didn't work
  at all. I just mapped them in the order they appear in the XML file,
  i.e. the first rule becomes MediaWiki's $pluralForm=0, the second
  becomes $pluralForm=1, etc. Not sure if there is a more rigorous way
  to do it.

Change-Id: I65ee788c1a8e5ee2ede2091990d86eb722749dd3

Fix "return to" display on Special:UserLogin.

- Do not use $wgRedirectOnLogin on permissions
  error message
- Added LoginForm::executeReturnTo() to remove
  code duplication
- Changed usage of OutputPage::returnToMain()
  to OutputPage::addReturnTo() since we already
  check these parameters ourself

Change-Id: Ie80d03f5fca7aa87017f1ad7804dda721a1132de

Merge "Make wfScript()/mw.util.wikiScript() respect $wgScript/$wgLoadScript"

Merge "[FileBackend] Allow enforcing POSIX file owner names."