Improve test coverage for ApiBase.php

One bug fixed: a timestamp of '00' or similar would get interpreted as
'now' by mistake instead of Unix timestamp 0, without throwing the
warning for using 0 instead of 'now'.  This is because it called
wfTimestamp() once to parse the input date, got a Unix timestamp of 0
back, and then tried passing that 0 back to wfTimestamp again to
reformat as a wiki date, but it got reinterpreted as 'now'.

Also fixed parameters with type "user" to validate usernames more
correctly.  This might be risky, though, if I missed any valid
usernames, or if API clients were for some reason relying on passing in
invalid usernames.  If we don't actually want to do this, we should add
a comment explaining why we're allowing any title without a fragment
rather than validating properly.

Still lots more work to do here.

Change-Id: I56b4290263df8698efdbddda71a7eabd9e303abc

Fix (MediaWiki)Site::normalizePageName return type

I checked all callers of these methods and almost all of them expect the
method to return false. It looks like this return type was known at some
point, but got lost. Let's add it back.

Bug: T191634
Change-Id: I43484835b8f26e07ada6a2b1452a99ccc6d9b438

tests: Enable PHPUnit 4/6 compat layer in some tests that need it

Change-Id: I27a21fa9e97414fae02acbefb28011f0275cba63

phpunit: Alias PHPUnit_Framework_Error for PHPUnit 6 compat

In PHPUnit 6, it's now PHPUnit\Framework\Error\Error.

Change-Id: Iaf0ef0a10a6a2cb7d7fc280d5ca873e77dbacb1d

RevisionTest: Fix getMockTitle()

There's no such function "getModel", it's probably supposed to be
"getContentModel". PHPUnit 6 emits warnings for trying to mock functions
that don't exist.

Change-Id: I0e5434e48958a8b75238b2ebffb5c1dc4e60c864

phpunit: Create class aliases for interfaces in PHPUnit 4/6 compat layer

Change-Id: If36bb3403f226b0399e37d2fd63b4338b1de10ae

Localisation updates from https://translatewiki.net.

Change-Id: I09ba70916875abf16faef899deb543689fbcfb42

Merge "Selenium: use default value for waitforTimeout"

Merge "Change registred users to autoconfirmed users in "MediaWiki:Semiprotectedpagewarning" and "MediaWiki:Semiprotectedpagemovewarning" messages"

Merge "installer: Fix setting a custom $wgMetaNamespace"

Remove deprecated function ApiBase::getProfileTime

Bug: T189330
Change-Id: I404766b53ad33d615557faa693a513914d959a63

Remove deprecated function ApiBase::getModuleProfileName

Bug: T189330
Change-Id: Ifa30d115b97e8109d11cb932a924d63afc6f81d9

Remove deprecated function ApiBase::getProfileDBTime

Bug: T189330
Change-Id: I7584024dff54608b138a7fbaced216ff9133ac2c

Change registred users to autoconfirmed users in "MediaWiki:Semiprotectedpagewarning" and "MediaWiki:Semiprotectedpagemovewarning" messages

Bug: T190882
Change-Id: Id56f5d8670b0e1363603a05e612be91a986f9698

Merge "Change ResultWrapper to IResultWrapper in pagers and special pages"

Merge "Use IResultWrapper in UserArray and TitleArray"

Merge "Fix return type for html strings"

Merge "Fix param doc of Skin::makeSpecialUrlSubpage"

Merge "Remove "probably you" from password reset email information"

Merge "Provide PHPUnit 4 and 6 compatibility layer"

installer: Fix setting a custom $wgMetaNamespace

`$textbox.removeProp('readonly')` has no effect and leaves the input
still as readonly, so instead use `$textbox.prop( 'readonly', false )`,
which does work.

Bug: T188415
Change-Id: I898c5605caf383cc992a948f14294193460f761b

Merge "Don't rely on magic __call in MWNamespaceTest"

Merge "Fix --memory-limit option to maintenance scripts"

Provide PHPUnit 4 and 6 compatibility layer

PHPUnit 6 removed some functions that were heavily used in version 4. To
be able to support both versions for a short time, we'll use a trait to
fill in the missing methods until we drop PHPUnit 4 support.

This trait is included in MediaWikiTestCase so most tests will be able
to benefit from it by default. Otherwise, anything that calls
setExpectedException() or getMock() will need to use it.

Change-Id: I707129e471e960e034e2aa994a467b9dc0239b69

Don't rely on magic __call in MWNamespaceTest

Change-Id: I32a7e7a55bc733f19d7c5ed1fbc6cfde748d4812

Localisation updates from https://translatewiki.net.

Change-Id: If0f967280101e28c85fbd85f290cc0ad219b94f7

Fix param doc of Skin::makeSpecialUrlSubpage

