Explicitly declare ApiFormatXml::addXslt() protected

Only used in private/protected context.

Change-Id: Ic44d6e70e89c3d229d79aa08d2a0092609e7d62e

Explicitly declare ApiFormatBase::initPrinter public

It's overridden three times, and in all cases it's public.

Change-Id: I1712319076b38bbb5418c60b954e174354831be3

Call static functions statically in API

Change-Id: I5952acb932c1e7337dc85226a898da3472e799e9

Use Elvis operator where possible in API

Change-Id: I0a26c04cf2ded2c1bd74d8dabd131ae8e20117f3

Remove unused local variables in API

Change-Id: Ia94ae232308ad47ee88dbd30675cfcbdb6b371ed

Localisation updates from https://translatewiki.net.

Change-Id: I11fb4866b2fc0c5daf9ee639d42feef2cdafe768

Merge "Improve change content model log entry for page creations"

Improve change content model log entry for page creations

There really isn't a previous content model because the page didn't
exist beforehand, so say the page was created with a non-default
content model.

Bug: T128645
Change-Id: Ibe0b124e3f3ceaaf3775f82e2646b7639a146a42

Localisation updates from https://translatewiki.net.

Change-Id: I683765945022afc8b9b4a839ab28f47f79d86f69

Merge "Instrument diff timing"

Localisation updates from https://translatewiki.net.

Change-Id: Ib49bf89dcc309086e521635db1198fe39e46b995

Merge "qunit: Don't require expect() anymore"

Merge "Don't treat configured but absent engine as executable"

qunit: Don't require expect() anymore

This hasn't been useful in QUnit for a while now with the improved
assertion context object and tracking of asynchronous tests without
shared global state.

Change-Id: Icaf865b4d6e85e739bf79c4d1bacb8a71ec5a3da

Instrument diff timing

Bug: T128697
Change-Id: I748286abac025092abc33b3b7b8a0d3dabafdd25

Don't treat configured but absent engine as executable

E.g. if you have $wgExternalDiffEngine = 'wikidiff2' but after
a PHP update you no longer have the module you still shouldn't attempt
to shell out to some nonexistent wikidiff2.

Bug: T74030
Change-Id: I745cd1cb2e152f4fbb95c8f782d70117f8c844f1

Merge "SpecialUserlogin: Update main RequestContext in addition to globals"

Localisation updates from https://translatewiki.net.

Change-Id: I62572d292497e21e7ffcaf6b8ab2524f469360a6

Merge "Ditch support for original wikidiff"

Ditch support for original wikidiff

It's been unmaintained for a while and does not support
various languages adequately.
Also, document $wgExternalDiffEngine.

Change-Id: Ia8aeffd79d550fb7a1a7121456940446eea8bd4f

Merge "Add additional tracking information to mediawiki.searchSuggest"

Add additional tracking information to mediawiki.searchSuggest

Adds a few pieces of information to improve tracking of autocomplete
usage.

* When using Special:Search 'go' feature forward wprov parameter to redirect
* Include a data attribute indicating autocomplete location to
  differentiate usage of the header and Special:Search content autocompletes
* Report exact query string that was used for impression-results
* Add handling to allow searchSuggest subscribers to append tracking
  information to generated article links
* Add a new hook, SpecialSearchGoResult, that can either change the url
  redirected to in the 'go' feature or cancel it entirely.

Bug: T125915
Change-Id: Iec7171fcf301f1659d852afa87ce271f468177c1

SpecialUserlogin: Update main RequestContext in addition to globals

Change-Id: I835bb77938f7e02c862563ea38341cf5840aa367

Merge "Follow-up e33d6f45a. Fix some mistakes w/ limit on QueryPage"

Merge "Revert "Allow auto suggestion for subpages of Special:BotPasswords""

Revert "Allow auto suggestion for subpages of Special:BotPasswords"

This is completely wrong. If you're going to auto-complete anything here (which is probably not really necessary), you need to auto-complete the "bot names" for the current user rather than every user name on the wiki.

This reverts commit aa8823ac4b58be611caf2595ad3c7b0ecacbc8ac.

Change-Id: I5d477198827c107f947d25dddc2513f12d2bfd62

Merge "Revert "Templatize Special:Contributions lines""

Revert "Templatize Special:Contributions lines"

Breaks RevisionDelete.

This reverts commit 1bce6db10e99840cfbd10fa074c93d8d96d88412.

Change-Id: I8f63e002e34a9d13e2597a27e3dd918a687d1e7c

