@@ -100,7 +100,7 @@ discard block |
||
100 | 100 | * |
101 | 101 | * It's important that img_media_type come first, otherwise the |
102 | 102 | * tables will be fragmented. |
103 | - * @return Array Fields to sort by |
|
103 | + * @return string[] Fields to sort by |
|
104 | 104 | */ |
105 | 105 | function getOrderFields() { |
106 | 106 | return array( 'img_media_type', 'count(*)', 'img_major_mime', 'img_minor_mime' ); |
@@ -169,8 +169,9 @@ discard block |
||
169 | 169 | * Output a row of the stats table |
170 | 170 | * |
171 | 171 | * @param $mime String mime type (e.g. image/jpeg) |
172 | - * @param $count integer Number of images of this type |
|
172 | + * @param integer $count integer Number of images of this type |
|
173 | 173 | * @param $totalBytes integer Total space for images of this type |
174 | + * @param integer $bytes |
|
174 | 175 | */ |
175 | 176 | protected function outputTableRow( $mime, $count, $bytes ) { |
176 | 177 | $mimeSearch = SpecialPage::getTitleFor( 'MIMEsearch', $mime ); |
@@ -757,6 +757,9 @@ discard block |
||
757 | 757 | WatchAction::doWatchOrUnwatch( $this->watch, $nt, $user ); |
758 | 758 | } |
759 | 759 | |
760 | + /** |
|
761 | + * @param Title $title |
|
762 | + */ |
|
760 | 763 | function showLogFragment( $title ) { |
761 | 764 | $moveLogPage = new LogPage( 'move' ); |
762 | 765 | $out = $this->getOutput(); |
@@ -764,6 +767,9 @@ discard block |
||
764 | 767 | LogEventsList::showLogExtract( $out, 'move', $title ); |
765 | 768 | } |
766 | 769 | |
770 | + /** |
|
771 | + * @param Title $title |
|
772 | + */ |
|
767 | 773 | function showSubpages( $title ) { |
768 | 774 | if ( !MWNamespace::hasSubpages( $title->getNamespace() ) ) { |
769 | 775 | return; |
@@ -193,6 +193,9 @@ |
||
193 | 193 | $this->getOutput()->redirect( $this->goToUrl ); |
194 | 194 | } |
195 | 195 | |
196 | + /** |
|
197 | + * @param string $title |
|
198 | + */ |
|
196 | 199 | function showLogFragment( $title ) { |
197 | 200 | $moveLogPage = new LogPage( 'pagelang' ); |
198 | 201 | $out1 = Xml::element( 'h2', null, $moveLogPage->getName()->text() ); |
@@ -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(), |
@@ -889,7 +889,7 @@ discard block |
||
889 | 889 | /** |
890 | 890 | * Show results from other wikis |
891 | 891 | * |
892 | - * @param SearchResultSet|array $matches |
|
892 | + * @param SearchResultSet|null $matches |
|
893 | 893 | * @param string $query |
894 | 894 | * |
895 | 895 | * @return string |
@@ -1348,7 +1348,7 @@ discard block |
||
1348 | 1348 | * @since 1.18 |
1349 | 1349 | * |
1350 | 1350 | * @param string $key |
1351 | - * @param mixed $value |
|
1351 | + * @param string $value |
|
1352 | 1352 | */ |
1353 | 1353 | public function setExtraParam( $key, $value ) { |
1354 | 1354 | $this->extraParams[$key] = $value; |