@@ -244,6 +244,12 @@ |
||
244 | 244 | ) ); |
245 | 245 | } |
246 | 246 | |
247 | + /** |
|
248 | + * @param string|null $referenceHash |
|
249 | + * @param string $snaksJson |
|
250 | + * @param string $snaksOrderJson |
|
251 | + * @param string $expectedErrorCode |
|
252 | + */ |
|
247 | 253 | protected function makeInvalidRequest( |
248 | 254 | $statementGuid, |
249 | 255 | $referenceHash, |
@@ -77,6 +77,10 @@ |
||
77 | 77 | $this->assertTrue( $changeOpResult->isEntityChanged() ); |
78 | 78 | } |
79 | 79 | |
80 | + /** |
|
81 | + * @param string $itemIdString |
|
82 | + * @param PropertyValueSnak $mainSnak |
|
83 | + */ |
|
80 | 84 | private function newItemWithClaim( $itemIdString, $mainSnak ) { |
81 | 85 | $item = new Item( new ItemId( $itemIdString ) ); |
82 | 86 |
@@ -81,7 +81,7 @@ |
||
81 | 81 | * @param EntityId $id |
82 | 82 | * |
83 | 83 | * @throws InvalidArgumentException |
84 | - * @return Item|Property |
|
84 | + * @return \Wikibase\DataModel\Entity\EntityDocument |
|
85 | 85 | */ |
86 | 86 | protected function makeEntity( EntityId $id ) { |
87 | 87 | if ( $id instanceof ItemId ) { |
@@ -153,7 +153,6 @@ |
||
153 | 153 | |
154 | 154 | /** |
155 | 155 | * @param string $flavor |
156 | - * @param EntityDocument[] $entities |
|
157 | 156 | * @param EntityId[] $redirects |
158 | 157 | * |
159 | 158 | * @return RdfDumpGenerator |
@@ -222,7 +222,7 @@ discard block |
||
222 | 222 | } |
223 | 223 | |
224 | 224 | /** |
225 | - * @param $itemId |
|
225 | + * @param ItemId $itemId |
|
226 | 226 | * @return \PHPUnit\Framework\MockObject\MockObject |
227 | 227 | */ |
228 | 228 | private function getOutputPageEntityIdReaderReturningEntity( $itemId ) { |
@@ -235,8 +235,8 @@ discard block |
||
235 | 235 | } |
236 | 236 | |
237 | 237 | /** |
238 | - * @param $itemId |
|
239 | - * @return MockObject |
|
238 | + * @param ItemId $itemId |
|
239 | + * @return EntityRevisionLookup |
|
240 | 240 | */ |
241 | 241 | private function getEntityRevisionLookupReturningEntity( $itemId ): EntityRevisionLookup { |
242 | 242 | $entityRevisionLookup = $this->createMock( EntityRevisionLookup::class ); |
@@ -38,6 +38,9 @@ discard block |
||
38 | 38 | */ |
39 | 39 | class MediawikiEditEntityTest extends MediaWikiTestCase { |
40 | 40 | |
41 | + /** |
|
42 | + * @param string $name |
|
43 | + */ |
|
41 | 44 | private function getUser( $name ) { |
42 | 45 | $user = User::newFromName( $name ); |
43 | 46 | |
@@ -120,7 +123,7 @@ discard block |
||
120 | 123 | * @param EntityId $entityId |
121 | 124 | * @param EntityTitleStoreLookup $titleLookup |
122 | 125 | * @param User|null $user |
123 | - * @param bool $baseRevId |
|
126 | + * @param integer $baseRevId |
|
124 | 127 | * @param bool[]|null $permissions map of actions to bool, indicating which actions are allowed. |
125 | 128 | * @param EditFilterHookRunner|null $editFilterHookRunner |
126 | 129 | * |
@@ -224,6 +224,7 @@ |
||
224 | 224 | } |
225 | 225 | |
226 | 226 | /** |
227 | + * @param ItemId $id |
|
227 | 228 | * @return MockObject|OutputPageEntityIdReader |
228 | 229 | */ |
229 | 230 | protected function newEntityIdReaderReturningEntityId( $id ) { |
@@ -189,7 +189,7 @@ discard block |
||
189 | 189 | } |
190 | 190 | |
191 | 191 | /** |
192 | - * @return LocalizedTextProvider|MockObject |
|
192 | + * @return LocalizedTextProvider |
|
193 | 193 | */ |
194 | 194 | private function newLocalizedTextProvider(): LocalizedTextProvider { |
195 | 195 | return $this->createMock( LocalizedTextProvider::class ); |
@@ -211,6 +211,10 @@ discard block |
||
211 | 211 | ); |
212 | 212 | } |
213 | 213 | |
214 | + /** |
|
215 | + * @param ItemId|null $itemId |
|
216 | + * @param string $editLinkUrl |
|
217 | + */ |
|
214 | 218 | private function newLinkingSpecialPageLinker( $itemId, $editLinkUrl ) { |
215 | 219 | $specialPageLinker = $this->newSpecialPageLinker(); |
216 | 220 | $specialPageLinker->expects( $this->once() ) |
@@ -221,7 +225,7 @@ discard block |
||
221 | 225 | } |
222 | 226 | |
223 | 227 | /** |
224 | - * @return MockObject|SpecialPageLinker |
|
228 | + * @return SpecialPageLinker |
|
225 | 229 | */ |
226 | 230 | private function newSpecialPageLinker() { |
227 | 231 | return $this->createMock( SpecialPageLinker::class ); |
@@ -635,6 +635,9 @@ |
||
635 | 635 | $this->assertTrue( $canBeMoved ); |
636 | 636 | } |
637 | 637 | |
638 | + /** |
|
639 | + * @param integer $propertyNamespace |
|
640 | + */ |
|
638 | 641 | private function newEntitySourceBasedFederationSettings( $itemNamespace, $propertyNamespace ) { |
639 | 642 | global $wgWBRepoSettings; |
640 | 643 |