Use LogPage::validTypes() instead of $wgLogTypes in a few places

This is preparation for T200385, which will add a hook to modify the
list of known log types, which will bypass the global variable.

Change-Id: I763cf8b71a98d1dba5f9964fc8d919a268c5d8a5

Enclose compact() call in error suppression

In PHP 7.3, compact() now emits warnings when a variable doesn't exist.
Because our language files aren't required to contain all the possible
variables, this results in massive spam trying to run tests.

Change-Id: Idab0340ec1cdebfca67cb448e350a408438bcbbc

API: Update query modules for MCR

MCR deprecated the Revision class in favor of the broadly similar
RevisionRecord, and more interestingly added the concept of multiple
content "slots" to revisions.

Thus, prop=revisions, prop=deletedrevisions, and so on gain a parameter
to specify which slots are wanted. When this new parameter is not
specified (and any content-related props are specified), a warning about
the legacy format will be issued.

The rest of the modules just needed to call methods or use constants on
RevisionRecord instead of Revision. ApiQueryDeletedrevs wasn't touched,
since it has been deprecated since 1.25 anyway.

This also updates a few non-query modules that don't depend on details
of editing, diffing, or viewing MCR revisions that haven't been figured
out yet.

Bug: T200568
Change-Id: I1327d1784f5cedb006cd74df834cf9a560a77a5d

Localisation updates from https://translatewiki.net.

Change-Id: Ibec891ce84ce50657b8b3486f5be27629e6404f5

Merge "Remove two unused OutputPage methods"

Merge "Disallow empty paths in LockManager"

Merge "Correct recent schema changes for MSSQL, Oracle"

Merge "Fix test database prefix in ParserTestTopLevelSuite"

Add explanation about JS editing changes to permission error

This does not handle the error message when viewing a non-existent
page (noarticletext-nopermission), but that seems complicated to
get right.

Change-Id: Ia4e798b9a646908bd84fec5a53db929040e61aa7

Merge "HistoryAction: Implement HistoryPageToolLinks hook for adding more links"

Merge "Check for right-* messages as a structure test"

Merge "Segregate right to edit sitewide CSS/JS"

Merge "Deprecate $wgExternalDiffEngine = 'wikidiff2'"

Check for right-* messages as a structure test

Structure tests run for extensions as well, which this test should. All
user rights should have right-* messages for display in the interface.

Bug: T143156
Change-Id: I23b8eb66bc68121b2ae17e73e705acd3e6f2d053

Prefill date in form on Special:Log when calling with old parameters

When the date is specified using &year=...&month=... URL parameters
rather than &wpdate=..., HTMLForm can't automatically fill it in,
but we can explicitly specify the default.

Bug: T199856
Change-Id: I71f4953d57a3893190778e5bd763db108dc433d3

ReverseChronologicalPager: Fix timezone confusion

(This change is best tested with Special:Log before changes from
38756eae4, since it seems the $mDay/$mMonth/$mYear values are not
directly used anywhere in core after that.)

The setTimezone() call did not actually change the offset used for
database lookup, because getTimestamp() returns a Unix timestamp,
which by definition does not include timezone information.

It did change the offset used for year/month/day fields in the
interface, but it changed it in the wrong direction: instead of
assuming the date is in local timezone and converting to UTC, it
assumed the date is in UTC and converted it to local timezone.

We don't actually want to do anything with timezones here. For better
or worse, the date selectors for pagers have always used UTC dates.
The timezone code was only introduced relatively recently in 53fa809a.

What we really want for the interface is to display the previous
date, relative to the database offset. The interface asks the user
to select the month/date up to which to display things (such as
"May 2018 (and earlier)"), but the database queries use strict
comparison (such as `log_timestamp < '20180601000000'`), so the
database offset is calculated to the next date after selected one.

The incorrect timezone calculation accidentally did so, but only if
the wiki timezone had a negative offset relative to the UTC. If the
wiki was using UTC or a timezone with positive offset, the correction
was not applied, causing the selected date in the interface to move
one day forward from the date that was set.

Bug: T171110
Change-Id: I8c2cd398d7c7ac68a8f46ee94cb9e7c6beed5694

Merge "Prefill username in form when calling Special:Log/<username>"

Merge "Handle $title === null in Title::newFromText"

Merge "SelectWithInput: Add 'disabled' property on main widget"

Merge "GlobalFunctions: Enable patch-coverage for wfExpandUrl test"

Merge "INSTALL: Don't warn against using PHP "as a CGI plugin""

Merge "Update jetbrains/phpstorm-stubs"

Handle $title === null in Title::newFromText

This relied on TitleCodec throwing MalformedTitleException in the
past, but that is fragile as other parts of the logic do not
expect null.

Bug: T200456
Change-Id: I1aca3971e2a9c0b1fe3adbcf34f3ee65b2271234

