Added new + updated old tests for preventing regressions in Parsoid

Change-Id: I2963d8f2a91ec473eb7c9b063c4c5a4c2de8f82d

Merge "Capitalize URL in api help"

Merge "Fix typo "relavent" in release notes"

Localisation updates from translatewiki.net.

Change-Id: I053c79ff1f095d49668e94a0e05a2b56e668a9ea

Fix typo "relavent" in release notes

Follows-up 8c328d030fc1f7f.

Change-Id: I7395d78b29ba4f754466ded6e0c3b14f35f92440

Localisation updates from translatewiki.net.

Change-Id: I6c67e5c688c85cb291255f67f8fe10c576474272

Capitalize URL in api help

As part of bug 34965

Change-Id: Ia1df9748b5de33f6ef5088103ccc6b886f9a17d5

Localisation updates from translatewiki.net.

Change-Id: Ifa64f58104e4e0fdeafd137079676a33698bb229

Merge "Allow SearchUpdate hook to abort core update call"

Allow SearchUpdate hook to abort core update call

Going to use this in CirrusSearch since the default text
handling is insane for Solr.

While we're at it, further move content handling to SearchEngine
so children can override behavior here.

Change-Id: I09d11b81c224d53609c57d75d54021e697b56629

Merge "Docs and drop silly m prefix from private variables"

Allow modification of flags on Recent Changes

Moved the message map in ChangesList::flag to a new setting named
$wgRecentChangesFlags and used it keys to iterate over the flags
Also allow setting the class to deal with minor vs. minoredit

Also make ChangesList::recentChangesFlags public to be used from
extensions, when they will build its own line

Changed one list of placeholders to use recentChangesFlags to have the
correct number of spaces there

Change-Id: If69de05c4edec4e84c3e059090b9cb89845e3269

Merge "mediawiki.api.parse: Specify wikitext as the content model"

mediawiki.api.parse: Specify wikitext as the content model

The parse API defaults to use the content model of the title, which
defaults to "API" in the main namespace which can be configured to have
a different content model.

Bug: 48201
Change-Id: I51143873e3782df4df500194a7ac71c4b446746b

Change @since and @deprecated notes to 1.22

Using the following command line, I have found doc comments mentioning
"1.21" when they should mention "1.22" instead, which I have fixed
manually:

git diff REL1_21 | grep --color=always -C 10 -iE \
'^\+.*(since|deprecated).*1\.21(\D|$)' | aha > oldver.html

I also moved the release notes for I1987190f ("Combine JavaScript and
JSON encoding logic") from RELEASE-NOTES-1.21 to RELEASE-NOTES-1.22
because I had reverted the commit on REL1_21 only (see Id3b88102 and
bug 47431 for the rationale).

Change-Id: I11b917a371e07267dfa98b8449776d0c1cb29b15
Follows-Up: I25cf5a94f6e47f85a9d0b80cc1c9c9f957288478
Follows-Up: I3d72e4105f6244b0695116940e62a2ddef66eb66
Follows-Up: I3faa9c3e8107c6e46cdf21f8c18adda1f42890d7
Follows-Up: I6aab19c8d68bf47beddad42632b0360a7b12f251
Follows-Up: I86368821fc2cd0729df5342b8572eb470c0f77a0
Follows-Up: Id3b88102e768318e3605a19e9952121091a40915
Follows-Up: Ie667088010e24eb6cb569f9e8e8e2553005223eb

mediawiki.js: Clean up and improve documentation coverage

Function addStyleTag was only documented as a private function,
since a while now it is also exposed as a public method.

Rename the private function so that they can both be included
in the JSDuck index (jsduck can't deal with name clashes between
public and private). Optimised by letting it inherit the
documentation instead of repeating it.

The following were missing documentation:
* mw
* mw.hook
* mw.legacy
* mw.loader#addEmbeddedCSS

Moved assignment of `done` in mw.loader#addScript to where it
was needed. The other if/else paths in that function don't use
it and don't need it to have a value.

With the introduction of mw.log#deprecate (fe46903) we no longer
need mw.legacy. mw.legacy was created for future use to move
legacy stuff into to make it easier to keep track of where they
are used (simple search for 'mw.legacy') however that never
happened and in retrospect was probably a bad idea as it would
require people to change code twice.

We now have a much cleaner solution now that we're able to
create little accessors on any host-object with mw.log#deprecate.
This makes mw.legacy obsolete and it should not be used for
anything. Just in case something is using it I won't remove it
just yet, instead I've deprecated the (what would have become
the) deprecator ;-). We should be able to remove this in 1.23.
Note: Can't use mw.log#deprecate here because that isn't defined
yet at this point, this is the core library which can't have any
dependencies.

