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

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

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"

Simplify autocomplete attribute in HTMLForm

Follows-up 7489a3e8

Change-Id: Ifb17c88e39df7031054b3bee83772172c64d0a6b

Improve (weak and strict) type hints invarious places

Change-Id: I3a42ec1547fae971d7b495c99dd144739d8521ab

Remove leading backslashes from "use \…" tags

Change-Id: I494b029de089a07e3b946ee78293a12d5036f63e

Remove self-explaining "section heading" comments from classes

I can see that "parent::__construct" literally calls the parent
constructor. I can see that stuff preceeded by the keyword "protected"
is protected. I really (really) don't need comments explaining such.

Change-Id: I7458e714976a6acd3ba6a7c93fdc27d03903df83

Remove duplicate getTitle call in RevisionStore::getNextRevision

Change-Id: I5b7cd0e2a9ce413e582b4bdeeebc94a143896728

[MCR] pass $queryFlags into RevisionStore::getTitle

Bug: T183716
Bug: T183717
Bug: T183505
Change-Id: I15e4663902e2cbfe15b0da2e46449330e313bd0d

Merge "Add @covers tags to database tests"

Merge "tests: Complete test coverage of HtmlArmor"

Merge "Remove useless StatusTest::testCanConstruct()"

Merge "Remove resolved todo from MWNamespace test"

Merge "Improve @covers for TemplateParserTest"

Merge "make 7zip wrapper usable for dumping text revisions again"

Merge "Change php extract() to explicit code"

Merge "Add HTML attributes lang and dir to toctitle"

Merge "maintenance: Add unit test for Benchmarker class"

tests: Complete test coverage of HtmlArmor

Change-Id: I3c50ea56721226ddb37bcf4778a7d71e9b56d0f7

Add @coversNothing in places where @covers does not apply

These tests apply to things that are not relevant to PHP code coverage,
such as testing presence of messages, JSON files, or the PHPUnit tests
themselves.

Using @coversNothing indicates that there is no code here to be covered,
and prevents warnings when using --strict-coverage mode (T152923).

Change-Id: Id89ee2c15a3ce3f10e34b13fb677cd1af75af9e6

Improve @covers for TemplateParserTest

The current @covers tags for TemplateParserTest are an incomplete call
stack for TemplateParser::processTemplate, and miss out on some
functions that are called. Those functions could be repeated in @covers
tags, but it would be burdensome to keep it up to date.

Instead, just use class level @covers for the whole test case. This also
takes care of the previously uncovered testEnableRecursivePartials().

Change-Id: I475ceba26b70a115b40d7735834a07f298f7bf99

Add @covers tags to database tests

Change-Id: Ic46bb961b41856d390ffe0e3632790c506f5655b

Remove resolved todo from MWNamespace test

The todo was added by Addshore in aea1b27db02 at the same time he added
all the @covers tags, so it appears to all be resolved.

Change-Id: I03d866fdd1ca382ca2b1699e01a0dd9a28eef1ec

Remove useless StatusTest::testCanConstruct()

Neither Status, nor StatusValue have constructors, so there's not much
point testing whether the object can be constructed.

Change-Id: I170940889b5da5198ef4576d04786f84c293dd01

Add @covers tags to languages tests

I removed comments that merely repeated the location of the class being
tested. There are other tests in this directory that don't have a
corresponding class and need further investigation.

Change-Id: Ic16f0887b5030ac53fab4382cfaedfb5426cdb08

Add @covers tags to miscellaneous tests (#2)

Change-Id: I9116598bee4f4917e02290d273644c13475ff721

Localisation updates from https://translatewiki.net.

Change-Id: Icc11f7ae67c7a70f4c59eec632c74926700a69c9

maintenance: Add unit test for Benchmarker class

Change-Id: I3f835fc07741fd42b8adb5b019f72589429cc14f

Merge "phpunit: Remove outdated comment about calling of LinkCache::clear()"

Merge "mw.rcfilters.ui.SaveFiltersPopupButtonWidget: Remove pointless option"

Merge "Add grep comment for js vars wgRestriction*"

phpunit: Remove outdated comment about calling of LinkCache::clear()

This is actually being called now, from resetDB() between tests.

Change-Id: I5aa15ce3fd57b6fc90b424fde8691ff33c7ddf45

Add HTML attributes lang and dir to toctitle

The toctitle is in user interface language. This change add the
HTML attributes lang and dir to the toctitle.

This allows to copy (a part of) the rendered HTML to a word processing
and the language for spell checking has the right language.

Change-Id: I17730869f81744560672d7d0acb67ced71e69c4e

Add @covers tags API tests without them

Change-Id: Ida0f954cb0a44719af3a3d0c6bda4587c9ca42b6

Add tests for TextboxBuilder

Change-Id: I305e0bce902fb027bf02cc8a28e8846808f6437c

Add @covers tags to ContentHandler tests

Change-Id: I4d0994e91f63a0921343f34ef81a82463b4ab969

Add @covers tags to jobqueue tests

Change-Id: I030634ac87139011e5b3d461064a8c10f10cea84

Add @covers tags to HTMLForm test

Change-Id: I41cc4867b071ee9a58a43760fc4a8b301cda558e

Add @covers tags to filerepo tests

Change-Id: I696e03f9ab3934b6cd04e88e1d611060846d32a4

Add @covers tags to LinksUpdate test

Change-Id: I65ebce434f00a42e06be5168acbaa8fbf637995f

Add @covers tags to includes/debug tests

Change-Id: Ida6562c87204f1680239d0bdbfde4d6fdc9bf748

Add @covers tags for RevisionStoreDbTest

Change-Id: Ie7ce23d5ad67816c1ae8c2c838f8263fca17b4f8

Merge "Add @covers tags to miscellaneous tests"