Merge "Selenium: record video of every test"

Selenium: record video of every test

In addition to two existing modes (headless and visible), this patch
introcues another mode, headless recording.

If DISPLAY environment variable is not set, Chrome will run in it's
built-in headless mode.
If it is set, and the value starts with colon (`:`), Chrome will
run in headless mode using Xvfb and record video of every test using FFmpeg.
If the value does not start with colon, Chrome will be visible.

Bug: T179188
Change-Id: Ic1723c5f2d57a28201caf6ba7056cb73fb74a957

messagecache: check overridable message array in getMsgFromNamespace()

Follow up to a3d6c1411dad.

This avoids extra queries for messages that have a software defined value.

Bug: T193271
Change-Id: I25aa0e27200a0b417721cf1fbd34a82095405b89

Merge "[Korean] Add 2 additional bookstoreList"

Localisation updates from https://translatewiki.net.

Change-Id: I44bc61d6e2147602329eeface6892e78db0388b2

Merge "API: Allow prop=info intestactions to return reasons"

Merge "Migrate some wfWikiId() callers to getLocalDomainID()"

API: Allow prop=info intestactions to return reasons

T194585 raises a use case for callers to be able to know why an action
is not allowed. We can make that possible easily enough. The default
remains to return only a boolean.

This also deprecates inprop=readable in favor of intestactions=read,
since they both just return `$title->userCan( 'read', $user )`.

(ApiQueryInfoTest added by David Barratt)

Bug: T194585
Change-Id: Ib880f0605880eac776d816ea04e0c7ab9cfbaab1
Co-Authored-By: David Barratt <dbarratt@wikimedia.org>

Merge "Update ImportableUploadRevisionImporter for interwiki usernames"

Merge "Have maintenance/updateExtensionJsonSchema.php copy documentation"

Have maintenance/updateExtensionJsonSchema.php copy documentation

It already skips version-1 style documentation keys that begin with "@".
It should also copy them into the "description" metadata key for
version 2.

Change-Id: I460568f074b9c0256eb6bbbcd0b046b16e0d8790

Update ImportableUploadRevisionImporter for interwiki usernames

This was somehow missed in I5401941c.

Bug: T206013
Change-Id: Ia618b05329e6cbfca7c95d9161f12ba4150705c8

Add session_write_close() calls to SessionManager tests

PHP 7.3 doesn't like it if session_id() is called when the session has
been started, so we need to be sure to close it first in a few tests.

Bug: T207112
Change-Id: Ief36c1bb7b5c9066f158b5bb0d6d785a7f7ddd3c

Avoid fatals when the filter tags is empty

Bug: T194164
Change-Id: I418cbe2259e7c1622a58a4a310a3cd56873f16c3

Migrate some wfWikiId() callers to getLocalDomainID()

Change-Id: I33fe222b7ca66babd61610febaebcf52d3806a7d

Merge "Simplify `list-style` property"

Merge "Fix warning in doEditSectionLink() caused by not-yet-unstubbed $wgLang"

Merge "API: Ignore expired blocks in ApiQueryBase::showHiddenUsersAddBlockInfo()"

Merge "LocalisationCache: Avoid use of compact()"

Merge "API: Remove long-deprecated methods (and one class)"

Merge "CREDITS: Bump for 1.32.0 release"

API: Remove long-deprecated methods (and one class)

* ApiBase::profileIn() (deprecated in 1.25)
* ApiBase::profileOut() (deprecated in 1.25)
* ApiBase::safeProfileOut() (deprecated in 1.25)
* ApiBase::profileDBIn() (deprecated in 1.25)
* ApiBase::profileDBOut() (deprecated in 1.25)
* ApiBase::dieUsage() (deprecated in 1.29)
* ApiBase::dieUsageMsg() (deprecated in 1.29)
* ApiBase::dieUsageMsgOrDebug() (deprecated in 1.29)
* ApiBase::getErrorFromStatus() (deprecated in 1.29)
* ApiBase::parseMsg() (deprecated in 1.29)
* ApiBase::setWarning() (deprecated in 1.29)
* ApiPageSet::getInvalidTitles() (deprecated in 1.26)
* ApiQueryLogEvents::addLogParams() (deprecated in 1.25)
* ApiUsageException::getCodeString() (deprecated in 1.29)
* ApiUsageException::getMessageArray() (deprecated in 1.29)
* UsageException (deprecated in 1.29)

Change-Id: Iabb2589a29cc3b46624d31358f3a6bf7b3ccbd57

LocalisationCache: Avoid use of compact()

In PHP 7.3, compact() now raises notices if the variable is undefined, which
is something that we expect. So we can check whether the key exists instead
of bothering with compat() and suppressing warnings.

