Merge "jquery.makeCollapsible: Clean up issues caused by wrong nesting"

Merge "jquery.makeCollapsible: Move functions out of the var statement"

jquery.makeCollapsible: Clean up issues caused by wrong nesting

The way it was done - switching first on action (expand/collapse), then on
elements - caused the logic to be split all over the file.

This caused:
* code duplication (e.g. computing the elements to be acted upon in
  the same way for expanding and collapsing, repeated same comments)
* regressions when the logic was changed for one but not for the other
  (this was the case e.g. with table expanding/collapsing).

As for the second, I fixed all spotted inconsistencies; as for the
first, I'll let the diffstat speak for itself.

Change-Id: I2d2592b4d00424f0c23c493f6de6c824d0714dfc

jquery.makeCollapsible: Move functions out of the var statement

Converts function expressions into function declarations.
A 230-line var statement is too much for me to handle.

Change-Id: I22dc787229f2150dbad8de4e19a7fa034dcdca3d

Merge "Fix ORMRow::insert() on PostgreSQL."

Merge "[JobQueue] Try to cut down on waitForBackups() calls in runJobs.php."

[FileBackend] Suppress missing container errors with "ignoreMissingSource".

Change-Id: I2b9f4764c4938dcee61f17c31ed4d4b84fbfbcb2

[LockManager] Various fixes to lock managers.

* Improved handling of corrupt values in cache for MemcLockManager.
  Also improved the use of Status warnings a bit.
* Removed broken special-case handling for SH->EX lock escalation.
  Updated MysqLockManager to compensate.
* Made FSLockManager only use one handle per file, which is more
  efficient and avoids errors when escalating locks (SH->EX).
* Made lock unit tests have more useful output on failure.

Change-Id: Ib304712fa2b6b3fd02bfc1b08b6f238c771960c2

Merge "Update docs"

Update docs

Change-Id: I65fb974f159a7b317843b44e90f8225ab1bf722d

[JobQueue] Try to cut down on waitForBackups() calls in runJobs.php.

Change-Id: I2fc97ef8dbc02d1184959ba962dcafdae9fae808

Rewrite user_touched comment, since apparently it is not clear enough

Change-Id: Ib993e5a6dc56963e9afbbdc2ada77762e1594f0b

Merge "IcuCollation::$tailoringFirstLetters: implement letter removal"

Fixed redis auth error spam in logs.

Change-Id: I52e074b530d4dba2a0a8ec9b7d32f96d4527c6f4

Fixed connection comparison in RedisConnectionPool::handleException().

* This was comparing a Redis with a RedisConnRef.

Change-Id: I6353736c8ad42a18d90e8474d38398d305950346

IcuCollation::$tailoringFirstLetters: implement letter removal

This is necessary for Swedish, where 'Þ' ("thorn") - considered a
separate letter by default in the first-letters-root.ser file - is
sorted as 'th', causing unexpected output on category pages - words
starting with 'th'..'u' were placed under a heading with the thorn.

There were three obvious ways to do this:
* somehow include information that this letter is to be removed in the
  string itself, as in 'sv' => array( "Å", "Ä", "Ö", "-Þ" ) - could
  potentially clash with valid uses
* create a separate array other than $tailoringFirstLetters to store
  this information - would cause the data to be fragmented all over
  the file
* include information about letters to be removed in a separate key
  "linked" to the regular one, as in '-sv' => array( "Þ" ) - I see no
  obvious downsides, so this is what I ended up doing

Bug: 45446
Change-Id: I57e07a2027c391c5baa767a68f4409b9de7b4618

Merge "Localisation updates from translatewiki.net."

Merge "Made backend sync script update the pos file periorically."

Localisation updates from translatewiki.net.

Change-Id: I285aebdacc930b2eeb97f56987c958ae3f4811cd

Merge "Reformatted some array parameter docs."

Merge "numRows on MySQL no longer propagates unrelated errors"

Reformatted some array parameter docs.

Change-Id: Ie9eefd3693e82f483a9ef0da3f972452ae7bf49e

Fixed @param tags to conform with Doxygen format.

Doxygen expects parameter types to come before the
parameter name in @param tags. Used a quick regex
to switch everything around where possible. This
only fixes cases where a primitve variable (or a
primitive followed by other types) is the variable
type. Other cases will need to be fixed manually.

Change-Id: Ic59fd20856eb0489d70f3469a56ebce0efb3db13

Merge "Make sure Parsoid doesn't get snobbish and treat non-html5 tags badly."

Merge "Fixes for e288e4036"

Merge "Fixed many small spelling mistakes and php docs, var decl."

(bug 45978) Correct case of Special:Userrights' canonical name

Fix for I50796216 (17e17ba).

The error is PHP Notice:  Found alias defined for Userrights when
searching for special page aliases for UserRights. Case mismatch?

Bug: 45978
Change-Id: Ib82bd38d0229af7571f2ab3c0470fa0f0384b635

IcuCollation::$tailoringFirstLetters: 'en', 'it', 'hu', 'pt', 'uk' verified