Disallow empty paths in LockManager

Fix broken FileBackend test that had paths normalizing to null.

Change-Id: I8a7e88ae44118cc9578e1b306a5a008ccefc23af

Update composer/spdx-licenses to 1.4.0 and mediawiki/mediawiki-codesniffer to 21.0.0

https://github.com/composer/spdx-licenses/compare/1.3.0...1.4.0

Change-Id: I39f7a1310455159866bfed5224536e800befec0d

Update jetbrains/phpstorm-stubs

Change-Id: I553afcc8f0126b9611dd9fce26029bd9c9e17071

Prefill username in form when calling Special:Log/<username>

Bug: T200446
Change-Id: I3f2fcedee8440d5943e556f7c1c06dfb1085e549

Merge "Upstream EasyDeflate library from VisualEditor"

Fix PerDbnameStatsdDataFactory metric prefix

This was reporting Mediawikiwikidatawiki for example as the '.'
was added inside the rtrim call

Bug: T196609
Bug: T200424
Change-Id: Ia8e51371b9a27ed14ec55bb34a52ec9e65ad55cf

Merge "Deprecate public access to some DifferenceEngine properties"

Deprecate $wgExternalDiffEngine = 'wikidiff2'

The value 'wikidiff2' for $wgExternalDiffEngine was obsoleted in
I32b670ebf6 but not formally deprecated. Let's do that now.

Change-Id: Ie5ccd3895544cf393885b84d38eca6003c79e8ad
Depends-On: I33c6207cfc5fb66987d31818bc27055f597d28c7

Deprecate public access to some DifferenceEngine properties

These had no business being public in the first place and there is
no usage in Gerrit. In case something does use them, it will probably
be broken by the refactoring that's soon to come, so the deprecation
warning makes it easier to figure out where the problem lies.

Change-Id: I72ba5cfbf91f4af16028ba4f8619df6a7168a786

Use setContentLang() instead of setMwGlobals()

This changes behavior in some tests by making them set $wgLanguageCode
as well as $wgContLang, but that seems like a good thing.

Bug: T200246
Change-Id: I936888f46ff9fefe2707efba837e2ce3a7ca5e3f

Remove two unused OutputPage methods

addMetadataLink and getMetadataAttribute were added in 2004 (22f8c4ce)
for functionality that was moved to extensions in 2011 (27c3b22b).
Those extensions, CreateCommonsRdf and DublinCoreRdf, now use
addHeadItem() directly.  There are no other callers in core or
extensions.

Change-Id: I4bc21fdaefc8f56969a2950eb53b340bf4aac399

Merge "Use 'default' instead of 'value' in HTMLTextField"

INSTALL: Don't warn against using PHP "as a CGI plugin"

This note is very old; it was added in r2832 (635388356be70130) and
r3285 (7ab7a50cf928861f). Modern versions of MediaWiki support CGI and
FastCGI installations nearly as well as mod_php installations. In fact,
Wikimedia currently uses HHVM's "fastcgi" server type, and intends to
use php-fpm with PHP 7 (see migration plan in T176370).

For a basic MediaWiki installation using the web installer, the only
difference I know of is the default URLs: /w/index.php?title=$1 for CGI,
as opposed to /w/index.php/$1 for mod_php. This issue is easy to fix by
changing $wgArticlePath, if the web server sets REQUEST_URI. Admins who
want to use short URLs (e.g. /wiki/$1) have to do this anyway.

Also, in img_auth.php and the "img-auth-nopathinfo" message, don't imply
that CGI and FastCGI server configurations generally can't provide path
information or support MediaWiki features that need it.

Change-Id: I89212e9d51f950a58f911083b9e109b69cd4e060

Merge "Add helper trait for deprecating properties"

Make MessageCache use APC for big messages

Bug: T118893
Change-Id: I418d0ca490911d7ee4213fe1728cfbc637ef8b10

Localisation updates from https://translatewiki.net.

Change-Id: I69852319aa363902ca532563bee3938103e1be41

Use 'default' instead of 'value' in HTMLTextField

Bug: T193087
Change-Id: I631970797a778caa77d3605e0a5b8ca00e2cb56e

GlobalFunctions: Enable patch-coverage for wfExpandUrl test

Change-Id: I3b4594dab0aba659cc47233f7be3da751066b7de

Merge "Make the 'other' option superessable in getSuggestedDurations"

Merge "Follow-up 9742d569b: Replace accidental \t in i18n with a space"

Merge "Deprecate $wgShowSQLErrors and $wgShowDBErrorBacktrace and make nonfunctional"

Deprecate $wgShowSQLErrors and $wgShowDBErrorBacktrace and make nonfunctional

