@@ -136,7 +136,7 @@ discard block |
||
136 | 136 | /** |
137 | 137 | * Returns an array of all valid error messages. |
138 | 138 | * |
139 | - * @return array |
|
139 | + * @return string[] |
|
140 | 140 | */ |
141 | 141 | public static function getValidErrorMessages() { |
142 | 142 | static $messages = null; |
@@ -1222,7 +1222,7 @@ discard block |
||
1222 | 1222 | * Display a "successful action" page. |
1223 | 1223 | * |
1224 | 1224 | * @param string $type Condition of return to; see `executeReturnTo` |
1225 | - * @param string|Message $title Page's title |
|
1225 | + * @param Message $title Page's title |
|
1226 | 1226 | * @param string $msgname |
1227 | 1227 | * @param string $injected_html |
1228 | 1228 | */ |
@@ -1735,7 +1735,7 @@ discard block |
||
1735 | 1735 | * Private function to check password expiration, until AuthManager comes |
1736 | 1736 | * along to handle that. |
1737 | 1737 | * @param User $user |
1738 | - * @return string|bool |
|
1738 | + * @return string|false |
|
1739 | 1739 | */ |
1740 | 1740 | private function checkUserPasswordExpired( User $user ) { |
1741 | 1741 | global $wgPasswordExpireGrace; |
@@ -232,7 +232,7 @@ |
||
232 | 232 | * $diff = new Diff($lines1, $lines2); |
233 | 233 | * $rev = $diff->reverse(); |
234 | 234 | * |
235 | - * @return Object A Diff object representing the inverse of the |
|
235 | + * @return Diff A Diff object representing the inverse of the |
|
236 | 236 | * original diff. |
237 | 237 | */ |
238 | 238 | public function reverse() { |
@@ -222,6 +222,13 @@ discard block |
||
222 | 222 | return $ranges; |
223 | 223 | } |
224 | 224 | |
225 | + /** |
|
226 | + * @param integer $bottoml1 |
|
227 | + * @param integer $topl1 |
|
228 | + * @param integer $bottoml2 |
|
229 | + * @param integer $topl2 |
|
230 | + * @param integer[] $snake |
|
231 | + */ |
|
225 | 232 | private function lcs_rec( $bottoml1, $topl1, $bottoml2, $topl2, &$V, &$snake ) { |
226 | 233 | // check that both sequences are non-empty |
227 | 234 | if ( $bottoml1 > $topl1 || $bottoml2 > $topl2 ) { |
@@ -490,6 +497,11 @@ discard block |
||
490 | 497 | */ |
491 | 498 | } |
492 | 499 | |
500 | + /** |
|
501 | + * @param double $M |
|
502 | + * @param double $N |
|
503 | + * @param double $limit |
|
504 | + */ |
|
493 | 505 | private static function findMostProgress( $M, $N, $limit, $V ) { |
494 | 506 | $delta = $N - $M; |
495 | 507 | |
@@ -609,6 +621,12 @@ discard block |
||
609 | 621 | /** @var int */ |
610 | 622 | public $rightlength; |
611 | 623 | |
624 | + /** |
|
625 | + * @param integer $leftstart |
|
626 | + * @param integer $leftend |
|
627 | + * @param integer $rightstart |
|
628 | + * @param integer $rightend |
|
629 | + */ |
|
612 | 630 | function __construct( $leftstart, $leftend, $rightstart, $rightend ) { |
613 | 631 | $this->leftstart = $leftstart; |
614 | 632 | $this->leftend = $leftend; |
@@ -112,6 +112,9 @@ |
||
112 | 112 | } |
113 | 113 | } |
114 | 114 | |
115 | + /** |
|
116 | + * @return string |
|
117 | + */ |
|
115 | 118 | function stripParameters( $text ) { |
116 | 119 | if ( !$this->stripParametersEnabled ) { |
117 | 120 | return $text; |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | } |
99 | 99 | |
100 | 100 | /** |
101 | - * @param string|bool|int $expand A PROTO_* constant or false |
|
101 | + * @param string $expand A PROTO_* constant or false |
|
102 | 102 | */ |
103 | 103 | public function setExpandURLs( $expand ) { |
104 | 104 | $this->expandUrls = $expand; |
@@ -189,6 +189,9 @@ discard block |
||
189 | 189 | return $options; |
190 | 190 | } |
191 | 191 | |
192 | + /** |
|
193 | + * @param boolean $isKnown |
|
194 | + */ |
|
192 | 195 | private function runBeginHook( LinkTarget $target, &$text, &$extraAttribs, &$query, $isKnown ) { |
193 | 196 | $ret = null; |
194 | 197 | if ( !Hooks::run( 'HtmlPageLinkRendererBegin', |
@@ -381,7 +381,7 @@ discard block |
||
381 | 381 | * Get all styles for a given context. |
382 | 382 | * |
383 | 383 | * @param ResourceLoaderContext $context |
384 | - * @return array CSS code for $context as an associative array mapping media type to CSS text. |
|
384 | + * @return string CSS code for $context as an associative array mapping media type to CSS text. |
|
385 | 385 | */ |
386 | 386 | public function getStyles( ResourceLoaderContext $context ) { |
387 | 387 | $styles = $this->readStyleFiles( |
@@ -631,7 +631,7 @@ discard block |
||
631 | 631 | * @param array $list List of file paths in any combination of index/path |
632 | 632 | * or path/options pairs |
633 | 633 | * @param string $option Option name |
634 | - * @param mixed $default Default value if the option isn't set |
|
634 | + * @param string $default Default value if the option isn't set |
|
635 | 635 | * @return array List of file paths, collated by $option |
636 | 636 | */ |
637 | 637 | protected static function collateFilePathListByOption( array $list, $option, $default ) { |
@@ -1007,6 +1007,7 @@ discard block |
||
1007 | 1007 | * the BOM character is not valid in the middle of a string. |
1008 | 1008 | * We already assume UTF-8 everywhere, so this should be safe. |
1009 | 1009 | * |
1010 | + * @param string $input |
|
1010 | 1011 | * @return string input minus the intial BOM char |
1011 | 1012 | */ |
1012 | 1013 | protected function stripBom( $input ) { |
@@ -104,7 +104,7 @@ discard block |
||
104 | 104 | * @since 1.18 |
105 | 105 | * @param string $feature |
106 | 106 | * @param mixed $data |
107 | - * @return bool |
|
107 | + * @return boolean|null |
|
108 | 108 | */ |
109 | 109 | public function setFeatureData( $feature, $data ) { |
110 | 110 | $this->features[$feature] = $data; |
@@ -301,7 +301,7 @@ discard block |
||
301 | 301 | |
302 | 302 | /** |
303 | 303 | * Find snippet highlight settings for all users |
304 | - * @return array Contextlines, contextchars |
|
304 | + * @return integer[] Contextlines, contextchars |
|
305 | 305 | */ |
306 | 306 | public static function userHighlightPrefs() { |
307 | 307 | $contextlines = 2; // Hardcode this. Old defaults sucked. :) |
@@ -507,6 +507,7 @@ discard block |
||
507 | 507 | * Process completion search results. |
508 | 508 | * Resolves the titles and rescores. |
509 | 509 | * @param SearchSuggestionSet $suggestions |
510 | + * @param string $search |
|
510 | 511 | * @return SearchSuggestionSet |
511 | 512 | */ |
512 | 513 | protected function processCompletionResults( $search, SearchSuggestionSet $suggestions ) { |
@@ -590,7 +591,7 @@ discard block |
||
590 | 591 | * settings, returning a list of index numbers. |
591 | 592 | * @deprecated since 1.27; use SearchEngineConfig::userNamespaces() |
592 | 593 | * @param user $user |
593 | - * @return array |
|
594 | + * @return integer[] |
|
594 | 595 | */ |
595 | 596 | public static function userNamespaces( $user ) { |
596 | 597 | return MediaWikiServices::getInstance()->getSearchEngineConfig()->userNamespaces( $user ); |
@@ -599,7 +600,7 @@ discard block |
||
599 | 600 | /** |
600 | 601 | * An array of namespaces indexes to be searched by default |
601 | 602 | * @deprecated since 1.27; use SearchEngineConfig::defaultNamespaces() |
602 | - * @return array |
|
603 | + * @return integer[] |
|
603 | 604 | */ |
604 | 605 | public static function defaultNamespaces() { |
605 | 606 | return MediaWikiServices::getInstance()->getSearchEngineConfig()->defaultNamespaces(); |
@@ -610,7 +611,7 @@ discard block |
||
610 | 611 | * and preferences |
611 | 612 | * @deprecated since 1.27; use SearchEngineConfig::namespacesAsText() |
612 | 613 | * @param array $namespaces |
613 | - * @return array |
|
614 | + * @return string[] |
|
614 | 615 | */ |
615 | 616 | public static function namespacesAsText( $namespaces ) { |
616 | 617 | return MediaWikiServices::getInstance()->getSearchEngineConfig()->namespacesAsText( $namespaces ); |
@@ -648,7 +649,7 @@ discard block |
||
648 | 649 | * - default: set to true if this profile is the default |
649 | 650 | * |
650 | 651 | * @since 1.28 |
651 | - * @param $profileType the type of profiles |
|
652 | + * @param string $profileType the type of profiles |
|
652 | 653 | * @return array|null the list of profiles or null if none available |
653 | 654 | */ |
654 | 655 | public function getProfiles( $profileType ) { |
@@ -163,6 +163,9 @@ |
||
163 | 163 | } ); |
164 | 164 | } |
165 | 165 | |
166 | + /** |
|
167 | + * @param string $table |
|
168 | + */ |
|
166 | 169 | private function batchDeleteByPK( $table, array $conds, array $pk, $bSize ) { |
167 | 170 | $dbw = $this->mDb; // convenience |
168 | 171 | $res = $dbw->select( $table, $pk, $conds, __METHOD__ ); |
@@ -64,7 +64,7 @@ discard block |
||
64 | 64 | /** |
65 | 65 | * Log CSP report, with a different severity depending on $flags |
66 | 66 | * @param $flags Array Flags for this report |
67 | - * @param $logLine String text of log entry |
|
67 | + * @param string $logLine String text of log entry |
|
68 | 68 | * @param $context Array logging context |
69 | 69 | */ |
70 | 70 | private function logReport( $flags, $logLine, $context ) { |
@@ -165,7 +165,7 @@ discard block |
||
165 | 165 | * Stop processing the request, and output/log an error |
166 | 166 | * |
167 | 167 | * @param $code String error code |
168 | - * @param $method String method that made error |
|
168 | + * @param string $method String method that made error |
|
169 | 169 | * @throws UsageException Always |
170 | 170 | */ |
171 | 171 | private function error( $code, $method ) { |