tests: mock parser tests file access

I eventually got tired of our parser tests creating and deleting fixture
files over and over.  This patch mock the files in memory and just
expose the file metadata which is all we need for parser tests.

The mocked classes are under /tests/phpunit/mocks/ and respect the
hierarchy of /includes/.

The wiki.png and headbg.jpg files are still copied on each test :/

Change-Id: Iccdff67222e66d48d01dd1596d09df2ea24b8c2a

Merge "Increasing z-index of personal toolbar from 1 to 100"

Merge "Removed unused join array from ActiveUser conds."

Merge "Change EXIF to Exif"

Change EXIF to Exif

Per https://en.wikipedia.org/wiki/Exchangeable_image_file_format. Spotted
by Shirayuki and documented on
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Sesp-property-exif-data/en

Change-Id: I92ba67ec60ccfe7a173d950593357b86792b8ed3

CREDITS: Spell my name right

That's been bugging me for a while.  :-)

Change-Id: If543ac24ee36fdd913ac2e2839a282a677d90152

Removed unused join array from ActiveUser conds.

* The table was not in the table array so this did not affect
  the SQL, but was still unused cruft.

Change-Id: Ia5aa934fb6e6de9daeb0072d902df6e681575fd7

Increasing z-index of personal toolbar from 1 to 100

Since the personal toolbar is at the top of the page and may include
children that need a high z-index, such as dropdown menus or overlays
we should set it to at least be equal to the highest potential
competitor (the Visual Editor toolbar).

The other option would be to not set a z-index at all, but this would
regress bug 37158.

Bug: 48078
Change-Id: I61daea50209d040f7e0f71207a4e90998c15083f

Merge "Fixed spacing"

Merge "Localisation updates from translatewiki.net."

Merge "echo is not a function"

Merge "Add attributes to the Parsoid-only tests."

echo is not a function

Removed parenthesis after echo

Change-Id: Ia533aedf63b11d15dcc6a5cf75a56134a4b11d86

Localisation updates from translatewiki.net.

Change-Id: I454062bca563af5a95bf57ada92a113532a7ed59

Add attributes to the Parsoid-only tests.

Needed to fix these up in order to get them running right when the Parsoid
test-runner begins to let through more attributes.

Several tests are make wt2html/wt2wt only because they involve templates
which we can't convert from html2wt without additional information.
The "SOL-sensitive wikitext tokens" test was previously wrong: the PHP
preprocessor generates lists from the template content.  Updated it to
the proper results while adding attributes, although it will currently
fail until we either fix the native Parsoid preprocessor or tweak our
parserTests to use the PHP preprocessor.

Co-Author: Mark Holmquist <mtraceur@member.fsf.org>
Co-Author: C. Scott Ananian <cscott@cscott.net>
Change-Id: I657e4869fb0c2ae66d6732d28c6fc6645ad8e534

print is not a function

Removed parenthesis after print

Change-Id: I1343872de7aa7c64952a3d86a63aaa091e46bda3

Fixed spacing

Added spaces around some parenthesis

Change-Id: If1e50e2a7b0046e91c0bbce8bf6641d8b2446a1d

[FileBackend] Added "adviseStat" option for the "listing followed by stat" case.

* Used this parameter to speed up copyFileBackend.php.
* Also added mtime checks to copyFileBackend.php and a few cleanups.
* Also fixed some incorrect getDirListPageInternal/getFileListPageInternal docs.

Change-Id: I424ef238f7adf4cf1f33b74e3a4e187dcb328a99

Merge "phpcs: Fix WhiteSpace.LanguageConstructSpacing warnings"

Merge "Refactored Special:ActiveUsers queries to remove filesort/temp table."

phpcs: Fix WhiteSpace.LanguageConstructSpacing warnings

Squiz.WhiteSpace.LanguageConstructSpacing:
   Language constructs must be followed by a single space;
   expected "require_once expression" but found
   "require_once(expression)"

It is a keyword (e.g. like `new`, `return` and `print`). As
such the parentheses don't make sense.

Per our code conventions, we use a space after keywords like
these. We appeared to have an unwritten exception for `require`
that doesn't make sense. About 60% of require/include usage
was missing the space and/or had superfluous parentheses.

It is as silly as print("foo") or return("foo"), it works
because keywords have no significance for whitespace between
it and the expression that follows, and since experessions can
be wrapped in parentheses for clarity (e.g. when doing string
concatenation or mathematical operations) the parenthesis
before and after basiclaly just ignored.

Change-Id: I2df2f80b8123714bea7e0771bf94b51ad5bb4b87

Merge "Remove taint support"

Merge "Remove hphpc support and deprecate related functions"

Remove hphpc support and deprecate related functions

