test: abstract parser test result

This patch introduce the new ParserTestResult class which is meant to
represent the result of a parser test.  I have refactored some methods
to take advantage of this new class.

It just hold the test description and the actual/expected parser output.
A short isSuccess() method is provided for convenience, we can later
improve the class to carry more methods.

Change-Id: Ifb86e09451875dc119633b52d3f7e4f47c67cc60

Moved some RedisConnectionPool docs to the factory function.

Change-Id: I4183b96e123882de604de1484456abcd10091fb2

Merge "Check for rev_len not rev_sha1"

Merge "Add two table parser tests"

Add mobile target to QUnit and its dependencies

This allows us to run mobile JavaScript tests in the environment (skin)
they were written for.

Change-Id: I84e0512590de9ff2dbdf519d983a0c1c2d007194

Add two table parser tests

One tests paragraph wrapping inside of table cells, the other mixed wikitext /
html syntax tables in Parsoid only.

Change-Id: I1e07e868d7f329ac1fa5d33b7fa07c7e701f41ba

Added profiling to MWHttpRequest and Http.

Added profile calls to various functions in
MWHttpRequest as well as to the wrapper
function Http::request.

Change-Id: I7f2819a075891dbc565a7524d56e7bf077ab089a

Localisation updates from translatewiki.net.

Change-Id: Ib5e4b598df854da0bf279190047508a5f276eaab

Remove spaces in function signature

Change-Id: I45aea7a7af88cd913b2f485913620a8af0ab2fed

Merge "Fix type of list=tags&tgcontinue"

Merge "(Bug 45775) Adjusted the margin of "userloginForm""

Merge "Use gettype only for debugging text"

(Bug 45775) Adjusted the margin of "userloginForm"

Change-Id: Ibede75a3f4e998d1972cacf1f5840882d39550ee

Merge "Added ScopedCallback functions for early execution or cancellation."

Merge "[FileBackend] Replaced redundant "disposition" param with "headers"."

Merge "Cleanup for ApiImageRotate"

acessor for MediaWikiTestCase::$useTemporaryTables

useTemporaryTables is marked private to prevent childs from overriding
the value which would cause a lot of madness whenever it forgot to reset
its status.  But that in turns prevents a child class from figuring out
whether we use temporary tables or not. The ActiveAbstract extension
test suite has such a requirement and hence need a proper accessor.

MediaWikiTestCase::usesTemporaryTables() is a public accessor to for the
private property MediaWikiTestCase::$useTemporaryTables.

Change-Id: I8016c49116a187d7523cc0d6190514f8a513eafb

Add array with watchlist options to SpecialWatchlistQuery

This data is very useful with the hook as it holds the
criteria to build the watchlist query.

Change-Id: I688ab0b35b301f180bbfde3ad3043b1ee602de10

Merge "Move plural rules of Samogitian(sgs) to plurals-mediawiki.xml"

Remove error suppression from Parser::formatHeadings

The array index looks always initialized

Change-Id: I63a1c264ded0c4a744ee990de9bef4f8f9d20d1d

Move plural rules of Samogitian(sgs) to plurals-mediawiki.xml

* CLDR does not define plural rules for sgs.
* Port the plural rules present in LanguageSgs.php to CLDR plural
  definition syntax
* Remove LanguageSgs.php
* Update the tests, reorder/rename the plural form names

Change-Id: I44658402d69a6805cdfd189fe780eadee94056c7

Remove error suppression in Exif::isRational

preg_match throws warning, when getting a array as input, but there is a
is_array check, so no need for error suppression

Change-Id: Ie7f180f45c04e8e396186cd6a5a9e40e2c94a80a

Merge "Revert "Refactored use of $wgMemc in JobQueueDB to use a field.""

Revert "Refactored use of $wgMemc in JobQueueDB to use a field."

This reverts commit 7293699561f8751c66d55697e63b13a18653ac6b

Unit tests fail.

Change-Id: Iac8025844b37c584e451a4120e54898ae4f2524e

[LockManager] Refactor lock TTL code to base class.

* Also made the TTL configurable.

Change-Id: I713979267fe7bd92ac6cd5fe4d4168d20e539c75

Refactored use of $wgMemc in JobQueueDB to use a field.

Change-Id: Ic2a0a16142abdc5a971729fe550c10c1b8fa42b9

