Merge "Fix Maintenance::output when called very early"

Fix Maintenance::output when called very early

If a maintenance script is called with --help (or without
required args), this is currently failing, as MediaWikiServices
is only loaded after Maintenance::setup is initially called.
Follows 336454104d1249439e3ff97746de69c5e7b4a5f9.

Change-Id: Ied283d362675b6b98bd3144132a240b432998991

Use namespaced version of IDatabase

Change-Id: I49c9b462d316763f90ebf5a95caa388716d6a278

Merge "Moved textbox1 building into TextConflictHelper on edit conflicts"

Follow-up 6f07389ef2eb: fix variable name

Caused Notice: Undefined variable: text

Bug: T184123
Change-Id: I950a02134b145a2928af33995ca37a6965f265e4

Merge "Cleanup live update feature flag and FilterWrapperWidget"

Merge "Remove deprecated back-compat HtmlFormatter class"

Merge "Add tests for ApiFormatRaw"

Merge "Add tests for ApiFormatBase"

Merge "Add some @noinspection flags to doPostOutputShutdown()"

Moved textbox1 building into TextConflictHelper on edit conflicts

Factored out some minor parts about building editor CSS classes.

getEditConflictMainTextBox() mainly mirrors showTextbox1 parts not
included were moved to the EditPage.

Change-Id: I671e095acc08382dd0a1c3d167fdaaa623ec5499

Add tests for ApiFormatRaw

Bug: T183767
Change-Id: I63ce42dd61f6e47f7278c436cad1e4f05e287b04

Merge "[MCR] fix RevisionStore::checkDatabaseWikiId for DB names with dashes."

[MCR] fix RevisionStore::checkDatabaseWikiId for DB names with dashes.

This is a quick hack, for a proper solution see I6a98684af1d7

Bug: T183914
Change-Id: I45a151a332145438c484e02bbe96cd85dd602b9e

Remove deprecated back-compat HtmlFormatter class

Change-Id: I9325ae046997812c607e93896e108053edca76d5

Merge "ApiComparePages: Add 'fromsection' and 'tosection' parameters"

Bump dev dependency composer/spdx-licenses to 1.2.0

Notably, this includes support for SPDX 3.0.

https://github.com/composer/spdx-licenses/compare/1.1.4...1.2.0

Deprecated license identifiers will still be accepted, but in the future we
can start warning that they are deprecated.

Bug: T183858
Change-Id: I819ce40f24d8447a20c266b2f1fa454e966dd5a5

Merge "HTMLTextAreaField: Add 'useeditfont' param"

HTMLTextAreaField: Add 'useeditfont' param

Change-Id: I7c76e911b4d71842b49272186e880051eee8de2b

Add tests for ApiFormatBase

Ensuring proper behavior of the base class lets comprehensive tests of
subclasses be simpler.

This also adjusts ApiFormatTestBase to be a bit more usable, passing an
array of options through to encodeData() instead of just a class name.
And removes the unused 'SKIP' from testGeneralEncoding, but allows
expecting an exception (for use in I63ce42dd).

Change-Id: Ib2a1fa0b04860b09105376881ff8411f9534c453

Merge "Fix RepoGroup caching bug"

Merge "Mark DBError as runtime exception"

Merge "Put menus/popups of infused PHP widgets into the default overlay"

Fix typo in PostgresUpdater in I30a3a983

Bug: T174028
Change-Id: I916c8e693e477e03295a2224320eb94599c34d65

Merge "Allow programmatic input in Command"

Allow programmatic input in Command

Bug: T182463
Change-Id: Ib68180c7af12558686f4864c24fd85f01201d6fb

Merge "Add message to the assert of ApiClearHasMsgTest"

Display the file sha1 value in the file info page

When a visitor accesses an info page of an existing file, base-36 sha1 value of this file will displayed in the table of basic information.

Bug: T181651
Change-Id: Ib8cae3e4070a186e28e8625556d3c510af0701c2

Add message to the assert of ApiClearHasMsgTest

This adds 'sanity check' message for one of asserts so it is more
clear that it is asserting a precondition.

Change-Id: I10d36c1bf69effd7bd83f4a2259144e9f6e114fc

Fix RepoGroup caching bug

'bypassCache' was renamed in I9f79e5942ced4ae13ba4de0b4c62908cc746e777
to 'latest' but the cache bypass code was not updated.

Also fix an unrelated phpdoc error.

