Merge "Avoid rebuilding database fixtures for every test run"

Avoid rebuilding database fixtures for every test run

This reduces the runtime of database-bound tests by about 40%
(on my system, from 4:55 to 2:47; results from Jenkins are
inconclusive).

The basic idea is to call addCoreDBData() only once, and have
a addDBDataOnce() that is called once per test class, not for
every test method lie addDBData() is. Most tests could be
trivially be changed to implement addDBDataOnce() instead of
addDBData(). The ones for which this did not work immediately
were left out for now. A closer look at the tests that still
implement addDBData() may reveal additional potential for
improvement.

TODO: Once this is merged, try to change addDBData() to
addDBDataOnce() where possible in extensions.

Change-Id: Iec4ed4c8419fb4ad87e6710de808863ede9998b7

Merge "Replace create_function() with a closure"

Replace create_function() with a closure

As in 266da0fb89ec4ae5 in extensions/Flow.

Change-Id: I0f6a38761cf1fa2956dad0f44fa023e1a8855df4

Convert timing to ms before we've recorded too much data

Change-Id: I0a4e18290cdc662e597484a94205a481415cbb7b

Merge "Respect undeclared command line options"

Merge "HTMLForm: Use OOUI\Element::configFromHtmlAttributes instead of rolling our own"

Merge "HTMLFormFieldWithButton: Don't make the button primary by default"

Merge "Remove some deprecated functions from ApiBase"

Localisation updates from https://translatewiki.net.

Change-Id: Iaea8b19d6e8c2cb24cb41c36f42c833c8cadf85a

Remove some deprecated functions from ApiBase

* ApiBase::addTokenProperties() was removed (deprecated since 1.24).
* ApiBase::getFinalPossibleErrors() was removed (deprecated since 1.24).
* ApiBase::getFinalResultProperties() was removed (deprecated since 1.24).
* ApiBase::getRequireAtLeastOneParameterErrorMessages() was removed (deprecated since 1.24).
* ApiBase::getPossibleErrors() was removed (deprecated since 1.24).
* ApiBase::getRequireMaxOneParameterErrorMessages() was removed (deprecated since 1.24).
* ApiBase::getRequireOnlyOneParameterErrorMessages() was removed (deprecated since 1.24).
* ApiBase::getResultProperties() was removed (deprecated since 1.24).
* ApiBase::getTitleOrPageIdErrorMessage() was removed (deprecated since 1.24).
* ApiBase::parseErrors() was removed (deprecated since 1.24).
* Remove related constants ApiBase::PROP_ROOT, ApiBase::PROP_LIST,
  ApiBase::PROP_TYPE, ApiBase::PROP_NULLABLE.

Patches were submitted for remaining uses in Gerrit extensions.

Change-Id: Idea70300874258fbcb9deef6504eb55f2ebe8d6c

HTMLFormFieldWithButton: Don't make the button primary by default

It results in multiple 'primary' buttons in forms, contrary to the
guidelines, for example on Special:Export.

Before: https://phabricator.wikimedia.org/F3605116
After: https://phabricator.wikimedia.org/F3605163

Changed Special:BotPasswords to explicitly use a 'primary' button,
since as far as I can tell it only displays a single button.

Change-Id: Ic647d2650ca635644245eb657cce045db9869fc5

Merge "Make UploadBase use TempFSFile to wrap the temporary file"

Merge "Fix mw.viewport.isElementInViewport with scrolled page"

Merge "build: Ignore phpcs in /skins but not /includes/skins"

Merge "composer-merge-plugin: Bump to latest upstream version"

Merge "Clean up XCFHandler::getImageSize()"

Use ipb_id when updating expiry in Block::updateTimestamp()

If Block::purgeExpired() hasn't been called due to low traffic, already
expired blocks might have their timestamps extended.

Bug: T128695
Change-Id: I21e7dca57bda77a755b54d0e09f340c3f0e75bd6

composer-merge-plugin: Bump to latest upstream version

v1.3.1 fixes issues with "self.version" constraints in included composer
manifests.

Depends-On: Ic34ea8b4456621e62810ad16692cc2ca7ac5f966
Change-Id: I2fa884a8ff4af42e1a62fa8e5d0799dcfc0f4547

Merge "user: Remove debug entries from User::isValidUserName() rejections"

Clean up XCFHandler::getImageSize()

No change in behavior:

* Moved setting of array elements into the initializer.
* Replaced sprintf() with variable interpolation.
* Removed a pointless assertion.

Change-Id: Ie77d26c80d592911b33bc544f831c4d34cc47d0e

Fix mw.viewport.isElementInViewport with scrolled page

With a scrolled page the function would return false since it assumed that
getBoundingClientRect returns the offset when in reality it returns the offset
relative to the window viewport.

This implementation keeps the ability to specify custom viewport rects. If we
wanted to just rely on the window viewport the implementation could be changed
with getBoundingClientRect to be much simpler.

Bug: T129466
Change-Id: I56142fbd1177fe47171a1874f73b7a8359c282ee

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

HTMLForm: Use OOUI\Element::configFromHtmlAttributes instead of rolling our own

Depends on I0e5c956b9358b510c8473b1cfe6465ea1b5c07ef in OOjs UI.

This mostly reverts dd04b310527c3ce69911789254939e9a2b627ada and parts
of e85bd04bcd26f9a05406cedf6ee0dcac0b54c435.

In addition to cleanup, it fixes bugs in HTMLFormFieldWithButton
(which did not add some attributes in OOUI mode) and HTMLMultiSelectField
(which did not do the mapping, losing some attributes in OOUI mode).

Change-Id: I0d1a5288e9edb73a0c3a8431feca9fcc67b72b6a

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"

user: Remove debug entries from User::isValidUserName() rejections

This reverts commit a8b32e0e3c (r31561).

Change-Id: I4ca7548c9fde5853a3ac9b8612a0e9a3241fd16b

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