Clarify and simplify exception output by deprecating
$wgShowSQLErrors and wgShowDBErrorBacktrace.
$wgShowExceptionDetails will now control most related output.
$wgShowHostnames will now solely control output of
MWExceptionRenderer::reportOutageHTML.

Bug: T165768
Change-Id: Idead2c11c499463dfa6293c3d4b33be3bde92e1a

Follow-up 9742d569b: Replace accidental \t in i18n with a space

Change-Id: Id8ad5916b7dd59a078fb600c7998e41f1b8abd3c

Merge "Make load* methods of DifferenceEngine idempotent"

Merge "cache: Minor docs for MessageCache interaction with WANObjectCache"

cache: Minor docs for MessageCache interaction with WANObjectCache

Follows-up 04bc03a29a.

Change-Id: I8fb810fc2e08277bb64fa18595483a2161cfb1cc

Merge "Fallback language for Santali should be English"

SelectWithInput: Add 'disabled' property on main widget

…also HTMLSelectAndOtherField and HTMLSelectOrOtherField now
pass the 'disabled' field when creating the input widget.

The OOUI versions of these HTMLFormFields would get enabled
right after infusion even they were disabled in PHP. This
makes sure that they remain disabled.

Change-Id: Iddd8ad81731dba7bdcb599d6fe104cb259b11733

Merge "ApiBase: Use prefixed parameter name for 'missingparam' error"

Merge "Make OutputPage::mPagetitle private"

Merge "objectcache: improve logging and error handling in BagOStuff"

Fallback language for Santali should be English

Bug: T200290
Change-Id: I65005b8b31bd022c7978628f69c5d56d735063e3

Localisation updates from https://translatewiki.net.

Change-Id: Idbc346a6c0d5bf2bdc2278bc2ef3440579500dc7

objectcache: improve logging and error handling in BagOStuff

Bug: T198239
Bug: T199383
Change-Id: I0b4110396d03b98e83a7b614caf57d7e7c284361

Merge "PageArchive: Pass correct overrides to newRevisionFromArchiveRow()"

Merge "Avoid a redirect loop when the request URL is not normalized"

Make the 'other' option superessable in getSuggestedDurations

Bug: T193364
Change-Id: Ic2dbc961f7eebad11da53724b9cce2f804ffad39

Make OutputPage::mPagetitle private

Code search revealed only one outside access, which I've submitted a patch for.

Depends-On: I001deecad241b091070483d882088d2d6361ed5a
Change-Id: I0f7e565f4eacc1074c619772b2d11a47a05e3584

thiemowmde's legal name changed

I was a co-author only, and my contribution was not really
noteworthy, so I choose to remove the outdated name (in this
case only).

Change-Id: I9d9592d5b114adae633f6a4bdac2dd60f329bd7a

Merge "LogEventsList: Correct typo"

Merge "$wgHttpsPort should only be used in very special cases"

Merge "LogPager: Add backwards-compatibility for hide_[type]_log URL params"

Localisation updates from https://translatewiki.net.

Change-Id: I8b632d4052c2634867c7d498305cf287d2fc58c9

Revert "Ensure LanguageCode::bcp47() returns a valid BCP 47 language code"

This reverts commit 8380f0173e79b66f0e2afd6c49cd88afb9f4f6f3.

Reason for revert: Caused T199941

Bug: T199941
Change-Id: I93af756a2d70d6bc91f828fe6ac19bf10ca8788f

Merge "Revert "Accept BCP 47 codes as aliases for nonstandard variants""

Revert "Accept BCP 47 codes as aliases for nonstandard variants"

This reverts commit 0818070c59eb4e8330496796b943951c01c7a573.

Reason for revert: Caused T199941

Bug: T199941
Change-Id: I24c178eb33890477de79cbb3122861c140578011

Merge "LogEventsList: Use DerivativeContext"

LogEventsList: Correct typo

The parameter to getTitleInputDesc() is actually unused, so this was
not breaking anything.

Change-Id: Ia301faf511e30b4fc4542ccb80b24cf807cf731c

ApiBase: Use prefixed parameter name for 'missingparam' error

Similar errors use the prefixed parameter name, this one should too.

Bug: T200155
Change-Id: Ia14d6a9c457af06e72428c1eae14bd3849b4595a

PageArchive: Pass correct overrides to newRevisionFromArchiveRow()

Revision::newFromArchiveRow took 'page' as an override for ar_page_id,
while RevisionStore::newRevisionFromArchiveRow() needs 'page_id'.

Thanks to sanity checks elsewhere in RevisionStore, this mistaken
override causes an exception to be thrown rather than undeleted
revisions potentially pointing to the wrong page.

Bug: T200072
Change-Id: I9d7543866c674f4d8aea9ec00fcc15cbf616ca66

LogEventsList: Use DerivativeContext

Bug: T200136
Change-Id: Ie2b7753684dc0257b0b53d9c9314feeb14d99182

