Fixes for e288e4036

Changed static calls to instance functions.

Make BitmapHandler::rotate() non-static.

We don't have a Bitmap class. It should have
been falling back to BitmapHandler class.
(JpegHandler ← ExifBitmapHandler ← BitmapHandler)

Change-Id: I17be410456b00cef2ded8d6e2282ae0de4785695

(bug 33186) image rotate api

add api action imagerotate to rotate images.
rotations are stored as a new version of the image.

Change-Id: Id15a92d19cda8256917e7e1e5ee4241012214102

Two typo-bugs, credits update

* ApiMain::addFormat() had a missed bug for custom formatters.
* ApiPageSet used incorrect variable
* Marked Roan as leading until 2009 (per IRC discussion)
* Updated my involvement to present

Change-Id: I5964bcdd3f82fedad06732df39f420796abe972b

Merge "Docs for gerrit 47419"

Merge "[FileBackend] Added a "ttl" option to getFileHttpUrl()."

Docs for gerrit 47419

Change-Id: I021b7250418e60397127e1778107ee3da77ffb18

Merge "Read full memcached response before manipulating data"

Merge "fix some spacing"

fix some spacing

Added/removed spaces around logical/arithmetic operator
Reduced multiple empty lines to one empty line
Removed wrong tabs before comments at end of line
Removed too many spaces in assigments

Change-Id: I2bba4e72f9b5f88c53324d7b70e6042f1aad8f6b

Fix align of block comments

Change-Id: I88ea33a125a71671886b49e4ebf4c1d0a1cce572

Merge "(bug 1495) Enable on-wiki message language fallbacks"

Read full memcached response before manipulating data

Memcached response when fetching data typically looks like this:
VALUE <the stored value for whatever key you requested>
END

What the code used to do is read the first line (the VALUE) and re-
assemble the data being fetched there (like unserializing serialized
data). After that, it will read the next line (END).

The value could be a serialized object, which could have a __wakeup.
This __wakeup could have code which in turn executes Memcached-
related stuff. The problem is that, while that object is being
unserialized already, it's wakeup code is attempting to read new
stuff from Memcached, but we have yet to read the END of the data
we're attempting to unserialize (when we'll read a new value from
Memcached, the first thing we'd get is the END we have not yet read..)

The correct way to go about this would be to first read the full
Memcached response, and only unserialize the read data after that.
This is exactly what this patchset does.

Change-Id: I902809c6dde657091c8161a09df823170bd41f7a

Changing position of colon

Change-Id: I5a973a04221f730c3b6cb4e1abf1a6d0bd95d107

Add full stop to full sentences

Change-Id: I3182c093ce582fb24db1fd8216aa9b9e756edf9a

Remove the mediawiki.api.titleblacklist module

Moved to the TitleBlacklist extension in I5bf7064a.

Bug: 38244
Change-Id: Ia595085371615da057aaa358d76929916a3e2186

Merge "Revert "(bug 45677) CSSJanus: support text-shadow and box-shadow flipping""

Revert "(bug 45677) CSSJanus: support text-shadow and box-shadow flipping"

See I5d24c7d8456e2. This is a wontfix per Trevor and myself.

This reverts commit e457d804eb6f854baf5edfd022ce217d53d6e946

Change-Id: I886a078cbeedd5a742e9c6dfedc9bd718bab089c

[FileBackend] Added a "ttl" option to getFileHttpUrl().

Change-Id: Ifc7b9c012276ed1f1e436ffafbd5615689680054

(bug 33304) Fix ApiQueryAllPages and old indef protections

In the query to find only protected pages, ApiQueryAllPages includes a
clause to make sure it doesn't find expired protections. But it doesn't
take into account the possibility that the protection might be an old
infinite protection where the expiry is represented as NULL instead of
'infinity'.

Note this may not close the whole bug; that remains to be determined.

Bug: 33304
Change-Id: If85591986a19bf78f7d56ad60d251fbc450ac451

Merge "Few minor parameter type hints in documentation"

Made syncFileBackend.php clearer with handling --postime.

