mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/Cite
synced 2024-11-27 16:30:12 +00:00
Improve function and property documentation
Removed some wrong @param from test function, the @dataProvider should be a enough here Change-Id: Ib84ce497fef4d48df7547ebc38515fc377e7de01
This commit is contained in:
parent
6aa96067ba
commit
e30f055226
|
@ -1,8 +1,6 @@
|
||||||
<?xml version="1.0"?>
|
<?xml version="1.0"?>
|
||||||
<ruleset>
|
<ruleset>
|
||||||
<rule ref="./vendor/mediawiki/mediawiki-codesniffer/MediaWiki">
|
<rule ref="./vendor/mediawiki/mediawiki-codesniffer/MediaWiki" />
|
||||||
<exclude name="MediaWiki.Commenting.PropertyDocumentation.MissingDocumentationProtected" />
|
|
||||||
</rule>
|
|
||||||
<file>.</file>
|
<file>.</file>
|
||||||
<arg name="extensions" value="php" />
|
<arg name="extensions" value="php" />
|
||||||
<arg name="encoding" value="UTF-8" />
|
<arg name="encoding" value="UTF-8" />
|
||||||
|
|
|
@ -16,6 +16,7 @@ use ResourceLoaderContext;
|
||||||
*/
|
*/
|
||||||
class CiteDataModule extends \ResourceLoaderModule {
|
class CiteDataModule extends \ResourceLoaderModule {
|
||||||
|
|
||||||
|
/** @inheritDoc */
|
||||||
protected $targets = [ 'desktop', 'mobile' ];
|
protected $targets = [ 'desktop', 'mobile' ];
|
||||||
|
|
||||||
/** @inheritDoc */
|
/** @inheritDoc */
|
||||||
|
|
|
@ -143,12 +143,6 @@ class FootnoteMarkFormatterTest extends \MediaWikiUnitTestCase {
|
||||||
* @covers ::getLinkLabel
|
* @covers ::getLinkLabel
|
||||||
*
|
*
|
||||||
* @dataProvider provideGetLinkLabel
|
* @dataProvider provideGetLinkLabel
|
||||||
*
|
|
||||||
* @param string|null $expectedLabel
|
|
||||||
* @param int $offset
|
|
||||||
* @param string $group
|
|
||||||
* @param string $label
|
|
||||||
* @param string|null $labelList
|
|
||||||
*/
|
*/
|
||||||
public function testGetLinkLabel( $expectedLabel, $offset, $group, $labelList ) {
|
public function testGetLinkLabel( $expectedLabel, $offset, $group, $labelList ) {
|
||||||
$mockMessageLocalizer = $this->createMock( ReferenceMessageLocalizer::class );
|
$mockMessageLocalizer = $this->createMock( ReferenceMessageLocalizer::class );
|
||||||
|
|
Loading…
Reference in a new issue