Merge "Set wgWellFormedXml for LinkerTest::testUserLink"

Merge "Set wgHtml5/wgWellFormedXml for XmlTests"

Set wgWellFormedXml for LinkerTest::testUserLink

Change-Id: I9ab370e2f23e0134741680ff620e8bb552496868

Set wgHtml5/wgWellFormedXml for XmlTests

Change-Id: I3cab843bad9461098cf3c6b574fb767c3267ab14

Fix SanitizerTest for wikis with wgUseTidy = true;

testRemovehtmltagsOnHtml5Tags needs wgUseTidy = false;

3) SanitizerTest::testRemovehtmltagsOnHtml5Tags with data set #2
('time', false)

Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'<time></time>
-'
+'<time>'

Change-Id: Ib7e156293e2efae053b055e40393e442719eb5bd

Merge "ParserOutput::addLanguageLink needs a string"

Merge "Tidy up some unbalanced returns in methods"

Merge "Remove static method from interface"

Merge "Adding missing @group Database to unit tests"

Merge "Remove unreachable lines"

Adding missing @group Database to unit tests

Reasons:
* Signature parsing
* GenderCache
* Using TestUser class (inside ApiTestCase)

Change-Id: Iea4e9f9cafaa3afe7da8b922644061d3d575afe3

Merge "[JobQueue] Make doIsEmpty return bool in all cases."

[JobQueue] Make doIsEmpty return bool in all cases.

Change-Id: Ic42f426ef3d13bd0726955fe29b7e7c3d0cae152

Merge "Add numerous missing @throws to method documentation"

Merge "Fixup a couple of uses of return values of functions that return void"

Fixup a couple of uses of return values of functions that return void

Change-Id: I7c8f08c5b436cb3209fd98b9479a50abe223cccc

Add numerous missing @throws to method documentation

Change-Id: Iba868e82a75fef7c7d011bc5be192bf059d037c0

Tidy up some unbalanced returns in methods

Leave a TODO

Change-Id: Ic49ac8630650b341aafb3c17abd22f3a07805d27

Remove unreachable lines

Change-Id: Ie7ec14f4ce65bc87b0cd49ec9266c394770ee7c4

Move insertUpdateRow call before return applyPatch to make it reachable

This might not be quite right... We might want to cache the result of apply patch, insert the row and then return the result...

Change-Id: Ie6a1bb16d2912ad43aa8fd45b2bf10a8bf8652ae

Fix condition for the entries with a null model.

A null model means ‘guess from page name’.

Change-Id: I63e80761e46cc7707e9c862e2e50a950c1766aa4

The error message should refer to $wgContentHandlerUseDB

Change-Id: I81bc50e6949cecdc7f3d7e0b89f54464b4a146c7

Merge "Remove broken test."

Remove broken test.

Calling the method does not raise an exception but a notice (depending on settings).
PHPUnit just happens to turn this into an exception. The test thus breaks if PHPUnit
does not do this, which apparently happens in some cases and caused bug 41491

Change-Id: I9d14fd875c70c8b3d164c0b8a4fa2667c5769682

Merge "prefix debug log when they end up in the main log"

Merge "(bug 38897) Fix unhandled exception when visiting e.g. MediaWiki:Somemessage/0"

Merge "Replace deprecated Title::userIsWatching()"

Merge "worthwhile tests in testTitleObjectStringConversion"

Merge "Memory setting for djvutext is now a constant"

Merge "Use MWNamespace method instead of binary arithmetic"

Merge "Allow SQL queries with long field list to wrap"

Merge "mediawiki.Uri.test: Clean up"

Merge "(bug 40585) Don't drop 'step="any"' in HTML input fields."

Allow SQL queries with long field list to wrap

Change-Id: I7ed64a246455a7f2c48e7e120d45d1ebeb929584

Use MWNamespace method instead of binary arithmetic

"& ~1" or "| 1" are not easy to read

Change-Id: I08ff92669d32f3663ea58e98d8c49f0a54dfeca4

Memory setting for djvutext is now a constant

This is a follow-up for change Ia735e1b7.
See also bug 42466.

Change-Id: I67d8ae9000713c349c7506e9a77d350c89992400

Merge "Adding fragment support to SpecialPage::getTitleFor()"

mediawiki.Uri.test: Clean up

* Grouped assertions together by logical test unit (method names).
* Re-ordered (constructor, properties, methods, special cases).

Change-Id: Ia8de947278cdf469859062af47e1333c7a9f827b

Merge "hideDeprecated( 'CategoryPage::getAutosummary' )"

Merge "Localisation updates from translatewiki.net."

Merge "parse argument for message 'ago' in MWTimestamp::getHumanTimestamp"

hideDeprecated( 'CategoryPage::getAutosummary' )

check-vars correctly detects the call to CategoryPage::getAutosummary as
deprecated by coming from Article::getAutosummary but needs an explicit
hideDeprecated() to not complain about it.