The $urlaction is passed to Title::getLocalURL which allows also
string[]

Change-Id: I9594f37b704403ffa96f490b079c4008153e9fb8

Merge "maintenance: Fix autopatrols typo in deleteAutoPatrolLogs"

maintenance: Fix autopatrols typo in deleteAutoPatrolLogs

Follows-up e1a23c9aad.

Change-Id: I43c00b2ac884a80f779be9eead50ad3de6d13fc6

ChangesListSpecialPageTest: Use Database::makeList() instead of makeshift DIY code

Otherwise these tests break completely when you add conditions like
$conds['rc_patrolled'] = [ 1, 2 ];
That maps to 'rc_patrolled IN (1,2)', but the DIY code in
normalizeCondition() was too simplistic to generate that.

Change-Id: I449317185f98e20b3e17f1b13610d872ae828171

Merge "rdbms: move assertOpen() call near the top of Database::query"

Merge "Add missing "use" clause to DumpTestCase"

mediawiki.jqueryMsg: Rename .parser class to .Parser

This class is marked @private, but nonetheless, there is at least
one use in MobileFrontend, so keeping a back-compat alias for now.

Also rename the HtmlEmitter utility class (also private, not used
anywhere outside this file), and fix its broken documentation by
repeating the @class and @private tags on its own constructor.
It previously had all its methods indexed by JSDuck as part of the
previous class in that file (Parser), which created doc pages
with examples that don't work.

Change-Id: I02d851d9b6eac89f1a2b85b438b982bd055bedee

Merge "Revert "Default installations to using RemexHtml for tidying""

Revert "Default installations to using RemexHtml for tidying"

This reverts commit efcef34d3da74613b01a320873bc59e134748183.

This is causing failures in CI for extensions

Depends-On: If9789a61d52f60882fc2f0226757c9d93e1c6362
Change-Id: I17cf305a951b2bf1f03285b12c3e131abcffd31d

Selenium: use default value for waitforTimeout

The default is 10000 (10 seconds). It was increased by mistake. 20000 is the value needed for mochaOpts timeout. There is no reason to have the double value for waitforTimeout.

Bug: T179190
Change-Id: I14f72367a36815d155cedc57757481f6294fdc51

Fix return type for html strings

Change-Id: Ifc1ae7740ad1b130186b4b970d3d84651b016177

Merge "Default installations to using RemexHtml for tidying"

Merge "tests: Reset Postgres sequences when cloning and truncating"

Merge "rdbms: Allow PostgreSQL schema-check functions to find temporary tables"

tests: Reset Postgres sequences when cloning and truncating

This improves the repeatability of the unit tests by making the ID
values generated depend less on what previous tests might have done.

It also prevents tests from using up sequence numbers for the live DB's
tables.

Change-Id: Iaa8ae1e5cef4b9099bd1b4b8fc806f5af372a7ff

Merge "Add DeleteUnknownPreferences hook"

rdbms: Allow PostgreSQL schema-check functions to find temporary tables

PostgreSQL puts temporary tables and such in a hidden, per-connection
"schema" that's checked for unqualified table accesses before the normal
search_path. We should check that in all the schema-checking functions.

Change-Id: I1194ac31f31133b177f624138afee19d00e454b9

rdbms: Add a deprecationLogger callback

Much like the existing errorLogger, but for logging deprecation
warnings.

The default in the RDBMS layer is to call trigger_error() with
E_USER_DEPRECATED. The default in MediaWiki (via MWLBFactory) is to log
to the 'deprecated' log group, much like wfDeprecated() does, although
unfortunately we can't effectively use that directly since we have no
idea of a proper $callerOffset to pass.

Change-Id: Id13625e249516e84d72b6310953bb338a90976da

Use IResultWrapper in UserArray and TitleArray

The result of IDatabase::select is directly passed around and that type
is IResultWrapper

Change-Id: I7668fcac960bb83e834d849aaef5fe90e5bc115b

Merge "rdbms: Remove support for PostgreSQL < 9.2, and improve INSERT IGNORE for 9.5"

Merge "rdbms: Roll back empty implicit transaction on error"

rdbms: Remove support for PostgreSQL < 9.2, and improve INSERT IGNORE for 9.5

MediaWiki doesn't support PostgreSQL < 9.2, so drop the support for
older versions.

At the same time, since we're messing with the DatabasePostgres::insert()
code anyway, let's start using ON CONFLICT DO NOTHING for PG >= 9.5.

And since we're doing that, let's do the same for
DatabasePostgres::nativeInsertSelect().

Change-Id: I7bf13c3272917ebafeaff11eb116714a099afdf3

rdbms: move assertOpen() call near the top of Database::query

This should be done before bothering with begin() calls or setting
the value of fields like "trxDoneWrites" or "lastWriteTime".