Merge "HTMLMultiSelectField: Improve compat with GET forms with no wpFormIdentifier"

Added non-parallel fallback to MultiHttpClient when curl is unavailable

If the curl extension is not available, fall back to the existing
HttpRequestFactory and associated classes. Also added related phpunit tests.

Bug: T139169
Change-Id: I2f9d4acbb491bce28d7105e124c5cee7e16e86d7

Merge "Make MessageCache use getWithSetCallback() for big messages"

Localisation updates from https://translatewiki.net.

Change-Id: I93c905909b83a89898922e241610275186e5346b

Segregate right to edit sitewide CSS/JS

To limit the likelihood of incidents like T189665, the ability to
edit sitewide CSS/JSON/JS is split out from editinterface, into
separate 'editsitecss', 'editsitejson' and 'editsitejs' rights.

editsitecss, editsitejs, and the right to edit another (potentially
more privileged) user's personal CSS/JS is removed from sysops, and
a new user group, interface-admin, is created specifically for that
task (along with a new grant group 'editsiteconfig').

interface-admin is granted to the first user of a new wiki, along with
sysop and bureaucrat.

Bug: T120886
Bug: T190015
Depends-On: Ia9b2ea1450aff6121dc0f3777bf029292c8aaad9
Change-Id: Ifefd872640642441e26f8b2f144ffe4b88d2eb12

Localisation updates from https://translatewiki.net.

Change-Id: I848c703ad10f39827730fcdacdd83b4ad47b346d

Merge "Explicitly mark Skin::makeVariablesScript() public"

Merge "Fix octal escape sequence overflow in FiltersTest.php"

Explicitly mark Skin::makeVariablesScript() public

Remove else. If contains an unconditional return

Change-Id: I57a04339395d947dffbc3595d321106d4fd8256c

Localisation updates from https://translatewiki.net.

Change-Id: I01a747cba68c60850761ac5dcef992bcebaaf6b5

Merge "Send API timing to statsd for all requests, not just writes"

Add maintenance script for resetting login/signup throttle

Bug: T194506
Change-Id: I6dda5f516ba47a452e438946379fcb696c8c87f9

Merge "filebackend: let any specific backend value of "readOnly" override the default"

Merge "JobQueueGroup: Allow readOnlyReason to be specified per JQ type"

Merge "Install cache/integration-tests as require-dev"

Install cache/integration-tests as require-dev

Bug: T199440
Change-Id: I1e50d7202a5d02088319ec9e90e0136519cba1ce

GlobalFunctions: use a closure instead of polluting global namespace

Change-Id: Ia587008de75a8bde90ad45d4932284f0b6401845

Send API timing to statsd for all requests, not just writes

The rate will be about 8k req/s, whereas currently graphite1001
receives about 350k metric updates per second, so we would expect
this to increase statsd load by about 2%.

This would have been very helpful in diagnosing API cluster overload
incidents such as the one on July 11.

Change-Id: I0a248e1bcec05915d98ec735f4cfb9800e64412d

Merge "language: Document sources of data and example usage"

Merge "resourceloader: CSSMin::getLocalFileReferences now strips anchors"

Merge "Language::listToText: Avoid PHP warnings from E_STRICT"

resourceloader: CSSMin::getLocalFileReferences now strips anchors

When processing:

  url( 'foo.svg#anchors' )

getLocalFileReferences() would yield a file 'foo.svg#anchors'. It might
occur with upstream libraries, such as WikiFonts used by the Refreshed
skin.

When a path is found, strip the anchor entirely. Skip the case if that
results in an empty file.

remap() would properly skip the behavior "#default#', also skip url that
are solely an anchor such as '#other'.

Add a few test cases to cover CSSMin::getLocalFileReferences and cover
anchors usage in CSSMin::remap.

Bug: T115436
Change-Id: I1749ddc2b89021807f42d64131931ad7a99a7b43

language: Document sources of data and example usage

Change-Id: I0c72069d010285d9b88ff83bc1478410961065d3

parserTests: re-enable save parse and fix MCR errors

* Re-enable parse on save as requested in T159014, instead hide
  the feature behind an option.
* Add the MCR tables to the list of tables that need to be cloned. Fixes
  unconditional exception.
* Fix some undefined member variables, remove assignment to unused
  variable $this->databaseSetupDone.

Bug: T159014
Change-Id: I6be577ebcbadeb4e08079a82543d521fc2bb5948

LogPager: Add backwards-compatibility for hide_[type]_log URL params

Also, the $default value in LogEventsList#getFiltersDesc was not
being generated properly and would have always been an empty array.

Bug: T199856
Change-Id: Id286c76259406521f12cda67a4a715032e022637

Merge "Convert Title::getTitleCache() to using MapCacheLRU (again)"