Fix parserTests.php script

 * Follow up to 60e4f3fd

Change-Id: If1370a720da21c3662fda4100c96b1758ddc1dc0

Follow-up e33d6f45a. Fix some mistakes w/ limit on QueryPage

Fix some rather embarrassing mistakes. See comments on e33d6f45a.

Also change the behaviour when someone sets an offset higher than
max allowed. Returning a page with 0 results and broken prev/next
links is poor UI imo.

Change-Id: Ibfc983675ae0c600eeccd5e361550e9b5f96f5fd

Merge "mediawiki.ui: Button group active buttons unchanged on user interaction"

Merge "Upstream `isElementInViewport` from MobileFrontend"

Merge "Templatize Special:Contributions lines"

Merge "MediaWiki Widgets: Add new SearchInputWidget"

mediawiki.ui: Button group active buttons unchanged on user interaction

Applying default cursor and making active button's hover state behave same
as normal state for improved user experience.

Bug: T77916
Depends-On: Ib89500d70b3e79edcf9ba989f05a311e0edec82b
Change-Id: I8fd74c28ccb89df1e257aa3d63d362697c4542ed

Templatize Special:Contributions lines

Bug: T122537
Change-Id: I11aac43de495881e10e393d075a231bd346ea547

Integrate Less helper `.mixin-screen-reader-text()`

Adding Less parametric helper mixin for screen reader users.
This will help with changes such as
I7db2d7e5387f1a4392d587764598dbd82dd0a5cd

Bug: T119444
Change-Id: Ie9bce512804385578361eb77579106c799d5ff57

Merge "Prevent "Failed to create..." warnings when session loading fails"

MediaWiki Widgets: Add new SearchInputWidget

SearchInputWidget is similar to a TitleInputWidget, but doesn't has
a visible loading indication, doesn't highlight the first result and
uses the opensearch api endpoint for suggestions, instead of
prefixsearch.

Extra points:
 * Improve documentation of mw.widgets.TitleInputWidget's configuration
   option validateTitle

Bug: T118443
Change-Id: I8b8098041fe2971389fa908d007d2e77255829ec

Merge "Add tags support to patrol, protect, unblock, and undelete"

Merge "installer: Update main RequestContext when updating $wgLang"

Merge "tests: Ensure main RequestContext reflects custom $wgLang"

Merge "maintenance: Remove odd use of $wgLang"

installer: Update main RequestContext when updating $wgLang

Change-Id: I1e3b76b2358a90a551616dc84eb6f2be6bb41c0a

tests: Ensure main RequestContext reflects custom $wgLang

Call RequestContext::resetMain() so that any subsequent call
to getMain() and getLanguage() is aware of the latest state
as set up by the setUp() and setupGlobals() methods.

The MediaWikiTestCase class for PHPUnit did this already, but
the parserTest suite and UploadFromUrlTestSuite didn't yet.

Change-Id: I6481176228944004091078704d0346c8f3fc0cf1

Merge "filebackend: Clean up TempFSFile and fix IDEA errors"

maintenance: Remove odd use of $wgLang

* testCompression.php: Called $wgLang, but didn't set it.
  It just assumed it was English. Use a local variable instead.

* update.php: Set $wgLang to English. Set main RequestContext as well.

* transstat.php: Rename $wgLanguages to $languages for less confusion.
  It's just a local instance of language.inc's Languages class.

Change-Id: I67865f4d8710946001f7586ab0a17e125366b184

filebackend: Clean up TempFSFile and fix IDEA errors

Change-Id: I4e25e3bf906fa3a918f4462fac1a6be5e85696aa

CSSMin: Remove file-existance filter in getLocalFileReferences()

Follows-up 8f5cd11d82fa.

The old getLocalFileReferences() method is no longer used anywhere.
Remove it and rename getAllLocalFileReferences back to it.

Change-Id: I864258aad128ba9b54464c7bc854543f2937f977

resourceloader: Don't cache CSSMin::remap() calls

The output of this function is not deterministic. This function
is what tranforms urls to include content hashes and also embeds
content as data URI. The calls to this function are expected to be
up to date and matching the version hash as constructed by the
current web request.

Previously, the content hash as constructed based on the collective
file hash of all references images was accurate but the generated
css associated with it could be stale.

Unless we abuse MemoizedCallable to take a separate cache buster
(with $localFileRefs?), it's best to remove this as it isn't
a deterministic function and shouldn't be memoized.

