Avoid fatal when finding no base revision for a null revision.

Bug: T205675
Change-Id: Iae67649a1be9597086033ad34d9d00556ba35730

Phabricator: Use Tddddd instead of Bug ddddd in comments

Change-Id: Ic9fe03cab270bd6be738af346164ad5d31a0d780

Merge "rdbms: clarfiy some comments about commitMasterChanges() methods"

Merge "rdbms: clarify IDatabase::setTransactionListener() comment"

Merge "MessageCache: do not store the EXCESSIVE array as it is only needed for HASH"

rdbms: clarfiy some comments about commitMasterChanges() methods

Change-Id: I2c9f3cb9b4dcd16f561e55d119849b526e9d402b

MessageCache: do not store the EXCESSIVE array as it is only needed for HASH

This saves space as the number of entries becomes larger.

Bug: T193271
Change-Id: I32d7512d7f05ae7c9a69b88cd19df64d7a21b4a1

Merge "Fix option name in maintenance/importDump.php"

Special:Preferences: Drop isOouiEnabled() before it gets released

Depends-On: Ic1df4e965df61c2d164b2ffd2fa07ed68b028ddf
Depends-On: If0e83751089d010d026a736040d77b9a0851ef42
Depends-On: Id9c840e979b723806883bb3e63d7f2f691fea629
Depends-On: I9d5dbf449c624233cef2ccf0e3e4200d9eb2b7c3
Change-Id: If2a21a6a40802ab187b30de78fb521f016dd10cd

Merge "Special:Preferences: Drop non-OOUI legacy form version"

Merge "Drop 'SpecialRecentChangesQuery' & 'SpecialWatchlistQuery' hooks, deprecated in 1.23"

Merge "Database: close() should not commit transactions"

Make UID clock drift error have more details.

Change-Id: Ic682e48b06161c2a286f5da9016f4831ee72a31e

Move test assertion to mirror parameter order

Follow up from I2df0551c5837adc578b27082ab6ba2ac95d937f8

Bug: T206130
Change-Id: Ib669c77fdb709846d0182cb28796cf53914114c4

Merge "SECURITY: Fix permissions check for patrol action"

SECURITY: Fix permissions check for patrol action

Return existing errors instead of empty array in checkUserConfigPermissions().
Returning an empty array wiped out previously-found errors.

Also add test coverage for patrol action.

Bug: T206130
Change-Id: I2df0551c5837adc578b27082ab6ba2ac95d937f8

Database: close() should not commit transactions

Transactional databases normally roll back when a connection is closed
with an open transaction rather than committing them, so MediaWiki
committing them is unexpected.

There are two cases being changed here: automatic transactions without
writes and manual transactions. For the former it shouldn't make a
difference if we commit or roll back since no writes were done anyway.
The latter has logged a message since MW 1.31 (I0992f9a8), and that
warning has not been logged in Wikimedia production in the past 60 days
so we should be ok there too.

Bug: T206147
Change-Id: Ieceef4deb49044db8f0622d38ee76c9d9f39704c

Fix option name in maintenance/importDump.php

Bug: T200138
Change-Id: Icc5d9878bc3ebdf03efec9161b220ad0bf55dc9a

Merge "Show copyright based on $output->hasCopyright()"

Merge "Remove unused function: isStructuredFilterUiEnabledByDefault"

Show copyright based on $output->hasCopyright()

Introducing setCopyright/hasCopyright in OutputPage to
make the showing of the copyright message controllable
by pages other than articles.

hasCopyright defaults to isArticle() to preserve the rule
that all article should show the copyright.

This is used by Flow to show the copyright
on various pages that contain user-generated
content.

Bug: T184960
Change-Id: I3a50dbcedc6b119b9262c50cb3a84b0dd230fb3d

Localisation updates from https://translatewiki.net.

Change-Id: Ic0af665f26aa1af0bd706ee26bb811d0a482ec2b

Deprecate Language::setCode as public method

setCode changes the language code for the Language object but it also
replaces the whole language codes for all Language objects.

> $lang = Language::factory( 'fr' )

> $lang2 = Language::factory( 'fr' )

> $lang->setCode( 'it' )

> print $lang2->getCode()
it
> $lang3 = Language::factory( 'fr' )