a43f751 removed the usage of $wgMiserMode

Change-Id: I5528dba582d218721324431015bd930b9b6ab57e

jquery.tablesorter: Fix explodeRowspans

$.tablesorter's explodeRowspans works ok for simple tables, but row
headers or unusual spanning structures will confuse it.

This rewrite makes it more robust.

Bug: 41889
Change-Id: Icb674f7eece053435ca9525d45709579df14cc74

jquery.tablesorter: Fix broken getElementSortKey()

Change I3e9898f3 introduced changes to
getElementText()/getElementSortKey() that broke certain behavior with
respect to tables having colspanned cells. Restore the old behavior for
these cases.

Change-Id: I18be2e6c1f0ab926ff4c795530600b52bc6488c0

(Bug 42461) Remove call to "new Database()"

 * Refactored LoadBalancer::reportConnectionError()
 * Removed PHP4-style reference from DatabaseError constructors
 * Allowed to not pass a database to the DBError family of exceptions.

Change-Id: If9570b45ea7285de2b8b2391e704bc01f76be48a

(bug 45054) Fix rsvg converter command line when thumbnailing.

Change-Id: Ia6c1d4e5d61629e9651495ee92fe1c7298a50ef1

Merge "Remove spaces between parentheses in function calls"

Merge "Set initial language and variant user preferences to user's preferred variant on user registration"

Localisation updates from translatewiki.net.

Change-Id: I36833fa0e34d515906005009fcfe7ffb7f59a3e9

Fix type of list=tags&tgcontinue

Was already interpretted as string before, but with a default of null,
this is easy to read and looks like all other 'continue' param

Change-Id: Ica972e927d4949bb0e8925f10471b9273c052011

Remove spaces between parentheses in function calls

Change-Id: If8ec665a01b566fa5189dfcc810b2d9a17f371e1

(bug 46239) jquery.makeCollapsible: don't fail if no options given

Also a typo elsewhere. That's what you get when coding at 2 AM.

Change-Id: I8dd4f0d06cc5dbf7fe67f2b584c9afe8f5d1b36a

Merge "Readd GENDER to english logentry messages"

Localisation updates from translatewiki.net.

Change-Id: Ie28bd2d1bce13b5a78eeb8573d8b5b9527b1afeb

Readd GENDER to english logentry messages

Was removed due to performance reason, but some fixed are applied in
meantime, so there should not be a problem on adding the GENDER back.

This is a (manually) partial revert of r110909

bug: 24156
Change-Id: I8139c51bb8a1ea4fa69a1327c46d347ea043cc9f

Special:RevisionDelete: DatabaseBase::makeList: empty input

bug:45211
Change-Id: I3777c1332aa2a0dd9e9cf4b643b53f31db3f5e91

Merge "Default $wgContentHandlerUseDB to true"

Check for rev_len not rev_sha1

Change-Id: Ida29357d1277ec1ddeada2aaca2d1b889ba742a8

(bug 44486) Rephrase password reset messages to remove 'reminder' language

Change-Id: Ie2a18ddb92b8c992bc72ea4263a43e6928681041

Merge "Cache Revision::selectFields() call result"

Cache Revision::selectFields() call result

Change-Id: Ia41208658b601f68e77a0e1fa36c13e28ea8b9be

Merge "jquery.makeCollapsible: add plainMode support"

Merge "jquery.makeCollapsible: add a few explicit options to the jQuery plugin"

Merge "jquery.makeCollapsible: move helper functions to the outer scope"

Merge "jquery.makeCollapsible: clean up the handler toggling logic"

jquery.makeCollapsible: add plainMode support

When .makeCollapsible() is used on a <ol>/<ul> list, the toggler is
inserted as the zeroth list item and clicking is causes all the other
list items to be hidden separately, one by one.

This hiding behavior is applied even when using custom togglers placed
outside of the list. This has some pretty serious performance
considerations when applied to large lists or tables (more than
~25 items/rows).

Enter plainMode. In combination with custom togglers, it makes all
types of elements to be treated the same and hidden/shown as a whole.

Change-Id: Idd9429b20d70741af72657feef26e1c95c76d51f

jquery.makeCollapsible: add a few explicit options to the jQuery plugin

Add a few options to the jQuery plugin to allow for explicit passing
of various options in addition to the conventions used now (via data-
attributes or magic id/class combinations).