hphpc has been superseded by hhvm, so support for hphpc is no longer
needed.

* Continue to use Preprocessor_Hash under HipHop since it is still
  faster under hhvm
* Keep $wgCompiledFiles for now, so that wikihiero doesn't give an error
  before Ic9d1e795 is merged
* Migrate the run-server script and associated configuration file to
  hhvm. Enable EnableStaticContentFromDisk since it doesn't seem
  ridiculously inefficient at first glance. Run from $IP rather than
  $IP/.. since hhvm is apparently not picky about sourcing files from
  outside of the current directory.

Change-Id: Ic3e769f1fbad4f7ad26dd819406796fee48c6b45

Merge "Add PLURAL support to benefits"

Merge "Localisation updates from translatewiki.net."

Merge "Exclude external_user table in DatabaseSqliteTest"

Merge "Remove Parser_LinkHooks and CoreLinkFunctions classes"

Merge "[FileBackend] Process cache negatives for SHA1 on file stat."

Localisation updates from translatewiki.net.

Change-Id: I1a7c7dc38c92525952f13bc0016f8d067f6036ce

Add content model to prop=info

Fields of page table are shown by default in prop=info

Change-Id: Id8b90c341dea19337558ed355975ab890f0663e2

Merge "(bug 46751) Make Buryat (Russia) (буряад) (bxr) fallback to Russian"

Merge "Fix bootstrap in unit tests"

Merge "Make Special:MovePage's "Reason" field an input, not a textarea"

tests: Don't set wgStyleSheetPath in addition to wgStylePath

Because 1) `$wgStyleSheetPath = &$wgStylePath;` in default
settings, so setting one sets the other. No need to set both
and 2) in wmf-branches this variable is unset, thus this
caused an E_NOTICE internally when Test::setMwGlobals is
trying to access it to preserve the current value,
and 3) wgStyleSheetPath is deprecated.

Follows-up I1362932db223.

Change-Id: Ibd3f28e460fef995f68dfe1292d25fb75950dcf5

parserTests: fix broken parsoid-only tests.

A few of the parsoid-only tests had broken/wrong output HTML, which was
being hidden by the normalization process.  In preparation for using
a stricter normalizer, fix up the bogus tests:

a) put a <tbody> inside all <table>s
b) it's <hr> not <hr/>
c) fix an orphaned </span>

Change-Id: I44961cd40f8f29eea005d9c6c7a458aadc4770d0

Merge "skin: Revert font-weight on successbox and move to preferences"

skin: Revert font-weight on successbox and move to preferences

Removed the font-weight: bold from the css rule of the error,
warning and successboxes and placed this css declaration in
mediawiki.special.preferences.css under a new class
"mw-sp-pref-successbox".

Follows-up 44e64fb.

Bug: 48197
Change-Id: I1a3d774c2daef158176479fba7828b383a195bb9

[FileBackend] Process cache negatives for SHA1 on file stat.

Change-Id: Idc9f82309d52818acf57adc9c4eb04370a7b422a

Merge "mediawiki.user: Use session ID instead of 1-year cross-session cookies"

Merge "mw.config: Add wgPageContentModel for content model of current page"

mw.config: Add wgPageContentModel for content model of current page

This field of the page table can be useful to determine the
corrosponding model for a page.

Change-Id: I0525ad764540b213b2697e1e9e64f7da73f6cf83

Merge "Localisation updates from translatewiki.net."

Merge "(bug 47950) Fix IndexPager detection of "is first" state"

Localisation updates from translatewiki.net.

Change-Id: I5b167dc4fe14b1de8dd84d912244a4f96d762a63

(bug 48212) Fix I720946e7: $this->opts['year'] shouldn't be modified.

Otherwise, the offset is set automatically based on the default 'year'
parameter, then ReverseChronologicalPager gets misled and assumes that
the user have already specified a starting point so there must be a
previous page, causing a 'prev' link returning nothing shown.

Bug: 48212
Bug: 45580
Change-Id: I5341ff8c0d0422b9b3184dce74f0f143822cfe93

Merge "Add test for comment-only line followed by a template transclusion"

mediawiki.user: Use session ID instead of 1-year cross-session cookies

No cookies should be assigned for longer than session expiry.

Bug: 44327
Change-Id: If2f096dadb639769d859e1596d84b3ad5775a01d

Update docs/skin.txt

Reflect the fact that five of the skins were removed. Some rephrasing
as well.

Change-Id: I1acfa872eb4601003762d29443265a49e2100cd6

Use __METHOD__ for default $fname params to database functions

Change-Id: I86cbdeabee9709cde15a6b7bd47a53229c2e3f92

Add note that wgActiveUserDays affects the NUMBEROFACTIVEUSERS magic word.

Change-Id: I312fb1d8d0ffa2a978366c96accedf2a7b42bec8

