tests: fix PHPUnit @requires for APCU

@requires is an annotation used by PHPUnit to flag a test as requiring a
PHP function or extension. c214c9a95 changed it to list two functions
separated by a slash:

   @requires function apc_store/apcu_store

However that is not recognized by PHPUnit and the test is always
skipped.

Change the @requires to require the APCU extension.

Repro case:

  phpunit.php --verbose includes/libs/MemoizedCallableTest.php

Follow-up c214c9a95255abf1032d389d0fe8bcbde38b55db

Bug: T140587
Change-Id: I1fdd52c8fc8ca1c7c0c7f914e6da8774799ba5d8

Merge "LanguageConverter tweaks to Pig Latin converter"

Localisation updates from https://translatewiki.net.

Change-Id: I23379ec2eb8320618f5be56618ed5c45587c8825

Add DeleteUnknownPreferences hook

This makes it possible for extensions to add to the WHERE clause
with which unknown preferences are deleted.

Bug: T188966
Change-Id: Ifb22ca42207956f2b07333da026ff0b215a83930

Merge "Add further test cases to deleteAutoPatrolLogsTest"

Add further test cases to deleteAutoPatrolLogsTest

Also make --batch-size work and make an if check nicer
(makes no functional difference AFAICT).

Bug: T189594
Change-Id: I3ce63386cb35441acfa226f313ec8aac1aa417a6

Merge "Introduce deleteAutoPatrolLogs maintenance script"

Introduce deleteAutoPatrolLogs maintenance script

It's useful to delete old patrol logs that are not useful

Bug: T189594
Change-Id: I605bb85f172eb25df45ed83ce50a3d1044f1c281

Merge "DB: Add img_description_id column"

Merge "build: Updating mediawiki/mediawiki-codesniffer to 17.0.0"

rdbms: rename and clarify getTransactionLagStatus method regarding begin()

Make sure any value from the last transaction is cleared and not used.

For sanity, make getLagFromPtHeartbeat() use this regardless of the age of
the transaction unless it returns null.

Change-Id: I52df6147f99736ad1a389ae70d347ae968e50c7f

Revert "rdbms: make getMasterPos() ignore GTIDs outside of gtid_domain_id"

This had a noticeable increase in LoadBalancer::doWait timeouts.

This reverts commit ceb7d61ee7ef3edc6705abd41ec86b3afcd9c491.

Change-Id: I7004d55a05c20f646f70d778d7b6496123e270a4

rdbms: avoid incorrect warnings in getLagFromPtHeartbeat()

Change-Id: I4c0d42be0ee0c518d7bacdae0f76feb53007861e

Merge "Make LanguageConverter roman-numeral cases consistent"

Populate ar_rev_id and make it non-nullable

Revisions deleted before MediaWiki 1.5 do not have a value in this
field. This is going to be a problem for migration to the MCR schema, so
provide a maintenance script to clean this up.

Then, for good measure, change the schema to make the field
non-nullable.

Bug: T182678
Change-Id: Ie2e11f12a30f379db32c3e074658012c6f93adb0

Merge "Selenium: update WebdriverIO configuration"

Localisation updates from https://translatewiki.net.

Change-Id: I35703b36f31021e7ba272c3b1869808cb95b580f

Special:Preference: Check for changes before enabling Save button

* Changing mousedown and keydown to mouseup and keyup
  …so that isPrefsChanged gets the updated value and can return
  something useful

* Changing one to on
  …so that if a value is updated and then changed back to the
  original, the button can be disabled again

Bug: T180538
Change-Id: I7146f8d3166e09e13b26321e7a6a7c8ac74f9aaf

LanguageConverter tweaks to Pig Latin converter

Add apostrophe to the set of valid word characters for the en-x-piglatin
converter, so that "don't" and "can't" are properly converted to Pig
Latin (eg, to "on'tday" and "an'tcay").

Add an optional `s` before `qu` so that "squish" is properly converted
to "ishsquay".

Change-Id: Ibc5cf2c007a42d9447688b857aa75f9a3d8ae152

