Remove deprecated Language::specialPage()

Was deprecated since 1.24. There were no uses in Gerrit extensions

Change-Id: If67456a62cb7d9688793d876508f0f168f3bb1ff

Remove deprecated Language::GetLanguageName()

Was deprecated since 1.20. Uses in Gerrit extensions have been replaced.

Change-Id: I623e82ec1d9ade4da05f2eae15afdf65aa04109b

Remove deprecated Language::getTranslatedLanguageNames()

Was deprecated since 1.20. There were no uses in Gerrit extensions

Change-Id: I0b16c0f146ecbee81921b5823b32776414f6945c

Remove deprecated Language::getLanguageNames()

Was deprecated since 1.20. Uses in Gerrit extensions have been replaced.

Change-Id: I2011d3e209915ddbbafdbdfff0279ea7cc5b9125

Merge "ChangesList: Use context language for LogPage::getName() message"

Merge "initSiteStats.php: Make it obvious you need to use --update"

Merge "DB: Correct argument documentation to match code."

Merge "Move EmailNotification::updateWatchlistTimestamp to WatchedItemStore"

initSiteStats.php: Make it obvious you need to use --update

Otherwise people will run the script a few times and be totally confused
as to why their statistics aren't being saved. The wording is taken from
updateArticleCount.php, which has a similar parameter.

Change-Id: I802553120c1bf4d8f6234ccced5f19ecd5cd9b66

Move EmailNotification::updateWatchlistTimestamp to WatchedItemStore

Flow is the only thing remaining that uses this deprecated method.

Change-Id: Iaa4e1e34cb3f2a91c163565fb0107c500e3852d7

DB: Correct argument documentation to match code.

Change-Id: I1cc3910932d8c4f04b1dbbc05ada88299c3ce10b

Replace uses of join() by implode()

All of core uses implode() consistently now.

Change-Id: Iba50898c64c43f356d1caf8869f484e90d9ff651

Merge "Hooks do not have to return true any more, they can return null"

Merge "Move WatchedItem logic to WatchedItemStore"

Merge "Add WatchedItemIntegrationTest"

Merge "Guard against allowing intermediate caching when cookies are present"

ChangesList: Use context language for LogPage::getName() message

Was calling escaped() directly on a Message object which resulted
in a $wgLang lookup, thus ignoring the context language given to the
class constructor.

Change-Id: Ia8ce739178924299ca559088fc40a2b048d7ed72

Merge "Use single quotes in API where possible"

Merge "MemoizedCallable: Add test for memoized closure"

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

MemoizedCallable: Add test for memoized closure

Follows-up da3443bfa2a.

It won't work across different requests due to use of uniqid()
internally (so we may want to use a HashBagOStuff for closure
instead to save memory in APC). But at least assert that it does
and is intended to work.

Change-Id: Id4b42e8717f62f733eee8fcc9a4ab6c0f5c1f2fe

Merge "Declare functions with access modifiers"

Merge "Explicitly declare ApiQueryImageInfo::getInfo() public"

Merge "Remove deprecated methods in ApiQuery"

Merge "Explicitly declare ApiImport::getData() public"

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

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

SpecialPageFactory: Update main RequestContext in addition to globals

Change-Id: I5fd7bbf0fea0e6bd8b23b8be6d47017682a31d69

Use single quotes in API where possible

Change-Id: I972e296f4820f78f5dfcecc27bc4912ca84a3178

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"

Declare functions with access modifiers

Parents have this visibility.

Change-Id: Icb4d0a112def0ac02f9aa42723c81bc63c085450

Explicitly declare ApiQueryImageInfo::getInfo() public

Used as such in core and TimedMediaHandler.

Change-Id: I0ec90d2cf9ae4cca31e93522e0f68e4d55a7bcfe

Remove deprecated methods in ApiQuery

No more uses found in core or Gerrit extensions for the following methods:

* ApiQuery::getGenerators() was removed (deprecated since 1.21).
* ApiQuery::getModuleType() was removed (deprecated since 1.21).
* ApiQuery::setGeneratorContinue() was removed (deprecated since 1.24).

Change-Id: I1d46adc9ecad598c7d7726a2243bba4f7a04531a

Explicitly declare ApiImport::getData() public

Used as such in this file.

Change-Id: I52cae5763132b606f0e4b67f1359baee9fdad1c8

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

Move WatchedItem logic to WatchedItemStore

This also removes assumptions that when a page
in one Namespace should be watched / removed
that the page in the talk / subject ns for the
page should have the same action applied

This should maintain all backward compatability
for the WatchedItem class

This also includes tests written by:
 - WMDE-leszek
 - Addshore

Bug: T127956
Change-Id: Iad9abafe4417bb479151a3bfbee6e1c78a3afe3c

Add WatchedItemIntegrationTest

Change-Id: I4760820fe075a29319e64c4dbc5d44f69848a351

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"

Hooks do not have to return true any more, they can return null

Via the argument given by Krinkle in
https://gerrit.wikimedia.org/r/#/c/274751/1/ImageMap_body.php

Quote: "[Returning true is] obsolete for a while and slowly disappearing
from existing code. Only 'return false' is an explicit signal. The
default is true. This was done because it very often is forgotten and
causes broke in production in catastrophic ways on numerous occasions.
This better reflects the mental model of intent and also makes it more
natural when dealing with hooks such as these, which can't be aborted
and as such don't have a sensible purpose in returning false, which
means returning true can be confusing."

Change-Id: I98308ed9105d904e47db3ac7899412f239c2bf9d

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