Fix I25cf5a94: It should be @since 1.21 now.

That patch was created before 1.20 release and has stayed there for a
long time...

Change-Id: Icd28f749d55dd820e142ba094c848b713d9548ab

qunit: Use jQuery.getScript instead of document.write

TestSwarm jobs were breaking because this runs after the
document is ready, thus causing the page to be blanked by
document.write.

Change-Id: I5e1c331d72ac364c9cd970a9b085f0b986b26a14

Removing exception in case rev is null

Change-Id: I2fe2961876dcdc9bde3bca9584fcbe2294629eba

Adding check to make sure rev isn't null

Change-Id: I4be3904ed413086d559c21f95fd35e9b1540ce7f

Add test for comment-only line followed by a template transclusion

https://bugzilla.wikimedia.org/show_bug.cgi?id=48179

Change-Id: I4e41dcea94bbb1376ffa96d7381029b2cee85ec3

Merge "mw.hook: Implement callback framework"

Merge "mediawiki.action.edit: Support dynamically inserted form fields"

mediawiki.action.edit: Support dynamically inserted form fields

This fixes CharInsert in LQT. LQT dynamically creates its own text boxes
and this code should take that into account.

Bug: 41220
Change-Id: Id9469f9dfcbb92854780c63252cd9c5069e94487

Merge "Localisation updates from translatewiki.net."

Merge "Reduced memory usage a bit for copyFileBackend.php with --missingonly."

Localisation updates from translatewiki.net.

Change-Id: Ia291dfa770302c485bcf584854ee020f8a01ce8d

Merge "Move DEADLOCK_* constants into DatabaseBase"

Update SearchEngine::searchText() and SearchEngine::searchTitle() return type hints

Matches updates to MWSearch that can now return Status objects

Change-Id: I056fdf9f290d0a17e49948be158dd881185c8345

Merge "Adding GetNewMessagesAlert hook and wgUserNewMsgRevisionId JS global"

Merge "Partly disable title conversion if variant == main language code"

Merge "Remove SkinLegacy.php"

Merge "Added a --backoff option for syncFileBackend.php"

Move DEADLOCK_* constants into DatabaseBase

Change-Id: I75ea4c19c757adc96ba3c5afe0b85d8579b02c80

Remove pointless vertical alignment

Change-Id: I57926bfb718fdb7e4f55a08f39fc6af9f2e4adf6

Remove SkinLegacy.php

Followup to Ia6d73c2d (the great skin removal event of 2013).

The two classes this file defined, SkinLegacy and LegacyTemplate, were
already removed from AutoLoader.php in the aforementioned commit, but
the file itself somehow stayed.

The Nostalgia extension includes its own copy, and I've rewritten
Cologne Blue to use SkinTemplate last year, which means that nothing
needs the legacy classes anymore. I grepped mediawiki/extensions just
in case and there were no hits apart from Nostalgia.

Change-Id: Ib36edff3f9b617357386914980b56fa5b1f6a83b

Remove taint support

Venema's taint feature was apparently never publicised beyond the
php-dev mailing list, and now there is a PECL taint extension which is
incompatible with the interface assumed here.

Grep confirms that wfArrayMap() is not used in any extensions.

Change-Id: I17d8d302947443262474c608829c7c7504bef589

Refactored Special:ActiveUsers queries to remove filesort/temp table.

Change-Id: I4f32bad5084aadd289ccd2eefd93fc3d0d585b11

Revert "Fixed behavior of Title::invalidateCache."

Causes jenkins build failure

This reverts commit 3b4c514efa08b0a68885b1a920e48f670cd06e15

Change-Id: Ieeb54f1e7a9cfb5ac02b8a9f49bbe0644a064fa0

Fixed behavior of Title::invalidateCache.

Removed actions that should not be in Title::invalidateCache,
specifically the clearing of the InfoAction cache, and added
those actions to their appropriate locations.

Added a new method InfoAction::invalidateCache that clears the
cache for a given title.

Bug: 46816
Change-Id: I38ac616d2e99fb339aca33425745368ba85b6350

Revert "Remove Special:ActiveUsers"

This reverts commit 27104686abe2aaf464d8d654a71e946326a0926d.

Revert "Remove link to Special:ActiveUsers from Special:Statistics"

This reverts commit 62949af09ea3ceb96d897ec8aab74afdadb02d4e.

I used a sledgehammer where a scalpel would've done--and I should
be whacked with the cluebat. This should be done per-wiki, plus
improvements.

Change-Id: Ib43b42057b8d9f714a8a71ba9cbe375385254b7c

Localisation updates from translatewiki.net.

Change-Id: I44a29f7dab2c2131e90e6cd8e910d25569bb0744

