@@ -106,7 +106,7 @@ discard block |
||
| 106 | 106 | /** |
| 107 | 107 | * Creates an action and supplies it with a fake web request. |
| 108 | 108 | * |
| 109 | - * @param string|Action $action The action to call, may be an action name or class name. |
|
| 109 | + * @param string $action The action to call, may be an action name or class name. |
|
| 110 | 110 | * @param WikiPage $page the wiki page to call the action on |
| 111 | 111 | * @param array|null $params request parameters |
| 112 | 112 | * @param bool $wasPosted |
@@ -259,6 +259,10 @@ discard block |
||
| 259 | 259 | return $result; |
| 260 | 260 | } |
| 261 | 261 | |
| 262 | + /** |
|
| 263 | + * @param string $comment |
|
| 264 | + * @param integer $flags |
|
| 265 | + */ |
|
| 262 | 266 | private function createTestItem( EntityDocument $entity, $comment, $user, $flags ) { |
| 263 | 267 | $store = WikibaseRepo::getDefaultInstance()->getEntityStore(); |
| 264 | 268 | $rev = $store->saveEntity( $entity, $comment, $user, $flags ); |
@@ -180,6 +180,9 @@ |
||
| 180 | 180 | $this->timestamp = (int)$timestamp; |
| 181 | 181 | } |
| 182 | 182 | |
| 183 | + /** |
|
| 184 | + * @param string $name |
|
| 185 | + */ |
|
| 183 | 186 | private static function getRdfWriter( $name, BNodeLabeler $labeler = null ) { |
| 184 | 187 | $factory = new RdfWriterFactory(); |
| 185 | 188 | $format = $factory->getFormatName( $name ); |
@@ -224,7 +224,7 @@ discard block |
||
| 224 | 224 | * @param string[] $entityTypes |
| 225 | 225 | * @param ExceptionHandler|null $exceptionReporter |
| 226 | 226 | * |
| 227 | - * @return EntityIdReader|SqlEntityIdPager a stream of EntityId objects |
|
| 227 | + * @return EntityIdPager a stream of EntityId objects |
|
| 228 | 228 | */ |
| 229 | 229 | private function makeIdStream( array $entityTypes, ExceptionHandler $exceptionReporter = null ) { |
| 230 | 230 | $listFile = $this->getOption( 'list-file' ); |
@@ -241,7 +241,7 @@ discard block |
||
| 241 | 241 | /** |
| 242 | 242 | * Returns EntityIdPager::NO_REDIRECTS. |
| 243 | 243 | * |
| 244 | - * @return mixed a EntityIdPager::XXX_REDIRECTS constant |
|
| 244 | + * @return string a EntityIdPager::XXX_REDIRECTS constant |
|
| 245 | 245 | */ |
| 246 | 246 | protected function getRedirectMode() { |
| 247 | 247 | return EntityIdPager::NO_REDIRECTS; |
@@ -45,8 +45,8 @@ |
||
| 45 | 45 | * @param bool $allowLocalShortDesc |
| 46 | 46 | * @param array $params |
| 47 | 47 | * @param array $requestedPageIds |
| 48 | - * @param array $localDescriptions |
|
| 49 | - * @param array $centralDescriptions |
|
| 48 | + * @param array $descriptions |
|
| 49 | + * @param array $descriptions |
|
| 50 | 50 | * @param int $fitLimit |
| 51 | 51 | * @param array $expectedResult |
| 52 | 52 | * @param int $expectedContinue |
@@ -455,6 +455,10 @@ |
||
| 455 | 455 | ); |
| 456 | 456 | } |
| 457 | 457 | |
| 458 | + /** |
|
| 459 | + * @param NamespaceChecker $namespaceChecker |
|
| 460 | + * @param MockRepository $mockRepo |
|
| 461 | + */ |
|
| 458 | 462 | private function newLangLinkHandler( $namespaceChecker, $mockRepo ) { |
| 459 | 463 | $settings = $this->newSettings(); |
| 460 | 464 | |
@@ -106,8 +106,8 @@ |
||
| 106 | 106 | |
| 107 | 107 | /** |
| 108 | 108 | * @param string $baseURL |
| 109 | - * @param string $text |
|
| 110 | 109 | * @param string|null $earliestRevTimestamp |
| 110 | + * @param string $titleText |
|
| 111 | 111 | * @return Title |
| 112 | 112 | */ |
| 113 | 113 | private function mockTitle( $baseURL, $titleText, $earliestRevTimestamp = null ) { |
@@ -58,6 +58,9 @@ |
||
| 58 | 58 | return $redirects; |
| 59 | 59 | } |
| 60 | 60 | |
| 61 | + /** |
|
| 62 | + * @param integer $revision |
|
| 63 | + */ |
|
| 61 | 64 | protected function resolveLogicalRevision( $revision ) { |
| 62 | 65 | return $revision; |
| 63 | 66 | } |
@@ -147,7 +147,7 @@ |
||
| 147 | 147 | |
| 148 | 148 | /** |
| 149 | 149 | * @param $inner |
| 150 | - * @param $key |
|
| 150 | + * @param string $key |
|
| 151 | 151 | */ |
| 152 | 152 | protected function spoilTheSignature( $inner, $key ) { |
| 153 | 153 | $value = $inner->get( $key ); |
@@ -33,7 +33,7 @@ discard block |
||
| 33 | 33 | * @param EntityRevision[] $entityRevisions |
| 34 | 34 | * @param EntityRedirect[] $entityRedirects |
| 35 | 35 | * |
| 36 | - * @return EntityLookup |
|
| 36 | + * @return CacheRetrievingEntityRevisionLookup |
|
| 37 | 37 | */ |
| 38 | 38 | protected function newEntityRevisionLookup( array $entityRevisions, array $entityRedirects ) { |
| 39 | 39 | $mock = new MockRepository(); |
@@ -160,6 +160,9 @@ discard block |
||
| 160 | 160 | ->map(); |
| 161 | 161 | } |
| 162 | 162 | |
| 163 | + /** |
|
| 164 | + * @param string $message |
|
| 165 | + */ |
|
| 163 | 166 | private function assertNonexistent( LatestRevisionIdResult $result, $message ) { |
| 164 | 167 | $shouldNotBeCalled = function () use ( $message ) { |
| 165 | 168 | $this->fail( $message ); |