Change-Id: I5d241e4284f07a6be271c04cec3a5833dbaeed91

Docs and drop silly m prefix from private variables

Change-Id: Ib2e5507e5735dd94423174f475e6a8e733f89fd7

Merge "displaytitle: Pass-by-reference fix for PHP 5.4"

displaytitle: Pass-by-reference fix for PHP 5.4

PHP 5.4 includes some pass-by-reference changes which apparently make
the '&' sigil necessary here.

Spotted by Mormegil <mormegil@centrum.cz>. Followup to Ie162535b.

Change-Id: I038fa7f8e1559c95dba7b9b498fc1d1f5eb76887

Return errors from WatchAction

Currently, WatchAction::doWatch and WatchAction::doUnwatch return true
always. Let's have them return a status object instead.

This also cleans up the handling of Status objects in some of the API
modules.

Change-Id: I9dd9f0fd499c37f29fa12bcdb6142238a1f11e4d

doc: speed up dot generation

dot supports generating multiple graphs in one run, that was not always
the case and Doxygen had a default of not using that feature.  Given it
got introduced in dot 1.8.10 and Wikimedia CI server has 2.26.x, I guess
it is safe to enable this.

End result, is less shell out overhead.

Change-Id: Ia07b430bd1af608bbe9c10eecbe2df3e0b9c74fc

doc: raise doxygen lookup cache

At the end of a run, doxygen emits a notice regarding the cache lookup
size being too small.  This cache is used to find symbols given their
name which would speed up processing a bit.

The notice before this patch:

lookup cache used 65536/65536 hits=2258467 misses=112595
Note: based on cache misses the ideal setting for LOOKUP_CACHE_SIZE is 1
at the cost of higher memory usage.

And after:

lookup cache used 104632/131072 hits=2131285 misses=105434

That is 7% less miss, not that much but still.  Note how the cache is
well filled (80%).

Change-Id: I0b97e8c5581a63c44a382891f8f1fc8d60975e80

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I213cd86665ecdbd77e7504e9cd77eea9cbd15b07

doc: sync with doxygen 1.7.6.1

The Wikimedia CI server runs doxygen 1.7.6.1, this patch sync our
configuration with that version.

TCL_SUBST and LOOKUP_CACHE_SIZE have been added.
IDL_PROPERTY_SUPPORT default value changed, unused by us though

Change-Id: Ic43d99c4675eb8f1772a72a9b9bfaf16057e19f1

Merge "Fixed doc typo"

Merge "PHPUnit now recognizes extension parser tests"

Fixed doc typo

Change-Id: I158c794539036d48a34dec9f2bbd202752da4fa4

Merge "Restore messages of old login and signup forms"

Restore messages of old login and signup forms

The change 44840d15929d8ecfc1d7c31d6f7e86d873ac15a2 "Only new vertical format login and signup forms" removed
some messages no longer used in core, but 'userlogin' is used by
extensions including MobileFrontend

This restores the messages but marks them as unused in core and deprecated.

Change-Id: I82c4fc37481a4a0dea1e839238188f6555e3c60d

Merge "Don't show the "permissions error" page after a user self-removes rights"

Merge "Pass content to SearchUpdate rather than raw text"

Pass content to SearchUpdate rather than raw text

This is still less than ideal, but at least consolidates the
logic so we can later implement more content-specific handling.

Change-Id: Ic24a7ea2818f6eeb67453877dc5a31f110541ecf

Merge "Use getFileContents() instead of incorrect file system call to fetch file contents."

Remove unused class, SearchUpdate is backend-agnostic

Change-Id: I8c91c28ff2bde361490691b4abc107329f0d8194

Merge "filebackend: throw exceptions during file iteration."

Merge "UserCache::getUserName: Docs said @since 1.21, but not in REL1_21"

UserCache::getUserName: Docs said @since 1.21, but not in REL1_21