Make LanguageConverter roman-numeral cases consistent

Add a look-ahead to ensure that the regex intended to match roman numerals
doesn't also match the empty string.  Tweak the regular expressions slightly
to ensure that Sr/Ku/Crh all have identical regular expressions.

Change-Id: If43bf99a21c42c6c5050f814c0bc99edec353228

Use fallback languages in ResourceLoaderImage

Bug: T191120
Change-Id: Ic72aead466c12d344bac359654e6dcc7f0e5c8cb

Add Hebrew alias for special page ApiSandbox

Change-Id: I09bf2951b5bc18bf9f7080d178ea74466d14e97e

Localisation updates from https://translatewiki.net.

Change-Id: I20bbb93b56df4246c31344f3929742af581c3118

Merge "MessagesBs.php: Update namespaces names"

Merge "rdbms: avoid lag estimates in getLagFromPtHeartbeat ruined by snapshots"

Localisation updates from https://translatewiki.net.

Change-Id: Id315b02132af92cc2d4b61a28da34be32e333e54

rdbms: avoid lag estimates in getLagFromPtHeartbeat ruined by snapshots

Bug: T190960
Change-Id: I57dd8d3d0ca96d6fb2f9e83f062f29b1d53224dd

Merge "rdbms: update IDatabase::getLag comments"

Merge "rdbms: add more error logging to DatabaseMysqlBase::masterPosWait"

MessagesBs.php: Update namespaces names

Bug: T172050
Change-Id: I3eb31ef0785138b14342c27a4232bec2b782c0bf

Selenium: update WebdriverIO configuration

No changes are made to the configuration file. It has just been
regenerated with the latest WebdriverIO version. Minor whitespace
cleanup has been made to the file after it has been generated so it
conforms to our ESLint configuration.

This is the first step in updating configuration so videos of failed
tests are recorded.

Bug: T179188
Change-Id: Id0db3743ae7d828357b1527770bb7dcd2e02f99b

Merge "mediawiki.action.view.rightClickEdit: Remove redundanat dom-ready handler"

rdbms: add more error logging to DatabaseMysqlBase::masterPosWait

Change-Id: Ia687ddeffc9c9427c6126d8ed831fb52b38e8260

Merge "Improve docs for revision, archive, and slots schemas in tables.sql"

Localisation updates from https://translatewiki.net.

Change-Id: I0c57d6f3f632ddd06f36f833c53f97d96e7dd645

rdbms: update IDatabase::getLag comments

Also add missing __METHOD__ to getLagFromPtHeartbeat()

Change-Id: I4257b1d47a88779b47d807a881561c331ff3aa30

Merge "rdbms: remove some dead code from Database::begin"

rdbms: remove some dead code from Database::begin

The "since" filed is computed after getLag(), so this was useless

Change-Id: I252602e85709b1cfc191d4783bd1c53d5b3e8fe8

Merge "Improve docs for archive schema in tables.sql"

Improve docs for revision, archive, and slots schemas in tables.sql

Follows up Id2257827.

Change-Id: Ifd61eaa51a6f7781522cecdbf8c563465e97e885

mediawiki.action.view.rightClickEdit: Remove redundanat dom-ready handler

This file was wrapped in a jQuery() dom-ready callback, but it doesn't
and shouldn't have that because it needlessly delays execution.

I probably added because back then, using the first parameter of jQuery()
was a common way for aliasing it to '$', but we use closures for that
nowadays.

Change-Id: I65768564ed556828fd193dfe99d11370fa745112

rdbms: clean up session/transaction loss logic in Database

* Refactor the code to be less cluttered, moving some logic
  to reconnect(), which is now named replaceLostConnection()
* Handle the case of a second connection loss on query retry
* Make canRecoverFromDisconnect() check for temporary tables
* Do not clear session-level variables on deadlock errors
  since only the transaction is lost, not the whole session
* Make sure any empty transaction is destroyed on deadlock
* Attempt reconnection *before* triggering the transaction
  callbacks since some of them  might want to use the database
