Merge "SpecialNewPages: Fix omitted Show/Hide redirect value"

Merge "resources: Upgrade jquery.i18n from 1.0.4 to 1.0.5"

Merge "Fix pingback on non-MySQL"

resources: Upgrade jquery.i18n from 1.0.4 to 1.0.5

Change-Id: I9b7eebaec8fea6c328543ebf18afa8062c54efbe

Merge "Convert file delete to use OOUI"

Convert file delete to use OOUI

Also make a few changes for consistency with normal delete form:
* Add `autofocus` attribute on the "Other/additional reason" text field
* Remove bold styling from "Suppress …" checkbox label
* Remove `class="wpReasonDropDown"` from the "Reason" dropdown
* Remove `id="wpDeleteSuppressRow"` from the "Suppress …" form row

Bug: T173997
Change-Id: Ia5a0412c959c6149b01afe7fc6cbf5e8591673ac

Merge "[MCR] Allow extensions to manipulate service instances"

Remove long-dead OutputPage methods set/getPageTitleActionText()

They were accidentally made non-functional in April 2009 by commit
e4d21170.  Subsequent commits 2d045fa1, 9e230f30, e275ea28, ae45908c
removed all callers by October 2011.  Needless to say, I found no
callers in core or extensions.

It seems we have the functionality in some other way, probably just by
directly calling setHTMLTitle(), so there's no need to revive this
feature.

Bug: T200643
Change-Id: Ifa2537b97cf19e7e91ee33d4ea4c131f7a38ee89

Merge "Update categoriespagetext message"

Convert remaining MagicWord:: calls to MagicWordFactory

Bug: T200247
Depends-On: Ie061fe90f9b9eca0cbf7e8199d9ca325c464867a
Change-Id: I49c507f3875e46a8e15fd2c28d61c17188aabffc

Merge "Update MagicWordArray to use MagicWordFactory"

Fix pingback on non-MySQL

MySQL has its own implementation of upsert(), but the others rely on
$uniqueIndexes being a field list, not a field/value associative array.
The previous code generated an SQL error when checking for existing
values.

Bug: T200864
Change-Id: Ifb56f7f350fbb84144bc6f5a1dd76939816338a6

Fix total breakage of SQLite web upgrade

Partial revert of 8d61cf2793abd2, since LBFactoryTest.php passes without
overriding selectDB() in the current master. It's unclear why it was necessary.

Bug: T200861
Change-Id: I7c0560d7890616899297f81a227db98996d5cb7f

Merge "Fixing linkTrail problem for Urdu language."

Merge "Disable "...able to change preferences" browser test"

Localisation updates from https://translatewiki.net.

Change-Id: I5cc354afe1211eb802a885358545a5a7fd08efff

Disable "...able to change preferences" browser test

It's incredibly flaky.

Bug: T199446
Change-Id: I5372a285dab4f5f032ae340d9fd30b9c7c8bf72a

Update MagicWordArray to use MagicWordFactory

Bug: T200247
Change-Id: Ie5a60b81382d7299beadc691fe4d27e931ebe0ed

Merge "Update CoreParserFunctions to use MagicWordFactory"

Merge "Add explanation about JS editing changes to permission error"

Merge "Return array without holes from User::getAllGroups"

Merge "Fix variable capitalization in message"

Merge "Fix documentation for $wgRightsText"

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

Merge "ReverseChronologicalPager: Fix timezone confusion"

Return array without holes from User::getAllGroups

Change-Id: I2933ffe2cddec3d595536f93df17b609cdabab9e

Fix variable capitalization in message

Change-Id: I96be2f3585dd1711f4aaf1a114dfef7c106de202

Fix documentation for $wgRightsText

Change-Id: I6e7b8501241276b91172df7f13dfc7a1227b23a5

Merge "SpecialPageExecutor, make sure Skin context is correctly set"

Update CoreParserFunctions to use MagicWordFactory

Bug: T200247
Change-Id: I122d8acf601581b18756a5b8d65e50953b28c21d

SpecialPageExecutor, make sure Skin context is correctly set

Bug: T200771
Change-Id: I6d642449a0962f0a6bc242d2121ad3f270d9df77

Merge "Fix PerDbnameStatsdDataFactory metric prefix"

Merge "Add parentheses around nested inline IF statements"

Introduce SpecialPageFatalTest

This test makes sure that special pages do not fatal
in their most basic form (anon user viewing the page).

Depends-On: I3a9f5b315eb114cb12ea4071f8da9079f797fcf6
Change-Id: Ic675e92d8dd8f11fa67914d2ce1dc00a379106ca

Fixing linkTrail problem for Urdu language.

Bug: T178400
Change-Id: I9194fcbfa3768155a1b0965b566defa12e65d18e

Merge "Update Parser to use MagicWordFactory"

Merge "MagicWordFactory to replace MagicWord static members/methods"

Merge "Use multibyte-aware truncation to avoid invalid UTF-8 in cl_sortkey_prefix"

Pass $title to 'GetLinkColours' hook

I think it's reasonable for link "colours" to depend on the page on
which they are shown. It seems similar to how self-links are handled.
The $colours variable is not cached, so title-specific link colours
will not "leak" to other titles.

Used in ProofreadPage in Ic910c2c33a6f1f8a70d9a122fbd2128428f29bd5.

Bug: T199288
Change-Id: I7378102a3e06544e9e695b255982c9bb0cfbf3a2

Update categoriespagetext message

Since Special:Categories now lists empty categories, the message will need to be updated.

