@@ -173,6 +173,9 @@ |
||
| 173 | 173 | $this->assertLastMessageContains( 'Q4' ); |
| 174 | 174 | } |
| 175 | 175 | |
| 176 | + /** |
|
| 177 | + * @param string $expectedString |
|
| 178 | + */ |
|
| 176 | 179 | private function assertLastMessageContains( $expectedString ) { |
| 177 | 180 | $messages = $this->progressReporter->getMessages(); |
| 178 | 181 | |
@@ -433,6 +433,9 @@ |
||
| 433 | 433 | )->map(); |
| 434 | 434 | } |
| 435 | 435 | |
| 436 | + /** |
|
| 437 | + * @param string $idString |
|
| 438 | + */ |
|
| 436 | 439 | private function getMockEntityId( $idString ) { |
| 437 | 440 | $entityId = $this->createMock( EntityId::class ); |
| 438 | 441 | $entityId->method( '__toString' )->willReturn( $idString ); |
@@ -86,7 +86,7 @@ |
||
| 86 | 86 | } |
| 87 | 87 | |
| 88 | 88 | /** |
| 89 | - * @return AliasesProvider|EntityDocument |
|
| 89 | + * @return EntityDocument |
|
| 90 | 90 | */ |
| 91 | 91 | abstract protected function getEntity(); |
| 92 | 92 | |
@@ -67,7 +67,7 @@ |
||
| 67 | 67 | } |
| 68 | 68 | |
| 69 | 69 | /** |
| 70 | - * @return DescriptionsProvider|EntityDocument |
|
| 70 | + * @return EntityDocument |
|
| 71 | 71 | */ |
| 72 | 72 | abstract protected function getEntity(); |
| 73 | 73 | |
@@ -65,7 +65,7 @@ |
||
| 65 | 65 | } |
| 66 | 66 | |
| 67 | 67 | /** |
| 68 | - * @return LabelsProvider|EntityDocument |
|
| 68 | + * @return EntityDocument |
|
| 69 | 69 | */ |
| 70 | 70 | abstract protected function getEntity(); |
| 71 | 71 | |
@@ -190,6 +190,10 @@ |
||
| 190 | 190 | $this->fail( 'Should throw specific exception' ); |
| 191 | 191 | } |
| 192 | 192 | |
| 193 | + /** |
|
| 194 | + * @param integer $revisionId |
|
| 195 | + * @param string $slotRole |
|
| 196 | + */ |
|
| 193 | 197 | private function newRevisionRecord( $revisionId, $slotRole ) { |
| 194 | 198 | $revision = $this->createMock( RevisionRecord::class ); |
| 195 | 199 | $revision |
@@ -714,6 +714,9 @@ |
||
| 714 | 714 | ); |
| 715 | 715 | } |
| 716 | 716 | |
| 717 | + /** |
|
| 718 | + * @param integer $langCount |
|
| 719 | + */ |
|
| 717 | 720 | private function generateLanguageValuePairs( $langCount ) { |
| 718 | 721 | $result = []; |
| 719 | 722 | $langCodes = WikibaseRepo::getDefaultInstance()->getTermsLanguages()->getLanguages(); |
@@ -25,6 +25,9 @@ |
||
| 25 | 25 | $this->termValidatorFactory = $termValidatorFactory; |
| 26 | 26 | } |
| 27 | 27 | |
| 28 | + /** |
|
| 29 | + * @param NullChangeOp $changeOps |
|
| 30 | + */ |
|
| 28 | 31 | public function add( $changeOps ) { |
| 29 | 32 | $this->innerChangeOps->add( $changeOps ); |
| 30 | 33 | } |
@@ -213,6 +213,9 @@ discard block |
||
| 213 | 213 | } |
| 214 | 214 | } |
| 215 | 215 | |
| 216 | + /** |
|
| 217 | + * @param string|false $termType |
|
| 218 | + */ |
|
| 216 | 219 | private function getEntityTerm( EntityId $entityId, $lang, $termType ): string { |
| 217 | 220 | if ( $termType === 'label' ) { |
| 218 | 221 | return $this->termLookup->getLabel( $entityId, $lang ) ?? ''; |
@@ -223,6 +226,10 @@ discard block |
||
| 223 | 226 | throw new InvalidArgumentException( "\$termType can only be 'label' or 'property'. '{$termType}' was given" ); |
| 224 | 227 | } |
| 225 | 228 | |
| 229 | + /** |
|
| 230 | + * @param string $code |
|
| 231 | + * @param EntityId $collidingEntityId |
|
| 232 | + */ |
|
| 226 | 233 | private function collisionToError( $code, $collidingEntityId, $lang, $label ) { |
| 227 | 234 | return new UniquenessViolation( |
| 228 | 235 | $collidingEntityId, |