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 "[FileBackend] Use strcmp() for listing comparisons."

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

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"

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"

(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

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"

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] 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

(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