Introduce PageHistoryPager::doBatchLookups hook.

The intention of the new hook is to allow extensions to prefetch
any information that may be needed for displaying history rows.

In particular, this is needed by Wikibase to allow us to inject
localized entity labels into the edit summaries.

Bug: T95672
Change-Id: Ie10ef99154da35713a4f583e2de2162fba28eef2

Merge "Avoid (s) for unknown plural in a message"

Merge "Fix warning, add new special page to MessagesEn.php"

Merge "Add $linkTrail for Western Baluchi (bgn)"

Fix warning, add new special page to MessagesEn.php

Followup I7d3ef927b5686f6211bc5817776286ead19d916b

Change-Id: I50fa932f2868c38216745b4f9a403b820cc372a5

Merge "mediawiki.jqueryMsg: Abort test if a language request fails"

Merge "Use vslow in tagUsageStatistics"

Merge "backupTextPassTest: Disable testCheckpointGzip test"

backupTextPassTest: Disable testCheckpointGzip test

Follows-up 3d4c0e93, 73c6f525.

Bug: T70653
Change-Id: I2fbb3e8a2ba14751dfc8942d6d1a379faa9c48bf

Use vslow in tagUsageStatistics

Change-Id: If3e76b1ba560eac6e4073f12c9dfa5becb75b6e0

The 'spambot_username' message is a reserved username

It is used by cleanupSpam.php

Change-Id: I34d9bba7bf33d363bdce410c24a21dacd84771b5

mediawiki.jqueryMsg: Abort test if a language request fails

Follows-up 365b6f3.

If one of the requests fails, stop the test. This avoids one test
from leaking into the next test if it times out. And also makes
the error in question easier to handle. Most subsequent errors
tend to be cascading failures from earlier ones in this case.

It will still run other qunit tests within this module and other modules,
it only aborts the subtest for "Match PHP parser" in mediawiki.jqueryMsg.

To test:
* Change getMwLanguage() and append a bogus string to the request url,
  e.g. 'load.fail.php'.
* It will fail as expected with 'Language "en" failed to load.' and QUnit
  reports that it expected 49 more assertions, and moves on to the next test.

Change-Id: Id77bf43ef0b7a031db2fdb1d6c1bb8b2c64e689c

Merge "ChangeTags: Hide tags whose description messages are disabled"

Merge "Tweaked CAS log message for User::saveSettings"

ChangeTags: Hide tags whose description messages are disabled

If an interface message describing the given tag exists, check if it has been
disabled. If so, take that as an indication that the tag should not be
displayed.

This amounts to an adequate (if somewhat low-level) interface for managing
which edit tags are shown in change list views like Special:RecentChanges and
Special:Contributions.

This patch is based on a suggestion made by jackmcbarn in a review comment on
commit 494989772cf. I did it in a separate change because I'd be interested in
hiding the HHVM tag even if we end up deciding against a generic mechanism for
hiding tags.

Change-Id: I515cf2118ab929da985d6b40481325640e800dcd

Merge "API: Overhaul ApiResult, make format=xml not throw, and add json formatversion"

Tweaked CAS log message for User::saveSettings

Change-Id: I82376561b3c7c15da6fe6374db0245caa5d79ce8

qunit: Abort unfinished AJAX requests in test teardown

Similar to what we do with animations already, ensure we give
the next test a clean start by aborting any requests that were
made during the test that are still pending.

Also log the details of the request (ajax options, e.g. url) to
the console.

To test:
* Add "sleep( 1 );" to LocalSettings.php.
* add "$.ajax( mw.util.wikiScript() )" to a test in mediawiki.util.test.js.
* Run Special:JavaScriptTest/qunit/plain?module=mediawiki.util

Change-Id: Iefef89effc092d296baa9df68a86d95497730708

Merge "ChangeTags was backported to 1.25"

Merge "ResourceLoaderImageModule: Implement cache invalidation"

Avoid (s) for unknown plural in a message

Change-Id: I365d6279310cb2455e078de5677bf7fffa54d60d

ChangeTags was backported to 1.25

So change the relevant references to 1.26.

Change-Id: I351de6a77747e3152052a448b8b6d2f25699e1d0

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

Localisation updates from https://translatewiki.net.

Change-Id: I655e14be267853043341809918d583003366bd7b

Merge "Remove MW_SPECIALPAGE_VERSION. Unused anywhere."

RELEASE-NOTES for change tagging changes

See I37275e0f and I7d3ef92.

Change-Id: I4ed12a2551241f3224d9994b66e3f1675c51d236

UI for adding and removing change tags on revisions and log entries

There is a new special page, Special:EditTags, which is very similar to
Special:RevisionDelete in a lot of ways. In fact, the SpecialEditTags class
started off as a copy-paste of SpecialRevisiondelete.

You invoke this special page by going to an article history page, checking
some revisions, and clicking "Edit tags of selected revisions". Then you
pick the modifications you want to make and click "Apply". Very much like
the revision deletion workflow.

I had to restructure some of the Action routing code, which was only
designed to handle revision deletion. Also removing some code from
SpecialRevisiondelete which didn't work as advertised in the first place,
and definitely doesn't work now.

Change-Id: I7d3ef927b5686f6211bc5817776286ead19d916b

Allow users to add, remove and apply change tags using the API

You can add tags at the same time as performing action=edit, as long as you
have the "applychangetags" right. Also, you can add or remove tags after
the fact from revisions and log entries using the API action=tags.