Change-Id: I4ad85ee36ed348e8b082b5a4dba11d7cb92d8ad0
Follow-up: ef915e37e3d872

Merge "vector: Add SVG versions of collapsible menu icons"

vector: Add SVG versions of collapsible menu icons

SVG with a PNG fallback is added for the collapsible tab icon.
It allows SVG-capable browsers to view images at full resolution
while keeping backwards compatibility with browsers that don't support
SVG.

Implementation similar to I914da06494.

Change-Id: I0448e34bd10aa5cf83f1530a44b1555caee271b1

Merge "Switched FileBackendStore to use ProfileSection when possible."

Merge "Avoid exceptions by first checking language code validity"

Improve documentation of action=parse (followup Ie4f7324c)

The documentation for action=parse needs clarification, particularly in
light of the changes made in Ie4f7324c which were too hastily merged.

Also, add the RELEASE-NOTES entry missing from Ie4f7324c.

Change-Id: Ia8847f7e99620a9f7af24956d972c882f78aae26

Restore API action=parse&text=foo functionality on wikidata.org

Bug: 48201
Change-Id: Ie4f7324c2a10700e52dadc4facd6b4fe6c76cf48

Merge "Only new vertical format login and signup forms"

Vector: Add the collapsibleTabs script from the Vector extension

* Move jquery.collapsibleTabs from the Vector extension
  to the Vector skin (see 097f387362)
* Register the module and its dependencies
* Merge the init script into the existing vector.js

Bug: 46513
Change-Id: I7bd77e6e4c384c9e2033f9d05ffcae86b0bb7a69

Switched FileBackendStore to use ProfileSection when possible.

* Also removed outer profiling layer.

Change-Id: Ice3b921fe0c9981ac16f005d66a78bf4f4a5c601

Merge "Support updating search index when page is deleted"

Support updating search index when page is deleted

Right now, if you delete a page then you end up with a stale
entry in the search index, this affects all core SQL-based
searches.

For extensions, this means they no longer have to implement
something like ArticleDelete, they can just add the delete()
method to their SearchEngine subclass.

Change-Id: I9d8a9878aeebc53f453ab1cbacc03fe73fcca949

Localisation updates from translatewiki.net.

Change-Id: I4e865f40a8c69fdc7520bd3a89ec61ca9af57337

Merge "Removed redundant doc comments."

Merge "Make rc_cur_id have proper value for upload log entries."

Merge "Link to mediawiki.org page in 'edithelppage' message"

Merge "Pass along tagFilter to ContribsPager, it already expects it"

Pass along tagFilter to ContribsPager, it already expects it

ContribsPager already has this below line:

    $this->tagFilter = isset( $options['tagfilter'] ) ? $options['tagfilter'] : false;

The reason the code already works fine is because, in ChangeTags.php, there's this fallback:

    if ( $filter_tag === false ) {
     $filter_tag = $wgRequest->getVal( 'tagfilter' );
    }

Which essentially boils down to the same result as passing along tagfilter, which in
SpecialContributions is read from $wgRequest, passed to ContribsPager, and (well, currently
not) passed to ChangeTags

Change-Id: I12c0336ae31c5524e41fa0087afe2ba104dc7d37

Only new vertical format login and signup forms

* The "VForm" templates replace existing templates.
* Remove useNew switch logic and $wgUseVForm{UserLogin,CreateAccount}.
* The CSS and JS files for forms lose the .vform qualifier, except for
  the CSS for styling vform elements.
* Merge tiny mediawiki.special.userLogin.signup.js into createAccount.js.
* Remove replaced and obsolete messages (see
  https://www.mediawiki.org/wiki/Account_creation_user_experience/Strings
  for details), and remove mention of "new" forms and useNew from
  MessagesQqq.php.

Bug: 46333
Change-Id: I50f25583c3aa9a4a263fb40f50256f92c18ddb4d

Merge "mediawiki.api: Implement login method"

Make rc_cur_id have proper value for upload log entries.

Primary motivation here is to make Special:Recentchangelinked work
correctly for uploads where the description page doesn't exist yet.
(Previously it was mostly only showing reupload log entries)

Mild downside to this approach is adding the recentchanges row is
now in a different commit...

Change-Id: I74c68b972c47c03066d3065e3748769a27a654fb

