rdbms: Document varargs for IDatabase::buildLike

This is needed in order for Phan not to consider calls to
IDatabase::buildLike as invalid. Interestingly, it does not
consider calls to Database::buildLike invalid.

Bug: T191668
Change-Id: I0e027f5ec66d20b1d11e3441086001f6a751e1f5

Various cleanups to MediaWiki::preOutputCommit

Do not send headers if they were already flushed. Split off some
chronology protection logic into a separate private method. Use
ILBFactory over LBFactory in a few places. Also, update various
code comments.

Bug: T225655
Change-Id: Iecb574e11d8ba09147ff7b84ad57d8845069ba99

Merge "Add link trail for Azerbaijani language"

Reduce selector specificity

Adapt to current Vector code.

Change-Id: Ie5a13f927810cea465fa984c68cdef3664042a41

Merge "shell: Remove documentation that doesn't add meaning to the method"

DevelopmentSettings: Document why wgRateLimits is disabled

Follows-up 9c52f982d8.

Bug: T225796
Change-Id: I6f7a75d58c61712233134a9d480ce68719d6cb6a

Merge "Remove unused/deprecated Replacer classes"

Merge "Installer: Use HTML/CSS instead of JavaScript to toggle the help message"

Merge "Add a report about slow PHPUnit tests"

Add a report about slow PHPUnit tests

johnkary/phpunit-speedtrap prints this information right before the
test results.

Bug: T225730
Change-Id: I288dcd6b3c9b2c0b31c6047eaf48a23e13af726e

Localisation updates from https://translatewiki.net.

Change-Id: I78a1cea264df6fa322b5df4a342882fb867a2c1a

Merge "Use [...] instead of array(...) in PHP comments and documentation"

Remove unused/deprecated Replacer classes

Change-Id: Ifec6c624810b2354f2fab2c1fd4a27267b625c4b

Merge "Installer: Make short header and normal header of WebInstaller equivalent"

Installer: Make short header and normal header of WebInstaller equivalent

Change-Id: I8ae0cca582c2dab87a4b3b050030f6c96897939e

Use [...] instead of array(...) in PHP comments and documentation

Change-Id: I0c83783051bf35fe785bc01644eeb2946902b6b2

Merge "Fix some issues with HTMLSelectAndOtherField default and validation"

Merge "resourceloader: Remove unused static $contexts var in ResourceLoaderImageTest"

Merge "FileRepo: Use Late Static Binding in File static constructors"

resourceloader: Remove unused static $contexts var in ResourceLoaderImageTest

Change-Id: I95abbfd6d2baafb9378072fdf1a1fc5ed046cec3

Merge "Filter out blocks with duplicate IDs when checking for blocks"

Merge "rdbms: clean up and simplify toString() handle for Database"

shell: Remove documentation that doesn't add meaning to the method

Change-Id: I924453f683f058586516bb12e54940449d29cfd9

Filter out blocks with duplicate IDs when checking for blocks

Bug: T225919
Change-Id: I76549072d53083e6057f4fd8fe963e8989daa25c

languages: Use "your" instead of "my" in the updatedmarker value

Bug: T222260
Change-Id: I2faf7bde8ebb79fb296ce5e52535f2290c290ef8

Merge "[bugfix] Fetch tag ID before calling undefineTag()"

Disable rate limiting in Development Settings

Bug: T225796
Change-Id: I2475a04066d4aaefeba372bd223ef68548a8cf18

Merge "Fix param type of search terms in search related classes"

Merge "rdbms: allow synchronous=OFF for sqlite"

Merge "search: clean up some type hints in search classes"

Merge "rdbms: do not close the connection in LoadBalancerSingle::__destruct"

rdbms: allow synchronous=OFF for sqlite

Change-Id: I58be09bb170399f3ad94fc3bcc4881932d357a9b

search: clean up some type hints in search classes

Change-Id: I357a78b42ea02a63c3b64e8982f04314cbbfdf9f

rdbms: do not close the connection in LoadBalancerSingle::__destruct

The connection instance is injected unlike with the parent class,
meaning that it does not own the connection. No-op the method.

Change-Id: I61914b986619f201ac9cc8c94d46873a3b9e8177

Only attempt to deduplicate if there is data in archive and revision

The idea is to avoid expensive calls to makeDummyRevisionRow, and speed up
installation of MediaWiki on CI.

