@@ -39,6 +39,9 @@ discard block |
||
39 | 39 | /** @var Config */ |
40 | 40 | protected $config; |
41 | 41 | |
42 | + /** |
|
43 | + * @param null|Title $title |
|
44 | + */ |
|
42 | 45 | function __construct( $title, Config $config = null ) { |
43 | 46 | if ( is_null( $title ) ) { |
44 | 47 | throw new MWException( __METHOD__ . ' given a null title.' ); |
@@ -913,6 +916,9 @@ discard block |
||
913 | 916 | return true; |
914 | 917 | } |
915 | 918 | |
919 | + /** |
|
920 | + * @param string|boolean $timestamp |
|
921 | + */ |
|
916 | 922 | private function showRevision( $timestamp ) { |
917 | 923 | if ( !preg_match( '/[0-9]{14}/', $timestamp ) ) { |
918 | 924 | return; |
@@ -1388,6 +1394,10 @@ discard block |
||
1388 | 1394 | return true; |
1389 | 1395 | } |
1390 | 1396 | |
1397 | + /** |
|
1398 | + * @param string|null $earliestLiveTime |
|
1399 | + * @param integer $remaining |
|
1400 | + */ |
|
1391 | 1401 | protected function formatRevisionRow( $row, $earliestLiveTime, $remaining ) { |
1392 | 1402 | $rev = Revision::newFromArchiveRow( $row, |
1393 | 1403 | array( |
@@ -1563,7 +1573,7 @@ discard block |
||
1563 | 1573 | /** |
1564 | 1574 | * Fetch image view link if it's available to all users |
1565 | 1575 | * |
1566 | - * @param File|ArchivedFile $file |
|
1576 | + * @param ArchivedFile $file |
|
1567 | 1577 | * @param Title $titleObj |
1568 | 1578 | * @param string $ts A timestamp |
1569 | 1579 | * @param string $key A storage key |
@@ -1599,7 +1609,7 @@ discard block |
||
1599 | 1609 | /** |
1600 | 1610 | * Fetch file's user id if it's available to this user |
1601 | 1611 | * |
1602 | - * @param File|ArchivedFile $file |
|
1612 | + * @param ArchivedFile $file |
|
1603 | 1613 | * @return string HTML fragment |
1604 | 1614 | */ |
1605 | 1615 | function getFileUser( $file ) { |
@@ -1622,7 +1632,7 @@ discard block |
||
1622 | 1632 | /** |
1623 | 1633 | * Fetch file upload comment if it's available to this user |
1624 | 1634 | * |
1625 | - * @param File|ArchivedFile $file |
|
1635 | + * @param ArchivedFile $file |
|
1626 | 1636 | * @return string HTML fragment |
1627 | 1637 | */ |
1628 | 1638 | function getFileComment( $file ) { |
@@ -214,7 +214,7 @@ |
||
214 | 214 | /** |
215 | 215 | * Show the main upload form |
216 | 216 | * |
217 | - * @param HTMLForm|string $form An HTMLForm instance or HTML string to show |
|
217 | + * @param UploadForm $form An HTMLForm instance or HTML string to show |
|
218 | 218 | */ |
219 | 219 | protected function showUploadForm( $form ) { |
220 | 220 | # Add links if file was previously deleted |
@@ -57,7 +57,7 @@ discard block |
||
57 | 57 | * |
58 | 58 | * @param string $subPage Subpage, e.g. in |
59 | 59 | * http://example.com/wiki/Special:UploadStash/foo.jpg, the "foo.jpg" part |
60 | - * @return bool Success |
|
60 | + * @return boolean|null Success |
|
61 | 61 | */ |
62 | 62 | public function execute( $subPage ) { |
63 | 63 | $this->useTransactionalTimeLimit(); |
@@ -78,7 +78,7 @@ discard block |
||
78 | 78 | * |
79 | 79 | * @param string $key The key of a particular requested file |
80 | 80 | * @throws HttpError |
81 | - * @return bool |
|
81 | + * @return boolean|null |
|
82 | 82 | */ |
83 | 83 | public function showUpload( $key ) { |
84 | 84 | // prevent callers from doing standard HTML output -- we'll take it from here |
@@ -157,7 +157,7 @@ discard block |
||
157 | 157 | * @param File $file |
158 | 158 | * @param array $params |
159 | 159 | * |
160 | - * @return bool Success |
|
160 | + * @return boolean|null Success |
|
161 | 161 | */ |
162 | 162 | private function outputThumbFromStash( $file, $params ) { |
163 | 163 | $flags = 0; |
@@ -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(); |