Merge "Localisation updates from translatewiki.net."

test: rework interwiki lookup in parser tests

Some of our parser tests lookup interwikis.  This was originally done
(parser/parserTest.inc) by inserting a set of interwikis in the database
and was later lamely copy pasted in the PHPUnit wrapped test suite
(phpunit/includes/parser/NewParserTest.php).

Since that time, we had duplicate code and had the test hitting the
database to fetch interwiki.  Nowadays, we can trick the Interwiki
lookup class by using the InterwikiLoadPrefix hook, that let us skip
database lookup entirely (by having the hook returning false) and get
rid of the duplicate code.

The good old parserTests.php still pass the tests :-]

Change-Id: I36865e3890e08a05b8a6aaafa309a87556e235b9

Localisation updates from translatewiki.net.

Change-Id: I32973f48e15c917221cefe5d6d36a122e30cf706

Merge "Do not output numberofviews, if $wgDisableCounters = true"

Do not output numberofviews, if $wgDisableCounters = true

Follow up I9975dceedeffafadf560ed71238c51b94eb8c3ad
But now for the parser function

Change-Id: I18261b884b7f3be57a42c0b10594384fab6f9c9a

Merge "Remove Selenium tests from core"

Merge "jquery.makeCollapsible: cleanup"

PHPUnit now recognizes extension parser tests

Parser tests are registeredd by appending one or more .txt files to the
$wgParserTestFiles global setting.  Since this is shared with MediaWiki
core, I have made MediaWikiParserTest a factory of PHPUnit testsuite
which would filter in/out extensions tests.

The `extensions` test suite now has a second test suite builder which is
simply a wrapper around MediaWikiParserTest factory.

Play cases:

  $ php phpunit.php --group Parser --tap

Runs any parser tests including the ones coming from extensions.

With an extension having parser tests such as Cite:

  $ php phpunit.php --testsuite extensions --tap
  // Extensions tests are run including parser tests.

bug: 42506
Change-Id: Icc3e9d30706b32149aa9dd18552e4241ec4af67e

Fixed mysql warning about expecting a resource.

Change-Id: I799d063fec7a8e96f49db34ce669453bd26c3fb3

Merge "Localisation updates from translatewiki.net."

Merge "Don't reinvent the wheel in SpecialRecentchangeslinked::getExtraOptions"

Localisation updates from translatewiki.net.

Change-Id: I352ed34fa912240395a72e84e60f19c42fe10fb3

ResourceLoader: Serve stylesheets with "Access-Control-Allow-Origin: *"

This allows Javascript to access our styling information (cssRules and
related cross domain protected dom information) even if it is hosted
on a different domain.

Bug: 25886
Change-Id: I8e05c13ae1a1589fd120d5c439b1a7128ce2b659

jquery.makeCollapsible: cleanup

* Ensure correct handling of togglers which hook to multiple elements
  and elements which have multiple togglers
* Move the `options === undefined` check outside the loop
  per CR on I8dd4f0d0
* Get rid of the old private toggling functions for various kinds of
  toggles (obsoleted, but left alone, in Id3f457a8)
* Don't use separate logic for initial toggling of custom togglers
* Consistently use collapseText/expandText (uppercase T) instead of
  collapsetext/expandtext (lowercase t) where possible
* Adjust comments and whitespace

Change-Id: Id8ecd8f577fb4cd3f640a7830a85436a301b3e2c

filebackend: throw exceptions during file iteration.

* This lets calling code be far more robust rather than just
  silently ignoring entries due to some temporary problem.

Change-Id: I3ce2ae34f6cff5e40a80b8da5688503a387ce2a6

Removed redundant doc comments.

Change-Id: Iced16c6d32ef0e4d58376ed3c092a500ec917446

Localisation updates from translatewiki.net.

Change-Id: Ice334feb284bd21c83c26ea92705a1345d10b225

Merge "Cleaned up variable/type order in docs."

Fix typo ('associed' => 'associated').

Change-Id: I5ea2eec92ebe397f2d269562f1650e59cb969a19

Merge "Localisation updates from translatewiki.net."

Fixed doc typo

Change-Id: I1e4ab410cd02a74e667cc8611e58ad791558896a

Localisation updates from translatewiki.net.

Change-Id: If58755fdcfc3c12f16436c4a245a520cb037d994

