Merge "Revert "merged master"" into Wikidata

Revert "merged master"

This reverts commit 67bfdc7a68940d901e585eadd984a2074bf0216a

clear magicword cache before testing cleanSig

Change-Id: I89dc2173f6a27dcd16b0548b06070ac4244fd3cb

merged master

Change-Id: Ib2b879c4daa17401eeeb50767c0e5a54254855c3

Merge "Fix error display on failing rollback" into Wikidata

Merge "Check prepareSave() before undeleting." into Wikidata

Replace usage of doEdit() with doEditContent()

Change-Id: I2c423744bd91044c37bbec53b35729bd1c09901b

Merge "EditPage cleanup - parser errors, etc" into Wikidata

Merge "Updating of redirect target in Content objects" into Wikidata

EditPage cleanup - parser errors, etc

Cleaned up EditPage, removing and fixing comments etc.

The most prominent changes are:
* improved handing for parse errors
* improved handling for image redirects
* better readability because one huge try/catch block was removed

Change-Id: Ie33720922eb05dda89a22ca1f5f0cba4b1d31129

Avoid deprecation warnings in test cases.

a) when testing deprecated functions, use $this->hideDeprecated() to
suppress warnings.

b) use doEditContent() instead of doEdit()

Change-Id: Ifa9e0ef373ed72ef7d4a3c2dd558483af4a3fd55

Updating of redirect target in Content objects

Content::updateRedirect may be used to change the redirect target
without knowing anything about the content's format.

Change-Id: Ifaf8f2bdd9d1cbf984e2ef3c60d0282c984c18c1

Merge "Fixing creation of DifferenceEninge" into Wikidata

Merge "Revert all changes to DairikiDiff." into Wikidata

Merge "Make LinkFilter handle Content objects" into Wikidata

Merge "Add Content::matchMagicWord" into Wikidata

Merge "cleaned up comments in DifferenceEngine" into Wikidata

Merge "Made ApiParse aware of ContentHandler" into Wikidata

Merge "Improved content model handling in ApiEditPage" into Wikidata

Merge "Revision::getContent must return clone if mutable" into Wikidata

Merge "Bug 39509: Function for running legacy hooks." into Wikidata

Fix error display on failing rollback

Rollback will now show the actual errors that caused it to fail,
instead of a misleading message about permissions.

Also, errores returned by WikiPage::doEditContent are now propagated
and shown.

Change-Id: I5a3f8cb7a0b881d07d52f63504dd3757192205a9

Check prepareSave() before undeleting.

This calls prepareSave() before undeleting a page, to make sure the restored
revision does not violate any constraints imposed by the ContentHandler.

To achieve this, several improvements are made to the error handling in
SpecialUndelete.

Change-Id: I41bab9892de7c604be6aa7f6db9dee47b3f0d27c

Fixing creation of DifferenceEninge

Creation of DifferenceEninge insteances was based on invalid assumptions
in several places.

Change-Id: Ib67ca893ea53a27d4ac7efd8a9ca0d3b2d93949f

Revert all changes to DairikiDiff.

Reverting unrelated changes to DairikiDiff. Slight refactoring would
make the diff classes much more reusable and flexible, but that needs
some more thought, and is unrelated to the task at hand.

Change-Id: I9f132985207096676cbb6139c7d37572a10b8806

Merge "cherry-pick bdaee6f6594532409ce55e9cde0f942f96ec4181" into Wikidata

cherry-pick bdaee6f6594532409ce55e9cde0f942f96ec4181

Change-Id: I5251c8f83f401731c3027718656cb65d8d910f22

add docs for getContentInternal()

Change-Id: I87a0c9908975c236794bad39209ad8bdef4699c3

Add Content::matchMagicWord

Allows Content objects to control how magic worlds are applied to it.
Useful mainly when magic words are used outside the parser.

Change-Id: I9711f580f8beadfb9f93a706f87c97c7cf667f6c

Make LinkFilter handle Content objects

Change-Id: Iced1c83642143faa79a5461ae6278267e0595aa9

cleaned up comments in DifferenceEngine

Change-Id: I939fa563a66357efb8eae8d8f60bf0ba08cd05eb

Made ApiParse aware of ContentHandler

Change-Id: I656e4cab6641eaae98b40700c7b3a7cb0eeed54f

