Merge "[JobQueue] Added support for approximate FIFO job queues."

Fix warnings and enforce conventions in ContentHandler tests.

Syntax:
* Call parent setUp from setUp.
* Set required globals for the test inside the test class instead
  of assuming the default settings.
* Data providers are called statically and outside setUp/tearDown
  ("public static function")
* Test function names should be prefixed with "test"
  ("testIsRedirect")
* Marked 2 functions as unused. JavascriptContentTest has 2 data
  providers for tests that don't exist in it (nor in TextContentText)
  but do exist in WikitextContentTest.

Style:
* Single quotes
* Remove odd comment "# =====" lines
* Consistent tree wrapping with arrays.
  array(
      array(
          .. ) );
  array(
      array(
          ..
      )
  );
  Some were closing on the previous line instead.
  Made it consistent now.
* Remove odd indentation to make nested arrays line up:
  array( 'foo' => array(  'bar' => true,
                          'baz' => array() ) )
  array( 'foo' => array(
      'bar' => true,
      'baz' => array()
  ) )

  We don't do this kind of indentation because it is fragile
  and becomes outdates when any of the earlier keys ("foo")
  change. Converted to a regular tree instead.

  Also triggered git warnings for mixing spaces with tabs, which
  is almost always an detector for this style.

* Not using @annotations in inline comments, reserved (and only
  parsed/meaningful) for block comments.

Follows-up 8b568be5e2dfb889e1eeedbdca50b304864174f7

Change-Id: Ic55d539b9a58f448b550bcd98894d389764e0694

Merge "Localisation updates from translatewiki.net."

Merge "(bug 38377) Fix MediaWiki:Continue-editing"

Localisation updates from translatewiki.net.

Change-Id: I015596c76b853ed572b9f4457fffd23970670b21

Merge "(Bug 41370) protect aginst content-less revisions"

Merge "Add additional job columns to PostgreSQL as well."

[JobQueue] Added support for approximate FIFO job queues.

Change-Id: Icd8c084174e26a2a69f370e7bf6f858eba76e520

Bug 11748: Handle optionally-closed HTML tags

* Fixes bug 11748 (Parser issue for HTML definition list) and similar
  issues for nested unordered / ordered lists

* Stops wrapping HTML-syntax definition lists into paragraphs
  for consistency with their wikitext variants

* Enables one previously disabled test and adds another for nested
  definition lists with HTML syntax

Change-Id: If75ed54e11452dbcf5e6213cc20923064f811715

add some brackets in RecentChange.php

Change-Id: I36bb75bcb9bcd4a91263cb7d79352ab7442b19f8

Merge "alter sql.php prompt on line continuation"

Merge "(bug 37020) sql.php with readline eats semicolon"

Merge "accessor for database delimiter"

Merge "sql.php in interactive no more exit on DBerror"

fix code style violations in RecentChange.php

Change-Id: I39e74dd23812f1e22991b3c316b6a29339123b17

Merge "(Bug 41298) partial fix: re-apply Id4d5504a"

Merge "(Bug 41298) partial fix: re-applying Id7b331e1b"

sql.php in interactive no more exit on DBerror

This patch make it so that after a DBerror a new prompt is shown to the
user. Save us from having to relaunch sql.php.

Change-Id: Id3df3df87f6fe7b2aea31e0526c5ff697bc5832c

alter sql.php prompt on line continuation

As long as a command is not ended with a database delimiter, sql.php
kept appending the line without noticing the users. That is a bit
confusing since you might not even know what you are going to run when
entering the delimiter.

This patch alter the prompt to '    ->' until the command is finished.
That is more in line with how MySQL cli handles it.

Example:
$ php sql.php
> SELECT
-> *
-> from
-> job;
Query OK, 0 row(s) affected
// repeal command:
> SELECT * from job

Change-Id: Ic18b39d75d4db48d37d485e66f36d691e95934fb

(bug 37020) sql.php with readline eats semicolon

The database delimitor is now appended to the command before it is saved
in readline history.

Change-Id: Ie6bbac2d74e2e5df3910c5612adbd373a9c4695b

accessor for database delimiter

The command delimiter is a protected propery which prevents us form
getting it outside of Database scope. This patch add a public accessor
getDelimiter() so we can access it.

