@@ -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', |
@@ -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 ) { |
@@ -440,7 +440,7 @@ |
||
| 440 | 440 | * Basic wikitext removal |
| 441 | 441 | * @protected |
| 442 | 442 | * @param string $text |
| 443 | - * @return mixed |
|
| 443 | + * @return string |
|
| 444 | 444 | */ |
| 445 | 445 | function removeWiki( $text ) { |
| 446 | 446 | $text = preg_replace( "/\\{\\{([^|]+?)\\}\\}/", "", $text ); |
@@ -173,6 +173,10 @@ discard block |
||
| 173 | 173 | return $this->searchInternal( $term, false ); |
| 174 | 174 | } |
| 175 | 175 | |
| 176 | + /** |
|
| 177 | + * @param string $term |
|
| 178 | + * @param boolean $fulltext |
|
| 179 | + */ |
|
| 176 | 180 | protected function searchInternal( $term, $fulltext ) { |
| 177 | 181 | // This seems out of place, why is this called with empty term? |
| 178 | 182 | if ( trim( $term ) === '' ) { |
@@ -377,7 +381,7 @@ discard block |
||
| 377 | 381 | * Converts some characters for MySQL's indexing to grok it correctly, |
| 378 | 382 | * and pads short words to overcome limitations. |
| 379 | 383 | * @param string $string |
| 380 | - * @return mixed|string |
|
| 384 | + * @return string |
|
| 381 | 385 | */ |
| 382 | 386 | function normalizeText( $string ) { |
| 383 | 387 | global $wgContLang; |