Bug: T225901
Change-Id: I6f69281568218c89eb18353c06cabf7eb1926de8

Localisation updates from https://translatewiki.net.

Change-Id: I84ab99cdcd91ee6dab8ae284e8b1c6aea53cefac

[bugfix] Fetch tag ID before calling undefineTag()

undefineTag() also deletes the tag if it's not used anywhere,
which is breaking the rest of deleteTagEverywhere() in that case.

Bug: T225564
Change-Id: I7ca5db9efd0088b266e33c0a9ce78d73a4fa87c9

Use secure protocols on Wikimedia sites

Change-Id: I3af0038202027a737a306ae677f7cda2b9857404

Merge "Reduce HashRing test load to avoid several seconds of CPU"

FileRepo: Use Late Static Binding in File static constructors

The FileRepo extension API allows to specify factory methods that
FileRepo implementations will use to instantiate File instances.
Currently, the default static constructors in LocalFile and OldLocalFile
do not use Late Static Binding, so every subclass is forced to re-implement them,
even if they would not need any custom logic. Switching to Late Static Binding
(available since PHP 5.3) allows File implementors to reduce boilerplate
if they do not need to overwrite the existing logic.

Change-Id: Id8f6f5362b68269c2a3232796a1703be14116dd5

rdbms: clean up and simplify toString() handle for Database

Change-Id: Ia95e25505785aad71b70a74f4ec7a07a73e419e1

Installer: Use HTML/CSS instead of JavaScript to toggle the help message

Changes in the behavior:
* The expanding/collapsing now works even with disabled JavaScript.
* There is no FOUC on loading anymore.
* The animation on show/hide is absent.

Change-Id: I1b88d8e2cdbb1c969670a7a8637dba10bf447330

Merge "resourceloader: Support 'versionCallback' for computed package files"

Fix some issues with HTMLSelectAndOtherField default and validation

Bring HTMLSelectAndOtherField in line with other HTMLFormFields by
ensuring that the default value is of the correct type and passes
validation checks.

Also make sure HTMLSelectAndOtherField::validate checks for the
final value, if the field has required set to true.

Bug: T222170
Bug: T225860
Change-Id: I949ee3df2b1f597982cf522b149c54b8e79d59bc

Merge "PHPVersionCheck: remove version_compare() existence check"

PHPVersionCheck: remove version_compare() existence check

It was added to PHP in 2001.

Change-Id: I3b14d726aa1cefd6b64e4bac39ef489b3c79ac1a

Merge "mw.Uri: Use more intuitive variable names"

Merge "Hard deprecate Preprocessor_DOM"

Merge "Use HTTPS for creativecommons.org"

Merge "resourceloader: Validate ResourceLoaderContext::getDirection() input"

Merge "Installer: Move style of background image from HTML to CSS"

Merge "user: Move idFromName cache truncation to before adding the new value"

resourceloader: Validate ResourceLoaderContext::getDirection() input

Only dir=ltr and dir=rtl are now allowed. Ignore other values.

Change-Id: Id39471e8a792c7c48ff7ca9d80be2e6dd4caee6b

Merge "Add a custom block message for composite blocks"

user: Move idFromName cache truncation to before adding the new value

This prevents a double query for the same User::idFromName() call
twice in a row.

Bug: T225843
Change-Id: I5beb0a3ad5e715b68dd8a450408fb089184fa6ce

mw.Uri: Use more intuitive variable names

Change-Id: I8e53d7a0e0b89909a5c3798a819a0690c757a060

Fix param type of search terms in search related classes

Change-Id: I036434268f1e7992f7f9f23d2c1c0fcd399823c8

Add link trail for Azerbaijani language

Bug: T224614
Change-Id: If17cd3906d17cf5ffee6f6e99f46eb09b28a7153

resourceloader: Support 'versionCallback' for computed package files

The use cases we've seen for using computed (or virtual) package files,
involve expensive computations to expand and transform data for the client
that we don't want to evaluate in full just to compute the module's version
hash (e.g. in the StartupModule, where we need to do this for 1000s of
modules).

For such cases, the module can specify a 'versionCallback' of which the
return value will be used to seed the module's version hash. The default
remains the same as before, which is to use the full content to seed the
version hash (via getDefinitionSummary).

Bug: T223260
Change-Id: I76f573239e6bd429287e7adb33a92ffd5e260c20

