Merge "Explicitly declare ImportReporter::reportPage() and override public"

Merge "Explicitly declare ApiFormatXml::addXslt() protected"

Merge "Explicitly declare ApiFormatBase::initPrinter public"

Merge "Call static functions statically in API"

Merge "Use Elvis operator where possible in API"

Merge "Remove unused local variables in API"

Explicitly declare ImportReporter::reportPage() and override public

Used as such in SpecialImport.php.

Change-Id: Ia8cb1f4b95f43a50ff7ddd9aa4cb3ffc90f916c8

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

Merge "Fixes to masterPosWait() for master switchovers"

Merge "Fix some JobQueue IDEA errors"

Fixes to masterPosWait() for master switchovers

* Clean up return value types and docs.
* Handle master switch-over better w.r.t the job queue due
  to binlog name changes (the host portion). Previously the method
  would fail and trigger read-only mode when waiting on former
  master positions. Assume the the switch-over was done properly
  and thus return immediately.

Bug: T126436
Change-Id: Ib8c05a5c72d03a5c98e41b23c5653fc194b6d130

Merge "Correcting comments regarding $wgCacheDirectory in DefaultSettings"

Fix some JobQueue IDEA errors

Change-Id: I624f65ee1ca97c1acae9b54ca36d910eb4e42a70

Merge "Add FileOperationReplication log to improve multiwrite backend logging"

Merge "Clarify FileRepo::concatenate() docs a bit"

Merge "context: Support setting title to null"

Correcting comments regarding $wgCacheDirectory in DefaultSettings

Both the i18n and Less caches fall back to wfTempDir(), but some
others don't, so we can't make any definitive statement about what
happens when this is set to false.

Change-Id: I72e8116e9e1a7ec251db555d820fc0684c8a33b3

Ignore name-less inputs on preference page confirmCloseWindow check

Ignore any fields that will not be submitted when checking
if the user has made changes to the Preferences form.

Bug: T122702
Change-Id: I1cf27267e60f2077180b7bb6a3bc29fded4b67f6

context: Support setting title to null

We have various entry points without a title set:

* load.php: Setup.php defaults $wgTitle to null. When messages
  are parsed, MessageCache::parse() will emit a warning and uses
  $wgTitle which is then casted to Special:Badtitle.

* Parser tests: Work all with context now (or parser options)
  without ever setting $wgTitle. However, in order to allow SpecialPageFactory
  to getTitle() and then setTitle() the same value from context, context needs
  to support setting null as title, the same way it can get null as title.

Change-Id: I770f449fe67beedc3f7d4009ed2208e9dde4491c

Merge "Log violations of load.php's no-session constraint"

Add FileOperationReplication log to improve multiwrite backend logging

Change-Id: I541eac740f4a07182067294c9597cdd49330e86c

Localisation updates from https://translatewiki.net.

Change-Id: I11fb4866b2fc0c5daf9ee639d42feef2cdafe768

Log violations of load.php's no-session constraint

Bug: T127233
Change-Id: I3d9538b36f707ed69c2d163ccf1f807af4b02fc0

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"

Clarify FileRepo::concatenate() docs a bit

Change-Id: I6d978a7e6c8a3671192c346eaef590aa3033cf99

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