Change-Id: I9462fb2aeef0597c3a7a0d1b0b4856ffea2460f6

Merge "Resolve config defaults in RedisConnectionPool in the singleton()."

Merge "Don't set wgPostEdit for null edits."

Merge "Maintain the page ID in WikiPage instead of relying on Title"

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I34195aefb7ae833e983cd0e2612d3af6c29414c1

Merge "Force $wgCategoryCollation to be uppercase in tests"

Don't set wgPostEdit for null edits.

Bug: 45781

Change-Id: Ia2c3e02304fad07425af520850fe30ca5eab0c8e

Merge changes I97273c52,I560d766f

* changes:
  adjusted comments for IcuCollation::$tailoringFirstLetters
  language-specific collations: be-tarask added; be, be-tarask, ru verified

Merge "Vector: Don't create an extra request for skins.vector module"

(bug 1495) Enable on-wiki message language fallbacks

The core function behind wfMessage() (MessageCache->get()) did not
apply the language fallback chain to on-wiki messages.

This patch has changed the behavior to iterate over all possible
on-wiki fallbacks (starting with the user's language) before
using the built-in language cache (CDB files). Previously we only looked
for the existence of an on-wiki message in the users's language.

Performance wise, using the 'ab' language ('ru', 'en' fallbacks)
MessageCache::get (Averaged over runs and calls)
New Code: ~8.5% TET (110us/call)
Old Code: ~6.5% TET ( 90us/call)

TET: Total Execution Time

Change-Id: Iaaf6ccebd8c40c9602748c58c3a5c73c29e7aa4d

Remove german special page alias for 'Version'

The english alias was also used years in german, because it means the
same, so there is no need for a alias/translation. But translatewiki.net
does not allow aliases, which are the same than the english word. That
means the field on Special:AdvancedTranslate is empty and animated
people to fill it.

Follow up Id1400249 (same release, so removing should not be a problem)

Change-Id: I7a451b5cf1c973b0b04270335ff31f81271f27a2

Merge "American spelling - recognize/customize"

American spelling - recognize/customize

Change-Id: I41b2f8113cf84e16430c1f59a76acbb539269833

Merge "Changing position of colon for consistency"

Merge "Use American English spelling for behavior"

Make default params to CategoryViewer usable as defaults.

If the '$from' and '$to' parameters to the CategoryViewer constructor are left
off, they default to empty arrays. This makes later checks for $from['page'],
etc. issue an 'Undefined index' notice.

Using 'empty' would be more concise but could hide bugs.

Change-Id: I78793d13745d1fc1010b8e6861bdc3a89a39a87f

Extend the SpecialStatsAddExtra hook for a more distinct display of extra items

The current approach leaves not much room for a meaningful
display of varying extra statistical items nor its ability to display
a special page link as item name.

The change proposed allows for varying header titles as well as
item descriptions using the standard language parser. The proposed
change uses is similar approach as in the InfoAction hook. A item
now is specified as [<header-message>][<item-message>] = number

Use === array() check instead of a count() and continue legacy support
for the single array structure used prio 1.21.

Change-Id: I78107c72953878c86d221fb6f83baff5f6c11f09

Merge "normalize filenames of maintenance scripts"

Merge "(bug 45201) Add option to pass defaultNamespace to ApiPageSet"

Merge "Add CategoryAfterPageAdded / CategoryAfterPageRemoved hooks"

Merge "Made runJobs.php wait for slave lag."

Merge "(bug 40326) Check if files exist with a different extension during uploading"

Add CategoryAfterPageAdded / CategoryAfterPageRemoved hooks

My first instinct was to derive the hook signature from the implementation and
have a generic 'PageCategoriesUpdated' hook that receives the WikiPage object
and two arrays ($added, $deleted). After some reflection I decided to iterate
through each array and call the hook once per category. I like this way of
doing things because a change in page membership could be thought of from two
directions: it is either the category that is gaining or losing a member, or
the page that is gaining or losing a category.

The category name is sufficient for instantiating a Category object without
additional queries being needed, so my preference was to pass objects rather
than string identifiers.

If implementors of handlers happen to be interested in the set of categories or
pages being updated, it is still the case that they only need to write one
function, and track state using a class property.

Change-Id: Ica4c5fb3acdea6c00678ec0794808fa50a1dd39a

Made runJobs.php wait for slave lag.

* Relying on waitForBackups() alone will not work with non DB queues.

Change-Id: I2124c5cd11a70d183cfde00ad21d6642a3a0ee9e

normalize filenames of maintenance scripts

Use lowerCamelCase.php format for all files (per [[mw:CC#File_naming]]),
and make filenames more specific:
- clear_stats.php -> clearCacheStats.php
- clear_interwiki_cache.php -> clearInterwikiCache.php
- initStats.php -> initSiteStats.php
- proxy_check.php -> proxyCheck.php
- stats.php -> showCacheStats.php
- showStats.php -> showSiteStats.php

Also changed the class names accordingly (per [[mw:CC/PHP#Naming]]),
and make class names more specific:
- clear_stats -> ClearCacheStats
- InitStats -> InitSiteStats
- CacheStats -> ShowCacheStats
- ShowStats -> ShowSiteStats

Updated files that made references to the changed files/classes:
- DefaultSettings.php and SpecialBlockme.php (proxy_check.php -> proxyCheck.php)
- maintenance/showSiteStats.php (initStats.php -> initSiteStats.php)
- maintenance/README and docs/memcached.txt (stats.php -> showCacheStats.php)
- docs/maintenance.txt and docs/memcached.txt (clear_stats.php -> clearCacheStats.php)

Thanks Hashar for the initial help and encouragement :)

Change-Id: I60f76fc971e06e1b710dcda35f9c2d931b93bdd7

Merge "(bug 37352) Boolean issue when using sqlite."

(bug 40326) Check if files exist with a different extension during
uploading

Added FileRepo::findFilesByPrefix( $prefix ), which does a img_name LIKE
'$prefix%' search and use this in UploadBase::getExistsWarning().

Change-Id: I21eddc5d08ca3c23b342ebc1c6947e36c4201a45

(bug 37352) Boolean issue when using sqlite.

using a type cast instead, otherwise an empty string gets converted to
true when set with bool type.

Change-Id: I4161de32151a649161ab4084e80cafe4a462c7d5

(bug 33304) Fix ApiQueryAllPages duplicate rows in protection query

When querying for protection expiry without also querying protection
type(s), DISTINCT is not added to the query. Fix that.

Change-Id: I47d3ac1e0cfa6fa1fb0dc0089f5a01698b334cd5

Fix API list=deletedrevs continuation in mode 1

r86782 introduced validation of parameter combinations for
list=deletedrevs, but screwed up in that it disallowed drcontinue in
"mode 1" (list all deleted revisions for the specified titles).

Apparently no one uses this much, because it doesn't seem to have been
reported as a bug despite being broken since April 2011.

Change-Id: I30691d04fab759ee56fb4156d475fbb21b81ab45

Vector: Don't create an extra request for skins.vector module

Blank MediaWiki page:

head>
 load.php:startup
 load([ top ])
body>
 load.php:skins.vector&only=scripts
 load([ bottom ])

Skin styles are loaded outside ResourceLoader from the <head>
(so they don't wait for startup, and don't rely on javascript).
Fair enough for basic page styling.

Then the real `mw.loader`-able part of the module should be
separated. There is no reason for it to also be loaded outside
`mw.loader`.

Change-Id: Ife69c5a443aac9eec8c2ec06d986b53b2379816f

Few minor parameter type hints in documentation

Change-Id: Ibcaad4427375b9aa39bb2b333b72a0beb9a3620c

Reverting temporary change Iedb304f8 as promised

Change-Id: Id2d0e2242cfb3fe84b68c5bfb8ab7c74fdf92f7d

Avoid instantiating ResourceLoader modules twice

Commit 13169c88 seperated out the call to getModule from the assignment to the
$modules array in order to add a check for private modules, so they can be
filtered out (see bug 34907). This had the (presumably unintended) side-effect
of causing modules to be instantiated twice.

Change-Id: Iccd7f3d3d473c5a43107842ed4e1cf08857a3886

Merge "Fixed ArrayUtils::pickRandom() when $rand=0."

Merge "Return empty string for trail/prefix if disabled"

Merge "Add ID to Nested Fieldsets"

Return empty string for trail/prefix if disabled

This is necessary so Parsoid can know when the API doesn't have the linktrail/
prefix feature and when the API is actually telling us that the trail/prefix
is/are disabled.

Bug: 41151
Change-Id: If0895d507fe1b473e42bceb135d4030106ded442

[JobQueue] Throttle refreshLinks2 jobs based on finishing the refreshLinks jobs.

* This should lower the rate of queue activity when major templates change.
* Also fixed a broken array_diff() call in nextJobDB.php.
* Additionally removed checkJob from nextJobDB.php. This is redundant to
  runJobs.php de-listing queues via JobQueueGroup::pop() when the queue
  is empty.

Change-Id: I1518a0de9e7ada22350d9993dd7ffe5f2ce23745

Add ID to Nested Fieldsets

I'd rather just get rid of fieldsets when using the div output
format, but since I can't do that; at least give me the option
to style everything -- all the way down :)

Change-Id: Id95b1d98aa6ac11c5030d1ac96d05d6b60b7077e

Changing position of colon for consistency

Change-Id: I5f89c7aea9c7922a8351860af33a2b6f84476bb0

Fixed ArrayUtils::pickRandom() when $rand=0.

Change-Id: Iba7db16a9ad08dd845fa1b4dd2bc02ffad8c5a32

Moved wfPickRandom to ArrayUtils.

Change-Id: I45e21e722245901ab2988be4892cdb393169c62c

Merge "[LockManager] Added ScopedLock::release function."

[LockManager] Added ScopedLock::release function.

* Added more lock unit tests.

Change-Id: Ic359307cf26cfb621ae8d2db867801735a375dde

Localisation updates from translatewiki.net.

Change-Id: I3d3772d6068a0bf380538afb726d9f14bb3273b1

Merge "Adding new subclass to HTMLForm for constructing a checkbox matrix"

Merge "(bug 45677) CSSJanus: support text-shadow and box-shadow flipping"

(bug 45677) CSSJanus: support text-shadow and box-shadow flipping

We just need to negate the horizontal offset value in both of them.

This only supports *a single shadow* per element; multiple shadows
are not supported (only the first will be flipped).

Also, to make it possible:
* don't mangle 5+ consecutive numeric values in the
  'four_notation_quantity' rule
* support rgb(a) and hsl(a) colors in the 'color' rule

Change-Id: I148229558e1b9a0516e413ffe86007235c3c3ef8

[Bug 24782]: fix API paging for recentchanges

Combine timestampt and rcid to form a uniformely increasing unique ID for paging.

The patch was originally submitted by Sam Reed, I fixed it up a bit.

Change-Id: Icc43b62ffa7f70f2eba36e9a07141b0ef2e02aa8

Force $wgCategoryCollation to be uppercase in tests

Bug: 40489
Change-Id: I920950e93f214961aa91d916e606dbb22c8dc2d9

Merge "Changing position of colon for consistency"

Merge "Vector: fix jump-to-navigation link href"

adjusted comments for IcuCollation::$tailoringFirstLetters

More information about what actually sits in that array.

Summary of modifications to the Mimer data so far:
* removed data for "traditional" variants of de (German) and es (Spanish)
* used code 'tl' instead of 'fil' for Tagalog/Filipino
* added be-tarask (Belarusian Taraškievica)

Change-Id: I97273c52599a5eda3f63366d697b077d6b17ba81

Changing position of colon for consistency

Change-Id: I54f5877dbd2f28726998b7ccec4ae5cad517ab68

(bug 45201) Add option to pass defaultNamespace to ApiPageSet

Change-Id: I47f0ce18fcb877ae88e5eb26eef4c1c74a6d6755

Merge "mw.Map: Avoid using 'undefined' to check for real existance."

mw.Map: Avoid using 'undefined' to check for real existance.

Instead use hasOwnProperty or arguments.length when checking
for existance of object propertys and arguments respectively.

That way the following work as expected:

var a = new mw.Map();

a.get( 'constructor' );
// Should be `null`
// Was `function Object() { [native code] }

a.get( 'something', undefined );
// Should be `undefined`
// Was `null`

Bug: 45330
Bug: 45331
Change-Id: I035e23f700e2120618ed4fbe5ce95c7f9b947e41

jquery.qunit.completenessTest: Remove redundant jshint override

Change-Id: I94aeff9c857d3a1929834fe6acdcb00d238ae250

Merge "Fix checkstyle issues in JavaScript."

Fix checkstyle issues in JavaScript.

Change-Id: I22ad0fb99a5895d1a16c5a89194356d906d42eaf

Gitignore: Clean up and ensure /includes/cache is not ignored.

In Git (at least in version 1.8.x) entries without leading slash
can be matches in any subdirectory.

Add a / in front of all those that should only be in the root.

Change-Id: I61e2be30ac24c5b369a1d36869fbd027f197bebf

Merge "Localisation updates from translatewiki.net."

Merge "Added --postime option syncFileBackend."

Localisation updates from translatewiki.net.

Change-Id: Ib88f09f813a596f0cc418624f86b748a4a2db40e

Add username for GENDER in LogFormatter only when visible

For sanity.

bug:42621
Change-Id: If133597f06edcfdda4ad7a47506635743c80cd53

Merge "(Bug 45355) Read of arbitrary files through mwdoc-filter.php"

(bug 43518) API action=unblock should return the user name, not the full user object

Change-Id: I830e210122a6608b4b599bb0b59c1c99b073c06d

Revert "Creating new WMF 1.21wmf11 branch"

This reverts commit 4a5fa0c4246adf52cd9351d82f4917aec19b7a92.

Change-Id: I0c9d813b0e82f399ab815007452a8cc6c9dde145

Revert "Commit of various live hacks"

This reverts commit 3d6851f66257d5df117688b51b7f3037c8ab6aaf.

Change-Id: I26c418d631047783ba945e6b6a7f707d876b3146

Revert "Applied patches to new WMF 1.21wmf11 branch"

This reverts commit 855d049c75fe303a7a0fcb06dba60170437a694d.

Change-Id: I68f75d9e9cb942f6dc4bdef97a5d5458c9b89875

Revert "Update WikiData extensions to correct points for 1.21wmf11"

This reverts commit dd9a9650eb7bd975577b2dbc7854a2c7010c424a

Change-Id: I9526696616c7f1d941cc02fb8b5749d36cc9022c

Update WikiData extensions to correct points for 1.21wmf11

Change-Id: I55645efb152388d6c84c2afaba270d049a1711f9

Applied patches to new WMF 1.21wmf11 branch

Commit of various live hacks

Disable of MessageBlobStore clear

Reset $wgAutopromote (should be moved to a config change!!)

Disable setting of wgStyleSheetPath

Throttle page_touched

Add apc htcp packet numbers to SquidUpdate

Disable set names binary/utf8

Commment out searchindex table indexes

Was c532e81d583d3d0439fe76eea4d105d675461b56

Original revision Change-Id I42c4f859e55eb198f6c6841e582b3552aad7b31f
https://gerrit.wikimedia.org/r/#/c/7606

Change-Id: I5ec8dd53188e9e4128f99ceaff38ebf9dcf570bb

Creating new WMF 1.21wmf11 branch

Change intval( User::getOption() ) to User::getIntOption()

Also changed some getOption with int or bool cast

Change-Id: Ia551a50e9de047c62be84065481fdf8c02e2ef96

Merge "Remove Lithuanian override of plural rules"

Remove Lithuanian override of plural rules

Same rules defined in CLDR plural rules

Change-Id: I2420720fe7dc42bb196bdd442839e41bf7b7c766