@@ -308,7 +308,7 @@ |
||
308 | 308 | * @param User $user |
309 | 309 | * @param array $oldGroups |
310 | 310 | * @param array $newGroups |
311 | - * @param array $reason |
|
311 | + * @param string $reason |
|
312 | 312 | */ |
313 | 313 | function addLogEntry( $user, $oldGroups, $newGroups, $reason ) { |
314 | 314 | $logEntry = new ManualLogEntry( 'rights', 'rights' ); |
@@ -360,7 +360,7 @@ discard block |
||
360 | 360 | |
361 | 361 | /** |
362 | 362 | * @since 1.22 Returns the HEAD date in addition to the sha1 and link |
363 | - * @return bool|string Global wgVersion + HEAD sha1 stripped to the first 7 chars |
|
363 | + * @return false|string Global wgVersion + HEAD sha1 stripped to the first 7 chars |
|
364 | 364 | * with link and date, or false on failure |
365 | 365 | */ |
366 | 366 | private static function getVersionLinkedGit() { |
@@ -934,6 +934,10 @@ discard block |
||
934 | 934 | } |
935 | 935 | } |
936 | 936 | |
937 | + /** |
|
938 | + * @param string $text |
|
939 | + * @param string $name |
|
940 | + */ |
|
937 | 941 | private function openExtType( $text = null, $name = null ) { |
938 | 942 | $out = ''; |
939 | 943 | |
@@ -1052,7 +1056,7 @@ discard block |
||
1052 | 1056 | * |
1053 | 1057 | * @since 1.23 |
1054 | 1058 | * |
1055 | - * @return bool|string False if no such file exists, otherwise returns |
|
1059 | + * @return false|string False if no such file exists, otherwise returns |
|
1056 | 1060 | * a path to it. |
1057 | 1061 | */ |
1058 | 1062 | public static function getExtAuthorsFileName( $extDir ) { |
@@ -1081,7 +1085,7 @@ discard block |
||
1081 | 1085 | * |
1082 | 1086 | * @since 1.23 |
1083 | 1087 | * |
1084 | - * @return bool|string False if no such file exists, otherwise returns |
|
1088 | + * @return false|string False if no such file exists, otherwise returns |
|
1085 | 1089 | * a path to it. |
1086 | 1090 | */ |
1087 | 1091 | public static function getExtLicenseFileName( $extDir ) { |
@@ -293,6 +293,9 @@ discard block |
||
293 | 293 | ); |
294 | 294 | } |
295 | 295 | |
296 | + /** |
|
297 | + * @param FormOptions $opts |
|
298 | + */ |
|
296 | 299 | protected function runMainQueryHook( &$tables, &$fields, &$conds, &$query_options, |
297 | 300 | &$join_conds, $opts |
298 | 301 | ) { |
@@ -516,6 +519,9 @@ discard block |
||
516 | 519 | $this->setBottomText( $opts ); |
517 | 520 | } |
518 | 521 | |
522 | + /** |
|
523 | + * @param FormOptions $options |
|
524 | + */ |
|
519 | 525 | function cutoffselector( $options ) { |
520 | 526 | // Cast everything to strings immediately, so that we know all of the values have the same |
521 | 527 | // precision, and can be compared with '==='. 2/24 has a few more decimal places than its |
@@ -625,7 +631,7 @@ discard block |
||
625 | 631 | * Count the number of items on a user's watchlist |
626 | 632 | * |
627 | 633 | * @param IDatabase $dbr A database connection |
628 | - * @return int |
|
634 | + * @return double |
|
629 | 635 | */ |
630 | 636 | protected function countItems( $dbr ) { |
631 | 637 | # Fetch the raw count |
@@ -303,10 +303,17 @@ discard block |
||
303 | 303 | } |
304 | 304 | } |
305 | 305 | |
306 | + /** |
|
307 | + * @param integer $level |
|
308 | + */ |
|
306 | 309 | protected function listStart( $level ) { |
307 | 310 | return Xml::openElement( 'ul', ( $level ? array() : array( 'id' => 'mw-whatlinkshere-list' ) ) ); |
308 | 311 | } |
309 | 312 | |
313 | + /** |
|
314 | + * @param Title $nt |
|
315 | + * @param Title $target |
|
316 | + */ |
|
310 | 317 | protected function listItem( $row, $nt, $target, $notClose = false ) { |
311 | 318 | $dirmark = $this->getLanguage()->getDirMark(); |
312 | 319 | |
@@ -405,6 +412,9 @@ discard block |
||
405 | 412 | return $this->getLanguage()->pipeList( $links ); |
406 | 413 | } |
407 | 414 | |
415 | + /** |
|
416 | + * @param string $text |
|
417 | + */ |
|
408 | 418 | function makeSelfLink( $text, $query ) { |
409 | 419 | return Linker::linkKnown( |
410 | 420 | $this->selfTitle, |
@@ -414,6 +424,9 @@ discard block |
||
414 | 424 | ); |
415 | 425 | } |
416 | 426 | |
427 | + /** |
|
428 | + * @param integer $prevId |
|
429 | + */ |
|
417 | 430 | function getPrevNext( $prevId, $nextId ) { |
418 | 431 | $currentLimit = $this->opts->getValue( 'limit' ); |
419 | 432 | $prev = $this->msg( 'whatlinkshere-prev' )->numParams( $currentLimit )->escaped(); |
@@ -32,6 +32,9 @@ |
||
32 | 32 | |
33 | 33 | public $current; |
34 | 34 | |
35 | + /** |
|
36 | + * @param ResultWrapper $res |
|
37 | + */ |
|
35 | 38 | function __construct( $res ) { |
36 | 39 | $this->res = $res; |
37 | 40 | $this->key = 0; |
@@ -265,7 +265,7 @@ discard block |
||
265 | 265 | |
266 | 266 | /** |
267 | 267 | * @param string $srcPath The source path |
268 | - * @return string|bool The real path if it was a virtual URL Returns false on failure |
|
268 | + * @return string|false The real path if it was a virtual URL Returns false on failure |
|
269 | 269 | */ |
270 | 270 | function getRealPath( $srcPath ) { |
271 | 271 | $repo = RepoGroup::singleton()->getLocalRepo(); |
@@ -1513,7 +1513,6 @@ discard block |
||
1513 | 1513 | * Check a block of CSS or CSS fragment for anything that looks like |
1514 | 1514 | * it is bringing in remote code. |
1515 | 1515 | * @param string $value a string of CSS |
1516 | - * @param bool $propOnly only check css properties (start regex with :) |
|
1517 | 1516 | * @return bool true if the CSS contains an illegal string, false if otherwise |
1518 | 1517 | */ |
1519 | 1518 | private static function checkCssFragment( $value ) { |
@@ -1887,7 +1886,7 @@ discard block |
||
1887 | 1886 | * format, presumably. |
1888 | 1887 | * |
1889 | 1888 | * @param ApiResult $result |
1890 | - * @return array Image info |
|
1889 | + * @return User Image info |
|
1891 | 1890 | */ |
1892 | 1891 | public function getImageInfo( $result ) { |
1893 | 1892 | $file = $this->getLocalFile(); |
@@ -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; |