> print $lang3->getCode()
it

Better assign a new Language object.

Also add more tests for Language::equals.

Depends-On: I61439bac82021344c3f9a6056cccd937b3450af2
Depends-On: I2d9e551d6eb33f28f42aeaf48160eba21b83881f
Change-Id: I201b479f58e63c9c40fb8a3ec9575a551fb35235

Merge "API: Handle empty xxnamespace parameter in ApiQueryBacklinksprop"

Merge "AutoloadGenerator: Filter PSR4-compliant classes instead of ignoring directories"

Merge "Migrate image descriptions from image_comment_temp"

Merge "RCFilters: better vertical alignment of checkbox and text in menus"

RCFilters: better vertical alignment of checkbox and text in menus

- Remove min-height so menu items are a little smaller and
  their content is vertically-aligned.

- Force `vertical-align: middle;` for checkboxes by overriding
  OOUI with extreme specificity.

Bug: T200364
Change-Id: If0b2b5c1b749f913badbd0c02a85852ec0007020

Merge "MessageCache: remove confusing and unused $isFullKey parameter from get()"

MessageCache: remove confusing and unused $isFullKey parameter from get()

Follows-up cba0fb1c1576324e87b27, which removed the last caller.

Change-Id: I00c17fedff39b1b35519cff2a0f8eac3e4d6f2ab

Language: Don't return aliases to namespaces that don't exist

On a multi-wiki server using a l10n cache, the cache will probably
include aliases from all extensions enabled for any wiki while the
namespaces themselves will only be defined on the wikis where the
extension is loaded.

Bug: T206030
Change-Id: I985dfa3eb17944d5fc0c5be0979be21504fb1341

Merge "rdbms: make * consistently act like  in select/insertSelect methods"

Merge "Add audience parameter to PoolWorkArticleView"

Migrate image descriptions from image_comment_temp

image_comment_temp was always intended to be temporary, until an
expensive schema change on Wikimedia Commons's image table could be
done. Now that that has been done, stop writing image_comment_temp and
add a migration script to copy existing data into img_description_id.

Ic8efeddc will remove the reads from image_comment_temp and drop the
image_comment_temp table.

Bug: T188132
Change-Id: Iab5f521577a415b2dc213b517ee8a0dca4fdd0aa

Merge "Use pseudo elements to style changelist-separators on Contributions"

Add audience parameter to PoolWorkArticleView

The old behavior was that the audience was RAW if the revision object
parameter got passed in, otherwise PUBLIC. This was undocumented and
not used outside core; this patch gets rid of it in favor of an
explicit argument.

Bug: T205578
Change-Id: Ic7cdb38f658f6d85c48ff13c7f84c64a45c9b1ee

Merge "Add type hint to invalidateImageDescriptions()"

Add type hint to invalidateImageDescriptions()

Change-Id: Ie6322f25c959f5de4bbccd8f47d40f112817d00b

Merge "Replace WikiExporter streaming (unbuffered) mode with batched queries"

Drop 'SpecialRecentChangesQuery' & 'SpecialWatchlistQuery' hooks, deprecated in 1.23

Depends-On: I9822bb2bc1bcc13eec7924208bf3819d3512ad26
Change-Id: I67c06bb99d9c9582a638f061f523661c821bca8d

Merge "ApiOptions: fix resetting some preferences to default"

Merge "Use varargs in global functions"

Special:Preferences: Drop non-OOUI legacy form version

Change-Id: I65b89385c3ec28ef01b86dd933dae3801e503631

Remove unused function: isStructuredFilterUiEnabledByDefault

Follow up Id3799fefd21cd9bea0e089a5e12576ee9ea1085e

Bug: T196033
Change-Id: Ib2ad41be8648f7ff8b4a218fd66c328dee3924d9

CategoryTagItemWidget: Drop backwards-compatibility "Capsule" name

Not used for months.

Change-Id: Ie85d2d0ef6ebc3e1e2502519a02b8fadd8d53b7a

Use pseudo elements to style changelist-separators on Contributions

Currently, these separators are unnecessarily hardcoded in HTML.
This is problematic, as it limits the extent to which skins such
as Minerva and Timeless can skin these pages.

If the pseudo selector is not available (e.g. legacy browsers)
this degrades nicely to whitespace and the page is still useable.

