Fixed bug where catching DB errors left ignoreErrors() on

* Also fixed an actual rollback loop possible in postgres

Change-Id: I41508127f74e1bbee4c020546fed85ab53318ab7

Merge "Report correct rev_id in missing-revision message"

Merge "convertExtensionToRegistration: Still convert $wgTrackingCategories"

Merge "mergeMessageFileList: Support reading extension/skin.json files"

Merge "installer: Read from extension.json if possible"

Merge "LocalFileTest does not require database access"

Merge "SearchUpdateTest does not require database access"

Merge "registration: Make it easier for other code to get extension metadata"

Merge "registration: Support 'namemsg'"

Merge "registration: Support ResourceModuleSkinStyles"

Merge "Add special pages aliases for Western Balochi (bgn) from translatewiki"

Merge "Fix description for MessagesBgn.php"

Merge "Added CAS-style logic to User::saveSettings as a final sanity check"

registration: Support 'namemsg'

Change-Id: I0f446b31c1d36a492997ef31fae6d2d2bea36517

registration: Support ResourceModuleSkinStyles

Structurally, it's set up exactly the same as ResourceModules, so we can
re-use the same code.

Bug: T91566
Change-Id: I7cd1330edc3e97b4658c81aac67071a90ac61fb6

Merge "resourceloader: Require $wgResourceModuleSkinStyles paths to be in the base definition"

Report correct rev_id in missing-revision message

When trying to fetch the current revision, it currently always reports
"0" even when it's trying to fetch some other revision.

Bug: T92046
Change-Id: Ia5eb73ac32de0f654ac28ee929d5c4dda71c2f1b

Merge "Make {{int:emailuserfooter}} use {{int:emailpage}} in the quotes"

Merge "Made Block::insert handle expired rows"

Merge "Removed "page-lastedit" cache key trick from WikiPage"

Merge "Log when User::saveSettings is called in read-only mode"

Merge "Made replaceSectionContent try to use DB_SLAVE"

Merge "Avoid master queries on SpecialBlockList"

Merge "Avoid DB_MASTER in replaceSectionAtRev()"

Merge "Made getWatchlistInfo use DB_SLAVE"

Removed "page-lastedit" cache key trick from WikiPage

* Just rely on chronology protected and edit conflict handling.
  The time a user spends looking at and editing pages is larger
  than any normal slave lag anyway.
* However make sure that pages just made in the request are visible.
* In "master" datacenters, the slave lag will low anyway, and
  callers make use of $flags when needed. In other datacenters,
  the cache will itself be subject to lag anyway.
* Logging (DBPerformance log) shows this case is very rarely
  hit anyway.

Change-Id: If34d67c02f9a7bf0a506ee8f3990697eb403a710

Merge "Output converted namespace name in nstab when nstab-* is not defined"

Merge "Fix list=logevents for new revision delete log entries"

Allow setting anonnotice to "" to prevent fallback to sitenotice

Previously we only used Message::isDisabled (non-existent, empty string,
or "-") to decide whether to show a notice. If 'anonnotice' was disabled
'sitenotice' was shown instead.

When addressing logged-in users only, wikis typically use 'sitenotice'
and then put invisible content in 'anonnotice' (so that it shadows the
'sitenotice'). Now that the DismissableSiteNotice extension supports
closing of the notice for anonymous users (as of I87df3301c), this
becomes problematic as it has no way of knowing the notice was meant to
be invisible (and thus renders bogus "[close]" to all users).

This supersedes hacks such as <p></p> or <span></span>. Instead, the
empty string can now be used to have no anonnotice content, but also
don't show the 'sitenotice'.

Disabling with '-' (default) can still used to fallback to 'sitenotice'.

Also changed the code to, for consistency, allow hiding $wgSiteNotice
for all users by setting the sitenotice message to "-".

Bug: T94536
Change-Id: I11b8b883d480d0e07d8b395dd92360cb15de7c5b

Merge "Fix param order of a Title::newFromText call in MediaWiki"

Merge "MessagePoster followup: Dependency and docs"

Fix param order of a Title::newFromText call in MediaWiki

Otherwise it throws an exception on the latest master (I2b36b7a3)

I'm kind of surprised this didn't break when it was introduced in I60f44a10

Change-Id: I62f12aa9267d0812c3689064f512298187ac3c84

MessagePoster followup: Dependency and docs

* WikitextMessagePoster depends on mediawiki.messagePoster
* Tweak how promises are documented

Change-Id: Ie48bde9d5995e86f22744d6b6f3773d125ccb1b0

Made replaceSectionContent try to use DB_SLAVE

Bug: T92357
Change-Id: I1626280c863a46d49215cc384e207458b2f8ec39

Avoid DB_MASTER in replaceSectionAtRev()

Bug: T92357
Change-Id: Iac4511176b5c70e9deaf71302bc1866b8b1a241a

resourceloader: Require $wgResourceModuleSkinStyles paths to be in the base definition