Change-Id: Ic498e7a3b2d2e964bda8d31a70f71cda08560cec

jquery.makeCollapsible: move helper functions to the outer scope

They do not directly use any variables defined inside
$.fn.makeCollapsible (although they used to), so let's move them to
the appropriate place.

Also indent the body of the immediately-invoked wrapper function.

Change-Id: I3d5f33b4a3ed8bd72c5f55c4d9fa545e4f3bd8c4

jquery.makeCollapsible: clean up the handler toggling logic

Merged three functions which did almost the same thing into one,
togglingHandler(), which takes a few options instead.

Left the old functions as wrappers for now, I'll clean them up in
another commit.

Change-Id: Id3f457a84064382f2d962e9ac4d926827de93126

Add id for domain list on user create form

The user login form already has this id for the domain list, so
I'm adding it for the create form as well.

Change-Id: I1eec31c461a5c76565acf1e0bbfa07f367591678

Use gettype only for debugging text

Changed some gettype == 'array', 'object' and similar to is_array,
is_object or similar

Output of gettype must not stable across versions and it is slow

Change-Id: I07bfc063b03be1200989dd6facee66b35ab51d77

Merge "Make update.php and install.php use wfPHPVersionError() and reorganise it"

Merge "Fixed importScopedSession() and moved exportUserSession() to RequestContext."

Fixed importScopedSession() and moved exportUserSession() to RequestContext.

* Renamed WebRequest::exportUserSession -> RequestContext::exportSession.
  Updated the only callers of this new function.
* Init the user with User::newFromId() instead of relying on the session
  (which breaks when things like CentralAuth are enabled).
* Made RequestContext::exportSession() include the user ID.
* Removed now-redundant user ID checks in upload jobs.
* Added unit tests for the session import function.

Change-Id: I543e6766f7a8a828ea5d270328c3bc7738c6fe94

Localisation updates from translatewiki.net.

Change-Id: I66bc1bec2e64fef9dee6562c9e2a7268c8cdb1cc

(bug 45436) Update autonym of the Pali (pi) language

Change-Id: I078eb210eacb0f4c89a8858636bd5b84da68e79d

Merge "API: Remove useless PARAM_REQUIRED = false"

$.tablesorter: remove and inline getTextFromRowAndCellIndex()

It introduced unnecessary indirection and was only called once.

Change-Id: I519cc1f04a581d41aa23229182c55466ed7757c8

Cleanup for ApiImageRotate

Follows up Id15a92d19cda8256917e7e1e5ee4241012214102

Minor tweak to filerevert too

Change-Id: I742c854ebb97a5a1f954388be4b15f992fa93bd8

API: Remove useless PARAM_REQUIRED = false

Also fix the condition in getPossibleErrors to avoid reporting of:
<error code="nosectiontitle" info="The sectiontitle parameter must be
set" />

Change-Id: Ifd808a040590ddffc0527da5ef0cc8cacdd008d9

Merge "Added a --cache option to mctest.php and mcc.php."

DefaultSettings: Add @since annotations.

Based on information from:
- https://www.mediawiki.org/wiki/Manual:$wgConf
- https://www.mediawiki.org/wiki/Manual:$wgVersion
- https://www.mediawiki.org/wiki/Manual:$wgCanonicalServer
- https://www.mediawiki.org/wiki/Manual:$wgUsePathInfo
- etc.

Change-Id: If359c57493de8efa3beb92b75e2ece9ca4a3a828

Merge "mwdocgen.php: Exit with exit code of passthru(doxygen)"

mwdocgen.php: Implement --version option.

So that the job that runs it can pass along what it should
display (e.g. branch, tag, hash etc. whatever is appropiate
for the context of that run).

Change-Id: I1d5b6266e49a71672b0a53069e6ea6bb4658c3d2

Added a --cache option to mctest.php and mcc.php.

* This lets the server list used be that of different object caches.

Change-Id: I88039370affec6e431c931337d220f7472f6eaeb

Fixed E_STRICT warning:
Strict Standards: Declaration of UploadFromChunks::stashFile() should be
compatible with UploadBase::stashFile(User $user = NULL)

Change-Id: If14d7aeacb23df16bff94c5989cde4ee31c1d105

Tweaked MessageCache locks to handle exceptions better.