Change-Id: I141b97300853c60092363a90af063c3fd4941d27

Mark DBError as runtime exception

Some tools (e.g. PHPStorm) can trace exceptions through the
call tree and show warnings when a call can throw an exception which
is neither handled nor acknowledged via @throws annotation.
This is quite useful but currently too noisy to use due to all methods
involving some kind of DB interaction potentially throwing a DBError.
To avoid false warnings, make DBError inherit from RuntimeException
to signal it to such tools that it is not meant to be handled.

Change-Id: I847d258791b128547e10672c5e06a9ce18458371

Convert Preferences class into PreferencesFactory service

This deprecates the Preferences class and replaces it with
a PreferencesFactory service. Basically, all code from Preferences
is moved into DefaultPreferencesFactory. All Prefereces methods
are now either shims calling DefaultPreferencesFactory or just
throw exceptions.

Bug: T178449
Change-Id: Id0b2db0c2de0890f6e1609a9a0dca207c4600f99

Merge "Add test for DBConnRef"

Put menus/popups of infused PHP widgets into the default overlay

With some special tweaks:
* Remove weird custom overlay handling in DateInputWidget that nobody
  actually used. I think if you need something special like that, you
  can just write two lines of JS code to attach the dropdown elsewhere.
* Also handle the CapsuleMultiselectWidget produced by HTMLForm's
  'multiselect' field in the same way.

Bug: T183069
Change-Id: I693f406194aeb826a3ab5bc78c97015b0b8a7fdb

OOjs UI: Fix font size for default overlay

Menus/popups inside the default overlay suffer from the same issue as
dialogs, as described in the comment here.

Bug: T183069
Depends-On: Ied0541209c4a9d7bf8c0574b0a864ba09fff1b70
Change-Id: I53888581f9e1da3b036166613c46cbc1085aa55e

Merge "Update OOjs UI to v0.24.4"

Follow-up Ic67a6f1af: RCFilters: Untypo the recurring namespaces typo

The param name is namespace, not namespaces. Do keep up, Schottlender.

Bug: T183910
Change-Id: Ib14a3918a85c22ddd2121c8dcc709ab53b5ff63f

Update OOjs UI to v0.24.4

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.24.4

Depends-On: Ia54fb8fddc159f1612ca3038cdbd7da63935aa83
Change-Id: I3e59bf24191ddb753602cd4221fe998b53d25d8c

Merge "MCR database schema"

RCFilters: Correct looking for items in namespace group

To look for the namespace items, we either search for things in the
'namesapces' view (plural) or the 'namespace' group (singular) so
the code had a typo. This is a recurring typo, but overall, it makes
more sense to look for items in the **group** of namespaces rather
than in the namespace view, conceptually.

Bug: T183910
Change-Id: Ic67a6f1afd3e95fe776fbc5c33ecb7af737055b4

Merge "Expand templates: Change 'text' to 'wikitext'"

Merge "Expand templates: Use editfont on wikitext inputs"

Pass $title to RevisionStore in Revision::newFromArchiveRow

Follow up to:
I60568b8ffd22d5e3f861c03b4b8ef14332e9015b

Change-Id: Icab36eecbaa2fdbc8832f2563786c0041930b608

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 28d7734fa2385a8fa599e3ce1bfbcd8582c05bc4

Change-Id: Iee7f3f0022c9c770c1a02b401699df0915ea7c86

Add test for DBConnRef

Change-Id: I510b2ed22b3866392592a5df53aa0f3ff6aab750

Merge "Fix border colour used for textarea"

Fix border colour used for textarea

Prevents FOUC when WikiEditor finishes loading.

Change-Id: Ib3610939cdb4b6a65f702688de9852a203ad6bab

Expand templates: Change 'text' to 'wikitext'

Change-Id: Idbddadc7187bbfe6214f313ac0205184bceb2d23

Expand templates: Use editfont on wikitext inputs

Change-Id: Ie8919b4d5caf13749d81b419aff276f7c57d7049

Merge "Add @covers tags for more tests"

Merge "HooksTest: Covers Hooks::callHook too"

Add @covers tags for more tests

Change-Id: If2254b3a9e61e056091779a1ea14237aa7b42eb6

Generalize non-digit-grouping of four-digit numbers

In some languages it's conventional not to insert a thousands
separator in numbers that are four digits long (1000-9999).
Rather than copy-paste the custom code to do this between 13 files,
introduce another option and have the base Language class handle it.