Bug: T206979
Change-Id: I612049db4debd850a2e6d10bc631d31aa17be898

Fix warning in doEditSectionLink() caused by not-yet-unstubbed $wgLang

In ParserOutput::getText(), $wgLang can be a StubUserLang object,
which causes a typecheck warning in doEditSectionLink().

Solution is to replace $wgLang with $context->getLanguage(),
which is always a Language object.

Change-Id: I9945e1615c3f1b7ed5c7897b9f67d7ca0450ee50

Simplify `list-style` property

Using `none none` value is also working on older IEs, which had
issues with shorthand only `list-style: none`.

Change-Id: I7d4b075ac90fd902e0c0fbc3e11449faf71fe9b2

Localisation updates from https://translatewiki.net.

Change-Id: I4b631605549ac3599ca01d59fa69e35fad6c879e

CREDITS: Bump for 1.32.0 release

Change-Id: I2187346003a53874c559fcfe23ee94d822829794

Use "break" instead of "continue" inside a switch

"continue" statements in a switch are equivalent to "break". In PHP 7.3, will generate a warning.

Bug: T206974
Change-Id: I54bcec013ff52ab81bff09f8f7ef02f3944a5b7d

API: Ignore expired blocks in ApiQueryBase::showHiddenUsersAddBlockInfo()

This probably went unnoticed for so long because expired blocks are
regularly cleared as long as new blocks continue being issued, so people
found that the problem "fixed" itself.

Bug: T206944
Change-Id: I6559e1ec23f4469d19684b4d931913acbccb56d4

Merge "EmailNotification: Add newline before minor edit text"

Merge "resources: Register jquery.client as foreign resource and update to v2.0.1"

Merge "Add MessagesShn.php"

Add MessagesShn.php

This localizes:
* namespaces
* special page names

Bug: T206917
Bug: T206921
Change-Id: I0d27746aca56849056098ffffa866954d97f133d

resources: Register jquery.client as foreign resource and update to v2.0.1

Change-Id: Iec6414a1675dec4aeb5d9b66635d28670f094e42

Localisation updates from https://translatewiki.net.

Change-Id: I27647ca9376c400143ba52074322c4b50960db53

docs: Remove outdated information from globals.txt

design.txt doesn't exist anymore.

Bug: T111283
Change-Id: Ibb67fb721060b325114e571ba3e470a607d98884

Merge "jobqueue: clean up JobQueueDB::getCacheKey() to use makeGlobalKey()"

Localisation updates from https://translatewiki.net.

Change-Id: I5958a3381bd070c1cb7d79e6819b638418b87165

jobqueue: clean up JobQueueDB::getCacheKey() to use makeGlobalKey()

Change-Id: I341720fda0d242c7822f78048cb49f63538d08bc

travis: Remove PHP 7.2 from allowed failures

It's now passing!

Bug: T204884
Change-Id: Idb17630cff54b803ca5fa942b2fcf17a775edd0d

Merge "Output only to stderr in unit tests"

Merge "SlotDiffRenderer: add utility method for parameter type checks"

Merge "Linker: Document parseComment() as returning HTML"

Merge "Log startAtomic()/endAtomic() to the query logger"

Log startAtomic()/endAtomic() to the query logger

Change-Id: Ife7e292ef42946ec1c5e9121b7be6a90301b3903

Merge "exception: Correct label "Notice" for E_USER_NOTICE, not "Warning""

Merge "Revert "rdbms: add domain sanity checks to LoadBalancer connection methods""

Revert "rdbms: add domain sanity checks to LoadBalancer connection methods"

This reverts commit b416e166a3eba1dcfc2e912e37e443981d7f60ef.

Fails on Travis CI / PHP 7.1 / Postgres:

> Creating administrator user account
>
> UnexpectedValueException:
> Got connection to 'traviswiki-mediawiki', but expected 'traviswiki'.
>
> Backtrace:
> #0 includes/libs/rdbms/loadbalancer/LoadBalancer.php(887): Wikimedia\Rdbms\LoadBalancer->openForeignConnection(integer, string, integer)
> #1 includes/libs/rdbms/loadbalancer/LoadBalancer.php(766): Wikimedia\Rdbms\LoadBalancer->openConnection(integer, string, integer)
> #2 includes/libs/rdbms/loadbalancer/LoadBalancer.php(851): Wikimedia\Rdbms\LoadBalancer->getConnection(integer, array, string, integer)
> #3 includes/jobqueue/JobQueueDB.php(780): Wikimedia\Rdbms\LoadBalancer->getConnectionRef(integer, array, string, integer)
> #4 includes/jobqueue/JobQueueDB.php(761): JobQueueDB->getDB(integer)
> #5 includes/jobqueue/JobQueueDB.php(187): JobQueueDB->getMasterDB()
> #6 includes/jobqueue/JobQueue.php(332): JobQueueDB->doBatchPush(array, integer)
> #7 includes/jobqueue/JobQueue.php(302): JobQueue->batchPush(array, integer)
> #8 includes/jobqueue/JobQueueGroup.php(154): JobQueue->push(array)
> #9 includes/user/UserGroupMembership.php(162): JobQueueGroup->push(array)
> #10 includes/user/User.php(3733): UserGroupMembership->insert(boolean)
> #11 includes/installer/Installer.php(1680): User->addGroup(string)
> #12 includes/installer/Installer.php(1601): Installer->createSysop(PostgresInstaller)
> #13 includes/installer/CliInstaller.php(176): Installer->performInstallation(array, array)

Change-Id: I269df89d47cc5a183af630447cd1642d3d4a3deb

Merge "rdbms: add domain sanity checks to LoadBalancer connection methods"

Merge "Avoid global $wgUpdateRowsPerQuery in WatchedItemStore"

Merge "Add OOUI for HTMLFormFieldCloner"

Linker: Document parseComment() as returning HTML

It's not entirely clear from the documentation whether this method
returns wikitext or HTML, so make it explicit.

Change-Id: I094e370e921c43dd8c5dc4376b3dddd803665d90

Merge "Pass LBFactory to WatchedItemStore"

Merge "HtmlTest: Perform multilingual tests"

Merge "Use a ScopedCallback to silence transaction profiler in SqlBagOStuff"

HtmlTest: Perform multilingual tests

The tests here were only testing when user and content language
are the same, but the way things are rendered when they differ
is perhaps more interesting as it allows ensuring the two are used
in the correct places.

Change-Id: I357f50b082882fee485a95323d3a0a7cadb9a63c

Use a ScopedCallback to silence transaction profiler in SqlBagOStuff

This affects most LCStoreDB which is the default cache
and spamming the log.

Bug: T154424
Bug: T97693
Change-Id: I68af177c7d0d88ab266eb3fefb4336eff86d861c

Deprecate $wgUseKeyHeader and OutputPage::getKeyHeader()

The `Key` header was a draft IETF specification which expired without
becoming a standard.  It does not appear to be in active use anywhere.

Change-Id: I3924a1b5ff428b107573d2827c40e4af8adaaeb1

Avoid global $wgUpdateRowsPerQuery in WatchedItemStore

Change-Id: Ibff69d16919a595c3edb8e2b431aa42d84a2777a

Pass LBFactory to WatchedItemStore

Instead of passing the LoadBalancer and than get the LBFactory from the
service,
store the LBFactory on creation

Bug: T206601
Change-Id: I4c0d3712da591b2c5ddf2a65855d2cc9142417d2

exception: Correct label "Notice" for E_USER_NOTICE, not "Warning"

Follows-up ef06b528d9, which fixed the severity of all PHP errors
to have severity "Warning" or higher.

However, it also accidentally changed the label of "Notice"
to "Warning", which is confusing and incorrect.

Change-Id: Iffd39aa23b7f2cbff5cdaf876b8d4d595dcd6f96

rdbms: add domain sanity checks to LoadBalancer connection methods

Bug: T193565
Change-Id: I94d905277e01b8e30ac3f6532ece07388bb20cce

Merge "Make MergeableUpdate jobs avoid the sub-queue so they can always merge"

Merge "Make merged MergeableUpdate items always go to the end of the queue"

Merge "Fix Setup.php file-scope test"

Make MergeableUpdate jobs avoid the sub-queue so they can always merge

Change-Id: I5b100fae29b785ab4524d165dad2e8ee46406b0c

Increase OutputPage test coverage to >45%

Also stop returning a value from OutputPage::adaptCdnTTL().  There are
no users and the value doesn't seem very helpful: one would probably
expect it to return the new value of mCdnMaxage, but instead it
returns the new value of mCdnMaxageLimit. Better to have no return
value than one that's easily misunderstood (*and* which nobody uses).

Change-Id: Ia9dab86923b839334eab9f6fde17c4aed52130ec

Merge "Add join conditions to ActiveUsersPager"

Merge "resourceloader: Throw exception when config serialization fails"

Make merged MergeableUpdate items always go to the end of the queue

Change-Id: I2e8aa28f7426470d9b8b9e11b902deea793069b3

resourceloader: Throw exception when config serialization fails