No UI is provided for either of these changes. The target audience is user
scripts, gadgets and similar tools.

Includes a new log parameter format type: "list", for a comma-separated
list of values.

Logging of change tag events is limited to those that do not accompany an
edit (i.e. those done after the fact), and is hidden from Special:Log by
default, similar to the patrol log.

Bug: T20670
Change-Id: I37275e0f73fa3127f55da0c320b892551b61ee80

Merge "Add namespace aliases for Luri (lrc)"

Remove MW_SPECIALPAGE_VERSION. Unused anywhere.

If we needed such a constant nowadays we'd probably
stick it on SpecialPage itself rather than a global
define.

Change-Id: I8a62c4af3f6185ddd651cbf0ff86ecf39ca44123

Merge "Moved $wgShowUpdatedMarker checks to the right place"

Merge "Remove zh-tw message "editing""

Merge "Message::inLanguage() shouldn't unstub StubUserLang"

Merge "Avoid deadlocks in User::incEditCount"

Moved $wgShowUpdatedMarker checks to the right place

Change-Id: Icee2ec24aac5fe78f7204aabf434640251791cae

Removed some code duplication in Title::getNotificationTimestamp

Change-Id: I02114258448c1dec894e51c2d4c84cb2517eccb7

Merge "Avoid page_restrictions field queries for templates on edit form"

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

ResourceLoaderImageModule: Implement cache invalidation

Bug: T94733
Change-Id: I6ac2ef8d897813e560daa133b89a7932f8c1a362

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"

Avoid deadlocks in User::incEditCount

* This makes concurrent editing less problematic

Change-Id: I930222d5e831bb3729194abbdcb3cab194c70494

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

Merge "Avoid edit stash log spam for bots"

Avoid page_restrictions field queries for templates on edit form

* Title::newFromRow() will actually load page_restrictions, which
  should normally be empty. This avoids queries in getRestrictions().
  Those queries could add 100-200ms to edit form views.
* getLinksFrom() has been simplified to use newFromRow(), thus
  loading the old restrictions in passing.

Change-Id: I225955589a82e4accbf059d17390a5e28e2fa1ca

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"

Message::inLanguage() shouldn't unstub StubUserLang

When a string is passed to Message::inLanguage(), it first checks
whether the message's current language's code is equal to the string, to
avoid a call to Language::factory(). But if the message's current
language is an instance of StubUserLang, it's probably less expensive to
just call Language::factory() than it is to unstub.

This also avoids a possible recursion warning from T56193, particularly
if inLanguage() is being used intentionally in an attempt to avoid that
warning.

Change-Id: Ia09adec05cfbb09c09e07c6be1e2d613435664d9

Corrected typos in function name unpackMetaData and getMetaData

Bug: T95873
Change-Id: I71dffaaed24c983ce07bc02f806294337ca086e2

Remove zh-tw message "editing"

The current string is incorrect (written in zh-hans); removing it to
let it fall back to zh-hant. Since zh-tw exporting has been disabled
on translatewiki.net, it's not possible to fix it there. Maybe a full
cleanup is needed for zh-(cn|hk|mo|my|sg|tw).json at some time.

Change-Id: I3cc7bbb2065ecae535986525367d26dfc82c49b1

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

Add namespace aliases for Luri (lrc)

Change-Id: Ifaf3a58cd5b525305f2d545f8a50751f85579756

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

Add $linkTrail for Western Baluchi (bgn)

Change-Id: Ie4e9b0c29e2502502c2460385fabbc2599ba8d1c

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

API: Overhaul ApiResult, make format=xml not throw, and add json formatversion

ApiResult was a mess: some methods could only be used with an array
reference instead of manipulating the stored data, methods that had both
array-ref and internal-data versions had names that didn't at all
correspond, some methods that worked on an array reference were
annoyingly non-static, and then the whole mess with setIndexedTagName.

ApiFormatXml is also entirely annoying to deal with, as it liked to
throw exceptions if certain metadata wasn't provided that no other
formatter required. Its legacy also means we have this silly convention
of using empty-string rather than boolean true, annoying restrictions on
keys (leading to things that should be hashes being arrays of key-value
object instead), '*' used as a key all over the place, and so on.

So, changes here:
* ApiResult is no longer an ApiBase or a ContextSource.
* Wherever sensible, ApiResult provides a static method working on an
  arrayref and a non-static method working on internal data.
* Metadata is now always added to ApiResult's internal data structure.
  Formatters are responsible for stripping it if necessary. "raw mode"
  is deprecated.
* New metadata to replace the '*' key, solve the array() => '[]' vs '{}'
  question, and so on.
* New class for formatting warnings and errors using i18n messages, and
  support for multiple errors and a more machine-readable format for
  warnings. For the moment, though, the actual output will not be changing
  yet (see T47843 for future plans).
* New formatversion parameter for format=json and format=php, to select
  between BC mode and the modern output.
* In BC mode, booleans will be converted to empty-string presence style;
  modules currently returning booleans will need to use
  ApiResult::META_BC_BOOLS to preserve their current output.

Actual changes to the API modules' output (e.g. actually returning
booleans for the new formatversion) beyond the use of
ApiResult::setContentValue() are left for a future change.

Bug: T76728
Bug: T57371
Bug: T33629
Change-Id: I7b37295e8862b188d1f3b0cd07f66ac34629678f

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