Change-Id: I8a51a3041cc4cab592c5e37862efb69b538821a0

Localisation updates from translatewiki.net.

Change-Id: Ieb8805eb3bdd1dfadbcacb1ef86a51610e1155f8

Merge "Preferences: Use explicit string as returnto param"

Use JobQueueGroup::singleton()

Replaces usages of the deprecated pop() and insert()/insertBatch()
methods of Job class.

Change-Id: I5572c9e31a14bd660d7e3b9aa7792d0be9a26103

(bug 40585) Don't drop 'step="any"' in HTML input fields.

The default value for "step" is "1" which effectively bans any decimal
number from being entered into an 'type="number"' field and makes
HTMLFloatField unusable.

Change-Id: I103fd4047814a7fb0dfdc174f36badd5b660b7a4

Replace deprecated Title::userIsWatching()

Change-Id: I74203e4ed51be6fe72414d1a0b95b8370aaf0720

(bug 38897) Fix unhandled exception when visiting e.g. MediaWiki:Somemessage/0

Change-Id: I9876e4fb21c2a1a3a0a14bff05633c7aef6a3fd3

Merge "Remove code duplication"

Merge "Leave some FIXME comments for Bug 36911 - Division by zero"

Merge "Add the required call to the parent setUp to DiffHistoryBlobTest::setUp()"

Merge "Fix method declaration in UploadFromStash"

Merge "Make sure MaintenanceFixup's shutdown is simulated"

Preferences: Use explicit string as returnto param

Change-Id: I783ced55d5f811d55b4c26300890a386bcf55da1

Merge "Reset $mConvRuleTitle before every language conversion run"

Make sure MaintenanceFixup's shutdown is simulated

Previously, MaintenanceTest created a MaintenanceFixup in setUp and
relied on each test to call MaintenanceFixup's simulateShutdown instead
of making sure this was called in tearDown. Change Iee1261e7 broke this
by introducing a meta-test in a parent class.

Fixing that also allows us to not have to special-case failed tests.

Change-Id: I67c8615dd94734bb55ca48aac0c66298accbe0b4

Merge "Also pass $doxygenInputFilter as parameter instead of global."

prefix debug log when they end up in the main log

Whenever we use wfDebugLog( "group", "message" ); and "group" is not
also set in $wfDebugLogGroups, the group information is lost. By
prefixing the message with the group name, it makes it obvious where the
message comes from.

Example output:

...
[EventLogging] wgEventLoggingSchemaIndexUri has not been configured.
[EventLogging] EventLogging requires memcached, and no memcached[...]
Fully initialised
Profiler::instance called without $wgProfiler['class'] set[...]
Connected to database 0 at
MessageGroups::init-nocache
SQL ERROR (ignored): no such table: revtag_type
MessageGroups::loadGroupDefinitions-end
[ContentHandler] Created handler for wikitext: WikitextContentHandler
[ContentHandler] Created handler for javascript: JavaScriptContentHandler
...

Change-Id: Id75401bec516b5111cd1e3d4a9b29843953ce12b

Reset $mConvRuleTitle before every language conversion run

When conversion is run more than once, and manual title conversion is
only used in some of them, the result of title conversion is incorrectly
applied to all output after where it's really used.

In the past nothing has been broken because the converter is run for at
most one page (so at most one page title) in one request but this is not
true anymore when parser tests come.

Change-Id: Id689feaeff93acb859f7ba8d2aec0d7dbb5d1b6c

worthwhile tests in testTitleObjectStringConversion

The previous tests were not reflecting their comment, I have tweaked the
assertions a bit to actually test something that reflect the comments.

Change-Id: Ie1764ee1a218ead0169958704ba8625ab8d56445

Merge "[JobQueue] Added __METHOD__ to delete() call."

ParserOutput::addLanguageLink needs a string

The Parser passed a string, not a Title object to that method, doing
this in the test also

Change-Id: I05d195c1dc3cd2366795334f81601007240d9f33

parse argument for message 'ago' in MWTimestamp::getHumanTimestamp

Change-Id: I8ec8052f949313f5a3bcef2b9e668d139d7cbf6d

Merge changes Ic85d486d,I95bfb886

* changes:
  Rename JavascriptContentTest to JavaScriptContentTest
  Use the canonical class name: s/JavascriptContent/JavaScriptContent/

Merge "(bug 42226) failure in sites tests"

(bug 42226) failure in sites tests

MediaWikiSiteTest::testRemove got a duplicate key error.

Change-Id: I7a40fe50b757e4162b7ccf1655d872cbd2c51f5d

Fix method declaration in UploadFromStash

1) UploadStashTest::testValidRequest
Declaration of UploadFromStash::stashFile() should be compatible with
that of UploadBase::stashFile()