* en: obviously
* it: per Nemo_bis in comments on change I97273c52
* hu: per Tisza Gergő in comments on bug 45596
* pt: 'uca-default' collation is deployed on pt.wiki, 'uca-pt' is the same thing
* uk: per Dmytro Dziuma in comments on bug 45444

Change-Id: Ia7568a9ad40ef991b73059b5269e6236f52681f1

lowercase second character in digraph letters in IcuCollation tailorings

This is *the* valid way for Hungarian (per bug 45596 comment 10), and
it's likely more appropriate for other languages as well.

I should have done it this way in the first place; the original data
source includes these forms along with the all-uppercase ones (I
checked them all), so they're certainly at least not wrong. Just an
overlooking on my part.

Change-Id: Ie0ca297a082ddba8d757beb85655f86b3ee70b02

Merge "Lots of spelling mistakes and phpdoc attributes"

Fixed many small spelling mistakes and php docs, var decl.

Change-Id: I1508ed7eb77e5e4f700fb63955d626c4f5915840

Lots of spelling mistakes and phpdoc attributes

@throw->@throws
@returns->@return
@seealso->@see
@cover->@covers
etc

Change-Id: I9ae6bc3034e9790e2d66cd96473b923fe9ee7953

Merge "Add "User rights management" link on user page toolbox"

Add "User rights management" link on user page toolbox

Bug: 34040
Change-Id: I507962161581c3d9ae928594d8d280be9a4ffbdd

Merge "(bug 25325) fix wlexcludeuser for ApiFeedWatchlist"

More job queue doc tweaks.

Change-Id: I06bae67f67da23190c0576902767dc38a0d8ca5b

(bug 25325) fix wlexcludeuser for ApiFeedWatchlist

The effect of setting this option was later overwritten in the code; I
just moved an if a few lines down.

Also some null comparison cleanup.

Change-Id: I2a85673330681c6ff7facc25a7bbfb6972fe8fd5

Localisation updates from translatewiki.net.

Change-Id: I5efb53366137b0e6dceccf76a9cdaa3f7d77194f

Merge "Kill off numerous unused variables"

numRows on MySQL no longer propagates unrelated errors

Bug: 42430
Change-Id: Ie2277c97177be34f4a48dfa0eac736a2b5716f22

Made backend sync script update the pos file periorically.

Change-Id: I5b4028c8d69a6bf78b372f0d1cdb258207465f30

Merge "API param validation: Add wrong value to error message on user params"

Merge "createaccount api action didn't handle no token and no cookie"

API param validation: Add wrong value to error message on user params

It is always nice to get the wrong value back to know it.
Refactor a bit to avoid the array wrapping and unwrapping for non-multi
params.

Adjust another variable, add doc comments, removed a empty line and
reorder a condition for easy reading.

Change-Id: Ia91aa5908b82ad1209dc7da1139f91e1f2b45fac

Remove the page from the LinkCache before adding it again in WikiPage::loadFromRow()

This makes sure the LinkCache doesn't list a page both as good and bad link at the
same time, e.g. when a page gets created or deleted in the request.

Change-Id: I6774132ba8e4d28deed1908150c02fbaa42111a5

Merge "Return user ID as userid in watchlist API module"

Use ContextSource::getLanguage instead of $wgLang

Found two places, where $wgLang is used inside a ContextSource.

Change-Id: I1881a20d3cd9b50eb4af0cd588451ed941f936fa

Localisation updates from translatewiki.net.

Change-Id: I60a4f38e6b3459cbc2fb758615a311d67800d54d

Merge "Use TargetLanguage of the Parser in ImageGallery"

Merge "(Bug 34798) Synced the FeedUtils.php page with mediawiki.action.history.diff.css"

Use TargetLanguage of the Parser in ImageGallery

If there is a parser set on ImageGallery, the target language of the
parser should be used, because this parsed piece of html can get into
the parser cache.

Change-Id: I2e47b316e3dcc6a3db4fb5d13f5065e35fc369dd

Sync messages.inc with MessagesEn.php

rotate: Followup to Id15a92d19cda8256917e7e1e5ee4241012214102
pageswith props: sometimes earlier...

Change-Id: I8da23c89d5b1ebe669b4e4ef1e0d67ce8064a860

createaccount api action didn't handle no token and no cookie

If you made a request to action=createaccount and this was
the first time (so no account creation token had yet been
generated for you) a "nocookiesfornew" error was returned.

This is incorrect, what is supposed to happen in that
case is the api returns what token to use. This is
how users are supposed to request tokens, so the
issue is very confusing.

No release notes since this module was introduced in
this version.

Also removed sessionfailure from possible errors,
as it isn't one (that status triggers the needtoken
result).

Change-Id: Ibfc3879fa89b1e11303aef65feb45a91afc215e6

Updated job queue README.

Change-Id: I8f884823cc5396e38bc5cd2f803f062a2dbdfac8

(bug 45922) Fix arbitrary inconsistency in page title and headings for message accmailtitle