* Define wasConnectionError() for postgres
* Remove unused return value from handleSessionLoss()

Change-Id: Ic1dcab03f087ce310637210e8e9bc0771e44f045

Merge "Allow schema overrides to drop tables."

build: Updating mediawiki/mediawiki-codesniffer to 17.0.0

Change-Id: Ib494b47c54fe6354d166055b1e1b31d3583bb992

Allow schema overrides to drop tables.

This allows unit tests to use schema overrides that drop tables,
in addition to overrides that create or modify tables.

Change-Id: I59761c7db7f83698749324ca6b9ffced86ab1249

Merge "Bump avro to 1.8"

Merge "Fix MockMessageLocalizer"

RCFilters UI: Remove CSS customization for help icon

This fixes most of the misalignment after the OOUI v0.26 upgrade.

Bug: T191014
Change-Id: I4623836d134fa52fb6b562e671a5da104b787b7d

Merge "Add protection for User: JSON pages in the same manner as JS & CSS ones"

Add protection for User: JSON pages in the same manner as JS & CSS ones

Also recognise MediaWiki: JSON pages (with the existing protection of
the editinterface right).

Bug: T76554
Change-Id: Idba166d82ee6dd507d7345c9bdbefc8ca78ed7b4

Fix MockMessageLocalizer

Add it to the autoloader for the tests, and remove the unneeded use
statements which cause test failures.

Change-Id: I7092525eae440c6969fc84e8979572957733cf03

Merge "Make SpecialWhatLinkshere::showIndirectLinks use buildSelectSubquery()"

Improve docs for archive schema in tables.sql

Also fix typo in migrateArchiveText.php from 295b055ea33d.

Change-Id: Id225782783007c4674c5b25860172c07273d1045

Bump avro to 1.8

Adss PHP7 support.

Bug: T177273
Change-Id: I38cf66d6602170cec3a307bfe44656db103c321f

RCFilters: Rename deprecated 'unClip' icon to 'bookmark'

'unClip' got deprecated in OOUI v0.26.1 and 'bookmark' is the new,
more appropriate name.

Bug: T183601
Change-Id: I797071327987a8032b9d7d1d9bcfa27f4fd9db2f

Merge "Normalize and lower the default DB lag wait timeout"

Make SpecialWhatLinkshere::showIndirectLinks use buildSelectSubquery()

This avoids the SPI log warning from d395dfb039

Change-Id: I193418a224a8c6c0cc919e29c66686947d5969bf

Normalize and lower the default DB lag wait timeout

Bug: T190960
Change-Id: I49aca118583b20314e6bf82f196f3413571f5bd9

RawAction: Remove unused $gen variable

Follows-up 697c423259 (MW 1.27), which removed use of these variables.
The '$this->gen' member is never used, and neither is local $gen.

Change-Id: I663d343d0255d5e13d7d87b1d4e565a7df667449

Merge "BaseTemplate: Remove code for unused 'ispermalink' toolbox item"

Merge "Add MockMessageLocalizer"

Merge "resourceloader: Replace ClientHtml 'target' param with 'options' array"

Merge "resourceloader: Add test for ClientHtml passing down 'target'"

Merge "Get ~100% test coverage for ApiEditPage.php and fix a couple of bugs"

Merge "Selenium: install bluebird, mwbot dependency"

Merge "Let built-in web server handle .php requests"

Let built-in web server handle .php requests

When using the PHP built-in webserver with maintenance/dev/start.sh,
requests made to regular files do show in the console output but
requests to .php files do not log anything in the console. It is a bit
annoying since one is left wondering whether the requests are actually
honored.

The reason is the request router maintenance/dev/includes/router.php
does a require of the PHP file and abort the router (return true). That
causes the built-in php server to not produce the access log requests.

Based on the comment note, some earlier PHP version had POST data
stripped preventing login. The related upstream bug is
https://bugs.php.net/bug.php?id=60159:

  "Router returns false, but POST is not passed to requested resource"

