Merge "Reduce outages due to master DB problems in doViewUpdates"

Merge "mediawiki.loader: Remove optional params in mw.loader.implement tests"

Merge "Don't bother buffering a counter update with a delta of zero."

Merge "Displaying protection expiry date and time in action=info"

Don't bother buffering a counter update with a delta of zero.

Counter updates with a delta of zero don't do anything, so there's no point in
sending them over the wire.

Change-Id: Iae82ee9c9a8544c94abfcbdf944bd713687dce9d

Fix link to Special:RevisionDelete in newer log revdel logs

Bug: T96049
Change-Id: I6817ac13e776a3e6fa766c3996173b5da02f54de
Follows-up: Ic149960f797d05fd088fa014390d791e3ae08687

Localisation updates from https://translatewiki.net.

Change-Id: I29bea8f29d49d43dc501daf22ba6d572be7901b0

Merge "Clean up handling of 'infinity'"

UploadForm: Don't set a null context

Breaking Special:Specialpages on master now.

Bug: T96057
Change-Id: Ieb21b4d07af2dc8ea18016568c98b78e390ac2d5

Reduce outages due to master DB problems in doViewUpdates

Change-Id: Ie28e8dcfc3bc402b3119d9cf43612acab2af1004

Merge "Handle missing returnto on Special:ChangePassword/ConfirmEmail"

Merge "API: Add license info to API help output"

Merge "Avoid edit stash log spam for bots"

Merge "Made WatchedItem use Revision::getTimestampFromId"

Merge "Avoid using local main page title in JobSpecification"

Merge "Made job factory() callers use Title::makeTitle"

Merge "wikibits: Mark importScript/importStylesheet utilities as deprecated"

Avoid using local main page title in JobSpecification

* They may not work well on the wiki that runs the jobs

Bug: T92789
Change-Id: I496974b11bd865ede9dd6d2cd534e0abdd7fa3f7

Made WatchedItem use Revision::getTimestampFromId

Change-Id: I3a895a69e8f4678b82a4e50f70d6bfc4df932d74

Made job factory() callers use Title::makeTitle

* The titles are not from user input and this simplifies the code

Change-Id: I067049cca5661b387076f0c28bc0b71d22162a0f

Localisation updates from https://translatewiki.net.

Change-Id: I0ed09090cdcf12ced0fe66f47045795f97a11b0b

Handle missing returnto on Special:ChangePassword/ConfirmEmail

A missing returnto results in a null passed to Title::newFromText which
gives a warning:
Notice: Title::newFromText: $text must be a string. This will throw an
InvalidArgumentException in future

Follow-Up: I2b36b7a3b96b37e29fe06f69c13a185b3ec592a7
Change-Id: I95e6023a2630cbae69f25d63611ebafb8b2bd74c

specials: Clean up redirect special pages ($subpage can be null)

$subpage being null is clearly documented in SpecialPage::run,
SpecialPage::execute, and most special page subclasses.

But all the redirect subclasses only copied part of the typehint,
making it look like it's always a string.

For SpecialMyLanguage, follows-up b1853bba0. Don't cast null to
empty string, and don't bother giving Title::newFromText an empty
string only to bail out with null again.

Also:
* Add visibility 'public' where missing.
* Add or correct relevant documentation comments.
* In SpecialMyRedirectPages, handle error first and avoid having
  final return inside a conditional; Remove redunant 'else'.

Change-Id: Ie3543f44011832b198bb3d3e32528b6a2868dee1

wikibits: Mark importScript/importStylesheet utilities as deprecated

The global variable loadedScripts and the import functions from wikibits.js
have been deprecated since 1.17 [1].

The global loadedScripts is used to keep track which scripts have
been loaded through importScriptURI. To do this, it only needs to be
a local variable, not a global.

For scripts using the global to check whether a particular script
has been loaded, here are some possible substitutes:

* Use mw.loader.getState() for scripts loaded through ResourceLoader
  (which includes Gadgets).
* Use hooks (mw.hook) to communicate between the scripts.
* Use $( 'script[src="..."]' ).length to check whether a script with
  a specific URL has been loaded.
* Use $( 'script[src]' ) for a list of all scripts (excluding inline
  scripts), and filter it for whatever you are looking for.