Change-Id: I676d7ed9765444366fdf3dddb07c535807dabf52

deprecate make(Known?)LinkObj methods

Restore deprecation warnings from 1.16. The methods Linker::makeLinkObj
and Linker::makeKnownLinkObj should be replaced by calls to link().

Change-Id: I658289ac2b52a193b76edd946368e228cfa389cb

(Bug 41370) protect aginst content-less revisions

getAutoDeletereason did not gracefully deal with revisions that would
return null as their content (e.g. for deleted revisions).

Change-Id: I4976f2d09b8666ca1bccebea346af37ee77b09f7

Make action=query&list=users use User::getRights()

Made action=query&list=users use User::getRights() if
usprop rights given. This not only removes redundant
code, but makes it execute the UserGetRights hook, so
that this now includes rights given by Extensions (eg.
CentralAuth does that).

Patch Set 2: Modified the User class to be able to
inject further data into User::newFromRow() and using
that to inject the groups taken out of one SQL query
(for performance reasons). Furthermore I've split up
the query in ApiQueryUsers.php into one for user data
and one for the groups, to only have one row for each
user.
After all the perfomance of this should now be ok, not
extremly good, but bearable (though I couldn't test it
deeply, as I don't have much data in my CentralAuth
environment).

Change-Id: Ie5b2924abb82ac254c77e1d04cc4d5b308962dad

Merge "Function for "pretty timestamps" that are human readable and understandable."

Merge "fix fatal error in HttpTest"

follow up for edit conflicts

Change-Id: I511b78664fd2c14d63a1d399cb5ddefa2d125a22

Merge "Bug 37209: change exception into debug entry"

Bug 37209: change exception into debug entry

Change-Id: Ic2d3f0b8b0b15828a8acbc8868f26102bb33e75d

(bug 41352) restore pre-ContentHandler version of mergeChangesInto()

New implementation using ContentHandler is buggy and needs repair

Change-Id: I4c2055bed3e660be255621e4d476d7c489f4c371

Localisation updates from translatewiki.net.

Change-Id: Iaa5f02bce9c0fdf1bc1745fee4628a64f622f816

[JobQueue] README file for job queue classes.

Change-Id: Ib3e3eb4905511ba4d5fdfdf4f32279a6b6c1ac51

Add additional job columns to PostgreSQL as well.

This ports I49824c7fa855fea4ddcac5c9901ece8c2c0101d0 and fixes
bug #41196.

Change-Id: I18e0ac3aee2c2627f89d28d4a34afd90c822af91

Merge "(bug 41349) mediawiki.searchSuggest: Reorder code + fix selectors"

(bug 41349) mediawiki.searchSuggest: Reorder code + fix selectors

The code for general suggestions wouldn't run at all if no primary
search box was found. I moved this check down, to where the box is
actually needed.

The selectors were hardcoded with no explanation. Added comments,
added selector .mw-searchInput (as a generic one to be used by
skins with multiple search boxes, currently only CologneBlue).

Change-Id: Ie22fb66926688b0fdb28a121cfa0fb61ead0246d

Some minor fixes to README file.

Change-Id: I93a1f2ef728791016956a44864e9456d3de102a3

Merge "Remove float from config-settings-block"

Merge "[FileBackend] Created README file for the file backend classes."

Merge "Use ParserOptions::newFromUserAndLang in tests"

(Bug 41298) partial fix: re-apply Id4d5504a

The changes in EditPage were apparently lost during merge.

Change-Id: I2628d73a35eaab45336986959a585c7381b78471

Use ParserOptions::newFromUserAndLang in tests

This avoids using $wgUser and $wgLang for the ParserOption, which is not
needed in the tests

Change-Id: Iea07e9a0f0898ec5f8eb98a40000f5d33b279c9e

(Bug 41298) partial fix: re-applying Id7b331e1b

Changes to EditPage.php apparently got lost during merge.
Wonder how that happened, there are no conflicts.

Change-Id: I5449d0331f5d60ed0296b69888e10272f47f9d42

(Bug 41298) partial fix: re-apply I9441a277

Apparently, this change got lost during merge. Oops.

Change-Id: I75ddf1d9f2abfa97fb76d47da6fadd63658c0d46