Improved content model handling in ApiEditPage

Change-Id: Ifd9e344864f7663de7263cb536e83dbdc03cf2ed

Revision::getContent must return clone if mutable

Revision::getContent must return a cloned instance of the Content object
if the Content object is mutable to avoid confusion.

Content::copy is used to achieve this, which is specified to return $this
for immutable Content.

Change-Id: Iace17b6ae8aa85a3500624441b69bc067c1ade00

Bug 39509: Function for running legacy hooks.

ContentHandler::runLegacyHooks can be used to run hooks that don't
supprot Content objects yet. runLegacyHooks will issue a warning and take
case of serialization/unserialization of the content as appropriate.

Changeset 2: rebased.

Change-Id: I31109061110f87c38bdeebf30d520c8e1241bb29

fixing long lines

Change-Id: Iebcd40a1a37a3a93cc571fa2c2b0a87f53dc97a4

Merge "merging latest master" into Wikidata

merging latest master

Change-Id: I36b7f2f63ab8c08f8412d521dc68ea45c8b67711

Added utility method taking an array of elements and wrapping each element in it's own array.

Useful for data providers that only return a single argument.

Change-Id: I220948865e3de395363bf8858098e61de3c8d35b

Merge "add Article::setParserOptions" into Wikidata

add Article::setParserOptions

Change-Id: Ica1c0a3c4b5b569c4911d4c1fb0167812f932c20

Added fixme for incorrect assumption being made

Change-Id: I40b83428019b32c39c529577bf3f55677a407693

merged master

Change-Id: I6afafe971afb3f38fc8f1e66ba409283b8a698f8

Revert "(bug 260) Wrap <pre> overflow properly."

Due to bug 38800. Just reverting to prevent accidental deployment in
1.20wmf9. I would set a fixme status instead if there was such a thing.

The problem with this is that it breaks formatting of thousands of
Wikipedia articles due to interaction with SyntaxHighlight_GeSHi. I'm
sure some satisfactory solution can be found which fixes both bugs, but
leaving this commit deployed is not a good interim solution.

This reverts commit 782b9f177404f02417c03abaf96b371e1462f160.

Change-Id: I1977f093563b5658def9ba57a6ea6cfa9796f795

Merge "[FileBackend] Skip over some illegal paths and output the error."

[FileBackend] Skip over some illegal paths and output the error.

Change-Id: Ib32aadc6398168e4a63279d7da61a0dd1e1a6483

Merge "Add localonly= to prop duplicatefiles and imageinfo"

Better inline styling for badge.

Change-Id: Ie92a18478c0f9b62fceff39f54a9524432eec921

Merge "bug 36073 Use mw-imagepage-linkstoimage-ns as a class instead of an ID"

Merge "Change File::compare to use getName, not getTitle"

bug 36073 Use mw-imagepage-linkstoimage-ns as a class instead of an ID

This avoids multiple defines of the same ID on a File page.

Change-Id: I0f8ae37d74f2ec519361b2af164e115f6efa4ac6

Merge "memcached: better error messaging"

(bug 38492) let doxygen document variables

We are using '@var' to document our variables and class properties,
which is unfortunately not working since '@var' is really meant to
document a function or method.

The way to fix it is to use an input filter that will rewrite our PHP
source code to pretends variables are typed. Aka something like:

 /**
  * A title object
  * @var Title
  */
 var $title;

Will be made:

 /**
  * A title object
  * @var Title
  */
 Title $title;

That is incorrect PHP code but it is properly recognized by Doxygen.

This patch as a side effect, all variables and properties will end up
being documented in addition of type hinting.

Use a hack authored by Goran Rakic at:
http://stackoverflow.com/a/8472180/276152

Change-Id: I4ead1bd1feace44496b45ed8c55f5e52c59e7694

Add localonly= to prop duplicatefiles and imageinfo

When looking only for local files, the localonly param skipped the
mabye expensive look up inside the file repos.

Change-Id: Ib8f38d6abf9238a349bbfd617a36933bdfe74b5c

Change File::compare to use getName, not getTitle

For files the file name with underscores is often used,
than the compare method should do that also.
This matches than the ORDER BY img_name done by some query.

Change-Id: Ia1b89577538bdfdf751c7b9584e7c008a8c2bb1e

memcached: better error messaging