If something puts a string that's invalid UTF-8 in a JS config variable,
JSON serialization will fail on the entire config blob. Currently, this
causes the entire config blob to be silently dropped, which breaks all
JavaScript because elementary variables like wgPageName are missing.

This change makes this scenario fail loudly rather than quietly, by
throwing an exception. This also makes bugs like these easier to track
down.

Bug: T206475
Change-Id: Ief2ae00228389a23627d440dc1cd9a54cf2b6926

Merge "MessageCache: replace should actually replace, not reload"

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

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

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

Localisation updates from https://translatewiki.net.

Change-Id: I17af87ea492359cd0c09dc3c62af8caa8755368a

Split out getSlotParserOutputUncached() method for the sake of profiling

Change-Id: I553dba13486982b4530d290c84dc1f53b4df6ff9

Merge "Allow and use type Language instead of string for $lang of doEditSectionLink"

Merge ".mailmap: Add a few new entries"

Merge "Drop ParserLimitReport, deprecated in 1.22"

Merge "Drop UnknownAction, deprecated in 1.19(!)"

Merge "Drop wfRunHooks, deprecated since 1.25"

Merge "Drop UserGetImplictGroups, deprecated since 1.25"

Add join conditions to ActiveUsersPager

We're (very slowly and somewhat unofficially) moving towards using join
conditions everywhere, and here they're needed to avoid errors once the
actor migration reaches the READ_NEW stage.

Bug: T204767
Change-Id: I8bfe861fac7874f8938bed9bfac3b7ec6f478238

Merge "Drop DoEditSectionLink, deprecated since 1.25"

Merge "tests: Add helper function for ini_set with automatic cleanup"

Merge "Load installer i18n when running update.php"

Merge "messagecache: avoid caching message pages that do not override"

Merge "Do not retry the ThumbnailRenderJob."

Load installer i18n when running update.php

Bug: T206765
Change-Id: Ie1c1a5dbf7c905eaca55777448e5aba00267ed31

Merge "Revert "Unwrap HTML loaded from parser cache""

.mailmap: Add a few new entries

Change-Id: Iaacd5eff9c01ae9c03cf002a5063cae78cfb9e37

Allow and use type Language instead of string for $lang of doEditSectionLink

Hard-deprecate that the parameters $tooltip and $lang are optional.
Hard-deprecate other types than Language for parameter $lang.

Change-Id: I3ce048e3d0de9a4e96e2aa05d75a4ce41c3ec964
Depends-On: Ibe295e7020e995eea52e319feaf59c635f8bb4dc

Do not retry the ThumbnailRenderJob.

The job is a warmup for the thumbnail cache, so loosing it is not
an issue. Most of the times thumbnail rendering fails because of
non-renderable or broken images, or because the thumbnail was
already created by a request. These conditions will not be fixed
by retrying, will only create more load on Thumbor.

Bug: T203135
Change-Id: Ib4640eb13ca93f7350ff0671fc559a76ebbbb1e9

MessageCache: replace should actually replace, not reload

Prior to I462554b30, MessageCache::replace() did just that: it took the
existing cache and updated the one entry.

In I462554b30, the rearrangement of work into a DeferredUpdate
introduced a bug: the in-process cache was updated, but when the shared
cache was loaded later the entry was never updated in there so the
shared caches kept the old value. This was found in code review and
worked around by reloading all the messages from the database instead of
updating the existing cache.

But all that extra work reloading everything from the database causes
major slowness saving any MediaWiki-namespace page when the wiki has
many such small pages. Let's go back and fix the bug so replace() can
again replace instead of reloading everything.

Bug: T203925
Change-Id: Ife8e1bd6f143f480eb8da09b817c85aadf33a923

Only expand `{{...}}` in messages once

If we're going to call `OutputPage::addWikiText` to parse the message,
we don't need to pre-expand `{{...}}` markup before passing it to the
parser.  This makes these callsites consistent with how
`OutputPage::addWikiMsg()` inserts messages.

Bug: T206574
Change-Id: I2f68f73365df876e6f9d017e4e3ddea39c4a1284

[Korean] Add 2 additional bookstoreList

* Aladin.co.kr has HTTPS!
* National Library of Korea has 302 http://www.nl.go.kr on
  https://www.nl.go.kr
* Naver has HTTPS.

Change-Id: I5559ae2b32cddb540bc712e4b55cfe7c26c9a0b0
Signed-off-by: Yongmin Hong <revi@pobox.com>

Merge "Drop File / MediaHandler::getStreamHeaders(), deprecated since 1.30"

Merge "Drop DeferredUpdates::setImmediateMode(), deprecated since 1.29"

Merge "Drop ChangesListSpecialPageFilters, deprecated in 1.29 and uncalled"