750eac7e41d introduced the workaround on Nov 24th 2011. PHP 5.4.0 got
released in March 2012 and refers to that bug.  The development of our
router thus must have used a development/beta version of PHP 5.4.

Since the fix made it to the PHP final release, there is no need for
workaround anymore.

Bug: T190503
Change-Id: Ia8c4f1610ea0ef5b04ea30d6e91199b8e597065b

Get ~100% test coverage for ApiEditPage.php and fix a couple of bugs

Bugs fixed:

* The major one was that appendtext and prependtext seemingly didn't
  work at all on empty pages in the MediaWiki namespace, because
  $this->getTitle() was being used in place of $titleObj.
* Negative undo/undoafter were not rejected in a correct fashion.
* If a logged-in user who was not allowed to upload images tried to
  create an image redirect, the error message used a nonexistent key.

I also replaced assertEquals with assertSame.

I also removed a bit of clearly dead code.  This includes a "break"
following dieStatus().  If we actually want this break so that nobody
adds a case after the default and then removes the dieStatus() so the
switch incorrectly falls through, it could be re-added with
@codeCoverageIgnore.

I put the fixes in the same commit as the test changes because I like to
keep fixes together with the tests for those fixes.

All code now shows up as covered locally, except for one line that seems
to be a PHPUnit bug.

Change-Id: I9375bc5f40268fd681a2d447c66a03f40b23390a

Merge "RCFilters: Bidi isolate tag names and descriptions"

Selenium: install bluebird, mwbot dependency

Bluebird is implicitly installed on my machine, probably because I have a recent npm version. It needs to be explicitly installed in MediaWiki-Vagrant, probably because of an older npm version.

Bug: T190914
Change-Id: I9c704502fb5e20abf9b21d483723eb55b6fb3875

Add MockMessageLocalizer

This MessageLocalizer may be used in tests. By default, it sets the
language for all messages to 'qqx', to make the tests independent of the
$wgLanguageCode of the wiki the tests are running in.

Change-Id: I7412ec49b6c0fab8146e09bb8aa599b35d88fc97

SizeFilterWidget: Reduce width

Change-Id: Ib4fd667ce6bfcd0dd86a19619a7610c6c92b0619

Fix --memory-limit option to maintenance scripts

Follow up 2f293bd0. Previously this option was marked as not
taking an argument, thus if present it would have the value of
true (aka 1). Setting memory limit to 1 results in setting to
256kb, which is way too small.

Change-Id: I2a6e512e384c5ef882f7ba2877091ecb8fe01e35

resourceloader: Replace ClientHtml 'target' param with 'options' array

In preparation for passing down 'safemode' from OutputPage.

Only used in one place in Wikimedia Git: OutputPage::getRlClient().

Bug: T185303
Change-Id: If01eca96986ff8d7dcdaab6910bf183ba7c7311f

Merge "This patch reduces the edit summary length to 500 characters"

resourceloader: Add test for ClientHtml passing down 'target'

This was not previously tested.

Change-Id: I25d9ec28b712b0b699f1369b284faf337b8af5aa

Revert "Localisation updates from https://translatewiki.net."

Overwrites new messages

This reverts commit a7c30ad5d91dee12958c2a29ff6ffdc06a65d6e3.

Change-Id: I40b6aae837f2d4c06e860d47a8255d6f7ab2cf59

Localisation updates from https://translatewiki.net.

Change-Id: Idf22ae14a5fb2d2a0e5615720e047c63f32cdfff

BaseTemplate: Remove code for unused 'ispermalink' toolbox item

This code path is never used.

When introduced in 2005 (bf0d92d5 / rSVN10709), the "Permanent link"
item in the MonoBook sidebar was a link to an oldid only when viewing
the latest version of an article (using the ID 't-permalink'), and
otherwise it was a greyed out link to nowhere to indicate you're
already viewing a permanent link (using the ID 't-ispermalink').