MemcachedClient output a generic error message: "Error parsing memcached
response\n" whenever it is not able to read from the socket. It is also
lacking the remote peer it is reading from.

This patch add a new message when fgets( <socket> ) return false, which
means we could not read from the file pointer. It also get the stream
remote name for debugging purposes.

Change-Id: I9b8a25a03af0d730aa3b4830a44b1ea739343274

Merge "autoloader entry for UserNotLoggedIn class"

(bug 37830) new $wgRequirePasswordforEmailChange

$wgRequirePasswordforEmailChange to control whether password
confirmation is required for changing an email address or not.

Change-Id: Iaef440ef56d391bf9e68d15899fc81c6050722fb

autoloader entry for UserNotLoggedIn class

I have forgot to add an autoloader entry for the new exception class
named UserNotLoggedIn (introduced by Id8127299 / b1c1448d).

Change-Id: Ib7226c8532fd695edf78f0e4c33b91edbf8ed52e

Merge "Namespace gender aliases for Lithuanian (lt)"

Namespace gender aliases for Lithuanian (lt)

http://translatewiki.net/wiki/Summary:Support/Gender_in_Lithuanian_namespace_names

Change-Id: If0a30180fb103ee1b56781a5fd1587b48e100da8

(bug 38151) Implement mw.user.getRights and getGroups

* Also deprecate name() and anonymous()
  in favour of getName() and isAnon() (to match User.php), just
  like getRights and getGroups are now as well.

* Added unit tests for all.

Change-Id: I5970be9e8593589018152f4878f168b9b85ca5d5

Merge "Revert "AJAXify watchlist editor""

Revert "AJAXify watchlist editor"

Doesn't properly work on Special:EditWatchlist, some of the titles being unwatched are not fading out and staying behind.

There's also a small code quality issue unaddressed. Please re-submit for further review.

This reverts commit b46ec8fde5dfd3e80fa3861a01f1aac6f5a7b5ba

Merge "(bug 30625) Return warnings, if they exist, despite ignorewarnings"

Merge "(bug 18834) Add an edit count to rollback link"

Merge "(bug 27395) Write PHPUnit test suite for CSSJanus"

Merge "AJAXify watchlist editor"

(bug 31644) [MERGE DEPENDENCY] Let WikiMap url builders return full urls instead of canonical urls

* This commit depends on a commit of globalusage https://gerrit.wikimedia.org/r/17117
* This allows GlobalUsage, CentralAuth and AbuseLog extensions to use protocol relative URLs.

Change-Id: I7e65bd029f359c36770aa48e6663f26a671b92c0

(bug 18834) Add an edit count to rollback link

Add an edit count to rollback link to show how many edits will be
rollbacked. When the count is over 10 the text "more than 10 edits" will
be outputed.

Change-Id: I5c4050e0a9197d4c505e85685a9780c97138d427

Merge "Some fixes to our jQuery UI skin for buttons"

Merge "[FileBackend] Made swift handle unknown content types better."

(bug 27395) Write PHPUnit test suite for CSSJanus

Change-Id: If40c5de54bffae6e5ac8fac81754a53a40c652b1

[FileBackend] Made swift handle unknown content types better.

Change-Id: I50a5c17c40152a9e469a54bdd70a8408c57ebd50

Merge "Add hint for pipe-separated list to action=help"

Merge "Fix logic error from Ia74ac399 (2ee4653)."

Merge "* oracle DB schema update to current (changes relevant to 1.19 will be backported) * fixed an updater bug from 1.19 (will be backported)"

Fix logic error from Ia74ac399 (2ee4653).

The "views" part of the table should only
be displayed if view counters are *not*
disabled (and not the opposite).

Change-Id: I4859f396cc9e2eb7b9f4eff6502efc459a3410fd

Localisation updates from translatewiki.net.

Change-Id: I6556bddc4204ecebff39db8c8d01e8a169e62fc0

Added placeholder-msg support and more documentation to jquery.localize

* Added examples of all options (prefix, keys and params)
* Added example of raw attribute being used to bypass escaping
* Added documentation to code where it seemed clarification was needed
* Reduce duplication and simplified  some unnecessarily complex statements
* Added test for new placeholder-msg attribute support
* Made adding more attributes in the future easier
* Moved message wrapper function out of the localize function so we aren't defining it each time localize is used