Change-Id: Ieace2870f5496fcf230c5c8381baca4b8038db1d

Localisation updates from https://translatewiki.net.

Change-Id: Id32906c9e0becbdec14faf77335f79ab4cd06b84

rdbms: Roll back empty implicit transaction on error

If we're not going to set trxStatus to an error state in this case, we
need to issue a rollback to be sure the database (i.e. PostgreSQL) isn't
still in an error state too.

Bug: T189999
Change-Id: Id6e203b216fff937b6a97d779b36c278e3366409

Merge "Fix SamplingStatsdClient for PHP 7.1+"

Default installations to using RemexHtml for tidying

This combines two changes – defaulting tidying to on, previously off, and
defaulting the tidying library to RemexHtml, previously the tidy binary.
Config options are going to be a bit of a mess until we drop support for
the old tidy binary config route.

Bug: T185753
Depends-On: I0a8973f508fbf65160177b003260831639828eea
Change-Id: I6879a77a78d780c7c056d807dde20682c6097d1a

Merge "mw.special.apisandbox: Use a real button to insert values in multi fields"

Merge "rdbms: make Database query error handling more strict"

mw.special.apisandbox: Use a real button to insert values in multi fields

Stuffing an indicator inside a normal widget is weird.

Also, always add it, even when we also handle Enter presses on the input.

Change-Id: I7191b4f31bfe4b42a524c786994150f318fd8cd9

Merge "Swap isset by null check in ApiSetNotificationTimestamp/ApiPurge"

Merge "Change default of $wgLogAutopatrol from true to false"

Merge "SizeFilterWidget: Reduce width"

Change default of $wgLogAutopatrol from true to false

"$wgLogAutopatrol = true" basically just spams the logging table
and isn't useful in most cases (or ever?)

Bug: T184485
Change-Id: I418cc2fcc7017e5d2cbc0c6159b4c459b9c77ac7

Swap isset by null check in ApiSetNotificationTimestamp/ApiPurge

Change-Id: I1ebcbacf476a716d23b183dcbf379555ee1519dd

Fix SamplingStatsdClient for PHP 7.1+

Bug: T182380
Change-Id: I74a8aef1230c56d0c6a7856f8397e657fc6d1c40

Api requests can throw a ApiUsageException, and that should be expected

Change-Id: I4aa6e7511747c0d811e2e617b88e681a6eda875d

rdbms: ignore inactive mysql GTIDs in replication position methods

If it is known that master writes will use GTIDs with a certain
domain and server ID, ignore other ones on masterPosWait().

This restores ceb7d61ee except it uses the same server variables
and MySQLMasterPos preserves the active server/domain information
across serialization.

Change-Id: I1a4f143adcbec642966d7d1a55edb0f414a7f0c4

Add missing "use" clause to DumpTestCase

Change-Id: I0420e95003509634a87b98df279242cdc05843b2

rdbms: make Database query error handling more strict

Handle all errors in query() that might have caused rollback by
putting the Database handle into an error state that can only be
resolved by cancelAtomic() or rollback(). Other queries will be
rejected until then.

This results in more immediate exceptions in some cases where
atomic section mismatch errors would have been thrown, such as a
an error bubbling up from a child atomic section. Most cases were
a try/catch block assumes that only the statement was rolled back
now result in an error and rollback.

Callers using try/catch to handle key conflicts should instead use
SELECT FOR UPDATE to find conflicts beforehand, or use IGNORE, or
the upsert()/replace() methods. The try/catch pattern is unsafe and
no longer allowed, except for some common errors known to just
rollback the statement. Even then, such statements can come from
child atomic sections, so committing would be unsafe. Luckily, in
such cases, there will be a mismatch detected on endAtomic() or a
dangling section detected in close(), resulting in rollback.

Remove caching from DatabaseMyslBase::getServerVariableSettings
in case some SET query changes the values.

Bug: T189999
Change-Id: I532bc5201681a915d0c8aa7a3b1c143b040b142e

build: Bump postcss-less up to 1.1.5 from 1.1.3

Problem over, we think.

Bug: T190269
Change-Id: I0a94c99fb75613b03f15954512744dec63fd779e

build: Bump devDependencies to latest

 grunt-stylelint             0.9.0  →  0.10.0
 stylelint                   8.2.0  →   9.2.0
 stylelint-config-wikimedia  0.4.2  →   0.4.3

Change-Id: I612cf96d99e2af67b3f7ef3284657f66f3e18d2e

Munge inline elements found in tidy.conf as well

Bug: T184900
Bug: T184228
Change-Id: I421c4c7cf1eeeb6c44bb64081b49ae05937d1a8b

Merge "User: Remove deprecated, unused auth functions"

User: Remove deprecated, unused auth functions