bug: 459222
Change-Id: If582242f0473bce81a80f05d77421bdf6e5a3f29

(Bug 34798) Synced the FeedUtils.php page with mediawiki.action.history.diff.css

->Added the background,border in diff-context,diff-addedline,diff-deleted line in $styles array in the FeedUtils.php

Change-Id: Icbfb6cfc659652c146e565b760d0815640fdad64

Merge "(Bug 45854) Unset the cgroup if we don't have a memory limit"

Merge "Fix more UnitTests for databases that do not use integer timestamps"

Localisation updates from translatewiki.net.

Change-Id: Icec7f2891935766ba5406d7932198b0f07a8a6d9

Merge "Add the global correctly"

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

Fix more UnitTests for databases that do not use integer timestamps

Change-Id: I6f8b19aa6e6a9efc3320bb18fa8c7ec8b72a0c17

Merge "Fixed $vCallback comment and removed unused return value."

[JobQueue] Made supportedOrders() protected.

Change-Id: I7237472d8281c06aff11c4644e9a59848c325263

Fixed $vCallback comment and removed unused return value.

Change-Id: If438d83e1b43c110b3c23e8d0a7705fa0795049f

fix MessageCacheTest for non-english wikis

MessageCacheTest::addDBData is called before setUp, so wgLanguageCode
and wgContLang are not set to 'en', which result in mediawiki pages
created as $key/en. When the test runs, the language is set correctly
and mediawiki is searching under $key, and does not find anything.

Set wgLanguageCode and wgContLang at the begin of addDBData

1) MessageCacheTest::testMessageFallbacks with data set #0
('FallbackLanguageTest-Full', 'ab', 'ab')
Message fallback failed.
Failed asserting that false matches expected 'ab'.

Change-Id: I8e70b7b424df6a9ad98bfb6c228809f4b4437d1e

Merge "Use output/error maintenance functions"

Add the global correctly

Change-Id: I03be0cf1e82fc9f8f00727bed5aca9d9d2758c2e

(Bug 45854) Unset the cgroup if we don't have a memory limit

We only create the cgroup if there's a memory limit, but it
tried to always cleanup it, regardless if it was used or not,
resulting in a «/sys/fs/cgroup/memory/mediawiki/job/32466/tasks:
No such file or directory» error, when run with MW_CGROUP='mediawiki';
MW_MEM_LIMIT=0

Change-Id: I7fdb545b336ead62aa743b1bc3afdbfba2c3113f

Merge "Add SpecialPage::getGroupName and use it"

[JobQueue] Cleaned up handling of orders a bit.

Change-Id: I83d060e0db97d5bebf042041b0bd20563b093418

Removed "poolSize" option from RedisConnectionPool.

* The size is now automatically based on the number of servers.

Change-Id: I6ea53378c9c8254e8556c0a1e4efe64fd71ba356

[JobQueue] Added JobQueue::getAllQueuedJobs() function.

* Also added a small MappedIterator class for convenience.

Change-Id: I7dac1001a8e048cb524f7fbfee50a20a32e598ba

Use output/error maintenance functions

Change-Id: I9f9ebee9e577ab3cc7a042e5280161d2b2bad57a

Kill off numerous unused variables

Change-Id: I7039f1328f37ee669b694f73ee282602186bffd1

Merge "mw.loader: Buffer cssText in addEmbeddedCSS."

Merge "mw.loader: Create new style tags instead of appending text"

Adding validation for checkmatrix (same as for multiselect)

Change-Id: I825ec8a6094405cf7d74597a0eeda3898981763b

Merge "Localisation updates from translatewiki.net."

Merge "Few more message parameter type hints"

(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

Localisation updates from translatewiki.net.

Change-Id: If4f63fa6338b3c1938b54a88857dfc5014d75372

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

Add SpecialPage::getGroupName and use it

Having all group mapping for Special:SpecialPages in the global
$wgSpecialPageGroups is not a good OO style.
Created a method SpecialPage::getGroupName, which than can be overridden
by each subclasses to the featured group name.

Added also SpecialPage::getFinalGroupName to get the groupname on
Special:SpecialPages to handle the customization and
to keep $wgSpecialPageGroups for b/c

Change-Id: I1de3a186f0a59ec5ecb8996c5f805cf164e9637f

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

mw.loader: Buffer cssText in addEmbeddedCSS.

I52252e699a improved the repaint situation (bug 45810) a
bit, but not as much as we hoped. Inserting a stylesheet and
applying it to the page is a fairly expensive operation.
Invalidating the previous one in the process didn't help, but
fixing that didn't make a lot of difference.

This is a much more significant improvement by reducing the
number of <style> tags in the first place.

When a load.php request is handled, it reponds with many
mw.loader.implement() calls, one directly after the other
(not asynchronous in any way).

In the long term we could change the server side to combine
these better but for now the easiest fix with the largest gain
is to buffer it, yield once, and then insert it all at once.

This means we insert only one <style> for each load.php request.

Bug: 45810
Change-Id: I430fba9998b133a85dd3ac38237dc44b38630a9c