No skin in gerrit uses the feature of per-module local or remote paths,
and not supporting it will make implementing extension/skin.json support
easier.

Change-Id: I22863ef29426dbacd65439a12aeae56d4d757951

Avoid master queries on SpecialBlockList

* Filter out expired rows and rely on the insert() pruning
  to keep the table size reasonable.

Bug: T92357
Change-Id: Icdbd606979d3d9ce9b2d923f574447e3e7dd72f7

Merge "Add pluggable talk page poster and use it for mediawiki.feedback"

Made Block::insert handle expired rows

Bug: T90780
Change-Id: Ife9bcc0ec329156dac0bd033de35b25430234764

Made getWatchlistInfo use DB_SLAVE

Bug: T92357
Change-Id: I3b7d5d20d3028e15651dbca453a8bc47010b8194

Add pluggable talk page poster and use it for mediawiki.feedback

The core implementation will only support wikitext.
Flow will add its own implementation, and it can be used for any talk
page system identifiable by content model.

Bug: T91805
Change-Id: Ic69acafb24aa737536fe3a074e1958690732f0a7

Merge "Some cleanups to MWExceptionHandler::handleException"

Separate Monolog shims from other logging shims

Move the non-namespaced classes referencing external Monolog classes to
an isolated PHP source file so that they aren't brought into scope by
the need to load another shim class.

Bug: T95220
Change-Id: I8270b8d5cd25db5a0f84fa94f59a6555052ae1ae

Merge "Fix ApiQuerySiteInfo with siprop=rightsinfo to not die if your wgRightsPage is null"

Localisation updates from https://translatewiki.net.

Change-Id: I61ce392554123f4e0ff300bed2e289cf02dd4cf6

Added CAS-style logic to User::saveSettings as a final sanity check

* This should prevent lag or race conditions from rolling back data

Change-Id: I5e70975f4e4010fea7af0801bc11dda887df55f4

Merge "Cut down on deadlocks in invalidatePages()"

Merge "Made User::newTouchedTimestamp handle clock skew a bit better"

Merge "Update special pages aliases for Luri (lrc) from translatewiki"

Log when User::saveSettings is called in read-only mode

Change-Id: I56a1760f2e6bbda889590777b553a8dfda587590

Merge "Update special pages aliases for Persian (fa) from translatewiki"

Made User::newTouchedTimestamp handle clock skew a bit better

* This does not handle race conditions, but is a prelude to using CAS
  style logic on save using the timestamp.

Change-Id: I9c31c272fcf77b686764b7c3a6a32ac29576347c

Merge "Catch rollback exceptions in API exception handling"

Fix ApiQuerySiteInfo with siprop=rightsinfo to not die if your wgRightsPage is null

Follows-up I2b36b7a3

Change-Id: I7d76cd48021a0097ce5e11f4ad4100cb85ee5ad0

Catch rollback exceptions in API exception handling

Like I92293b22, but for the API. Needed for I41508127f fixing ignore
handling in the DB.

Change-Id: I1f2b70c80c7496f463f678b950c08de22476ea66

Merge "Parser: Say tildes instead of ~~~ in comment to fix Doxygen fatal"

Track request method in dbperformance.log

This will allow us to avoid needlessly investigating master requests that
only happen on POST (which is already what we want for multi-DC).

Bug: T92357
Change-Id: Ia7437d00f5b89a8e318d85659d60e2f9f9f26149

Merge "Add a breadcrumb to EditPage refactor"

Parser: Say tildes instead of ~~~ in comment to fix Doxygen fatal

Doxygen was unable to parse the file past validateSig().

> Parser.php:6397: warning: reached end of file while inside a ~~~ block!
> The command that should end the block seems to be missing!

Change-Id: I3d1b547968302611d2bd78a7c11dd0738b40d23a

mergeMessageFileList: Support reading extension/skin.json files

No more hacky reading of PHP files!

Bug: T94756
Change-Id: I0a92f8904bbc422d5c893f7c0af2daedf7576cf0

Merge "ResourceLoaderImage: Allow shorthand syntax"

Merge "resourceloader: Omit empty parameters from mw.loader.implement calls"

Merge "objectcache: Add @covers for BagOStuffTest"

Fix description for MessagesBgn.php

Add author names which I forgot

Change-Id: I18358f9e90276bf9db5eb03baf7813c72a3a2736

Update special pages aliases for Luri (lrc) from translatewiki

Change-Id: I17d71f47f8b7b7710261ab3c0a50bb8c45583b9d

Add special pages aliases for Western Balochi (bgn) from translatewiki

Change-Id: Iadc088653b1b0e5188ec0f106d278448edbcf17e

Localisation updates from https://translatewiki.net.

Change-Id: I13e36a6ca3c35959e1e5912ac27193cc13d9c74f

Merge "Mock error event firing in mw.loader test"

objectcache: Add @covers for BagOStuffTest

Change-Id: I93a8074ba79b5ea66a984edabe009cc828e6fc4d