This also fixes an issue in several languages where this logic
previously would not work for negative or fractional numbers.

To implement this, a new option is added to MessagesXx.php files,
`$minimumGroupingDigits = 2;`, with the meaning as defined in
<http://unicode.org/reports/tr35/tr35-numbers.html>. It is a little
roundabout, but it could allow us to migrate the number formatting
(currently all custom code) to some generic library easily.

Bug: T177846
Change-Id: Iedd8de5648cf2de1c94044918626de2f96365d48

Merge "build: Updating mediawiki/mediawiki-codesniffer to 15.0.0"

Localisation updates from https://translatewiki.net.

Change-Id: I4e4a02ce4bfd4b5a73d87c57b3a9e8ef12db5087

HooksTest: Covers Hooks::callHook too

Change-Id: Id2b97b031b2fcf82a6fba66dae353db1acf340c3

Merge "Chinese Conversion Table Update 2017-6"

Merge "Add test for ApiClearHasMsg"

Add test for ApiClearHasMsg

Bug: T183769
Change-Id: I60c66a7cddaf03a83afd29e536281916ef3eb6d5

Bump RemexHTML version to 1.0.2

Depends-On: I9fb9978c69f37bafc0049cd7ed343c033dd50ef8
Change-Id: I25bfac2c71b50db8d64f29f4b84af0a2732f245b

build: Updating mediawiki/mediawiki-codesniffer to 15.0.0

Clean up use of @codingStandardsIgnore
- @codingStandardsIgnoreFile -> phpcs:ignoreFile
- @codingStandardsIgnoreLine -> phpcs:ignore
- @codingStandardsIgnoreStart -> phpcs:disable
- @codingStandardsIgnoreEnd -> phpcs:enable

For phpcs:disable always the necessary sniffs are provided.
Some start/end pairs are changed to line ignore

Change-Id: I92ef235849bcc349c69e53504e664a155dd162c8

Use MediaWikiCoversValidator for tests that don't use MediaWikiTestCase

Change-Id: I8c4de7e9c72c9969088666007b54c6fd23f6cc13

Localisation updates from https://translatewiki.net.

Change-Id: Ib05ba7e02b1e4fdc06ebdc819f076106ad835d0c

Merge "Add tests for ApiCheckToken"

Add tests for ApiCheckToken

Bug: T183768
Change-Id: I63ab0413252c7333f73b881995869454c4881a57

Merge "Revision: Handle all return values of Title::newFromId"

Revision: Handle all return values of Title::newFromId

In Revision::getKnownCurrent, fail early when Title::newFromId returns
null to avoid an uncatchable fatal error being triggered.

RevisionStore#getKnownCurrentRevision requires that the title parameter
be an instance of Title.

This follows on from Ia4c20a91.

Change-Id: I9bddafcc5df630d1dff1e2526194186cab7097e5

Fix typo in word "callback"

Change-Id: If47b9d80df4189317ec41998b35546a6118288c2

Localisation updates from https://translatewiki.net.

Change-Id: I7a68758df8af2b9ffdf0e8d5a2c84fdbae221b9d

Merge "Remove self-explaining "section heading" comments from classes"

Parser: extract $title, follow-up 3d560be428

In the conversion away from extract(), the $title variable was missed. This
broke LabeledSectionTransclusion.

Change-Id: If4c140aedf16fc16a4ae2361f465798055748255

ApiComparePages: Add 'fromsection' and 'tosection' parameters

The deprecated diffing in prop=revisions and such used a 'rvsection'
parameter. To enable migration of things using that, action=compare
needs corresponding section parameters.

Bug: T183823
Change-Id: If1a0e2df614c083b37640418c69cca367ce0e895

Merge "Improve (weak and strict) type hints invarious places"

Merge "Decapitalise services folder in tests/phpunit"

Merge "Append leading \ to BlobStoreFactoryTest covers"

Localisation updates from https://translatewiki.net.

Change-Id: I38e3c7018119fe60f30e1f0830eea7508d0061d5

Append leading \ to BlobStoreFactoryTest covers

Change-Id: I9b85aa9808c40aa728c6ef186c50bc8cdda08bc6

Try to opportunistically flush statsd data in maintenance scripts

This helps to avoid OOMs from buffer build-ups in the statsd
factory object. This piggy-backs on to the same checks used
for deferred update runs. In addition, the output() method
checks if the data size is getting large and emits if needed.