Change-Id: I3066d8dbebc97abcc0567d71625f995d62549b4c

mwdocgen.php: Exit with exit code of passthru(doxygen)

Previously it just naturally exited which means even if the
doxygen bin doesn't exist, it'll return "success".

Bug: 46151
Change-Id: Ie1e3d956ce5d3e1f28c7c829bb57596113f07c4a

Removed user table JOIN from ActiveUsers query.

Change-Id: I2d6d2154a277d256db09e2a851424ba026862be9

Merge "Fix default value of the SpecialWatchlistFilters hook"

Merge "Localisation updates from translatewiki.net."

Merge "Format jquery.ui.button.css so it's easier to work with"

Added ScopedCallback functions for early execution or cancellation.

Change-Id: I6493cfa81d778c47aa7283dc3196989384235f0a

Localisation updates from translatewiki.net.

Change-Id: I04422727b54789ca326360ec86c630076b9210ae

jquery.badge: Treat non-Latin variants of zero as zero as well.

Bug: 45143
Change-Id: Ib8a36913df1ec4ae966551d1d5b8de712c34b96c

Cleaned up upload job error reporting.

Change-Id: I2379d235eb9fed58412c9fefb9cabe56868395b9

mark UtfNormal::CleanUpTest as expensive

The CleanUpTest are a bit long and reach the medium timeout of 10
seconds.  Mark them as large would give them a 60 seconds timeout,
hopefully enough to generate code coverage.

Change-Id: Id6d13204428ec1810a9b56db76bed351bf45b950

typo in @cover statement

PHP Code Coverage uses the @cover comment statement to limit code
coverage to some specific function.  The method reference lacked the
trailing 's' leading to an error when running coverage:

PHP_CodeCoverage_Exception: Trying to @cover not existing method
"Title::checkReadPermission".

Change-Id: Ie09e804039271ec8f0165c32dc7849ea4be8a3b9

Fix composer.json to prevent infinite loops.

So it turns out you're not supposed to specify the repository in
composer.json, as that's only for dependencies. Otherwise it will loop
infinitely.

The repository tag isn't for the current application, it's for
dependencies of the application. I've tried doing "composer install"
with MediaWiki master and it loops infinitely because it goes to the
repository, sees that it has a composer.json, reads it, sees that it has
a repository, etc.

This patch thus Remove the 'repositories' statement pointing to
Wikimedia gerrit installation which as added with 0ee7f355

Change-Id: I9a77d28f33a0b966f42c8c2e952b922809acc20d

Merge "Implement htmlEqual QUnit helper (QUnit.assert.htmlEqual)"

Allow having no serializer in RedisConnectionPool.

Change-Id: If367c26b6b19374d1445c406851e2c5d3e5cc634

Merge "Localisation updates from translatewiki.net."

Changing OutputPage param to a Context param in PageHistoryBeforeList

Change-Id: I348b449043fefba181e28486a07c4aa761bcef24

Localisation updates from translatewiki.net.

Change-Id: Idb52d9bc126193a3afe66ca68bfbb3c4ca375fff

Merge "servers => serves"

servers => serves

Change-Id: Idff55d11850fe68c2ba0938e5944dfe53beee390

Merge "Filtered "LOCK IN SHARE MODE" from SQLite"

Merge "Adjusting badge size per Vibha"

Fixes to Special:LinkSearch form

- Use Html class instead of Xml where possible
- Use Html class for <fieldset> generation, for consistency
- Add line breaks to the output for better readability
- Don't use $GLOBALS to access $wgScript

Change-Id: Ib11aa8ba3e3ef0e1ca07e0e3ac9605daa33f08a4

Merge "getBools doesn't exist in Translate anymore"

Merge "Fix Ifb93e49b"

Fix Ifb93e49b

This commit caused a PHP notice if you tried to use diff=prev with an
oldid which was the first revision of the page.

Change-Id: I4c4ccb9a0c8d82e104723a2e72f1c699e2fc249b

Merge "Handle certain errors in action=createaccount api module"

Merge "(bug 45937) API: Check amlang in meta=allmessages"

(bug 45937) API: Check amlang in meta=allmessages

Language::factory() throws an exception if given a syntactically invalid
code. Check the code beforehand to generate a proper error message.

Bug: 45937
Change-Id: I521e6f7ffc44becb302fde33c1df3879baa4045a