Merge "DefaultSettings: Remove "~~~~" from comment to fix Doxygen parse error"

Add a breadcrumb to EditPage refactor

Change-Id: I4cc398e712f499fdf35ac9384912e68ac8e9010b

Html::srcSet: allow density to be specified either with or without trailing 'x'

$wgLogoHD is meant to contain high-density alternatives for $wgLogo, but its
keys include the trailing 'x' (e.g., '1.5x'), making it unusable with
Html::srcSet(). Fix that by normalizing all density values to have a
single trailing 'x'.

Change-Id: I62cc3a9e4aeff3a7cb102de2965b8b40fd106c37

Localisation updates from https://translatewiki.net.

Change-Id: Ie48757c02285cbc60158862e2b73be9fe0d8b141

Output converted namespace name in nstab when nstab-* is not defined

In languages without converter implemented, FakeConverter just invokes
Language::getFormattedNsText(), which is the function used originally.

Change-Id: Ieb34a65776cf69bcd67f1ed8402eb3d38ece0f0c

Merge "Verify parameter for MapCacheLRU::has() can be passed to array_key_exists()"

Update OOjs UI to v0.9.7

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.9.7/History.md

Change-Id: I6548deccf1bce60873ed16229905cfa5790ec4a0

Make {{int:emailuserfooter}} use {{int:emailpage}} in the quotes

Change-Id: Iacbae715e24f783a4483dc739a6d88c6b24e53b8

jquery.suggestions: Convert documentation to JSDuck format

Change-Id: I39aba7f3f0c9d397f26934446e3a5ef686d84d86

Merge "jquery.suggestions: Document the callback context and parameters"

Move MWLogger classes to MediaWiki\Logger namespace

Move the MWLogger PSR-3 logging related classes into the
MediaWiki\Logger namespace. Create shim classes to ease migration of
existing MWLoggerFactory usage to the namespaced classes.

Bug: T93406
Change-Id: I359cc81fbd2dcf8937742311dcc7d3dee08747b0

Merge "PHPCS lint fixes"

PHPCS lint fixes

Change-Id: I16288db03f34439bdb16940a86720d2511f46467

Localisation updates from https://translatewiki.net.

Change-Id: I94574a6af0fbcc11b1ec6e7fc6adee2650bf2224

Cut down on deadlocks in invalidatePages()

Bug: T94992
Change-Id: I70f9d931b624e46ccc19f890d61de8d11326e686

Merge "Made rollbackMasterChanges catch exceptions, throwing the last one"

Made rollbackMasterChanges catch exceptions, throwing the last one

Change-Id: Ida36a302b35434d1af464cb77a0084ec441d038a

Merge "Add namespaces for Western Balochi (bgn) from translatewiki"

Merge "Add special pages aliases for Luri (lrc) from translatewiki"

Better file size limit message display logic

Only display upload size limit differentiation message
if there are 2 upload methods.

Bug: T94727
Change-Id: I23c5a5c5e7a30484c242005db831eec5c8c1f4a7

Skin: Account for User::newFromName returning false

Follows-up e41f5a07f893cf.

Bug: T92357
Change-Id: I00edfaed92af7d16894453f28999bfce2de2e37a

Made Skin::getRelevantUser use READ_NORMAL

Bug: T92357
Change-Id: I5089c99ae3a21bb8d41d488d18dd63fe1eaefea9

Merge "Removed BloomFilter classes"

Removed BloomFilter classes

* This ends up being more complex than its worth
  and even more so for multi-DC support

Bug: T93006
Change-Id: Iaa774fe69061e42955b11dc82d30dba93208e606

SkinFallback: Recommend using wfLoadSkin() if possible

Change-Id: I4f3841029578305ab692d853c45678f487adbc78

resourceloader: Omit empty parameters from mw.loader.implement calls

Follows-up ebeb29723, 1f393b6da, 0e719ce23.

Also:
* Add tests for ResourceLoader::makeLoaderImplementScript().
* Apply ResourceLoader::trimArray to makeLoaderImplementScript (new in c0c221bf).

This commit changes the load.php response to omit empty parameters.

These parameters were required until recently. The client has been
updated (1f393b6da and 0e719ce23) to make these optional, thus supporting
both the old server format and the change this commit makes

Clients with a tab open from before 0e719ce23 are naturally not
compatible with load.php responses from after this commit. Ensure
this is deployed several days after 0e719ce23 to reduce race
conditions of this nature.

(This is a re-submitted version of 4ce0c0da4)

Bug: T88879
Change-Id: I9e998261ee9b0b745e3339bc3493755c0cb04b6a

Have back-compat shim actually return the value

Bug: T94958
Change-Id: I75fbd11c8bb357b83079402592e8863acef18fb1

Merge "Fixed class name typo in docs"

Fixed class name typo in docs

Change-Id: Ifc73af824b31fc2f709b777397896f605fe6dfff

Merge "Check return value of preg_match in Sanitizer.php"

Merge "Add checks to try to catch T92046"