@@ -423,7 +423,7 @@ discard block |
||
423 | 423 | /** |
424 | 424 | * Return the list of revision fields that should be selected to create |
425 | 425 | * a new revision. |
426 | - * @return array |
|
426 | + * @return string[] |
|
427 | 427 | */ |
428 | 428 | public static function selectFields() { |
429 | 429 | global $wgContentHandlerUseDB; |
@@ -454,7 +454,7 @@ discard block |
||
454 | 454 | /** |
455 | 455 | * Return the list of revision fields that should be selected to create |
456 | 456 | * a new revision from an archive row. |
457 | - * @return array |
|
457 | + * @return string[] |
|
458 | 458 | */ |
459 | 459 | public static function selectArchiveFields() { |
460 | 460 | global $wgContentHandlerUseDB; |
@@ -485,7 +485,7 @@ discard block |
||
485 | 485 | /** |
486 | 486 | * Return the list of text fields that should be selected to read the |
487 | 487 | * revision text |
488 | - * @return array |
|
488 | + * @return string[] |
|
489 | 489 | */ |
490 | 490 | public static function selectTextFields() { |
491 | 491 | return array( |
@@ -496,7 +496,7 @@ discard block |
||
496 | 496 | |
497 | 497 | /** |
498 | 498 | * Return the list of page fields that should be selected from page table |
499 | - * @return array |
|
499 | + * @return string[] |
|
500 | 500 | */ |
501 | 501 | public static function selectPageFields() { |
502 | 502 | return array( |
@@ -511,7 +511,7 @@ discard block |
||
511 | 511 | |
512 | 512 | /** |
513 | 513 | * Return the list of user fields that should be selected from user table |
514 | - * @return array |
|
514 | + * @return string[] |
|
515 | 515 | */ |
516 | 516 | public static function selectUserFields() { |
517 | 517 | return array( 'user_name' ); |
@@ -756,7 +756,7 @@ discard block |
||
756 | 756 | /** |
757 | 757 | * Returns the base36 sha1 of the text in this revision, or null if unknown. |
758 | 758 | * |
759 | - * @return string|null |
|
759 | + * @return string |
|
760 | 760 | */ |
761 | 761 | public function getSha1() { |
762 | 762 | return $this->mSha1; |
@@ -838,7 +838,7 @@ discard block |
||
838 | 838 | /** |
839 | 839 | * Fetch revision's user id without regard for the current user's permissions |
840 | 840 | * |
841 | - * @return string |
|
841 | + * @return integer |
|
842 | 842 | * @deprecated since 1.25, use getUser( Revision::RAW ) |
843 | 843 | */ |
844 | 844 | public function getRawUser() { |
@@ -1533,7 +1533,7 @@ discard block |
||
1533 | 1533 | * Lazy-load the revision's text. |
1534 | 1534 | * Currently hardcoded to the 'text' table storage engine. |
1535 | 1535 | * |
1536 | - * @return string|bool The revision's text, or false on failure |
|
1536 | + * @return string The revision's text, or false on failure |
|
1537 | 1537 | */ |
1538 | 1538 | protected function loadText() { |
1539 | 1539 | // Caching may be beneficial for massive use of external storage |
@@ -1734,7 +1734,7 @@ discard block |
||
1734 | 1734 | * |
1735 | 1735 | * @param Title $title |
1736 | 1736 | * @param int $id |
1737 | - * @return string|bool False if not found |
|
1737 | + * @return string|false False if not found |
|
1738 | 1738 | */ |
1739 | 1739 | static function getTimestampFromId( $title, $id, $flags = 0 ) { |
1740 | 1740 | $db = ( $flags & self::READ_LATEST ) |
@@ -1790,7 +1790,7 @@ discard block |
||
1790 | 1790 | * @since 1.20 |
1791 | 1791 | * @deprecated since 1.24 |
1792 | 1792 | * |
1793 | - * @param IDatabase|int $db The Database to perform the check on. May be given as a |
|
1793 | + * @param integer $db The Database to perform the check on. May be given as a |
|
1794 | 1794 | * Database object or a database identifier usable with wfGetDB. |
1795 | 1795 | * @param int $pageId The ID of the page in question |
1796 | 1796 | * @param int $userId The ID of the user in question |
@@ -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; |