Bug: T128668
Change-Id: Icb87ddc585d7320ac48619446bb8b9cbe5f4780e

Prevent "Failed to create..." warnings when session loading fails

If there is an existing session for a given ID but loading it fails,
there is no point in trying to create a new empty session with that
ID. Just fail silently (the reason for not loading the session
should be logged elsewhere), don't spam the logs and don't slow
down execution by throwing and catching an exception.

Change-Id: I8299872cc29c32cb245686ed0bca6b9a5902cdc1

Linker: Avoid passing false to Title::newFromText

This is needed because the call to substr() returns false when the string
is only 1 character long, which occurs when parsing degenerate links like
[[:]] and [[::|foo]].

The seemingly unnecessary test for $match[1] !== '' is for forwards
compatibility with PHP 7.

Bug: T121706
Change-Id: Icc19ee990d01958d64b938d298e9a7e1df7181b5

Upstream `isElementInViewport` from MobileFrontend

As a useful utility function, we've copied this method several times
across multiple extensions, which is a pretty good sign it should
actually live in core.

Changes:
 * Add `mediawiki.viewport` module
 * Rewrite method to be more robust and accept any viewport
 * Add `mw.viewport` to jsduck categories file
 * Add method for checking if an element is close to the viewport
 * Add unit tests

Bug: T124317
Change-Id: I38eec4f1e568f51e7e212b2b3f10b8da8d36f316

api: Fix "PHP Notice: Undefined variable: config"

Follows-up 362c220aa1, which split the method up but didn't
re-create a local $config variable.

Change-Id: Ic32ac3c600d0b4d69e8129fa50ea47d9a6a54b20

Localisation updates from https://translatewiki.net.

Change-Id: Ib781f4c698a70de2afd5db1c3a0c58bf8f4285f0

Merge "Add ContentHandler::supportsCategories method"

Merge "AutoloadGenerator: Account for PHP 5.5 syntax like "self::static""

