@@ -59,7 +59,7 @@ discard block |
||
59 | 59 | /** |
60 | 60 | * Fetch dupes from all connected file repositories. |
61 | 61 | * |
62 | - * @return array Array of File objects |
|
62 | + * @return File[] Array of File objects |
|
63 | 63 | */ |
64 | 64 | function getDupes() { |
65 | 65 | return RepoGroup::singleton()->findBySha1( $this->hash ); |
@@ -67,7 +67,7 @@ discard block |
||
67 | 67 | |
68 | 68 | /** |
69 | 69 | * |
70 | - * @param array $dupes Array of File objects |
|
70 | + * @param File[] $dupes Array of File objects |
|
71 | 71 | */ |
72 | 72 | function showList( $dupes ) { |
73 | 73 | $html = array(); |
@@ -184,6 +184,9 @@ discard block |
||
184 | 184 | } |
185 | 185 | } |
186 | 186 | |
187 | + /** |
|
188 | + * @param File[] $list |
|
189 | + */ |
|
187 | 190 | function doBatchLookups( $list ) { |
188 | 191 | $batch = new LinkBatch(); |
189 | 192 | /** @var File $file */ |
@@ -35,7 +35,6 @@ |
||
35 | 35 | /** |
36 | 36 | * Implement by redirecting through Special:Redirect/file. |
37 | 37 | * |
38 | - * @param string|null $subpage |
|
39 | 38 | * @return Title |
40 | 39 | */ |
41 | 40 | public function getRedirect( $par ) { |
@@ -233,6 +233,9 @@ |
||
233 | 233 | } |
234 | 234 | } |
235 | 235 | |
236 | + /** |
|
237 | + * @param string $sourceName |
|
238 | + */ |
|
236 | 239 | private function getMappingFormPart( $sourceName ) { |
237 | 240 | $isSameSourceAsBefore = ( $this->sourceName === $sourceName ); |
238 | 241 | $defaultNamespace = $this->getConfig()->get( 'ImportTargetNamespace' ); |
@@ -144,6 +144,9 @@ discard block |
||
144 | 144 | return $subpages; |
145 | 145 | } |
146 | 146 | |
147 | + /** |
|
148 | + * @param string $par |
|
149 | + */ |
|
147 | 150 | private function parseParams( FormOptions $opts, $par ) { |
148 | 151 | # Get parameters |
149 | 152 | $parms = explode( '/', ( $par = ( $par !== null ) ? $par : '' ) ); |
@@ -215,6 +218,9 @@ discard block |
||
215 | 218 | } |
216 | 219 | } |
217 | 220 | |
221 | + /** |
|
222 | + * @param string $formcontents |
|
223 | + */ |
|
218 | 224 | private function getActionButtons( $formcontents ) { |
219 | 225 | $user = $this->getUser(); |
220 | 226 | $canRevDelete = $user->isAllowedAll( 'deletedhistory', 'deletelogentry' ); |
@@ -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' |
@@ -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 |