The functions importScriptURI and importStylesheetURI are used to
load scripts and stylesheets resp. from a URI. Use mw.loader.load() instead.
Note that there are minor differences between the deprecated functions
and mw.loader.load(), see [1] for details.

The importScript and importStylesheet shortcuts have been deprecated as well.

[1] https://www.mediawiki.org/wiki/RL/LJS#wikibits.js

Change-Id: Icc87243a8213841bfe46e48a9c074301c241041c

Merge "Change Language::timeanddate to userTimeAndDate in RevisionList"

Corrected typos in function name unpackMetaData and getMetaData

Bug: T95873
Change-Id: I71dffaaed24c983ce07bc02f806294337ca086e2

Localisation updates from https://translatewiki.net.

Change-Id: I27277eaf911553a14ed87622ca36a6212829a2b5

Merge "Add page_restrictions to readlock in lockSearchindex"

Merge "Only need one check for is_dir"

Update OOjs UI to v0.9.8

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

Change-Id: I369a24f74c455cd42571718a7b0078fd24c5f25e

Add second param to Title::newFromText warning so that called from works correctly

Change-Id: Ib977acb9abfcc6e07019013c6f9fc20ece16245d

Change Language::timeanddate to userTimeAndDate in RevisionList

This avois using $wgUser when formatting the timestamp

This was done in the overrides with
I30d88822d8ede5c138dd5403a998dd722f950bbe
The htmlspecialchars was added with
Ie1f16029020f980abba35d6322b89c95326f107b to the overrides.

Change-Id: Iaa12064502b343f0f47178f4848eed72662b422c

Merge "installer: Clear module_deps cache in updater"

Merge "Use Language::userTime in Preferences::datetimePreferences"

Merge "Fix most of IPTest to use data providers"

Merge "Set context earlier in UploadForm::__construct"

installer: Clear module_deps cache in updater

Unless --nopurge is set in update.php, clear module_deps cache
tables.

We already clear the objectcache via purgeCache(), and
msg_resource and msg_resource_links via MessageBlobStore::clear().

The module_deps table should be included as well.

Bug: T37472
Change-Id: I7cb63bb39a62b7f5be3855d022a8c76e175e95b5

Merge "Enable entity loader and handle errors nicely in WikiImporter constructor"

Localisation updates from https://translatewiki.net.

Change-Id: If81891b3a9bd94e4e60979f16798625a32b19640

Display warning message in Special:ListFiles when invalid username entered

A warning message is to be displayed when an invalid username
is entered in Special:ListFiles. This patch does the same

Bug: T72784
Change-Id: I35fef559db812d347af7a313778e25f775e1bfb6

Avoid edit stash log spam for bots

Change-Id: I2f9cc86794f8eee282c5efdbe96d4ff981b1977b

Enable entity loader and handle errors nicely in WikiImporter constructor

Two issues being addressed here:
* Slightly friendlier message (instead of fatal) if libxml is not present
* Need to make sure the entity loader is enabled when opening XML documents

Also provide an error message when XMLReader::open fails, as otherwise,
the user sees cryptic errors from code that tries to use the (unopened)
XMLReader.

Bug: T45868
Bug: T86036
Change-Id: Ibcccce9f09f87b17c3093fd0c3c3ff74d7dc6cb7

Localisation updates from https://translatewiki.net.

Change-Id: Ib5c98b70064d107f2299169a33cc1f4d43e35953

Merge "debug: Add missing "use DateTimeZone" in LegacyLogger.php"

Merge "Reorder backlinks indexes"

debug: Add missing "use DateTimeZone" in LegacyLogger.php

Seen in fatal.log in production: ErrorException from line 264 of
/srv/mediawiki/php-1.26wmf1/includes/exception/MWExceptionHandler.php:
Fatal Error: Class undefined: MediaWiki\Logger\DateTimeZone

Bug: T95727
Change-Id: Icb303314caaef47ac767fbf593e92d09e818f147

Use Language::userTime in Preferences::datetimePreferences

This avoids the indirect use of $wgUser

Change-Id: I951f5c4399dff6bb17c04e20dd9a7f763027b873

Remove unused "swap" global function

Deprecated in 26e1e083e8bfd4, unused since r12411.

Change-Id: If328f09114ac5a5d23a300d9b0bbcd7d67e051c8