resourceloader: Migrate use of 'raw' modules to 'raw' requests

The ResourceLoaderModule::isRaw() feature and the ability to magically
switch a regular load.php request into raw mode is being removed soon.

Instead, specify raw=1 in the request url where that behaviour is needed.

Bug: T201483
Change-Id: Ie4564ec8e26ad53f2de1a43330d18a35b0498a63

Hard deprecate Preprocessor_DOM

The Preprocessor_DOM implementation doesn't interact well with PHP memory
profiling, and has some limitations not present in the Preprocessor_Hash
implementation (see T216664).  There is no reason to keep around two
versions of the preprocessor: it just complicates on-going wikitext
feature development.

Hard deprecate use of Preprocessor_DOM, so we can remove the redundant
code in a future release.

Bug: T204945
Depends-On: Id38c9360e4d02b570996dbf7a660f964f02f1a2c
Change-Id: Ica5d1ad5b1e677542962fc36d582a793f941155e

resourceloader: Remove redundant 'group' setting for StartupModule

This feaure exists for controlling cache fragmentation of modules
that are queued for loading. However, the startup module is never
"queued" for loading.

It's loading is hardwired into ResourceLoaderClientHtml and doesn't
use any batching and couldn't logically, given that by design it
has to be the first thing that loads and cannot be combined with
other code (because we need to be able to return early and control
what else gets loaded from there).

[0] https://www.mediawiki.org/wiki/ResourceLoader/Architecture#Balance

Change-Id: If8900c538ce623f6a9a68a3bbef0927b4b6ae346

Add a custom block message for composite blocks

Give a custom reason and include the user's IP address and the
expiry of the longest block.

Bug: T225748
Change-Id: Ie147cca625365c1a01e2ed18819e5d6e3946865e

Reduce HashRing test load to avoid several seconds of CPU

Bug: T225719
Change-Id: I358383e99d7950c4747b48583dc8faf00b3deeab

Merge "Migrate remaining usages of Title::userCan() to PermissionManager"

Merge "Defines: Drop DB_SLAVE, deprecated since 1.28"

Defines: Drop DB_SLAVE, deprecated since 1.28

Change-Id: I9b48524bfa071c21e03c50e9e8231e4661917c78

Use HTTPS for creativecommons.org

http://creativecommons.org/ redirects to https://creativecommons.org/

Change-Id: I625a09b32a30df2043620c0ce0756b2a963ba0c7

Installer: Move style of background image from HTML to CSS

Change-Id: I5fb8a9fc87290c407be123238397accbc5a886dc

REST: Rename attributes to path params

Change-Id: I1cd7297715bf0f9902949a5117ea7ab94b689a37

REST: Testable EntryPoint

* Split EntryPoint into a static main() and a non-static execute()
* Add tests for execute()

Change-Id: I025356b04ddc5a16494f98c446d785d6bb05ab10

REST: HeaderContainer should start empty not null

Change-Id: I30b68862307f2b44e4f5708c06464e0a218c1ed5

StringStream and PathMatcher tests

Change-Id: Ic4c6e15ef9a107608fe8e72d469987ccce21f98f

Fix HHVM StringStream test errors

Change-Id: I3eebd14db7157bb423ee88af7b49d92f628bc771

Merge "Fix "succesful" typo"

Merge "StringStream::copyToStream() should adjust the offset"

Merge "REST: tests for HelloHandler and HeaderContainer"

Merge "REST: Implement 405 responses"

Fix "succesful" typo

Change-Id: I1f49ce9ce104263adc8763947dbd30634f18697d

Merge "build: Add missing dashes in .travis.yml (restore PHP 7.1/7.2 tests)"

Merge "Fix doc type of Installer::addInstallStep"

Merge "Clarify MediaWikiTestCase::getSchemaOverrides()"

Merge "Storage: Type against ILBFactory and ILoadBalancer in storage classes"

Merge "libs: Remove unused deprecated BufferingStatsdDataFactory::getBuffer()"

Merge "SpecialEmailUser: Remove validate/get target without sender specified"

Merge "Revert "Separate MediaWiki unit and integration tests""

Update monolog/monolog from 1.22.1 -> 1.24.0

https://github.com/Seldaek/monolog/releases/tag/1.23.0
https://github.com/Seldaek/monolog/releases/tag/1.24.0
https://github.com/Seldaek/monolog/compare/1.22.1...1.24.0