Merge "Fix visibility of error handlers"

Adding GetNewMessagesAlert hook and wgUserNewMsgRevisionId JS global

This hook allows extensions to disable or modify the new messages
alert ('orange bar of doom') while still allowing the user_newtalk
table to be updated.

The wgUserNewMsgRevisionId JS global allows gadgets and extensions
to create their own new message alerts on the client side.

I also threw in a few comment updates for good measure!

See also Echo change I3f35a56b which utilizes this.

Bug: 47962
Change-Id: I2105bdd2bcd5b27f6f36ec8d8fa7fa99d60a2d82

Localisation updates from translatewiki.net.

Change-Id: I9b379ba2d392a74200050e45efcc62010def9a32

Merge "swap enhanced watchlist/RC arrow images for prettier ones"

Fix visibility of error handlers

I've hit this error when $wgDBuser didn't have access to $wgDBname.
I've also checked all other error handlers in core.

An alternative approach would be to use an anonymous function but it doesn't support $this until PHP 5.4 so it would be messier.

Change-Id: I89db2e8c88b84d9b53c4b828ac0208b05a3d3783

Merge "Kill some references to legacy skins"

Merge "deleteEqualMessages: Include list of pages in dry run"

Code style fixes in Language::isValidCode()

Fix issues from https://gerrit.wikimedia.org/r/#/c/62123

Change-Id: I97e826bb9aa9e3a14d07aab7cf02a09ef2c84ef7

Merge "Implement mw.log.warn and mw.log.deprecate"

Merge "mediawiki.log: Add to jsduck index"

Include user name in weird User insertion exception.

Change-Id: I70fea9819e968ae306de5c3c3eee4292d3fa17d3

Kill some references to legacy skins

More cleanup after the great skin removal of 2013.

Change-Id: Ia91e9467afac4a0e755869b8791d4fc9a6fb5a56

mw.hook: Implement callback framework

We have various interface components that are extended, enhanced
or manipulated in some other way by extensions, gadgets and even
in core itself.

This framework helps streamlining the timing of when these other
code paths fire their plugins (instead of using document-ready,
which can and should be limited to firing only once).

Features like navigating to other wiki pages, previewing an edit
and editing itself – without a refresh – can then retrigger these
hooks accordingly to ensure everything still works as expected.

For the server side this has been possible already, but for the
client side we've been limited to running only once on the
original page load.

Further documentation and examples are in the code documentation.

Bug: 23580
Bug: 30713
Change-Id: Ic73a3efe53d6fb731e7f1e531d5f51530cd7e4fe

Fix bootstrap in unit tests

- Remove check for version, that version is already enforced in phpunit.php,
  so there is not point showing a warning for it is useless
- Remove call to MessageCache::destroyInstance(), there is no need for it,
  since $wgMessageCacheType is set in phpunit.php before running Setup.php
- Remove includes of bootstrap.php in LanguageSrTest.php and LanguageUzTest.php

Change-Id: I4b2db6b3e6f001175e1a407c5add2972aade5e60

Merge "Factorise call to getSuggestedDurations() in SpecialBlock::getFormFields()"

Merge "Pass correct number of parameters to Database constructor from installer"

Pass correct number of parameters to Database constructor from installer

DatabaseBase::__construct() accepts 6 parameters, installer passes 7.

Change-Id: I96d84f645f8f3899d7a99756b441957f1b234de9

Make Special:MovePage's "Reason" field an input, not a textarea

This reverts r45571 (207e3e64), with some whitespace changes.

The original reason for making it a <textarea> was to avoid confusion
with the <input> for new page name. This is invalid now that the page
name field has been split into a namespace dropdown and an actual page
title field.

Bug: 13627
Change-Id: I5ab2092c32682e7a8a1494bf58553971568fa6cf

deleteEqualMessages: Include list of pages in dry run

Also clarify that the talk page number is in addition to the
other number. So it says "10 pages are equal (+ 2 talk pages)"
instead of "10 pages are equal (2 talk pages)".

Document better that the default is a dry run.

Change-Id: I95edb08d7b92c393ed226bef2575e52a6135f8df

Merge "Monobook: Remove OperaNFixes.css files"

Merge "jquery.makeCollapsible: Enable passthru for links in premade toggle"

jquery.makeCollapsible: Enable passthru for links in premade toggle

Clicking on a regular link inside a premande toggle is supposed
to follow the link instead of collapsing/expanding the
appropriate element.

Broken during a rewrite in Id3f457a8 (2638b16c) - the comment
reflected the intended behavior, but the code was wrong.

Regression test included.

Bug: 46848
Change-Id: Id571f8d612e5131845d381121604c2b1116a077f

Merge "SpecialPreferences: Remove invalid <strong> tag in successbox"