Merge "Correct documentation for EditFilter hook parameter"

Correct documentation for EditFilter hook parameter

Change-Id: I1f5dbc834c44c33fce495a8c4ca5fc0f2b61b4c7

Consistency tweak: Monobook -> MonoBook

Spotted by Shirayuki
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Monobook.css/en

Change-Id: I408bcf361ef242c2cf522ce2288bafd053943ddb

Merge "Swap isSpecialPage for canExist()"

Swap isSpecialPage for canExist()

Add another check in ApiBase::getTitleOrPageId

Change-Id: I2123e3a3034cb815cf35f66f2fbf2b94fb27069f

Merge "(bug 26585) Detect CSV/array values in $_SERVER['REMOTE_ADDR']."

(bug 26585) Detect CSV/array values in $_SERVER['REMOTE_ADDR'].

Changed WebRequest::getRawIP to check for values of
$_SERVER['REMOTE_ADDR'] that are either an array
or a comma separated list of IP addresses, and throw
an exception.

Original patch by Ilmari Karonen. Adapted from original
patch to work with newer MediaWiki version.

Change-Id: I4b3c56adf46b336c5032db3f2a1e621c873f0d83

Merge "fix bug 41280, show correct content when displaying edit conflicts"

Merge "per bug 41244 and 41303, handle null or false params in TextContent constructor"

Localisation updates from translatewiki.net.

Change-Id: Ie47548a4b18f0f73a9e8e691633882caa99282d7

Merge "Removed reference to * from docs."

Merge "Remove duplicate files, no point in loading them twice"

Merge "Moved core Job classes under includes/job/jobs."

Merge "Stop php strict error warnings from MemcachedClient::_flush_read_buffer()"

Merge "tests: always call parent setUp"

tests: always call parent setUp

Some class extending MediaWikiTestCase did not call its setUp method. We
most probably always want to do it since MediaWikiTestCase::setUp() does
garbage collection and might do more in the future.

Change-Id: I68dde370a62c8f4a779836ca0c4ad06844fdc916

Merge "Additional parser tests to spec parser behavior and test Parsoid."

Merge "Localize parentheses in Linker::formatTemplates()"

Additional parser tests to spec parser behavior and test Parsoid.

Change-Id: Ife9750c5904cac1557b631f34a4ca8542f389226

Merge "Don't try and run a special page through action=parsex"

Merge "More informative debugging messages"

per bug 41244 and 41303, handle null or false params in TextContent constructor

Instead of throwing exception in these cases (which there are use cases for),
create an empty TextContent object.

Change-Id: Ice30ea78428d95805aa56cd3a35405aa67ad10ab

More informative debugging messages

Change-Id: Ib83dde07e8935e9ae3d864a12af7f93b68ee8f85

Don't try and run a special page through action=parsex

Change-Id: I1b4ca09b1674dbbde91acc6cade4dc1ff810bee5

Localize parentheses in Linker::formatTemplates()

Change-Id: I27c4c26206ba1e9c27a5c120a95491eb30e55522

fix bug 41300, restore the continue-editing bit of the edit page preview note

Change-Id: I39fa7a0951f8e2afe7e25cb61cd1716429228371

fix bug 41280, show correct content when displaying edit conflicts

Change-Id: I52522c06d20ddab85799dc33a0d07c10603a20a7

Merge "(bug 41292) Fix "containing..." search option"

Fix SpecialPage::isRestricted()

Fix SpecialPage::isRestricted(), broken by I41edb091 (22dd67e) which
inverted the group permission check. Seems to only affect
Special:SpecialPages. Bug 41294.

Change-Id: I8685eaeb4a688cc8e205afdb1639c08e4a2bd3da

(bug 41292) Fix "containing..." search option

Line 132 of jquery.js checks '<input type="hidden"/>' against
rsingleTag. The 'type="hidden"' part causes the HTML to no longer match
the regex, so jQuery fails to add the braced attributes. The "val: '1'"
part was likely wrong anyway; the correct attribute name is "value".

The bug causes the absence of the fulltext=1 query string argument,
which leaves the title near-match logic enabled even when the user
tries to override it. This commit fixes the problem by just putting all
the attributes inside the HTML string.