Change-Id: If94ab8b718e23b50a1b67ba0fac3ea24926d5950

Merge "[JobQueue] Added a sanity check to catch ack() breakage."

[JobQueue] Added a sanity check to catch ack() breakage.

* Also make sure the "id" field is set even if broken
  subclasses do not set it as the base class does.

Change-Id: I2f617187e225df6c1b02b4062c835cd6581c5d99

Remove code duplication

Remove trailing whitespace

Change-Id: I31b381c45ccc1631bd17c4d62a1cc1985a4770ba

Merge "[FileBackend] Removed obsolete "allowStale" parameter."

Leave some FIXME comments for Bug 36911 - Division by zero

Change-Id: I994c482cad1d195cf29d18f7ccf1d8816b1e7330

[JobQueue] Added __METHOD__ to delete() call.

Change-Id: If2644e2d0ef12b6ad68b7229f1e60a3d50fc139d

Merge "Register WikitextContentTest in the test autoloader."

Merge "Add missing wfProfileOut()"

Merge "[JobQueue] Reduced the change of deadlocks in recycleStaleJobs()."

Also pass $doxygenInputFilter as parameter instead of global.

Change-Id: I63fa0125576c4694f63d271f9824d783869f25d1

Rename JavascriptContentTest to JavaScriptContentTest

Better match the original class name.

Change-Id: Ic85d486d55c2bc58189e5d5b411d296e872fc577

Use the canonical class name: s/JavascriptContent/JavaScriptContent/

Change-Id: I95bfb8860f77d01bad027d1c2baf2d871e093758

Add missing wfProfileOut()

Change-Id: Id9c4958aa6659d66bb3486d17186424fbd9b5758

Register WikitextContentTest in the test autoloader.

Change-Id: I3c3b0fab9850196b0aad7c0bee06c4ca0b8e2fc6

Add parent::setUp() to DatabaseSQLTest

1) DatabaseSQLTest::testMediaWikiTestCaseParentSetupCalled
DatabaseSQLTest::setUp() must call parent::setUp()
Failed asserting that an array has the key 'setUp'.

Change-Id: I6aa5cb09999e0cee9b36fedd9193cb5aa08914ad

[JobQueue] Reduced the change of deadlocks in recycleStaleJobs().

* Do the DELETE by primary key to avoid gap locks. This should cut
  down on deadlocks, though their frequency is still fairly modest.
* Also use GET_LOCK() to assure that two processes are not doing this
  update at the same time.

Change-Id: Ic3de3b7cb3acd5294a151b4a00bfed65c33d0f1e

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I857041254a52acb1ba791211048840168c120384

Merge "Purge videos from squid on action=purge"

Merge "(bug 42466) Allow djvutxt to use more memory"

Purge videos from squid on action=purge

While we don't want to re-encode videos on purge, they should still
be removed from squid; compute the list of all urls before extensions
can filter it to remove files that should not be deleted from the
file backend.

Change-Id: Id837843b5a203e49a8594439ce3620da9b3c7b6a

Merge "[Upload] Added async upload concatenation support."

Merge "LinkerTest needs to set the wiki language to en."

LinkerTest needs to set the wiki language to en.

The localisation of Special:Contributions break the
testUserLink otherwise.

Change-Id: I25a40356a96e03975d8ceca975449f7b064bca2b

Merge "While testing, turn off DataBase::ignoreErrors."

Add the required call to the parent setUp to DiffHistoryBlobTest::setUp()

Change-Id: I598064cc7690f8dcc2fc5ef7a5d7c75935566d26
DiffHistoryBlobTest::testMediaWikiTestCaseParentSetupCalled
 DiffHistoryBlobTest::setUp() must call parent::setUp()

Merge "mediawiki.Uri: Don't raw encode the 'title' query parameter."

Adding fragment support to SpecialPage::getTitleFor()

Change-Id: I2007ad5166117b90af5560fd9bb1d0ee26e45cef

Merge "test: detects parent setUp not being called"

Localisation updates from translatewiki.net.

Change-Id: Ib48025b8cdc6296b429f398e37f8bca54b8703ee

Merge "Update misleading variable names in docs"

Merge "Update docs for DifferenceEngine::getDiff()"

Revert UIDGenerator commits.

Changes seemed to be merged without anyone really looking at it
and the authorship in git is wrong as well. All related changes
can later be batched as one commit.

Following commit / change are reverted:

 - 7ac3edd https://gerrit.wikimedia.org/r/36803
 - 036d03b https://gerrit.wikimedia.org/r/36813
 - 2b168e3 https://gerrit.wikimedia.org/r/36812
 - 6030d92 https://gerrit.wikimedia.org/r/36801

See wikitech-l thread
http://lists.wikimedia.org/pipermail/wikitech-l/2012-December/064801.html

Change-Id: Ie39fde2ee4121372b6d790943f306e56fbacc2b5