Change-Id: I83b4b2e0b2cb0061daacb52e029dbab2f4e1cc95
Depends-On: I32633c85a4b1eb7919a1ce8c9b6693227f2ae55b

Revert "Separate MediaWiki unit and integration tests"

This reverts commit 0a2b996278e57a8b8c5377cd3a3eaa54f993d4a9.

Reason for revert: Broke postgres tests.

Change-Id: I27d8e0c807ad5f0748b9611a4f3df84cc213fbe1

build: Add missing dashes in .travis.yml (restore PHP 7.1/7.2 tests)

Follows-up f638c599d5, in which I accidentally removed the leading
dashes from the entries in the 'include' array.

Apparently when you do that, it just assumes you intended to
have a dash in front of the first line and create a single-item
object in which you have the same key four times, where the last
one wins. Thus, testing only 1 variant: php 7.0.

Change-Id: I3fd77749017d15cc60daf9a531417b55c604e7ff

Merge "Expand ResponseFactory"

Merge "REST API initial commit"

libs: Remove unused deprecated BufferingStatsdDataFactory::getBuffer()

Usage
=====

https://codesearch.wmflabs.org/search/?q=%5CbgetBuffer%5Cb&i=nope&files=&repos=

Bug: T220656
Change-Id: I3d9637e1802e0d34698940f400c628f29f0aab3b

SpecialEmailUser: Remove validate/get target without sender specified

This was deprecated in 1.30 and per usage below, these has been cleaned
up. Passing "null" is deprecated and no longer used anywhere. All usage
now pass a user object that is the sender.

Usage
=====

* validateTarget():
https://codesearch.wmflabs.org/search/?q=%5CbvalidateTarget%5Cb&i=nope&files=&repos=

* getTarget():
https://codesearch.wmflabs.org/search/?q=%5Cb(-%3E%7C%3A%3A)getTarget%5Cb&i=nope&files=&repos=

Change-Id: I22ed2bc42216712602f8b4685035c3396d3d1e5c

Storage: Type against ILBFactory and ILoadBalancer in storage classes

Instead of the LBFactory/LoadBalancer implementations.

Change-Id: Ia58d9056bd60f3f094fbb16f7b9afa64e11143c8

Merge "Separate MediaWiki unit and integration tests"

build: Remove dbtype=postgres from Travis CI matrix

This has since been added to  WMF CI (and voting).

What remains is just secondary confirmation that the tests can
all pass a third-party system (e.g. not the WMF-specific Linux/Docker
environment), for supported PHP versions.

Change-Id: Ic257a34630c61d5328afeaa6dc5685581e03a528

Separate MediaWiki unit and integration tests

This changeset implements T89432 and related tickets and is based on exploration
done at the Prague Hackathon. The goal is to identify tests in MediaWiki core
that can be run without having to install & configure MediaWiki and its dependencies,
and provide a way to execute these tests via the standard phpunit entry point,
allowing for faster development and integration with existing tooling like IDEs.

The initial set of tests that met these criteria were identified using the work Amir did in
I88822667693d9e00ac3d4639c87bc24e5083e5e8. These tests were then moved into a new subdirectory
under phpunit/ and organized into a separate test suite. The environment for this suite
is set up via a PHPUnit bootstrap file without a custom entry point.

You can execute these tests by running:
$ vendor/bin/phpunit -d memory_limit=512M -c tests/phpunit/unit-tests.xml

Bug: T89432
Bug: T87781
Bug: T84948
Change-Id: Iad01033a0548afd4d2a6f2c1ef6fcc9debf72c0d

Merge "rdbms: avoid LoadBalancer::getConnection waste when using $groups"

resourceloader: Skip modules with circular deps in tree optimiser

Either the server needs to omit these from the registry with
state=error output to the client (and server-side error logging),
or it needs to detect them, and transport them unchanged, so that
the existing client-side logic can handle it.

This patch does the latter.

Without the source code change in this patch, the added test case
fails due to "top" and "middle1" then being registered with
an empty array as dependencies.

Bug: T223402
Change-Id: I57502d7c4e434de4737759aed325dd4200ca89bf

Localisation updates from https://translatewiki.net.

Change-Id: I2557496565cbdfee6bba829b3d2fdd36875aa5f7