mediawiki.loader: Remove optional params in mw.loader.implement tests

Follows-up 1f393b6da8736c. Done in a separate change to ensure
those tests work both ways.

Change-Id: I7e185db56e36fdfbb88c2d0938cdf55881e8c313

Merge "Switched RevDelList to using start/endAtomic()"

Merge "Made deleteTagEverywhere use start/endAtomic()"

Merge "Made QueryPage::preprocessResults also use the vslow group"

Merge "Made JobRunner avoid slave lag more aggressively"

Fix most of IPTest to use data providers

Change-Id: I7a41af445549f6688c559adbc5f16bdb428dae60

phpdoc: make references to \MediaWiki\Logger\Spi fully qualified

Change-Id: I4a5dbc3894ea175053e1238e7fc5de4c4cf085ca

Merge "Replace use of assertType with assertInternalType and assertInstanceOf"

Merge "Removed SpecialStatistics query group; vslow is enough"

Made QueryPage::preprocessResults also use the vslow group

Change-Id: If5893ef9e2a18684c8cd1ce1b6149742b392a3c3

Merge "Update namespaces for Luri (lrc) from translatewiki"

Merge "Add fallback language for Luri (lrc) to Persian (fa)"

Made JobRunner avoid slave lag more aggressively

Bug: T95501
Change-Id: Ibba6d2947638a17c86edcdaadf484c7aa45cd1c6

Add fallback language for Luri (lrc) to Persian (fa)

It's an Iranian language and fallback was not added I think back in 2008.
It's required for the creation of the Luri Wikipedia.
All Iranian languages speakers are also native Persian.
It's already added for languages such as Baluchi (bgn), Bakhtiari (bqi),
Gilaki (glk) and Mazanderani (mzn).

I discussed this with the community as well.

Change-Id: Idf3ec73f9115343fde2e8240db867877122864de

Update namespaces for Luri (lrc) from translatewiki

Change-Id: I241af41f278e6e117bcabd74d114a546ca69fb87

Reduce begin/commit activity in startAtomic with DBO_TRX

Change-Id: I6c1334967aeff54fb18091c23749fcb0b49624b9

Merge "Set time and date formats for ce"

Merge "Update special pages aliases for Westerm Baluchi (bgn) from translatewiki"

Set time and date formats for ce

Bug: T94665
Change-Id: I29b23d523785f49280c9d7add590340c31835e4d

Removed SpecialStatistics query group; vslow is enough

Change-Id: I1841f5c64ebeeed7c3e54a11216add5c104ad69d

Switched RevDelList to using start/endAtomic()

Change-Id: I2c950b558390e3fd9917414998f576e4e3b0593d

Made deleteTagEverywhere use start/endAtomic()

Change-Id: I54b697bc8c0f84d49bfcc20ec2753e18d24bf988

Localisation updates from https://translatewiki.net.

Change-Id: Ie915e0ffc24328785764b98bb47a5164162a2320

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

Set context earlier in UploadForm::__construct

The parent constructor sets the $context for the form, but
getSourceSection is using getUser before the call, which results in a
warning for using RequestContext::getMain().
Just set the context before use.

Change-Id: I88af41b22cf05ce18d3d2852325bc97cd2544556

Fixed ApiQueryBase::selectNamedDb() return value

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

Change-Id: Ibe62ff4275201382243c4861cc4fd15cf0316e43

Replace use of assertType with assertInternalType and assertInstanceOf

These are available in phpunit since 3.5.0, which I think
is reasonable to expect people to have at this point,
especially when we actually require 3.7.0 or higher in phpunit.php:

Use assertSame instead of assertArrayEquals in UIDGeneratorTest.
assertSame (and assertEqual) care about sortorder of the array,
and is perfectly sufficient in this case to check they array is correct.

if ( $puVersion !== '@package_version@' && version_compare( $puVersion, '3.7.0', '<' ) ) {
        die( "PHPUnit 3.7.0 or later required; you have {$puVersion}.\n" );
}

Change-Id: Ic32ab45110e4c4304ef046ae8d0e98c741255559

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

Update special pages aliases for Westerm Baluchi (bgn) from translatewiki

Change-Id: I15e4cb75d2fe1b2719334fb74135843b86690bc3

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"