Later, in 2007, when the behavior of "Cite this page" item (now part
of the CiteThisPage extension) was corrected for old revisions
(1a60bd4c / rSVN21819), the "greying out" of the permalink was removed
to be consistent with the new item. Only half of the feature was
removed though: permalink was no longer a link to nowhere, but the
code in MonoBook to make permalinks that are links to nowhere greyed
out (by using different ID) stayed.

This code was then dutifully preserved when BaseTemplate, MonoBook was
rewritten, and then through several refactorings.

I think we can safely remove it. Leftover styles in MonoBook are being
removed in I58aca1f4255df14d6289e167e180dad31b546a8c.

Change-Id: I2fb3d7e0ecb3bbbd0c09774e2d86f86502a48bf1

Merge "Revert "Pass revision being reverted to edit code""

Merge "mediawiki.user: Implement mw.user.stickyRandomId"

Merge "Make WikiPage::doViewUpdates use a PRESEND deferred update"

Merge "Overhaul site_stats table"

Overhaul site_stats table

The site stats table holds a bunch of metric fields, two of which are of
data type "bigint unsigned", 3 are "bigint" (signed) and one is int
(signed).  Also the default values differ widely: It is 0 on the
"unsigned" fields and the "int" field, but -1 on the three others. This
patch makes all of this more consistent:

Set all fields (except the ss_row_id, which isn't changed) data type to
"bigint unsigned". Also set NULL as the default value for all those
fields. Obviously -1 isn't a possible default value any more. Also, 0
can easily be mistaken for a real value (e.g. ss_active_users=0 -->
"there is nobody active on this wiki"). NULL, by it's definition, is the
value of choice for a value to insert into fields of which we don't know
a correct value.

The respective patch files were tested locally against MySql, Sqlite,
Postgres and SQL Server 2016. Neither oracle nor the upgrade with
update.php was tested.

Bug: T56888
Change-Id: I7d42aae434852a56b6f8dd559d8a5f3bce416021

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit ad7c4322d4dd7903065f066d8d96ead875b5126b

Change-Id: Ica20c20ce8f40786f9b2b8ec4c3021f49843354f

Merge "Fix whitespace trimming in headings"

Merge "tests: Remove some unused stuff in phpunit tests"

tests: Remove some unused stuff in phpunit tests

Change-Id: Ica20e4f4ac0887b7cac12700ffcd15f587b3ba01

Merge "Cleanup ProfilerOutputDb try/catch logic for DBErrors"

Merge "mediawiki.api.upload: Fix test assumption about <iframe>"

Revert "Pass revision being reverted to edit code"

Too many regressions, doing it the different way.

This reverts commit b7737ebed1e8940df4dfd4c50c1224d0edc8639d.

Bug: T190374
Change-Id: I12cb7470aeb7852ddea319c7e499ca3a7b45c84c

Merge "rdbms: ignore DBO_IGNORE if passed into Database::__construct()"

Merge "rdbms: add "use" statement for UnexpectedValueExcetion in Database"

Merge "Create save/publish buttons labels with ellipses"

Merge "Bump required Postgres version to 9.2"

Bump required Postgres version to 9.2

This was approved by TechCom in the "PostgreSQL schema change for
consistency with MySQL" RfC (T164898).

Bug: T164898
Bug: T190539
Change-Id: I1b17c18c513e4edcd9219e36e873c55dd7f33034

RCFilters: Bidi isolate tag names and descriptions

Tags can be untranslated in wikis; most often remain in English
even in an RTL wiki. Since that text is inserted by external
input that can be outside normal translation, we should expect
any directional string in there, and these should be bidi isolated.

Bug: T190718
Change-Id: Iebc3176c5d0f5097badd0dde565a5a6cbfa3a3c2

Merge "RCFilters: Hide 'reload watchlist automatically' setting if RCFilters is enabled"

Make WikiPage::doViewUpdates use a PRESEND deferred update

This is better than a dubious try/catch which had the off chance
of letting partial implicit transactions be committed.

Change-Id: Ied4870df166dc5a8224866284ea6e1958e65f499

Merge "Added heartbeat for pingback."