Bug: T181385
Change-Id: I598be98a5770f8358975815e51380c4b8f63a79e

Decapitalise services folder in tests/phpunit

Change-Id: If3bf8fd704eccdde435cb3bfd99ba1c1d3caee1f

Stop testing hhvm 3.12 on travis

WMF is using 3.18, which we're testing

Change-Id: I7749d4fe81433a1fd2e07447af1912c30d7c9330

Merge "Special:ProtectedPages: Add fields directly in $formDescriptor"

Verify that all @covers tags are sane when running tests

PHPUnit only verifies that @covers tags are correct when actually
running coverage - but that's too slow to do on every commit. And if
even a single tag is incorrect, the entire coverage job will fail. It
also has some different requirements compared to normal tags, like all
namespaced classes must use their absolute name. All of those things
combined make it easy for developers to accidentally break the coverage
job.

There are some external tools that also do this, but those tools are
incompatible with our unusual PHPUnit setup (the phpunit.php wrapper
script).

The MediaWikiCoversValidator trait just calls the same method that
PHPUnit does to validate @covers tags. It is implemented as a trait so
that test cases that implement PHPUnit_Framework_TestCase directly can
still use this.

Bug: T171899
Change-Id: I1d564bcae2bfbedb004c440b90db6341148ed4ba

Merge "API: Fix doc comment for ApiFormatNone"

Merge "[MCR] Revision::newFromArchiveRow remove recently added $title param"

Merge "Revert "[MCR] Add and use $title param to RevisionStore getPrevious/Next""

Merge "Revert "[MCR] Add optional $title param to Revision byId methods""

API: Fix doc comment for ApiFormatNone

Change-Id: I42812402b91b2775657e68189fb662a1a3d33c08

Localisation updates from https://translatewiki.net.

Change-Id: I3656774b88a5d31eff6b78779528c270b484ddd7

Merge "Make ImagePage#getThumbPrevText and ImagePage#makeSizeLink protected instead of private"

Special:ProtectedPages: Add fields directly in $formDescriptor

…instead of having separate functions.

Change-Id: Ife9ca83b97b24e4372d1b43b0eea41c3ac5c0fe2

Merge "Remove duplicate getTitle call in RevisionStore::getNextRevision"

Merge "[MCR] pass $queryFlags into RevisionStore::getTitle"

Merge "Remove leading backslashes from "use \…" tags"

[MCR] Revision::newFromArchiveRow remove recently added $title param

The $title param was added as part of
Ieabca1cf157fb667c75fc907b9da2917f71c61b3 and is not yet used
by any code.

The underlying issue with title fetching in RevisionStore has been fixed
in commit 4de36baa63afc23398f5d1747f08fc47309c314c
I15e4663902e2cbfe15b0da2e46449330e313bd0d

This patch removes the param that is not needed, but adds duplicate
logic to fetch a Title object to pass into the Revision object that
is created.
This logic is pulled directly from
RevisionStore::newRevisionFromArchiveRow.

Change-Id: I60568b8ffd22d5e3f861c03b4b8ef14332e9015b

Revert "[MCR] Add and use $title param to RevisionStore getPrevious/Next"

This reverts commit a760526bb41df8a46f70b498a2059323a99a683c.
This is a PARTIAL revert and the changes to the follwing methods remain:
 - Revision::newFromArchiveRow

This is no longer needed as the underlying issue has been fixed
in commit 4de36baa63afc23398f5d1747f08fc47309c314c
I15e4663902e2cbfe15b0da2e46449330e313bd0d

Change-Id: I3e0ae29e62d59bcb4be871ef5b389a673ce8f103

Revert "[MCR] Add optional $title param to Revision byId methods"

This reverts commit 7bfb4f195121008471974490ae24a86f2a450951.
and 56b7ba03a286c0997752fb61b14662347ea0e9de

This is no longer needed as the underlying issue has been fixed
in commit 4de36baa63afc23398f5d1747f08fc47309c314c
I15e4663902e2cbfe15b0da2e46449330e313bd0d

Bug: T183505
Change-Id: I194a558625d15fd4f7929e363557847f8bebde4f

Merge "Add @covers tags API tests without them"

Merge "Add @covers tags to languages tests"

Merge "Add @covers tags to miscellaneous tests (#2)"

Merge "Add @coversNothing in places where @covers does not apply"