MailAddress->toString(): Reduce complexity by inverting ifs

Change-Id: I5185c0231a0944f36d960211e78cf36273c94d52

Merge "Don't pass a MailAddress pass the email to mail()"

Merge "Fix typo in includes/watcheditem/WatchItemStoreInterface.php"

Localisation updates from https://translatewiki.net.

Change-Id: I508681048d303f4814be18462689688b93443813

Fix typo in includes/watcheditem/WatchItemStoreInterface.php

Comment didn't quite make sense, fixing the typo by removing "the"
made it a little better and clear.

Change-Id: I33abdb3489ab704d50722a8df6acb402592908b2

Don't pass a MailAddress pass the email to mail()

Bug: T207541
Change-Id: I1516023907e9773cb093010c6b67279f695abb1a

Include IP address in "Login for $1 succeeded" log entry

Bug: T207540
Change-Id: Iab4f2f2ddc8e64ead2f33356d03fa7beed399415

Merge "Convert Special:EmailUser to use OOUIHTMLForm"

Merge "RCFilters: Fix highlight circles for unseen changes"

Merge "Deprecate $wgSquidPurgeUseHostHeader"

Merge "Deprecate $wgUseESI"

Merge "objectcache: avoid using heavily time-drifted microtime() values in WAN cache"

Merge "During special page transclusion, save and restore context's WikiPage too"

RCFilters: Fix highlight circles for unseen changes

These are supposed to be unfilled, but the CSS wasn't being applied
because it didn't have -color in the class name. Put it back into the
&-color block where it belongs.

Bug: T207472
Change-Id: Ic517b9a1a0e8e95015f5b2bd3517c36bff3b20e8
Follows-Up: I76fec57ff237ba02afefcea28916d8348bac9b0e

During special page transclusion, save and restore context's WikiPage too

Setting the Title by calling setTitle clears the WikiPage, and the
next time getWikiPage() is called, it will be lazy-initialized to a
different instance of WikiPage.

This is mostly okay (the behavior has been like this for years and no
one noticed any problems), but it turns out that some extensions
(ConfirmEdit) use custom properties on the WikiPage object to pass
data between different hooks, which are lost when it's re-initialized.

Bug: T207065
Change-Id: I2881895f337bcfb1f86d5fc5a994fa9b0dcc768a

Localisation updates from https://translatewiki.net.

Change-Id: Ic7961893e9cd4f67fdfdeb36c00ce414444c3f15

RCFilters: Fix completely broken highlight circles

- Correct highlights class name (mw-rcfilters-ui-highlights, not
  mw-rcfilters-ui-changesListWrapperWidget-highlights)
- Remove -circle class suffix that the circles don't actually have
- Go back to using float: right; instead of text-align: right;
  (you can't use text-align for things that aren't inline)
- For seen/unseen watched items, apply circle mixin to the actual
  circles themselves, not their parents

Bug: T207472
Change-Id: Iade1af4299826a4e8a2108425ace9a406f0aa8cb
Follows-Up: I76fec57ff237ba02afefcea28916d8348bac9b0e

Deprecate $wgSquidPurgeUseHostHeader

This header supports Squid in forward-proxy mode using HTTP/1.0
HTTP headers.  It is not used in production.

Change-Id: I99646c9c5519bd55b3d4988306e379f89d413bdc

Deprecate $wgUseESI

This was an experimental Squid 3 feature which is now defunct.

Change-Id: I799d87309b0dc77100900b75040fa9b85a35fc9a

Merge "Watchlist: Commit after each batch watchlist insertion and removal"

Merge "Add docs and minor cleanup in PHPVersionCheck.php"

Add COALESCE for image deletion and imgcomment_description_id

I have no idea why this suddenly started raising an error rather than it
doing so since February (I0dd7258fe). But this should fix it.

Bug: T207419
Change-Id: Id97e1c7c2655d90928c777bc3377e5ea23f49f6b

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

Merge "Add ICU mapping for versions 62 and 63"

Localisation updates from https://translatewiki.net.

Change-Id: Idb8c9ff1533aa8470455dba149333a6212c509a5

Merge "Sort allowed log actions when displaying"

Add ICU mapping for versions 62 and 63

Change-Id: I5e1238e856d4149c30806e6b2cb3619c0c9c1dbf

objectcache: avoid using heavily time-drifted microtime() values in WAN cache

Change-Id: Ifcb9e4b4a1d5adfdaaa6ea505e34956516b871de

Watchlist: Commit after each batch watchlist insertion and removal

With this change, adding large numbers (500+) of items to one's Watchlist via
Special:EditWatchlist/raw will no longer trigger DBPerformance warnings
for "max affected rows expectation not met". The same code mechanism is added
for bulk removal of watchlist items.

