Respect undeclared command line options

This was broken by I847d45684ccd. We should really give an error when an
undeclared option is given (not ignore the option as in the previous
code), but most commandLine.inc users rely on undeclared options being
passed through to the $options global, and the change in I847d45684ccd
was apparently accidental.

Change-Id: Icec654a386bc79bf3e4ba81644c3c51ac5e093ba

HttpFunctions: Log in English

Bug: T127233
Change-Id: Ie61030a21f5a5f5e46452c1b6b15db594d741b71

Allow passing a language into Status::getWikiText() etc.

This also corrects a bug in MessageCache::parse() where it takes a
language code string but the Parser expects a Language object in
ParserOptions.

Bug: T128809
Change-Id: I3508c9e515e505890b3c62bc4a0982aa2a313782

Check User::isSafeToLoad() in LanguageConverter

Ideally LanguageConverter shouldn't be relying on global state at all.
But as a first step let's make it not try to use the global state when
that global state isn't even there.

Bug: T127233
Change-Id: I391cef3ec211d648b078fc509e0139daa58eb875

Localisation updates from https://translatewiki.net.

Change-Id: Ifd102e2f148caf1caea83af03b8e42eff8ff72cc

Merge "OutputPage::out() was removed"

Merge "OutputPage::setAllowedModules() was removed"

Merge "OutputPage::getHeadItems() was removed"

Merge "OutputPage::getScript() was removed"

Log a backtrace with "sessions are supposed to be disabled" message

Bug: T127233
Change-Id: I27ca561ab5967cde270f2b665b51da687f0c081a

Merge "phpunit: Abstract user-lang override in MediaWikiTestCase"

phpunit: Abstract user-lang override in MediaWikiTestCase

Removed redundant set up in these classes (same as their paren
class MediaWikiLangTestCase does already).
* BlockTest
* ExportTest
* MWTimestampTest
* TitlePermissionTest

Change-Id: I28d18cb797bb249981727b02dffce4f0d8682b02

StubObject: Remove redundant _call() method in StubUserLang

Already exists the same way in the base class.

Change-Id: If6c89cf42f11655ea45a1d6e1660f6e5e2593169

mediawiki.inspect: Expose total size in bytes from mw.inspect

To make it easier for the Performance inspector plugin lets expose
the total size in bytes from inspect, so it can be used to calculate
the total size and graph the data.

There's one drawback with the fix: the total size is
also included in the console log, adding one extra field when you
log the data as a table. The method is generic that logs the data
and I don't want to change that.

Bug: T129319
Change-Id: Ic7ccbe624f07443942974a71f8a0b3f94074bb55

Don't quote assert expressions in DairikiDiff

Per HHVM issue 5128, it is not possible to use '$this' in string-literal
assert() expressions. We can either wait for this to be fixed (unlikely to
happen soon, since it involves deep interpreter internals), comment out or
remove the asserts, or simply unquote them, so that they are actual expressions
rather than strings. The downside to this is that assertions will always be
evaluated (but so what, they are extremely cheap), and that when an assertion
fail the error message will simply read 'assert(): Assertion failed in
/path/to/file on line XXX' as opposed to including the expression in the
output. Fair trade, IMO.

See: https://github.com/facebook/hhvm/issues/5128

Bug: T124163
Change-Id: Ib458b1b0c28f8d38e9df427196ae79814f6dc0c2

OutputPage::out() was removed

Deprecated since 1.22. Six use remain in Gerrit extension
SemanticPageMaker, that's been unmaintained since MediaWiki 1.17. They
were not replaced.

Change-Id: I8d68657562f683a4933cc5321d5df04c80cdfc29

OutputPage::setAllowedModules() was removed

Deprecated since 1.24. No uses in core or Gerrit extensions remaining.

Change-Id: I58af2d94f31a3d5eecc21dab2813d4f27789b511

OutputPage::getHeadItems() was removed

Deprecated since 1.24. No uses in core or extensions remaining.

Change-Id: Ic038a287dab00468b08f41f31b0c8aedffc03c64

OutputPage::getScript() was removed

Deprecated since 1.24. One use remains in Gerrit extension
SemanticPageMaker, that's been unmaintained since MediaWiki 1.17. It was
not replaced.

Change-Id: I50afa8217e7bf1b47aef3951ef2ee7295d6db21e

Merge "Minor update on login form on layout and buttons"

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

Merge "Update OOjs UI to v0.16.2"

Merge "Remove deprecated Language::getLangObj()"

Merge "Remove deprecated Language::specialPage()"

Merge "Remove deprecated Language::GetLanguageName()"

Update OOjs UI to v0.16.2

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

Change-Id: I5c7402606bbc5ea1c20548b7eae657338b4ae151

Merge "Remove deprecated Language::getTranslatedLanguageNames()"

Remove deprecated Language::getLangObj()

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

Change-Id: I7ca08416d10fbba5222cb1a1fde75b697d354b09

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

Merge "Remove deprecated Language::getLanguageNames()"

Merge "Use header_register_callback to avoid caching responses with Set-Cookie headers"

Merge "Hygiene: Add wrapHTML method"

Localisation updates from https://translatewiki.net.

Change-Id: I4d6429a915ca693c26bdce85c0fc70ffab7af876

Merge "Remove use of "successful" in strings"

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: Add support for context source in ChangesList::flag()"

Hygiene: Add wrapHTML method

Put the code that wraps the parsed output html
in a method to reduce size of prepareQuickTemplate function

This allows skins to redefine the function if they wish.

Change-Id: I8a9e732d0682af89112869a2b30f61f10f531219

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

Use header_register_callback to avoid caching responses with Set-Cookie headers

This change mirrors logic that has been in use on the Wikimedia
Foundation production cluster's Varnish cache system for over 2 years to
guard against accidentally caching backend responses which include
Set-Cookie headers.

Bug: T127993
Change-Id: Ic79cf6c959dd870d6458874a9bffe9e25aba4919

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

Minor update on login form on layout and buttons

Changing current `constructive` button to `progressive` as part of the
consolidation task T110555. Also bringing some minor layout improvements,
aligning values to CSS coding guidelines and cleaning up typos and
superflous (initial) style values.

Bug: T89860
Bug: T126307
Change-Id: I2cc642ddc78f1b436e47261568601be592fe1fa4
Depends-on: I095eddfe5ec2f1fbc81939b67ed467484e15092b

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"

ChangesList: Add support for context source in ChangesList::flag()

This way the messages are generated in the correct language instead
of relying on the user language from global context.

This should ideally become a non-static method at some point,
but currently there currently many out-of-class callers.

Change-Id: Ifb1756c1a3bddc717387ed66a58dedd4c1a7dab9

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

Remove use of "successful" in strings

Remove uses of (un)successful(ly). The words only adds bytes and nothing
to the message that something succeeded of failed. In case "unsuccesssful"
was used, this was changed to "failed" for clarity.

Change-Id: I99974855dc9d8b73a3ea43af34475c1cceab9d05

Localisation updates from https://translatewiki.net.

Change-Id: I11fb4866b2fc0c5daf9ee639d42feef2cdafe768

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

Bug: T127233
Change-Id: I3d9538b36f707ed69c2d163ccf1f807af4b02fc0