@@ -163,6 +163,9 @@ |
||
163 | 163 | return $this->existingPropNames; |
164 | 164 | } |
165 | 165 | |
166 | + /** |
|
167 | + * @param integer $limit |
|
168 | + */ |
|
166 | 169 | protected function queryExistingProps( $limit = null, $offset = 0 ) { |
167 | 170 | $opts = array( |
168 | 171 | 'DISTINCT', 'ORDER BY' => 'pp_propname' |
@@ -178,7 +178,7 @@ discard block |
||
178 | 178 | |
179 | 179 | /** |
180 | 180 | * @param string $sizetype "min" or "max" |
181 | - * @param mixed $size |
|
181 | + * @param integer $size |
|
182 | 182 | * @return string Formatted HTML |
183 | 183 | */ |
184 | 184 | protected function getSizeLimit( $sizetype, $size ) { |
@@ -279,6 +279,12 @@ discard block |
||
279 | 279 | public $mForm, $mConds; |
280 | 280 | private $type, $level, $namespace, $sizetype, $size, $indefonly, $cascadeonly, $noredirect; |
281 | 281 | |
282 | + /** |
|
283 | + * @param SpecialProtectedpages $form |
|
284 | + * @param null|string $type |
|
285 | + * @param null|string $level |
|
286 | + * @param integer|null $namespace |
|
287 | + */ |
|
282 | 288 | function __construct( $form, $conds = array(), $type, $level, $namespace, |
283 | 289 | $sizetype = '', $size = 0, $indefonly = false, $cascadeonly = false, $noredirect = false |
284 | 290 | ) { |
@@ -189,7 +189,7 @@ discard block |
||
189 | 189 | * @param float $rand Random number between 0 and 1 |
190 | 190 | * @param int $offset Extra offset to fudge randomness |
191 | 191 | * @param bool $up True to get the result above the random number, false for below |
192 | - * @return array Query information. |
|
192 | + * @return string Query information. |
|
193 | 193 | * @throws MWException |
194 | 194 | * @note The $up parameter is supposed to counteract what would happen if there |
195 | 195 | * was a large gap in the distribution of cl_timestamp values. This way instead |
@@ -231,7 +231,7 @@ discard block |
||
231 | 231 | /** |
232 | 232 | * @param float $rand Random number between 0 and 1 |
233 | 233 | * |
234 | - * @return int|bool A random (unix) timestamp from the range of the category or false on failure |
|
234 | + * @return integer A random (unix) timestamp from the range of the category or false on failure |
|
235 | 235 | */ |
236 | 236 | protected function getTimestampOffset( $rand ) { |
237 | 237 | if ( $rand === false ) { |
@@ -158,6 +158,9 @@ |
||
158 | 158 | ); |
159 | 159 | } |
160 | 160 | |
161 | + /** |
|
162 | + * @param string $randstr |
|
163 | + */ |
|
161 | 164 | private function selectRandomPageFromDB( $randstr, $fname = __METHOD__ ) { |
162 | 165 | $dbr = wfGetDB( DB_SLAVE ); |
163 | 166 |
@@ -173,7 +173,7 @@ discard block |
||
173 | 173 | * Return an array of conditions depending of options set in $opts |
174 | 174 | * |
175 | 175 | * @param FormOptions $opts |
176 | - * @return array |
|
176 | + * @return string[] |
|
177 | 177 | */ |
178 | 178 | public function buildMainQueryConds( FormOptions $opts ) { |
179 | 179 | $dbr = $this->getDB(); |
@@ -535,7 +535,7 @@ discard block |
||
535 | 535 | * Creates the choose namespace selection |
536 | 536 | * |
537 | 537 | * @param FormOptions $opts |
538 | - * @return string |
|
538 | + * @return string[] |
|
539 | 539 | */ |
540 | 540 | protected function namespaceFilterForm( FormOptions $opts ) { |
541 | 541 | $nsSelect = Html::namespaceSelector( |
@@ -561,7 +561,7 @@ discard block |
||
561 | 561 | * Create an input to filter changes by categories |
562 | 562 | * |
563 | 563 | * @param FormOptions $opts |
564 | - * @return array |
|
564 | + * @return string[] |
|
565 | 565 | */ |
566 | 566 | protected function categoryFilterForm( FormOptions $opts ) { |
567 | 567 | list( $label, $input ) = Xml::inputLabelSep( $this->msg( 'rc_categories' )->text(), |
@@ -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 ) { |
@@ -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 ) { |