Change-Id: I77593acf4c1b3974d502a1a8382cca9267af2d41

Merge "(bug 37390) Clean up QUnit test suites"

(bug 37390) Clean up QUnit test suites

* Use new QUnit 1.8.0+ pattern:
 - no-globals (only QUnit itself)
 - assert-object pattern
 - Remove pointless '-- Initial check' assertions.
   I introduced this pattern last year for no particular reason and
   everybody just copied it. There is no reason for it, it works fine
   without it. And the rest of the world that uses QUnit doesn't do
   it either. Exceptions are caught by QUnit and error handling there
   is much better than a dull '!ok()' assertion.

* .jshintrc:
 - Tolerate "mulistr" (this is used in many extensions, test suites
   as well as in MediaWiki core)
   "foo\
   bar"
 - Use "jquery: true" instead of "jQuery" so that JSHint can optimize
   for jQuery edge cases.

Misc issues fixed:
 * Trailing comma's in jquery.tablesorter.test
 * Missing semi-colon in testrunner.js
 * Remove backwards-compatible line for "equals()" and "same()".
   These have been deprecated in QUnit since 2009 (which is way
   before we even started looking at QUnit).
   Inside QUnit itself is a deprecation notice, I don't know why
   I added this compatibility line, it is stupid and overrides the
   nice deprecation-warning that QUnit has built in for it).
 * Use QUnit.push inside assertTrue and assertFalse.
   QUnit assertion methods must not call each other, in order to keep
   the call stack length as expected so that QUnit can report the
   file and line number of the caller of the assertion in case of
   failure etc.

Change-Id: I5cded6e8c32dba39170a02bdd3236c8b7b04d269

Merge "Fix at end navigation condition for query pages"

Merge "Realign $magicWords array in LanguageEn.php"

Merge "Move some readonly around"

Merge "Make MWHttpRequest::__construct() protected."

AJAXify watchlist editor

This patch makes the watchlist editor to use pagination. That would
avoid old browsers crashing (bug 20483).
Patch also add some AJAX operations to the editor, for example to remove
items from watchlist (bug 32151).

The AJAX support is an ALTERNATIVE to the form based method, to keep
support for non javascript users.

This change contains a required change in the API for watch operation,
to allow batch operation, by support titles parameter. The old title
(single page) parameter is still used to keep backward compatibility.

Change-Id: I1d8c66db9ba6456858ef655397935a2b3a421632

Clean up badger.{css,js} from PageTriage, generalise it and move it into core.

Change-Id: I0f5e754146359448eb125456e240cf5768d4f541

(bug 37331) Modules sometimes execute twice in Firefox

This seems to have been caused by a weird race condition that caused the
loader to run while the module in question was also running; could also
have been caused by the module calling mw.loader, but that didn't seem
to be the case here.

The fix is to set .state='ready' before executing the module rather than
after. We can't do this in debug mode (where we're loading raw files and
can't execute things at will), but we have to do this in production
mode to prevent these freakish double executions.

Change-Id: I7e8dbd361fb265e520d3935e3a1fc7e3b6710b66

Move some readonly around

Checking first permission and than read only is better, than user
without permissions does not retry it when read only is over.

Moving disallowUserJS to top so javascript is always not possible, same
is already done on Special:ChangePassword

Change-Id: Ia27ee177b67f1151fd1178b24744b65da8e756fd

Merge "Localize comma separators in jump-to-nav."

Merge "(bug 38362) Special:Listuser now includeable"

Make MWHttpRequest::__construct() protected.

This class is not for direct initialization, and
new MWHttpRequest(...)->execute() doesn't work at all.

Change-Id: I557723c218cd8e8f27283d01141c890263da095d

Add @since 1.20 to getDirMarkEntity

Change-Id: Ib74b68a85e470ef23685802af93c3ec66d50e1ed

Merge "Follow-up Ifa9c55b7 (58852d8): Change one more item to getWatchedItem()"

Merge "No need to call parseTitle() directly in MediaWiki::__construct()."

Merge "Don't unset() $wgOut in AjaxDispatcher."

Merge "Fix for Ic74c7ba7 (c8c50eb): there's no reason to pass $this by reference."

Merge "Use local context to get messages and time formatting methods of Language class."

Merge "Store the Title and User objects passed RecentChange::notify*()"