@@ -333,6 +333,9 @@ |
||
| 333 | 333 | return $storeStatus; |
| 334 | 334 | } |
| 335 | 335 | |
| 336 | + /** |
|
| 337 | + * @param integer $index |
|
| 338 | + */ |
|
| 336 | 339 | private function getChunkFileKey( $index = null ) { |
| 337 | 340 | if ( $index === null ) { |
| 338 | 341 | $index = $this->getChunkIndex(); |
@@ -40,7 +40,7 @@ |
||
| 40 | 40 | private $repo; |
| 41 | 41 | |
| 42 | 42 | /** |
| 43 | - * @param User|bool $user Default: false |
|
| 43 | + * @param User $user Default: false |
|
| 44 | 44 | * @param UploadStash|bool $stash Default: false |
| 45 | 45 | * @param FileRepo|bool $repo Default: false |
| 46 | 46 | */ |
@@ -69,7 +69,7 @@ |
||
| 69 | 69 | /** |
| 70 | 70 | * Get a database connection for the bot passwords database |
| 71 | 71 | * @param int $db Index of the connection to get, e.g. DB_MASTER or DB_SLAVE. |
| 72 | - * @return DatabaseBase |
|
| 72 | + * @return DBConnRef |
|
| 73 | 73 | */ |
| 74 | 74 | public static function getDB( $db ) { |
| 75 | 75 | global $wgBotPasswordsCluster, $wgBotPasswordsDatabase; |
@@ -30,7 +30,7 @@ |
||
| 30 | 30 | /** |
| 31 | 31 | * Do a prefix search of user names and return a list of matching user names. |
| 32 | 32 | * |
| 33 | - * @param string|User $audience The string 'public' or a user object to show the search for |
|
| 33 | + * @param string $audience The string 'public' or a user object to show the search for |
|
| 34 | 34 | * @param string $search |
| 35 | 35 | * @param int $limit |
| 36 | 36 | * @param int $offset How many results to offset from the beginning |
@@ -146,7 +146,7 @@ |
||
| 146 | 146 | * |
| 147 | 147 | * @param string $database |
| 148 | 148 | * @param bool $ignoreInvalidDB If true, don't check if $database is in $wgLocalDatabases |
| 149 | - * @return IDatabase|null If invalid selection |
|
| 149 | + * @return DatabaseBase|null If invalid selection |
|
| 150 | 150 | */ |
| 151 | 151 | public static function getDB( $database, $ignoreInvalidDB = false ) { |
| 152 | 152 | global $wgDBname; |
@@ -171,10 +171,16 @@ |
||
| 171 | 171 | return is_object( $datum ) ? get_class( $datum ) : gettype( $datum ); |
| 172 | 172 | } |
| 173 | 173 | |
| 174 | + /** |
|
| 175 | + * @param string $expected |
|
| 176 | + */ |
|
| 174 | 177 | public static function wrongType( $expected, $datum ) { |
| 175 | 178 | return "Expected $expected, but recieved " . self::typeOf( $datum ); |
| 176 | 179 | } |
| 177 | 180 | |
| 181 | + /** |
|
| 182 | + * @param integer $datum |
|
| 183 | + */ |
|
| 178 | 184 | public static function outOfRange( $min, $max, $datum ) { |
| 179 | 185 | return "Expected value between $min and $max, but recieved $datum"; |
| 180 | 186 | } |
@@ -97,7 +97,7 @@ discard block |
||
| 97 | 97 | } |
| 98 | 98 | |
| 99 | 99 | /** |
| 100 | - * @param array $condition Query conditions suitable for use with |
|
| 100 | + * @param array $conditions Query conditions suitable for use with |
|
| 101 | 101 | * IDatabase::select |
| 102 | 102 | */ |
| 103 | 103 | public function addConditions( array $conditions ) { |
@@ -105,7 +105,7 @@ discard block |
||
| 105 | 105 | } |
| 106 | 106 | |
| 107 | 107 | /** |
| 108 | - * @param array $condition Query join conditions suitable for use |
|
| 108 | + * @param array $conditions Query join conditions suitable for use |
|
| 109 | 109 | * with IDatabase::select |
| 110 | 110 | */ |
| 111 | 111 | public function addJoinConditions( array $conditions ) { |
@@ -216,6 +216,7 @@ discard block |
||
| 216 | 216 | |
| 217 | 217 | /** |
| 218 | 218 | * @see self::generate() |
| 219 | + * @param integer $bytes |
|
| 219 | 220 | */ |
| 220 | 221 | public function realGenerate( $bytes, $forceStrong = false ) { |
| 221 | 222 | |
@@ -351,6 +352,7 @@ discard block |
||
| 351 | 352 | |
| 352 | 353 | /** |
| 353 | 354 | * @see self::generateHex() |
| 355 | + * @param integer $chars |
|
| 354 | 356 | */ |
| 355 | 357 | public function realGenerateHex( $chars, $forceStrong = false ) { |
| 356 | 358 | // hex strings are 2x the length of raw binary so we divide the length in half |
@@ -510,8 +510,8 @@ discard block |
||
| 510 | 510 | * Wait till the current timestamp reaches $time and return the current |
| 511 | 511 | * timestamp. This returns false if it would have to wait more than 10ms. |
| 512 | 512 | * |
| 513 | - * @param array $time Result of UIDGenerator::millitime() |
|
| 514 | - * @return array|bool UIDGenerator::millitime() result or false |
|
| 513 | + * @param integer[] $time Result of UIDGenerator::millitime() |
|
| 514 | + * @return resource UIDGenerator::millitime() result or false |
|
| 515 | 515 | */ |
| 516 | 516 | protected function timeWaitUntil( array $time ) { |
| 517 | 517 | do { |
@@ -571,7 +571,7 @@ discard block |
||
| 571 | 571 | } |
| 572 | 572 | |
| 573 | 573 | /** |
| 574 | - * @return array (current time in seconds, milliseconds since then) |
|
| 574 | + * @return integer[] (current time in seconds, milliseconds since then) |
|
| 575 | 575 | */ |
| 576 | 576 | protected static function millitime() { |
| 577 | 577 | list( $msec, $sec ) = explode( ' ', microtime() ); |