Bug: T185066
Change-Id: Ifaebeb0e95bc37aa51d0ddd426b8850bb40d9dbb

Merge "API: Update query modules for MCR"

Localisation updates from https://translatewiki.net.

Change-Id: I2a9594ddac8152a66caadade3c0293a16942d42a

Use multibyte-aware truncation to avoid invalid UTF-8 in cl_sortkey_prefix

The invalid UTF-8 could cause incorrect sorting of affected pages in
category lists on wikis using UCA collations. On my local testing
wiki, the generated cl_sortkey was just 0x30 regardless of the value
of cl_sortkey_prefix.

This doesn't fix existing bad data in the database. It will only be
updated when the affected page is edited (or null-edited).

The cl_timestamp field will also be updated when that happens, which
apparently may affect Wikinews' DynamicPageList extension, according
to comments on T27254. This is not easily avoidable.

Bug: T200623
Change-Id: I4baa9ea3c7f831ff3c9c51e6b8e5d66e7da42a91

Update Parser to use MagicWordFactory

Bug: T200247
Depends-On: Ie061fe90f9b9eca0cbf7e8199d9ca325c464867a
Change-Id: Iab6a4cb32c491caf2685cdd68f9465ef1dfa3c4c

MagicWordFactory to replace MagicWord static members/methods

Static members of MagicWord have been removed.

Static methods are soft-deprecated and forward to the factory.  They
will be hard-deprecated when all callers are removed from core.

MagicWord::clearCache() has been removed.  Instead, call
resetServiceForTesting( 'MagicWordFactory' ) on your MediaWikiServices
object.

Change-Id: Ie061fe90f9b9eca0cbf7e8199d9ca325c464867a
Bug: T200247

Merge "Special:NewPages: Change to use OOUI"

Merge "Improve test coverage for OutputPage"

Merge "Treat all time values on Special:Watchlist as floats"

Improve test coverage for OutputPage

Raises coverage locally from 16.10% to 26.81%.

Change-Id: Ib564624c644ee6620ac06872f5684831acaaaadb
Depends-On: I79c4e37092958c63a693194b27a9eafae70cb2f8

Treat all time values on Special:Watchlist as floats

PHP division may return float or int, depending on the operands. [1]
Make sure all numbers are of the same type (floats), because
XmlSelect compares values with ===.

[1] http://php.net/manual/en/language.operators.arithmetic.php

Bug: T199566
Change-Id: I37df6fd425f47d9a4562d83e04fcb50c3b97e0da

Add parentheses around nested inline IF statements

To improve readability of the code. Done using the NestedInlineIfsSniff
I wrote for MediaWiki-Codesniffer (T171520, I0ddf05d9).

Change-Id: I89ac6e9b5eab1f599fec3686b40a3e01d29d0250

Add @since tag for MediaWikiServices::getConfigRepository

Change-Id: I9c92b9640526c5a8dee2e994a5d0b1bc27d43381

Merge "Move utility methods from ApiTestCase to MediaWikiTestCase"

Merge "Use array_merge() for OutputPage::$mLanguageLinks, not +"

Special:NewPages: Change to use OOUI

Bug: T117739
Change-Id: I802d215374c0b5247247dd4ae2facce251e8d369

Localisation updates from https://translatewiki.net.

Change-Id: I3d04523e2e1357fb5f5b69bb91b5d51d65dcdf1b

Use array_merge() for OutputPage::$mLanguageLinks, not +

Based on documentation together with inspection of some callers, the
intent seems to be that this is an indexed array, not associative.  +
will therefore do totally the wrong thing, ignoring any new values that
have the same key as an existing item (e.g., '0' or '1').  Even if it
was an associative array, + keeps the values on the left-hand side,
which means you normally want to do $foo = $bar + $foo instead of $foo
+= $bar if you want to overwrite old values with the new ones.

Before this change, calling addLanguageLinks() or
addParserOutputMetadata() would generally not add all of the links it
was supposed to if there were already links defined.  (It could still
work if the arrays' keys didn't conflict for some reason, e.g.,
something passed an associative array or an indexed array with a hole.)
I don't know if anything actually hits this bug, because it's likely
that callers usually add all their links at once.  I find no uses of
addLanguageLinks() at all.

I found this bug while working on adding more tests for OutputPage, and
the tests for this change will be submitted later in
Icdc0288c04b8c4ba841f9fbb3e05a0cdc8a20fa5.

Change-Id: I53f6e7ea94417b0034371e56e733e8c86af21658

Move utility methods from ApiTestCase to MediaWikiTestCase

Change-Id: I79c4e37092958c63a693194b27a9eafae70cb2f8

Merge "GlobalFunctions: Hard-deprecate wfUseMW, unused"

GlobalFunctions: Hard-deprecate wfUseMW, unused

Change-Id: Ifa532f2fb7d482238e4b922f622a79b9ab6540e8

Make sure to not unpack an associative array into parameter list

Bug: T200394
Change-Id: I9c28e1cadeb76275d24eb7725f1578bf5ba43ad0

Merge "Guard against uncountable tag values"

Guard against uncountable tag values

As of PHP 7.2 counting false or null raises a "Warning: count(): Parameter must be an array or an object that implements Countable".

Bug: T182377
Bug: T196793
Change-Id: I7ca38bc55ae04f68106fe0d27c7d496da1538459

Localisation updates from https://translatewiki.net.

Change-Id: I8b1b0ee78ffc63ec22a29175249ec4eae1c8afdc

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