@@ -131,7 +131,7 @@ discard block |
||
131 | 131 | * When implemented in a descendant class, receives an array of titles as strings and returns |
132 | 132 | * either an unmodified array or an array of Title objects corresponding to strings received. |
133 | 133 | * |
134 | - * @param array $strings |
|
134 | + * @param string[] $strings |
|
135 | 135 | * |
136 | 136 | * @return array |
137 | 137 | */ |
@@ -164,6 +164,10 @@ discard block |
||
164 | 164 | return $this->strings( $this->handleResultFromHook( $srchres, $namespaces, $search, $limit ) ); |
165 | 165 | } |
166 | 166 | |
167 | + /** |
|
168 | + * @param string $search |
|
169 | + * @param integer $limit |
|
170 | + */ |
|
167 | 171 | private function handleResultFromHook( $srchres, $namespaces, $search, $limit ) { |
168 | 172 | $rescorer = new SearchExactMatchRescorer(); |
169 | 173 | return $rescorer->rescore( $search, $namespaces, $srchres, $limit ); |
@@ -102,7 +102,7 @@ |
||
102 | 102 | * With conflicting expectations, the most narrow ones will be used |
103 | 103 | * |
104 | 104 | * @param array $expects Map of (event => limit) |
105 | - * @param $fname |
|
105 | + * @param string $fname |
|
106 | 106 | * @since 1.26 |
107 | 107 | */ |
108 | 108 | public function setExpectations( array $expects, $fname ) { |
@@ -207,6 +207,10 @@ discard block |
||
207 | 207 | return $result; |
208 | 208 | } |
209 | 209 | |
210 | + /** |
|
211 | + * @param string $filter |
|
212 | + * @param string $data |
|
213 | + */ |
|
210 | 214 | private static function applyFilter( $filter, $data ) { |
211 | 215 | $data = trim( $data ); |
212 | 216 | if ( $data ) { |
@@ -309,7 +313,7 @@ discard block |
||
309 | 313 | * @throws MWException If a duplicate module registration is attempted |
310 | 314 | * @throws MWException If a module name contains illegal characters (pipes or commas) |
311 | 315 | * @throws MWException If something other than a ResourceLoaderModule is being registered |
312 | - * @return bool False if there were any errors, in which case one or more modules were |
|
316 | + * @return boolean|null False if there were any errors, in which case one or more modules were |
|
313 | 317 | * not registered |
314 | 318 | */ |
315 | 319 | public function register( $name, $info = null ) { |
@@ -572,7 +576,7 @@ discard block |
||
572 | 576 | /** |
573 | 577 | * Get the list of sources. |
574 | 578 | * |
575 | - * @return array Like array( id => load.php url, .. ) |
|
579 | + * @return string Like array( id => load.php url, .. ) |
|
576 | 580 | */ |
577 | 581 | public function getSources() { |
578 | 582 | return $this->sources; |
@@ -382,7 +382,7 @@ discard block |
||
382 | 382 | * Get all styles for a given context. |
383 | 383 | * |
384 | 384 | * @param ResourceLoaderContext $context |
385 | - * @return array CSS code for $context as an associative array mapping media type to CSS text. |
|
385 | + * @return string CSS code for $context as an associative array mapping media type to CSS text. |
|
386 | 386 | */ |
387 | 387 | public function getStyles( ResourceLoaderContext $context ) { |
388 | 388 | $styles = $this->readStyleFiles( |
@@ -629,7 +629,7 @@ discard block |
||
629 | 629 | * @param array $list List of file paths in any combination of index/path |
630 | 630 | * or path/options pairs |
631 | 631 | * @param string $option Option name |
632 | - * @param mixed $default Default value if the option isn't set |
|
632 | + * @param string $default Default value if the option isn't set |
|
633 | 633 | * @return array List of file paths, collated by $option |
634 | 634 | */ |
635 | 635 | protected static function collateFilePathListByOption( array $list, $option, $default ) { |
@@ -312,7 +312,7 @@ |
||
312 | 312 | * Convert passed image data, which is assumed to be SVG, to PNG. |
313 | 313 | * |
314 | 314 | * @param string $svg SVG image data |
315 | - * @return string|bool PNG image data, or false on failure |
|
315 | + * @return string|false PNG image data, or false on failure |
|
316 | 316 | */ |
317 | 317 | protected function rasterize( $svg ) { |
318 | 318 | /** |
@@ -135,14 +135,14 @@ |
||
135 | 135 | * NOTE: This ONLY works for getTitleInfo() and isKnownEmpty(), NOT FOR ANYTHING ELSE. |
136 | 136 | * In particular, it doesn't work for getContent() or getScript() etc. |
137 | 137 | * |
138 | - * @return IDatabase|null |
|
138 | + * @return DatabaseBase |
|
139 | 139 | */ |
140 | 140 | protected function getDB() { |
141 | 141 | return wfGetDB( DB_SLAVE ); |
142 | 142 | } |
143 | 143 | |
144 | 144 | /** |
145 | - * @param string $title |
|
145 | + * @param string $titleText |
|
146 | 146 | * @return null|string |
147 | 147 | */ |
148 | 148 | protected function getContent( $titleText ) { |
@@ -23,6 +23,10 @@ |
||
23 | 23 | * Item class for a filearchive table row |
24 | 24 | */ |
25 | 25 | class RevDelArchivedFileItem extends RevDelFileItem { |
26 | + |
|
27 | + /** |
|
28 | + * @param RevDelArchivedFileList $list |
|
29 | + */ |
|
26 | 30 | public function __construct( $list, $row ) { |
27 | 31 | RevDelItem::__construct( $list, $row ); |
28 | 32 | $this->file = ArchivedFile::newFromRow( $row ); |
@@ -26,6 +26,9 @@ |
||
26 | 26 | /** @var File */ |
27 | 27 | public $file; |
28 | 28 | |
29 | + /** |
|
30 | + * @param RevDelFileList $list |
|
31 | + */ |
|
29 | 32 | public function __construct( $list, $row ) { |
30 | 33 | parent::__construct( $list, $row ); |
31 | 34 | $this->file = RepoGroup::singleton()->getLocalRepo()->newFileFromRow( $row ); |
@@ -39,6 +39,9 @@ |
||
39 | 39 | return LogPage::DELETED_ACTION; |
40 | 40 | } |
41 | 41 | |
42 | + /** |
|
43 | + * @param integer $target |
|
44 | + */ |
|
42 | 45 | public static function suggestTarget( $target, array $ids ) { |
43 | 46 | $result = wfGetDB( DB_SLAVE )->select( 'logging', |
44 | 47 | 'log_type', |