@@ -110,6 +110,9 @@ discard block |
||
110 | 110 | } |
111 | 111 | } |
112 | 112 | |
113 | + /** |
|
114 | + * @param string|null $statementGuid |
|
115 | + */ |
|
113 | 116 | protected function makeValidRequest( $statementGuid, array $hashes ) { |
114 | 117 | $params = array( |
115 | 118 | 'action' => 'wbremovequalifiers', |
@@ -125,6 +128,9 @@ discard block |
||
125 | 128 | $this->makeInvalidRequest( $statementGuid, $hashes, 'no-such-qualifier' ); |
126 | 129 | } |
127 | 130 | |
131 | + /** |
|
132 | + * @param string $expectedError |
|
133 | + */ |
|
128 | 134 | protected function makeInvalidRequest( $statementGuid, array $hashes, $expectedError = null ) { |
129 | 135 | $params = array( |
130 | 136 | 'action' => 'wbremovequalifiers', |
@@ -122,6 +122,9 @@ |
||
122 | 122 | $this->fail( "Failed to find submit element" ); |
123 | 123 | } |
124 | 124 | |
125 | + /** |
|
126 | + * @param string $messageText |
|
127 | + */ |
|
125 | 128 | protected function assertHtmlContainsErrorMessage( $html, $messageText ) { |
126 | 129 | $assertions = [ |
127 | 130 | [ $this, 'assertHtmlContainsFormErrorMessage' ], |
@@ -101,6 +101,8 @@ |
||
101 | 101 | |
102 | 102 | /** |
103 | 103 | * Gets a "lagged" database connection: We always leave out the first row on select. |
104 | + * @param integer $selectCount |
|
105 | + * @param integer $selectRowCount |
|
104 | 106 | */ |
105 | 107 | private function getLaggedDatabase( Database $realDB, $selectCount, $selectRowCount ) { |
106 | 108 | $db = $this->getMockBuilder( Database::class ) |
@@ -14,7 +14,6 @@ |
||
14 | 14 | use DataValues\UnboundedQuantityValue; |
15 | 15 | use DataValues\UnknownValue; |
16 | 16 | use Deserializers\Deserializer; |
17 | -use Language; |
|
18 | 17 | use MediaWikiTestCase; |
19 | 18 | use RequestContext; |
20 | 19 | use Serializers\Serializer; |
@@ -166,6 +166,9 @@ |
||
166 | 166 | return $mockDataTypeLookup; |
167 | 167 | } |
168 | 168 | |
169 | + /** |
|
170 | + * @param string $string |
|
171 | + */ |
|
169 | 172 | private function fixLineEndings( $string ) { |
170 | 173 | return preg_replace( '~(*BSR_ANYCRLF)\R~', "\n", $string ); |
171 | 174 | } |
@@ -413,7 +413,7 @@ discard block |
||
413 | 413 | * @param SkinTemplate $skinTemplate |
414 | 414 | * @param array $links |
415 | 415 | * |
416 | - * @return bool |
|
416 | + * @return null|boolean |
|
417 | 417 | */ |
418 | 418 | public static function onPageTabs( SkinTemplate &$skinTemplate, array &$links ) { |
419 | 419 | $entityContentFactory = WikibaseRepo::getDefaultInstance()->getEntityContentFactory(); |
@@ -741,7 +741,7 @@ discard block |
||
741 | 741 | * @param Title|null $title use for further information |
742 | 742 | * @param bool $local shall links be generated locally or globally |
743 | 743 | * |
744 | - * @return bool |
|
744 | + * @return boolean|null |
|
745 | 745 | */ |
746 | 746 | public static function onFormat( &$comment, $pre, $auto, $post, $title, $local ) { |
747 | 747 | global $wgLang, $wgTitle; |
@@ -859,7 +859,7 @@ discard block |
||
859 | 859 | * @param string $modelId |
860 | 860 | * @param ContentHandler|null $handler |
861 | 861 | * |
862 | - * @return bool|null False on success to stop other ContentHandlerForModelID hooks from running, |
|
862 | + * @return false|null False on success to stop other ContentHandlerForModelID hooks from running, |
|
863 | 863 | * null on error. |
864 | 864 | */ |
865 | 865 | public static function onContentHandlerForModelID( $modelId, &$handler ) { |
@@ -112,6 +112,9 @@ |
||
112 | 112 | ); |
113 | 113 | } |
114 | 114 | |
115 | + /** |
|
116 | + * @param string $languageCode |
|
117 | + */ |
|
115 | 118 | protected function getHeadingHtml( |
116 | 119 | $languageCode, |
117 | 120 | EntityId $entityId = null, |
@@ -61,7 +61,7 @@ |
||
61 | 61 | * @param string $type |
62 | 62 | * @param TermSearchResult[] $returnResults |
63 | 63 | * |
64 | - * @return ConfigurableTermSearchInteractor|\PHPUnit_Framework_MockObject_MockObject |
|
64 | + * @return ConfigurableTermSearchInteractor |
|
65 | 65 | */ |
66 | 66 | private function getMockSearchInteractor( $search, $language, $type, array $returnResults = array() ) { |
67 | 67 | $mock = $this->getMock( ConfigurableTermSearchInteractor::class ); |
@@ -9,7 +9,6 @@ |
||
9 | 9 | use Wikibase\DataModel\Entity\PropertyId; |
10 | 10 | use Wikibase\DataModel\Services\Statement\StatementGuidParser; |
11 | 11 | use Wikibase\DataModel\Statement\Statement; |
12 | -use Wikibase\Repo\WikibaseRepo; |
|
13 | 12 | |
14 | 13 | /** |
15 | 14 | * API module for creating a qualifier or setting the value of an existing one. |