Right now, these changes are scoped to pages uses the ContribsPager
(used by Special:Contributions)
but we can adopt them in other pages once we have this approach set in
stone (CSS)

Bug: T205581
Change-Id: Ia2c485e6058ebf4282ed2f7eb7b046b2deb03f57

Hard-deprecate password handling in User

Add wfDeprecated for User::checkPassword(), User::setPassword(),
User::setInternalPassword() and User::checkTemporaryPassword().
With AuthManager mediating between the caller and a set of
authentication providers in a potentially multi-step process,
a password check or change now cannot always be expressed
with a single-step succed-or-fail process. Callers should use
AuthManager::beginAuthentication() with a PasswordAuthenticationRequest
for password checks, and AuthManager::changeAuthenticationData()
for a password change.

Bug: T91699
Change-Id: Ib0ae8f1ff10ae6c2655d529db8b3a32e0cb489b0

Merge "Remove temporary rcfilters variables"

ApiOptions: fix resetting some preferences to default

For preferences like 'skin' that have a limited number of values, null
is not a valid value, thus attempts to reset them fail with
"Validation error for \"skin\": This value is required."

Bug: T65080

Change-Id: I86554a6d30c8ab970740d8682fb2261476de0677

rdbms: make * consistently act like  in select/insertSelect methods

This now matches the documentation of insertSelect()

Bug: T202553
Change-Id: Ie6602fdd3b48b9136de7c65289c85ced5f5f2f1d

Merge "Remove __METHOD__ in ServiceWiring return array"

rdbms: clarify IDatabase::setTransactionListener() comment

Change-Id: I6f494934b3602206943581c3c62f565bf42bd26f

Fix caller name in JobRunner::commitMasterChanges

Use the given fname for all places.
The __METHOD__ inside the unlock closure would be shown as {closure} in
logs

Change-Id: I87ef26e893af858f58d1a77dcb2d8ee192456f5c

Merge "Preserve whitespace in search index text content"

Merge "Fix [[MediaWiki:Config-install-success/en]] translation issues"

Merge "Make messages [[MediaWiki:Config-db-web-account-same/qqq]] and [[MediaWiki:Config-db-web-create]] clearer"

Merge "rdbms: Document a bunch of stuff about query verbs"

API: Handle empty xxnamespace parameter in ApiQueryBacklinksprop

If the xxnamespace parameter is specified but empty, we don't need to do
anything because there will be no results.

Bug: T205810
Change-Id: Iae5d766a5408368cadd01583d18b745661304a7f

Merge "Add waitForReplication in DeleteLocalPasswords"

Add waitForReplication in DeleteLocalPasswords

It almost brought commonswiki down when it was run

Bug: T201009
Change-Id: Ia825f9572b8c71c5627eb627c58f51a689c2f8aa

