@@ -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; |
@@ -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 | /** |
@@ -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', |
@@ -26,6 +26,9 @@ |
||
| 26 | 26 | /** @var Revision */ |
| 27 | 27 | public $revision; |
| 28 | 28 | |
| 29 | + /** |
|
| 30 | + * @param RevDelRevisionList $list |
|
| 31 | + */ |
|
| 29 | 32 | public function __construct( $list, $row ) { |
| 30 | 33 | parent::__construct( $list, $row ); |
| 31 | 34 | $this->revision = new Revision( $row ); |
@@ -57,7 +57,7 @@ discard block |
||
| 57 | 57 | /** |
| 58 | 58 | * Get the internal type name of this list. Equal to the table name. |
| 59 | 59 | * Override this function. |
| 60 | - * @return null |
|
| 60 | + * @return string |
|
| 61 | 61 | */ |
| 62 | 62 | public function getType() { |
| 63 | 63 | return null; |
@@ -216,7 +216,7 @@ discard block |
||
| 216 | 216 | |
| 217 | 217 | /** |
| 218 | 218 | * Get the timestamp in MW 14-char form |
| 219 | - * @return mixed |
|
| 219 | + * @return string|false |
|
| 220 | 220 | */ |
| 221 | 221 | public function getTimestamp() { |
| 222 | 222 | $field = $this->getTimestampField(); |
@@ -120,6 +120,7 @@ |
||
| 120 | 120 | * Returns an array where the element of $array at index $key becomes |
| 121 | 121 | * the first element. |
| 122 | 122 | * @param int $key key to pull to the front |
| 123 | + * @param string[] $array |
|
| 123 | 124 | * @return array $array with the item at $key pulled to the front |
| 124 | 125 | */ |
| 125 | 126 | private function pullFront( $key, $array ) { |