Fix trivial PHPCS violations in */skins/* directories

Due to T127238, files in various */skins/* directories are not checked
by PHPCS. Temporarily removed the exclude rule from phpcs.xml and ran:

  composer fix includes/skins/* tests/phpunit/includes/skins/* tests/phpunit/skins/*

Change-Id: I9240c1cee825920b6634903282be6252cce55686

Merge "Revert "Deprecating: Consolidating `progressive` & `constructive` buttons""

Revert "Deprecating: Consolidating `progressive` & `constructive` buttons"

I merged it too early, sorry. Deployment is planned for a week later.

This reverts commit cb8fa5fd173f4cfdb9945019fa9ac894eafa3ffe.

Bug: T110555
Change-Id: I369fa6d2b4f6531f7ec475c08cba283d323a0ffb

Merge "Add help link for Special:ChangeContentModel"

AutoloadGenerator: Account for PHP 5.5 syntax like "self::static"

Bug: T128625
Change-Id: I905b549a738b76c07c56ec92f500553c1d48f558

Merge "Allow FSFile objects for src in FileBackend::do*Operations"

Allow FSFile objects for src in FileBackend::do*Operations

Convenience aside, this lets multiwrite backends do async replication for
"store" operations safely, buy keeping a handle to the source file that
prevents it from getting prematurely deleted before the post-send writes
to the secondary backends can happen.

Bug: T91869
Change-Id: I1254de527c47835c35fed6e526b42953c1b2b2ca

Add help link for Special:ChangeContentModel

Bug: T128556
Change-Id: Id55a463cf6eac4f20e800ae49448e6007f68a3c4

Fix use of RawMessage in Status::getMessage()

RawMessage::__construct() takes a single array of params, it's not
varargs-style like wfMessage().

Change-Id: I63950ee16e359aa2627228350e27d9e94bb6e8ce

Merge "Deprecating: Consolidating `progressive` & `constructive` buttons"

Add ContentHandler::supportsCategories method

and check for this in WikiPage::doEditUpdates before
inserting a new CategoryMembershipChangeJob.

Some content models like the Wikibase ones do not
have categories and it's wasteful to add these jobs
for all Wikibase edits.

Bug: T126977
Change-Id: I2c54a4ba1546445dc41101e15cb83a2c6cc2b1c9

Merge "Fix comment in SearchEngine.php"

Merge "objectcache: Fix MultiWriteBagOStuff docs ('asyncHandler', not 'asyncCallback')"

objectcache: Fix MultiWriteBagOStuff docs ('asyncHandler', not 'asyncCallback')

Follows-up 1171cc00cd.

Change-Id: I97189e916d0aba507273216e3e9ab28ce0d5ffec

Merge "Convert remaining array() to [] in API"

Merge "mw.Upload.BookletLayout/Dialog: Add determinate progress bar"

Merge "findHooks.php: Use recursive directory iterator"

Merge "WebRequestUpload: Use canonical header name in getHeader()"

findHooks.php: Use recursive directory iterator

The following dirs were not previously included:
* includes/compat/
* includes/compat/normal/
* includes/composer/
* includes/config/
* includes/db/loadbalancer/
* includes/debug/logger/
* includes/filebackend/filejournal/
* includes/filebackend/lockmanager/
* includes/jobqueue/aggregator/
* includes/jobqueue/jobs/
* includes/jobqueue/utils/
* includes/libs/
* includes/objectcache/
* includes/pager/
* includes/poolcounter/
* includes/profiler/output/
* includes/profiler/
* includes/registration/
* includes/templates/
* includes/tidy/
* includes/title/
* includes/utils/iterators/
* includes/widget/
* maintenance/ subdirectories

Change-Id: I6c01b8768f9f70152e320d254f8025a0ffde8c62

user: Remove obsolete note in docs/memcached.txt

The User object cache hasn't been that since b3acd4f.

Change-Id: Ia7ab78ef22c24a7421ea25db1440e7267f0a725d

Update OOjs UI to v0.16.1

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

Change-Id: I4bd51d4c105a09d71a911e04538b51f58c3ea721

Localisation updates from https://translatewiki.net.

Change-Id: I7c15b259b0f88dbc05ce1520ecb319a66eed6f73

Merge "Add mobile target to mediawiki.special"

Merge "Add access modifiers to MagicWordArray.php"

Merge "Add access modifiers to MagicWord.php"

Merge "Move MagicWordArray class to own file"

Merge "Rm unused $config from SpecialRecentChanges::getDefaultOptions"

Rm unused $config from SpecialRecentChanges::getDefaultOptions

Change-Id: I5dc8f091c06b0d32d5d84af885608a57ca9c054a

Add mobile target to mediawiki.special

Bug: T126935
Change-Id: I1135397d150b5ad978da88bfd9d85c1d9e294eb7

Merge "Reset scoped session for upload jobs after deferred updates"

Merge "Allow API action=logout with BotPasswords"

Merge "Unpersist the session on session load failure"

Add access modifiers to MagicWordArray.php

Change-Id: Ib5fe0fd22a6f24142f1e87338298c1bdef45cc4a

Add access modifiers to MagicWord.php

Change-Id: I3588d9d2c17203ec3ff11736cd6ce84f687677a5

Move MagicWordArray class to own file

Change-Id: I9ba602da77f1c8d4270d8ae10d03e82a3197e5b1

Fix comment in SearchEngine.php

Change-Id: Ib00b42d2210be5bc1125fa7ba74c27a5d7fbf36c

Merge "Unpersist the session on logout"

Allow API action=logout with BotPasswords

Since login is handled specially, do the same for logout.

Bug: T128335
Change-Id: Ib3b12d7045449b5c44f5ff7d1ecfce14416e8400

Merge "Use static::class instead of get_called_class()"

Cache user data in memory

Stores user data looked up in WAN cache in memory so that lookups in the
same request do not result in new memcached lookups.

Quick and dirty solution, but nicer ones are more difficult:
* no idea what replacing WANObjectCache::get/set with getWithSetCallback
  (which has its own in-process cache) would do, the code is complex
  and completely different
* would be nice to wrap the logic into a proxy object (like CachedBagOStuff)
  but WANObjectCache calls set() internally (and marks it final), so
  inheriting is not safe; the interface and implementation should be
  separated, and that means updating all external callers which do a type
  check.
* ObjectCache::getInstance('hash') cannot be used because it has no
  item limit and this could eat up the memory with a script that iterates
  through lots of users

The patch does not attempt to replicate tombstoning for
User::clearSharedCache('refresh').

Based on Iec1504700a and Idef9a9d3.

Change-Id: I419f356b0c306d16711b433da95dccdb44645154
Co-Authored-By: Ori Livneh <ori@wikimedia.org>
Bug: T128157

Merge "Revert "Add type hint against LinkTarget""

Merge "Add CategoryMembershipChangeJobTest"

Merge "CategoryViewer: Do opportunistic category recounts for mismatched file counts, too"