Use Unicode notation (U+XXXX) instead of HTML notation (&#xXXXX;) in comments

Performed using:
sed -i 's/&#x\([0-9a-f]*\);/U+\U\1/gi' languages/messages/* languages/classes/*

Change-Id: Id489ae5ffb10bc5b047f72bc6834dd0488b5f940

Merge "Change the Unicode numbers in the comments in MessagesPs"

Change the Unicode numbers in the comments in MessagesPs

The numbers are supposed to be the same as in MessagesFa.php.

Change-Id: Icd3e8c8ac1d5beb194dcbce36100a373816bc506

Update CoC file in Core

Update CoC file in core to have the "Special:MyLanguage" so
it links to the user's preferred language CoC page.

Bug: T202047
Change-Id: Iacecf59c9f3b3df171233aeb7bdde1bc1e9444ae

Remove __METHOD__ in ServiceWiring return array

It is in a callback and would provide only {closures}.
It is logged to a channel, where the context is known.

Change-Id: I15ee43ff9383970fc6d5b5cc45678397d2096604

Merge "HTMLInfoField: Support 'rawrow' in OOUI mode"

Merge "Create UserGetRightsRemove hook"

Fix logged method in LoadBalancer

Seeing {closure} in the logs as caller is not helpful

Change-Id: I45d9be25acdcc70c7d9755c15f68f7bc6887e166

Fix caller name in DatabasePostgresTest

Seeing {closure} in the logs as caller is not helpful
Also add some missing caller name, which avoids Database::query as
caller in the logs
Change-Id: I3cab186cd63b295d1e4d1ca5232bc81925a85eb0

Fix caller name in DatabaseMysqlBase::getMasterServerInfo/getServerId

Seeing {closure} in the logs as caller is not helpful

Change-Id: I63adbfcf4944747670a3a30182f6bd5cb6d0e48b

Fix logged method in MessageCache::replace/loadCachedMessagePageEntry

Seeing {closure} in the logs as caller is not helpful

Change-Id: Iddfb6042bba2a59a9f21f533b748c96478a87b97

Fix logged method in NameTableStore::getName

Seeing {closure} in the logs as caller is not helpful

Change-Id: I6a54ebf87024706b7890c7fe003b5380933fe861

Fix caller name in PurgeJobUtils::invalidatePages

Seeing {closure} in the logs as caller is not helpful

Change-Id: I252b23e46983b359cbad32323c3a063e7a7afcb5

Fix caller name in LocalFile::recordUpload2

Seeing {closure} in the logs as caller is not helpful

Change-Id: Idbf6ac9606d0f75cae452afffe133c5e3973f334

Fix caller name in File::getDescriptionText

Seeing {closure} in the logs as caller is not helpful

Change-Id: I415e2bb691650bc34adfb750c7a70a5d29176849

Fix caller name in LocalPasswordPrimaryAuthenticationProvider

Seeing {closure} in the logs as caller is not helpful

Change-Id: I8dc921feb13432ed636686f918d8747ae71cf62b

Fix caller name in ApiStashEdit::parseAndStash

Seeing {closure} in the logs as caller is not helpful

Change-Id: Id77253bdd4b65b0673214e7bf6ca3cc26bd155fa

Fix caller name in SiteStats::numberingroup/pagesInNs

Seeing {closure} in the logs as caller is not helpful

Change-Id: I0102cddc447257319fe8db7f8cbd837838653e99

Fix caller name in PopulateArchiveRevId::checkMysqlAutoIncrementBug

Seeing {closure} in the logs as caller is not helpful

Change-Id: Ie9091805bb4d9c05de93aef065cbbfef56c19e70

Fix caller name in User::addToDatabase

Seeing {closure} in the logs as caller is not helpful

Change-Id: I876877046ae4bd1756c13e04892c1381904566de

Fix caller name in WatchedItemStore::updateNotificationTimestamp

Seeing {closure} in the logs as caller is not helpful

Change-Id: I13a6632523d0e7e96adf3c9af89c71f38d5342f5

Fix caller name in ClassicInterwikiLookup::load

Seeing MediaWiki\Interwiki\{closure} in the logs as caller is not
helpful

Change-Id: I39af68a02d40667e428c7f30c88aacab8a02cd86

Fix [[MediaWiki:Config-install-success/en]] translation issues

The first sentence in this message should not be splited in two lines.
Too, content of page for FAQ, should be displayed on the user's language.

Bug: T205753
Change-Id: I7b75485411f6b6022bdc773521fbad1366dd8643

Deprecate class LegacyFormatter

No uses anywhere, was used by the now removed function wfErrorLog().

Change-Id: Ib4561157e776c64ab17f014fbd8603b0934acb2c

Merge "skins: Remove 'usemsgcache' and deprecate getDynamicStylesheetQuery"

Merge "Document some understanding of MessageCache in RawAction/EditPage"

Merge "Drop wfErrorLog, deprecated in 1.25"

Merge "user: Remove use of Message:text() from User::isUsableName()"

Drop wfErrorLog, deprecated in 1.25

Change-Id: I29131f3c677ab5747ac0ae5121669af63a1d5bd8

skins: Remove 'usemsgcache' and deprecate getDynamicStylesheetQuery

For several years now, action=raw urls are automatically purged
after edits, and receive the same cache allowance as page views.

The vast majority of action=raw requests come from the
importScript() function in wikibits.js, which only specififes
'title', 'action' and 'ctype'.

The 'usemsgcache' parameter is a relic (pre-2009) for optimising
something we haven't used in a very long time.  Nothing uses the
'usemsgcache' parameter, except for the one edge-case of
third-party wikis loading Filepage.css from Commons on local
file description pages. This commit removes the handling logic,
in favour of letting the normal Revision-logic happen instead,
which is compatible and produces the same response.

Before 2009, this probably made sense as an optimisation because
we didn't have Revision text cache in Memcached yet at that time,
to avoid DB-load from InstantCommons-based file page views.

The way FileRepo was using it needlessly allowed Filepage.css
to return stale responses (given non-canonical query parameters
variations are not purged on-edit). It also caused responses to
be cached longer or shorter than they should be, due to it being
based on the client wiki's configuration, instead of the repo's
configuration (e.g. Wikimedia Commons). By omitting these
'maxage' parameters, the $wgSquidMaxage config of the repo is
used instead, which makes more sense.

Bug: T193271
Change-Id: Ie52195b56b7f8fc17dec0cbedd6d77968e3bc040

Document some understanding of MessageCache in RawAction/EditPage

Bug: T193271
Change-Id: I8d110dd7b84faf6b1b64553e8088185de5aac7f2

user: Remove use of Message:text() from User::isUsableName()

None of the current translations use of "&", "{" or "[", which
presumably means they don't require tranformation. I'd also
very much prefer such dynamic constructs not be added into
this system which would make it rather unpredictable given these
reserved usernames shouldn't change from time to time.

The only match for the below command is qqq.json,

> $ git grep -E '"(double-redirect-fixer|usermessage-editor|\
> proxyblocker|sorbs|spambot.username|autochange-username)"\
> ' | grep -E '[{&[]'

This code was originally introduced with r37928 (5ad5cb4f0a64),
which used wfMsgForContent() and made no explicit mention of
transformation, it just happened to be the default. Later, when
that function was deprecated, it got batch-replaced by
->inContentLanguage()->text().

Bug: T189966
Change-Id: Ia4ddf215e83f15de552f8311b9e737559c72b49a

Don't use deprecated function wfArrayFilterByKey()

Change-Id: Ifdc67f8b9b8617e6398aa0d2b6ff4bec1e7507ce

rdbms: Document a bunch of stuff about query verbs

The decision to treat COMMIT/BEGIN as a "read" in isWriteQuery()
for the benefit of ChronologyProtector was first introduced
in r47360 (8653947b, 2009).

* Re-order strings in isTransactableQuery() to match the regular
  expression in isWriteQuery() for quicker mental comparison

* Add missing visibility to DatabaseSqlite->isWriteQuery, matching
  the parent class implementation.

Bug: T201900
Change-Id: Ic90f6455a2e696ba9428ad5835d0f4be6a0d9a5c

Merge "exception: Add FIXME about code duplication"

Replace deprecated use of $wgUseTidy in Special:ExpandTemplates

This use managed to escape my notice in
I2e7d990da1da378eb6e828d4b3c0f5a41791dd92.  Note that $wgUseTidy
has actually been false in production since
Idf246d05d116f63a73105b50a1929a7721fbe7b9.

Bug: T175706
Change-Id: I684be210c6efde5b5eb4dca163d45b2701df7555

exception: Add FIXME about code duplication

These methods aren't identical, so consolidation isn't
immediately obvious, and creating dependencies is problematic
in error handling code given there is a lot of pressure on this
code to not by itself cause additional errors.

This means that maybe it's best to keep these inlined without
duplication, but at the very least we then need to remember
to keep these in sync. This duplication has been around for
a while now, but documentation can never come too late...

Change-Id: I60160f61c13c8e115d839acce222f110e30bc2f2

Make messages [[MediaWiki:Config-db-web-account-same/qqq]] and [[MediaWiki:Config-db-web-create]] clearer

Bug: T205745
Bug: T205749
Change-Id: I511385b6bf64bb7326ac04a07592657a96af9bf3

Only strip newline in OrderedStreamingForkController

The controller should pass through lines of input exactly as
they were provided, only stripping the trailing newline that
delimits items. The trim was making `door` and `door ` equivilant
but for the use case in search the distinction is important.

Additionally check that the line is actually empty, don't throw
away inputs like '0' which are falsy.

Change-Id: Ifac910543fdb46a27da021e831e3e18befefcfc5

Merge "Handle empty revision table in populateArchiveRevId.php"