@@ -151,6 +151,9 @@ |
||
| 151 | 151 | ); |
| 152 | 152 | } |
| 153 | 153 | |
| 154 | + /** |
|
| 155 | + * @return string |
|
| 156 | + */ |
|
| 154 | 157 | private function executeSpecialEntityRedirect( array $params, User $user = null ) { |
| 155 | 158 | if ( !$user ) { |
| 156 | 159 | // TODO Matching the token of a non-anonymous user is complicated. |
@@ -123,7 +123,7 @@ |
||
| 123 | 123 | |
| 124 | 124 | /** |
| 125 | 125 | * @param Title[] $titles |
| 126 | - * @param int|null $continue |
|
| 126 | + * @param integer $continue |
|
| 127 | 127 | * |
| 128 | 128 | * @return array |
| 129 | 129 | */ |
@@ -89,7 +89,7 @@ |
||
| 89 | 89 | /** |
| 90 | 90 | * @param User $user the user performing the edit |
| 91 | 91 | * @param EntityId|null $entityId the id of the entity to edit |
| 92 | - * @param bool|int|null $baseRevId the base revision ID for conflict checking. |
|
| 92 | + * @param integer $baseRevId the base revision ID for conflict checking. |
|
| 93 | 93 | * Use 0 to indicate that the current revision should be used as the base revision, |
| 94 | 94 | * effectively disabling conflict detections. true and false will be accepted for |
| 95 | 95 | * backwards compatibility, but both will be treated like 0. Note that the behavior |
@@ -17,9 +17,9 @@ |
||
| 17 | 17 | private $buffer; |
| 18 | 18 | |
| 19 | 19 | /** |
| 20 | - * @param array $entityIds |
|
| 21 | - * @param array|null $termTypes if null, defaults to labels and descriptions only |
|
| 22 | - * @param array|null $languageCodes if null, defaults to de and en |
|
| 20 | + * @param \Wikibase\DataModel\Entity\ItemId[] $entityIds |
|
| 21 | + * @param string[] $termTypes if null, defaults to labels and descriptions only |
|
| 22 | + * @param string[] $languageCodes if null, defaults to de and en |
|
| 23 | 23 | */ |
| 24 | 24 | public function prefetchTerms( array $entityIds, array $termTypes, array $languageCodes ) { |
| 25 | 25 | if ( $termTypes === null ) { |
@@ -91,6 +91,10 @@ |
||
| 91 | 91 | $this->assertSame( $expected, $entityContent->getTextForSearchIndex() ); |
| 92 | 92 | } |
| 93 | 93 | |
| 94 | + /** |
|
| 95 | + * @param string $languageCode |
|
| 96 | + * @param string $text |
|
| 97 | + */ |
|
| 94 | 98 | private function setLabel( EntityDocument $entity, $languageCode, $text ) { |
| 95 | 99 | if ( !( $entity instanceof LabelsProvider ) ) { |
| 96 | 100 | throw new InvalidArgumentException( '$entity must be a LabelsProvider' ); |
@@ -57,6 +57,9 @@ discard block |
||
| 57 | 57 | ); |
| 58 | 58 | } |
| 59 | 59 | |
| 60 | + /** |
|
| 61 | + * @return EntityDocument |
|
| 62 | + */ |
|
| 60 | 63 | private function newUnsupportedEntity() { |
| 61 | 64 | return $this->createMock( EntityDocument::class ); |
| 62 | 65 | } |
@@ -68,6 +71,9 @@ discard block |
||
| 68 | 71 | $writer->deleteTermsOfEntity( $this->newUnsupportedId() ); |
| 69 | 72 | } |
| 70 | 73 | |
| 74 | + /** |
|
| 75 | + * @return EntityId |
|
| 76 | + */ |
|
| 71 | 77 | public function newUnsupportedId() { |
| 72 | 78 | return $this->createMock( EntityId::class ); |
| 73 | 79 | } |
@@ -57,6 +57,9 @@ discard block |
||
| 57 | 57 | ); |
| 58 | 58 | } |
| 59 | 59 | |
| 60 | + /** |
|
| 61 | + * @return EntityDocument |
|
| 62 | + */ |
|
| 60 | 63 | private function newUnsupportedEntity() { |
| 61 | 64 | return $this->createMock( EntityDocument::class ); |
| 62 | 65 | } |
@@ -68,6 +71,9 @@ discard block |
||
| 68 | 71 | $writer->deleteTermsOfEntity( $this->newUnsupportedId() ); |
| 69 | 72 | } |
| 70 | 73 | |
| 74 | + /** |
|
| 75 | + * @return EntityId |
|
| 76 | + */ |
|
| 71 | 77 | public function newUnsupportedId() { |
| 72 | 78 | return $this->createMock( EntityId::class ); |
| 73 | 79 | } |
@@ -25,6 +25,11 @@ |
||
| 25 | 25 | */ |
| 26 | 26 | trait DatabaseTermStoreWriterTestGetTermsTrait { |
| 27 | 27 | |
| 28 | + /** |
|
| 29 | + * @param string $termsTable |
|
| 30 | + * @param string $termInLangField |
|
| 31 | + * @param string $idField |
|
| 32 | + */ |
|
| 28 | 33 | private function getTerms( Int32EntityId $entityId, $termsTable, $termInLangField, $idField ): Fingerprint { |
| 29 | 34 | $loadBalancer = new FakeLoadBalancer( [ |
| 30 | 35 | 'dbr' => $this->db, |
@@ -85,6 +85,7 @@ discard block |
||
| 85 | 85 | } |
| 86 | 86 | |
| 87 | 87 | /** |
| 88 | + * @param EntityNamespaceLookup $namespaceLookup |
|
| 88 | 89 | * @return WikiPageEntityMetaDataLookup |
| 89 | 90 | */ |
| 90 | 91 | private function getWikiPageEntityMetaDataLookup( $namespaceLookup = null ) { |
@@ -163,6 +164,8 @@ discard block |
||
| 163 | 164 | |
| 164 | 165 | /** |
| 165 | 166 | * Gets a "lagged" database connection: We always leave out the first row on select. |
| 167 | + * @param integer $selectCount |
|
| 168 | + * @param integer $selectRowCount |
|
| 166 | 169 | */ |
| 167 | 170 | private function getLaggedDatabase( IDatabase $realDB, $selectCount, $selectRowCount ) { |
| 168 | 171 | $db = $this->getMockBuilder( IDatabase::class ) |