Bug: T171898
Depends-On: Ia0f496b8bfb2b68217d0f45f892045538494bfdc
Change-Id: I832b1843d1341b05227cdee2549bdcefa21eb300

Add docs and minor cleanup in PHPVersionCheck.php

- Consistency of the acronym PHP instead of "php".
- Add documentation to $functionsExtensionsMapping.

Change-Id: I06b507128bc3b95605a764cd63f621dbae3b2dc3

Merge "Fix jqueryMsg breaking when passed an undefined param"

Merge "Revert "Introduce Special:RedirectExternal""

Fix jqueryMsg breaking when passed an undefined param

This restores the behavior for undefined params as it was before
I445f9194bb8b2ed35baafbda30d1d0d008b64e2c

Bug: T207397
Change-Id: I751f01a9b3cd8bb87d7d7def98c918a87ddf81cd

Sort allowed log actions when displaying

This makes searching in the dropdown on Special:ApiSandbox simpler
because it keeps actions of the same log type together.

Change-Id: I240f5e19938393d5c15a1f5e7dd27780957231c0

Revert "Introduce Special:RedirectExternal"

This reverts commit b83cf88837d589ae82a41fffb68dd82855aff2e5.

Reason for revert: Security concerns, needs some more nuance.

Change-Id: I5823c3da77accca9839f9632611315223e05351b

Merge "ImagePage: Inherit parent's handling for action=render"

Merge "Reword 'prefixindex-strip' to "Hide the prefix in results""

ImagePage: Inherit parent's handling for action=render

ImagePage::render() was calling parent::view() instead of
parent::render(), thus skipping Article::render() entirely.
Therefore the logic to disable section edit links (and also,
to add an 'X-Robots-Tag: noindex' header) was not being used.
This fixes T65891 and T21415 for pages in 'File:' namespace.

Bug: T206546
Change-Id: I36ae716c9a363ae29b7a785cc41430301250baba

New hook ApiOptions

We need it to inform API clients they're changing globally overridden
preferences.

Bug: T198913
Change-Id: I692a1fb42b73d72814f31bfe9bd400c65f59d83c

Reword 'prefixindex-strip' to "Hide the prefix in results"

"Strip" is a very jargony term for removing something.

Translations of this message to the languages I can vaguely understand
all use the equivalents of "hide", "remove", or "display without".
Several translations to small languages I don't understand include the
word "strip" verbatim, which is almost certainly a mistake.

Bug: T76972
Change-Id: I6bf690773bfb2e5c6c35b3101733d1cd3e936e70

Database: Allow selectFieldValues() to accept SQL fragments

The documentation says "This must be a valid SQL fragment", but as
written it breaks if given anything other than a field name. It's easy
enough to fix by adding an alias to the internal select() call.

Bug: T201781
Change-Id: I76428af6d3aadc266254fdb24109a0ac2db3761f

Merge "Introduce Special:RedirectExternal"

Merge "mw.feedback: Remove a hack with passing message parameters"

Introduce Special:RedirectExternal

Special:RedirectExternal is an unlisted special page that accepts a URL as
the first argument, and redirects the user to that page.
Example: Special:RedirectExternal/https://mediawiki.org

At the moment, this is intended to be used by the GrowthExperiments project in
order to track outbound visits to certain external links. But it could be
extended in the future to provide parameters for showing a message to the user
before redirecting, or explicitly requiring a user to click on the link, which
could help improve security when users follow on-wiki links to off-wiki sites.

Bug: T207115
Change-Id: I822af14a84569aab22249e2f16a662a60e60f76a

Localisation updates from https://translatewiki.net.

Change-Id: If357bae8816570dbccf5bbb42b040ad6de159e72

mw.feedback: Remove a hack with passing message parameters

