Merge "Add @group Database tags to tests that need it"

Merge "Fixed ApiQueryBase::selectNamedDb() return value"

Have phpunit.php exit with 1 in case it couldn't run the tests

Just makes more sense not to claim we could complete everything
if we actually couldn't.

Change-Id: I4ec3a2da8913ca546782919813eb18a05a5cca0a

Fixed ApiQueryBase::selectNamedDb() return value

The doc specifies that this func should return
named database connection, but it does not. Fixed.

Change-Id: Ibe62ff4275201382243c4861cc4fd15cf0316e43

Merge "Use XML localName when importing"

Merge "Set context earlier in ImageListPager::__construct"

Merge "Skip file count on Special:Statistics when no files and no upload"

Merge "PLURAL support for list lengths in default-skin-not-found"

Merge "Add help link to three rather important pages"

Merge "Added LogEntryBase::extractParams/makeParamBlob"

Merge "Do not pass null to Title::newFromText in Special:MyLanguage"

Merge "update.php: Correctly report durations >= 1 hour"

Merge "Warn when jobs do large DB writes at once"

Merge "Removed unusable maintenance script"

Warn when jobs do large DB writes at once

Change-Id: I57e9bb630accd5b262188ab16b17b558cd3a2bc1

Removed unusable maintenance script

* Change left out of 4bdbfe6c1b

Change-Id: I2169586742653d0b43e2f581f28f79af960e7ada

ApiParse: Replace isCurrent() check with getLatest() comparison

Because the $pageObj state is what actually ends up getting used.
If $pageObj thinks an old rev is the latest one, then we are in
trouble, even if $rev knows what's really going on.

Bug: T95466
Change-Id: I4d6ba4f18adaaad052d3bee1a575ba034aaf112b

Add array type hints to minor methods in the Html class

I'm aware that adding these type hints does have the potential of beeing
a breaking change if a caller misuses it. Note that it really is a misuse
in this case because all these parameters are documented as "array" and
nothing else.

I double-checked the usages of all methods I touched and could not find
any caller that does not fulfill the contract of these methods - in other
words, all callers I can find in my local code base (which includes all
major extensions like Echo, Flow, Parsoid, VisualEditor and so on) pass
arrays to these parameters.

I left the main methods openElement, rawElement and so on untouched
because they are called way to often (500 times and more).

Change-Id: I5ca13b26fb08d732ce4cadc4ee3d38314e606fd3

Merge "Set a query group on SpecialRecentChanges query"

Merge "Partially revert increased wikitable padding"

Merge "Made wfReadOnly() more robust at handling the lagged-slave case"

Merge "Make RedisBagOStuff support tag name hashing"

Merge "Made WikiPage::doPurge() avoid calling commit()"

Merge "Use wfWaitForSlaves in upload stash cleanup script"

Localisation updates from https://translatewiki.net.

Change-Id: I52aa3fcab215c84aa499cf2c8f1ff67d3fff47c3

Merge "Log huge write queries in CLI scripts"

Merge "Made mergeActiveUsers use "recentchanges" query group"

Made wfReadOnly() more robust at handling the lagged-slave case

* Previously if it was called before a DB_SLAVE connection, it would not
  be set. This is fixed now.
* Also set mLaggedSlaveMode in LoadBalancer as appropriate. Before, it
  was only set in the "too lagged for ChronologyProtector" case.

Change-Id: Ic4dc555cf762653f157df1795f53f3577c1e587a

Merge "Removed unused wgDBAvgStatusPoll var"

Fix I9e63789e: Release notes should refer to old versions in HISTORY

Change-Id: Ibedf45f4a0a5b16f8fa494412b471e266404d9b3

Made WikiPage::doPurge() avoid calling commit()

* Calling commit() can cause imbalanced begin/commit problems

Change-Id: I55fec065de20653b216c8dfac7c361124c8fe10a

Merge "Made User::invalidateCache() use touch() instead of the DB query"