According to a github search they're not used anywhere anymore and
already deprecated since 1.27:

 ::getEditTokenTimestamp
 ::getPasswordFactory
 ::passwordChangeInputAttribs

Change-Id: I904e46e7343baee28dc960f8fd5437ae9d17fc4b

Merge "CSSMin: Don't match empty string as remappable url"

Merge "Convert HTTP links to MediaWiki.org to HTTPS"

Merge "rdbms: rename and clarify getTransactionLagStatus method regarding begin()"

CSSMin: Don't match empty string as remappable url

The empty string being matched causes an undefined array index
notice in production, seen from various random gadgets, but spiked
after a change in MonoBook from last week that introduced a
broken background-image rule with empty string as url.

In browsers, that is actually interpreted as valid and "expands"
to the current url and re-fetches as Accept:image/*, silly, but
still broken. The broken icon was fixed in MonoBook, but we still
need to avoid trying to remap empty string as url.

Two changes:

1. Fix regex used by remap() to not match empty string.
   This was already fixed for the 'url()' case without the
   optional quotes, but with quotes, it was being matched as
   non-empty. This is now fixed by using '+' instead of '*'.
   Added tests to confirm they produce output, and PHPUnit
   is configured to also assert no Notices are emitted (which
   it converts to fatal exceptions).

2. Fix processUrlMatch() as sanity check to throw if the key
   is missing.

Bug: T191237
Change-Id: I0ada337b0b4ab73c80236367ff79c31bcd13aa7d

Localisation updates from https://translatewiki.net.

Change-Id: I56c13502176ea25a03148afc4f2256079e76dc82

Merge "Change index on rc_namespace, rc_title to rc_namespace, rc_title, rc_timestamp"

Merge "Fix parsing of <pre> tags generated by extension tag hooks"

Merge "Revert "Work around Firefox 57 attribute parsing regression""

Merge "Special:Preference: Check for changes before enabling Save button"

Convert HTTP links to MediaWiki.org to HTTPS

Bug: T189687
Change-Id: I846d3a74f617b98c8237f2a8babef85c6c128b14

Merge "Update OOUI to v0.26.2"

Update OOUI to v0.26.2

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.26.2

Depends-On: If570668a7fa615b2ffc3474ce5a161515af05f3b
Change-Id: I0131533e91fc0e308f90aea2555ead07740b093d

Merge "Use fallback languages in ResourceLoaderImage"

Merge "rdbms: clean up DBO_TRX behavior for onTransaction* callbacks"

Merge "Special:ProtectedTitles: Use HTMLForm"

Merge "Use new API for descriptions in TitleWidget"

Merge "Remove outdated comment for .background-image-svg"

Merge "RCFilters: Followup Iebc3176c5d: Bidi isolate filter tags"

Use new API for descriptions in TitleWidget

Bug: T191333
Change-Id: I5682f2fe3000d61c3760d22e9a567c762cf36710

Remove outdated comment for .background-image-svg

Since we dropped prefixed versions, we are no longer embedding
multiple times.

Change-Id: I58ad6f732289253c22cf12c74e340cd81c26e943

Merge "Convert Maintenance to the new shell framework"

Merge "Switch Installer to the new execution framework"

Merge "Switch ServiceWiring to the new execution framework"

Convert Maintenance to the new shell framework

Change-Id: Ibec8a84071fdf061f9831b2964143d99464cd4cc

Switch Installer to the new execution framework

Change-Id: Id04ba0751ff94e9614852e351377405f65b6ea1c

Switch ServiceWiring to the new execution framework

To test:

$wgMimeDetectorCommand = 'file -bi';
$services = \MediaWiki\MediaWikiServices::getInstance();
echo $services->getMimeAnalyzer()->guessMimeType('/vagrant/mediawiki/README');

Change-Id: Iaf124901c68292736e5588636f5ec746147f6a54

Change ResultWrapper to IResultWrapper in pagers and special pages

There is no need for a specific type, the interface implementation is
enough

Change-Id: I22ec4d3a9a19fc86efc9a2deb06f810695d38c85

RCFilters: Followup Iebc3176c5d: Bidi isolate filter tags

Bug: T190718
Change-Id: I8e556624ddbc4e824a3fb9724f6fb721c18d2e85

Let install.php detect and inject extensions [2]

For CI, I went with a dirty trick to find extensions and either
wfLoadExtension() or include them. That has since been ported to
Installer::findExtensions() and LocalSettingsGenerator::getText().

The WebInstaller() relies on that to detect extensions and let the user
tick the ones to be installed.

Add --with-extensions to install.php so one can include all extensions
from the command line.

Note: The CliInstaller always wfLoad() any skin it can find.

The original patch 53926d9d was broken and got reverted. This is the
proper one.

Bug: T189567
Change-Id: I41820fce12e01224d64ab711e016835ef4d4febc