@@ -140,7 +140,7 @@ |
||
140 | 140 | /** |
141 | 141 | * @param ItemId $id |
142 | 142 | * @param string[] $labels |
143 | - * @param Statement[]|null $statements |
|
143 | + * @param Statement[] $statements |
|
144 | 144 | * @param SiteLink[]|null $siteLinks |
145 | 145 | * |
146 | 146 | * @return Item |
@@ -178,7 +178,7 @@ |
||
178 | 178 | /** |
179 | 179 | * @param string $msgKey |
180 | 180 | * |
181 | - * @return boolean |
|
181 | + * @return string |
|
182 | 182 | */ |
183 | 183 | private function isBceMsg( $msgKey ) { |
184 | 184 | return strstr( $msgKey, '-BCE-' ); |
@@ -103,7 +103,7 @@ |
||
103 | 103 | abstract protected function applyChanges( Item $item ); |
104 | 104 | |
105 | 105 | /** |
106 | - * @return Item|null |
|
106 | + * @return null|\Wikibase\DataModel\Entity\EntityDocument |
|
107 | 107 | */ |
108 | 108 | private function getItem() { |
109 | 109 | $params = $this->getParams(); |
@@ -176,6 +176,9 @@ discard block |
||
176 | 176 | $changeOp->apply( $item ); |
177 | 177 | } |
178 | 178 | |
179 | + /** |
|
180 | + * @param string $itemIdString |
|
181 | + */ |
|
179 | 182 | private function makeNewItemWithClaim( $itemIdString, $snak ) { |
180 | 183 | $item = new Item( new ItemId( $itemIdString ) ); |
181 | 184 | |
@@ -189,6 +192,9 @@ discard block |
||
189 | 192 | return $item; |
190 | 193 | } |
191 | 194 | |
195 | + /** |
|
196 | + * @param string $propertyId |
|
197 | + */ |
|
192 | 198 | private function makeSnak( $propertyId, $value ) { |
193 | 199 | if ( is_string( $propertyId ) ) { |
194 | 200 | $propertyId = new PropertyId( $propertyId ); |
@@ -144,7 +144,7 @@ |
||
144 | 144 | * @param array $data |
145 | 145 | * @param EntityId|string|null $id |
146 | 146 | * |
147 | - * @return object |
|
147 | + * @return EntityDocument |
|
148 | 148 | */ |
149 | 149 | public function unserializeEntity( array $data, $id = null ) { |
150 | 150 | if ( $id !== null ) { |
@@ -43,7 +43,7 @@ discard block |
||
43 | 43 | * @param object $listener |
44 | 44 | * |
45 | 45 | * @throws InvalidArgumentException |
46 | - * @return mixed The listener key, for removing the listener later. |
|
46 | + * @return integer The listener key, for removing the listener later. |
|
47 | 47 | */ |
48 | 48 | public function registerWatcher( $listener ) { |
49 | 49 | if ( !is_subclass_of( $listener, $this->interface ) ) { |
@@ -59,7 +59,7 @@ discard block |
||
59 | 59 | * Unregisters a watcher using its registration key. The watcher will no longer |
60 | 60 | * be called by dispatch(). |
61 | 61 | * |
62 | - * @param mixed $key A watcher key as returned by registerWatcher(). |
|
62 | + * @param integer $key A watcher key as returned by registerWatcher(). |
|
63 | 63 | * |
64 | 64 | * @throws InvalidArgumentException |
65 | 65 | */ |
@@ -210,7 +210,7 @@ |
||
210 | 210 | /** |
211 | 211 | * Returns a map of namespace names to URIs |
212 | 212 | * |
213 | - * @return array |
|
213 | + * @return string[] |
|
214 | 214 | */ |
215 | 215 | public function getNamespaces() { |
216 | 216 | return $this->vocabulary->getNamespaces(); |
@@ -18,6 +18,9 @@ |
||
18 | 18 | */ |
19 | 19 | class ApiJsonFormatTest extends ApiFormatTestCase { |
20 | 20 | |
21 | + /** |
|
22 | + * @param string $moduleIdentifier |
|
23 | + */ |
|
21 | 24 | private function getExpectedJson( $moduleIdentifier ) { |
22 | 25 | $json = file_get_contents( __DIR__ . '/../../data/api/' . $moduleIdentifier . '.json' ); |
23 | 26 | $json = json_decode( $json, true ); |
@@ -211,7 +211,7 @@ |
||
211 | 211 | * @param int $before insert into the internal entry list before this position. |
212 | 212 | * |
213 | 213 | * @throws MWException |
214 | - * @return array|bool the cache entry created by inserting the new chunk, or false if |
|
214 | + * @return integer|null the cache entry created by inserting the new chunk, or false if |
|
215 | 215 | * there is no more data to load from the source at the given position. |
216 | 216 | * The cache entry is an associative array containing the following keys: |
217 | 217 | * - start: the key the chunk starts at |