Prevent SiteStats::loadAndLazyInit from reloading in $wgMiserMode

* Large sites can use a chron to handle accumulated drift

Bug: T95426
Change-Id: I59603a7aa329a10f8a7e4d11f3496e8f8243ca42

Use wfWaitForSlaves in upload stash cleanup script

Bug: T95382
Change-Id: I8d93eb15064693634076e864ed6724392448cb45

Track search events

Tracking works only for the skin-provided search box (top right search
box on desktop). Also, add the 'update' option the jQuery suggestions
extension.

Change-Id: Icc0afcd6c9fa8c82a95d9decf988a19ae3edc165

Add test for IRC line for type protect/move_prot

All other log types already have a test, just move_prot is missing.

Change-Id: I97fdb21d3cb58847a883917b06294bc92dac9410

Log huge write queries in CLI scripts

* This is already done for web scripts, but with lower limits

Bug: T95382
Change-Id: I0fe14bcdaebc3568b0cb7af8084b7bfefeb82826

Made User::invalidateCache() use touch() instead of the DB query

* This method is used for clearing the User cache as well
  as bumping the value of getTouched() for HTTP 304 logic.
  These do not need to do the actual user_touched update.
* This also avoids problems with setting mTouched but
  deferring the update. That confused the CAS update
  logic since it expects mTouched to be in the DB.

Change-Id: I96b30f5c9c8b4714e6663b187f741954a13312cf

update.php: Correctly report durations >= 1 hour

Bug: T67772
Change-Id: I61492a1ad8bf7f574b198af82799b030221bb58b

Made mergeActiveUsers use "recentchanges" query group

Change-Id: I9b91fe5d81f7315d531c48530cfbcf779f3adec2

Set a query group on SpecialRecentChanges query

Change-Id: I2525aa9d504027aec6bb3f83c8a48ce818e252cd

Removed unused wgDBAvgStatusPoll var

Change-Id: Ib1abf72add4d62bb41f1e2061caf9de0abd84e43

Added LogEntryBase::extractParams/makeParamBlob

Centralize the serialize and unserialize call in the new logging system
into two functions to allow reuse by tests.

Change-Id: Ibedca39558cbd566661b20e7ebb9d2b6537e4f24

Begin 1.26 development cycle

Change-Id: I9e63789e7bbec95e72fca6dfc1a79f0ae1fba9b0

Merge "mime.types: allow bzip2 upload"

Merge "Remove form when including empty Special:WhatLinksHere"

Merge "Kill Dwimmerlaik"

Merge "Revert "Hide HHVM tag on Special:{Contributions,RecentChanges,...}""

Merge "Made User::loadPasswords respect "queryFlagsUsed""

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

Localisation updates from https://translatewiki.net.

Change-Id: I59a85ff5f09d46967e59feb3c65d76f1d04ec84d

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

Made User::loadPasswords respect "queryFlagsUsed"

* Also cleaned up a few var names

Change-Id: I3a835e3980dfc3205716cf5bb1a4c04aa037f36d

Revert "Hide HHVM tag on Special:{Contributions,RecentChanges,...}"

This reverts commit 494989772cfce8805a10c2cd497880f6f6793896.

Bug: T1205
Change-Id: If47af4a663530d9a32c1d24b6b55a63954f53fcb

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"

Do not pass null to Title::newFromText in Special:MyLanguage

This currently breaks phpunit integration test in Translate.

Change-Id: Ibdc04e9e20c0264076d7a8766ac9a4826033371e

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

Make RedisBagOStuff support tag name hashing

Bug: T95126
Change-Id: I266dd30c93cfc44395eba5bca64f451ca77c23a4

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

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

This broke a few things, and the debug line was basically pointless.

Instead, continue to only throw an exception if $text is an object, but only
warn if it's otherwise not a string.

This reverts commit 372ded2fea436bf0e61318991f903d61bde26366.

Change-Id: I060da9191cdbd00c4873caba875bfb77c917bcd7

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