I'm not sure what bug this is trying to work around. It mentions
T49395 but that doesn't look very relevant. Maybe the real problem was
confusion and bugs related to passing "raw params" (T33260#4675605).
Regardless, it's no longer needed now, because this code also works.

Change-Id: I580d46a5e4b7ff58989cb5fd6b7770e781fb5319

Merge "jqueryMsg: Don't fall back to simple parser when jQuery params passed"

Merge "rdbms: do not treat SAVEPOINT and RELEASE SAVEPOINT as write queries"

Merge "Allow getRevisionText to function without the text table."

Allow getRevisionText to function without the text table.

Without this patch, getRevisionText would fail silently (by
returning false) when the text table no longer gets joined,
due to the switch to the new MCR schema.

Bug: T205808
Change-Id: Iffc25c82a5d2b865c28070c76156d39d390cc675

Merge "Inject Config into Parser instead of using globals"

Merge "Force type Language for $lang of doEditSectionLink"

Merge "Force type Language for $lang of DifferenceEngine::setTextLanguage"

rdbms: do not treat SAVEPOINT and RELEASE SAVEPOINT as write queries

Change-Id: Ib5d15d90260f30d08f3641a7fb36d82d363b012c

Merge "Add OutputPage::wrapWikiTextAsInterface() to safely wrap wikitext"

Force type Language for $lang of doEditSectionLink

Also make the parameter $tooltip mandatory to ensure that a mandatory
parameter does follow an optional parameter. ParserOutput always fill
all parameters of doEditSectionLink.

The call of wfGetLangObj( $lang ) is not necessary anymore.

This change is a follow-up to Id49201957e716ef8a2ea930b3616ca4fe6e35633.

Change-Id: Ic49d76c7d9357f99b462471696f7e1eb86e70e3d

Force type Language for $lang of DifferenceEngine::setTextLanguage

Change-Id: I77e7116f81072fa42f580552e7b94323cfe31856

Add OutputPage::wrapWikiTextAsInterface() to safely wrap wikitext

This patch introduces a new method,
OutputPage::wrapWikiTextAsInterface(), which wraps the result of
OutputPage::addWikiTextAsInterface() in a tidy and robust way that
won't break if the wrapped message contains double newlines, extra
</div>, or other nasties.

This replaces a common unsafe pattern:
  $output->addWikiText( '<div class="..."' .... '</div>' );

Bug: T205624
Change-Id: I1040c7cf0ec1f5c4bef7c06d4486f50d85f2dc0f

Merge "INSTALL, UPGRADE: Re-wrap to 80 chars and enforce"

Merge "RELEASE-NOTES-1.32: Add entries for all changes to DefaultSettings.php"

INSTALL, UPGRADE: Re-wrap to 80 chars and enforce

Change-Id: Idabcc4db7a0b25448e7ad30641763e2b3e1a3160

Merge "Prepare for REL1_32 cut, labelling master as 1.33-alpha"

Merge "Follow-up 3481e3b2: Drop disablability of ExpiryInputWidget before 1.32 is cut"

Change to “LESS” as it's the official name (again)

Make it unified with comments below.

Change-Id: I05e7a4269ed88a497a0072c5838a99fd15167f5d

Merge "rdbms: re-add DB domain sanity checks to LoadBalancer"

rdbms: re-add DB domain sanity checks to LoadBalancer

Also clean up empty schema handling in DatabaseDomain

This reverts commit f23ac02f4fcf156767df66a5df2fa407310fe1d2.

Bug: T193565
Change-Id: I95fde5c069f180ca888a023fade25ec81b846d44

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()"

Follow-up 3481e3b2: Drop disablability of ExpiryInputWidget before 1.32 is cut

No longer used anywhere(?); we'd rather not have to explain the temporary
variable in the MediaWiki 1.32.0 release notes if we can instead just not ship
it.

Bug: T192620
Change-Id: Icfb82f228512ed45f1a27ce3e565fbc5fc09f39c

jqueryMsg: Don't fall back to simple parser when jQuery params passed

jqueryMsg parsing is required when the message body contains wikitext,
but it is also required when one of the parameters is a jQuery object.

Without this change, the following doesn't work:
mw.messages.set( 'foo', 'Hello $1' );
mw.message( 'foo', $( '<a>' ).text( 'World' ) ).parse();

(But if the message contained e.g. {{PLURAL:}} syntax, it did work.)

Change-Id: I445f9194bb8b2ed35baafbda30d1d0d008b64e2c

RELEASE-NOTES-1.32: Add entries for all changes to DefaultSettings.php

Change-Id: Iae1122ef69940ca208d30dcd9554af8b7f60ac74

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"

Convert Special:EmailUser to use OOUIHTMLForm

It already used HTMLForm for the main form, but the form, which asked for
the username didn't.

Converted the "Enter username" form to use HTMLForm and show it, if no user-
name was passed to the form, show the main "Send e-mail" form, instead.

Extra points:
 - Let HTMLForm::setSubmit*() function return it's own HTMLForm instance

This basically is a squashed version of these changes:
I6231577047c93781496e0f8af6809e2ef49e662a
I3e0c02155428ae400bc3a6d3ed2e66e69ee441fa

Bug: T117791
Change-Id: If8bf42b2bd092706f2e580083b2400121d35c41c

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)"

Prepare for REL1_32 cut, labelling master as 1.33-alpha

Change-Id: I41c19a63d20dc4e734430ad4fe989fa5a9e18d62

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