Change-Id: Ibcc56f64967e55740c10fdca30c525d4afc8fb25

Moved core Job classes under includes/job/jobs.

Change-Id: I9257bb0fce9791190f731d72b2e882a1ae400f35

Removed reference to * from docs.

Change-Id: Iec55a1d009f8e89e5c165f282be4dfda0b2b28a7

Merge "* (bug 41267) mediawiki.action.edit.preview is not loaded"

Merge "Add additional specs + Parsoid-centric tests to keep Parsoid in line."

Merge "check for null content in ApiParse.php"

check for null content in ApiParse.php

Change-Id: I32b3c02163f2295f91b680e3a5636105960b971a

Localisation updates from translatewiki.net.

Change-Id: I6cbab03f0e9af894789a03879afda65aec946f8f

Merge "Use Html class instead of Xml where possible in Special:Contributions"

[FileBackend] Give warnings when metadata is missing from swift.

Change-Id: I2ec2429c5fca02fbd8b5130b3f0db31faf6e365e

Use Html class instead of Xml where possible in Special:Contributions

Also simplified the code by using Html::rawElement() instead of
openElement() and closeElement() and added some lines breaks to
the output for better readability.

Change-Id: Ie7ce5c65ba34b30799580acda0a9d82eb38d257f

[FileBackend] Created README file for the file backend classes.

This patch adds some documentation for the MediaWiki FileBackend system.

Change-Id: I3bf8690f97be783c056c3daf39ff200766e7c8e0

* (bug 41267) mediawiki.action.edit.preview is not loaded

Change-Id: Icfea6f25416ba276ebca5e095e37155a945b8492

Merge "Use Linker::link() instead of Linker::linkKnown() when having options"

Merge "Setting $wgContLang without changing $wgLanguageCode accordingly is a very bad idea."

Merge "doxygen now recognizes README and .txt files"

miscellaneous doxygen warnings

* @licence -> @license
* Protects inline HTML by using double quotes, our inline comments uses
  elements such as <h1> or <firstnameLastname@gmail.com>
* Commands in lowercase (@TODO -> @todo, @NOTE -> @note)
* removes @abstract and @static since doxygen detects them from PHP
  code.
* various undocumented function parameters
* typos in parameters declarations

Change-Id: I62ad6fc124c355bf31acc780b9614a59cf79a421

doxygen now recognizes README and .txt files

The default doxygen configuration file only accepts source files such
as php or python.  Some of our documentation is in flat files which we
might want to include in our Doxygen inline documentation. The culprit
is that those document files needs to respect Doxygen syntax, i.e. the
text content need to be enclosed in a comment block such as:

 /*!
 \page page_title My Page Title

 My awesome documentation
 */

Doyxgen 1.8 supports markdown syntax, we might want to switch to that
eventually.

Change-Id: I78a0ce36314b2a7b9f89395e3ae3902b17cf5038

Localisation updates from translatewiki.net.

Change-Id: I0455c85c4e5166a3520eee90cf4fa0aaa0e170cd

Restore executable bit to bin/ulimit4.sh

Restore executable bit to bin/ulimit4.sh, accidentally wiped in the
Wikidata (ContentHandler) branch.

Change-Id: I49696bce45bbc3f362a6560ace38797c566930c1

Remove duplicate files, no point in loading them twice

Change-Id: I52df1ee1cfeda240e8546fa303b65116183d09e7

Merge "When deleting old thumbnails, actually increment $i"

Merge "Cache performer of a LogEntry to load it only once from the database"

Merge "RecentChangesLinked: 'tagfilter' option already set in parent"

Merge "CologneBlue rewrite: fix talkLink() to use generic nav links"

CologneBlue rewrite: fix talkLink() to use generic nav links

* introduce processNavlinkForDocument() for repeated links
* remove wfFindFile() check for NS_FILE namespace: Linker::link() already
  checks this
* redo the way the message for the link is selected
* comment everything

Change-Id: Ibeb0a1bfa1f34b9fdd9f6e5f082d6973d1ba09b9

Cache performer of a LogEntry to load it only once from the database

Change-Id: If920ba0a29b8c9b04705a20db5d8f9528b251fe5

Merge "Add wfSuppressWarnings(); to CologneBlue."