Remove Selenium tests from core

This never really worked properly, doesn't conform to coding
conventions, and isn't actually used by anyone for testing.

Our selenium stuff these days are in the qa/browsertests repo

Change-Id: I8f1efaa118fe41821fb3f4a6099d75a33681f17b

(bug 47449) {{#special:}} should always return a usable title

{{#special:}} should always return a usable title in NS_SPECIAL,
even when given an unknown special page title. Currently, it returns
"No such special page" text for such unknown names, which is
surprising.

Bug: 47449
Change-Id: I7f184956ec9d56c849f201f1230736fcedd0fadf

Merge "Fix FOUC on preferences."

Merge "(Bug 48952) Show debug output in toolbar"

API: Fix action=parse without any page or title or text

action=parse was raising an error about missing 'text' with 'title' even
when 'title' wasn't actually passed, due to a fix for bug 33865. But
this broke using action=parse to parse an edit summary without also
giving it wikitext to parse.

Instead, let's give the client a warning (unless it seems clear they
know what they are doing) and assume the empty string for 'text'.

Also, while we're editing the file, add some more examples as also
requested in bug 48319.

Bug: 48319
Change-Id: I03c1fbcb0bd31dea8bd84e164104f7ced0ace449

Merge "Added parsoid-only tests for parsoid's native cite implementation"

Made purgeDeletedFiles.php delete files still in the public zone.

* This makes purges more useful since the file won't come back.
* This only applies for files that are confirmed to already be
  in the deleted zone too. This is for sanity.

Change-Id: I60316c1d9323347d09607e36a334fc5eb0b6a2e7

Added parsoid-only tests for parsoid's native cite implementation

Change-Id: If5cf091d7c6c864f6bd90165086a19ae38ec257e

Fix patrolling where NP patrol isn't explicitly enabled

Hot fix! Follow-up fc5bcf1147f9

Change-Id: Iba7e70e1a1ddd9826f6e5f4022b88ccafc5f571d

Localisation updates from translatewiki.net.

Change-Id: I4e12c5f98fb8da8d7b83e167363668b2504b26b2

Cleaned up variable/type order in docs.

* A few other minor doc cleanups and obvious type hints.

Change-Id: I454fad0c1942068b4f3632ebd1b605bfbb9dfe7c

API: Fix IPv6 handling in list=blocks

The current handling of the bkip parameter assumes IPv4, and breaks for
IPv6 CIDR ranges (it also isn't working right for IPv6 non-CIDR, but not
in an obvious way).

This rewrite handles IPv6 correctly. It also necessarily adds validation
for the bkip parameter, which would formerly return (not very sensible)
results when passed invalid values.

Bug: 48129
Change-Id: I02471bb32c3a217004d07a79d9f98b62133b31ef

Merge "Add section preview mode to API action=parse"

Avoid exceptions by first checking language code validity

Bug: 49423
Change-Id: I3fd98ba08393856311a48fa40769027460c72ef9

Merge "Ensure <h5> and <h6> have different computed font size"

(Bug 48952) Show debug output in toolbar

The debug output written with the function wfDebugLog should  be
printed to the debug toolbar in the same way as wfDebug does.

Change-Id: Ib979df080332e16c8e0e6a89eab8741986ab39c3

Prepare DatabaseMysql for mysqli

PHP extensions mysql and mysqli have lots in common.
In order to support mysqli, this diff separates common MySQL behavior to abstract class DatabaseMysqlBase and implements the mysql-extension specific in DatabaseMysql.
Outside behavior remains the same.

I've tried to minimize future code duplication by introducing mysql*() methods even if the parent method just unwraps the result.

Bug: 45288
Change-Id: I905d4a4550377bc849a860f0962dad710d9dc71f

Fixme comment for bug 49504

Change-Id: Ie229e84b299c357cfe9a2db0a5ba07713aac7597

Add section preview mode to API action=parse

In https://gerrit.wikimedia.org/r/#/c/67266/, it was enabled automatically
when a section was specified in preview mode. However, this only works for
previews of a saved page's content, failing for user input previews which
might contain several sections of various degrees of nestedness.

Change-Id: I987144e5e315ed9e0a67f599083461a78879c0df

Merge "Remove non UTF-8 chars from debug output"