@@ -275,8 +275,6 @@ discard block |
||
275 | 275 | |
276 | 276 | /** |
277 | 277 | * Clean up a field array for output |
278 | - * @param ApiBase $module For context and parameters 'mergerequestfields' |
|
279 | - * and 'messageformat' |
|
280 | 278 | * @param array $fields |
281 | 279 | * @return array |
282 | 280 | */ |
@@ -315,6 +313,7 @@ discard block |
||
315 | 313 | * Fetch the standard parameters this helper recognizes |
316 | 314 | * @param string $action AuthManager action |
317 | 315 | * @param string $param... Parameters to use |
316 | + * @param string $param |
|
318 | 317 | * @return array |
319 | 318 | */ |
320 | 319 | public static function getStandardParams( $action, $param /* ... */ ) { |
@@ -172,7 +172,7 @@ discard block |
||
172 | 172 | } |
173 | 173 | |
174 | 174 | /** |
175 | - * @return array |
|
175 | + * @return string[] |
|
176 | 176 | */ |
177 | 177 | function __sleep() { |
178 | 178 | $this->loadDependencyValues(); |
@@ -228,6 +228,9 @@ discard block |
||
228 | 228 | private $name; |
229 | 229 | private $value; |
230 | 230 | |
231 | + /** |
|
232 | + * @param string $name |
|
233 | + */ |
|
231 | 234 | function __construct( $name ) { |
232 | 235 | $this->name = $name; |
233 | 236 | $this->value = $GLOBALS[$name]; |
@@ -252,6 +255,9 @@ discard block |
||
252 | 255 | private $name; |
253 | 256 | private $value; |
254 | 257 | |
258 | + /** |
|
259 | + * @param string $name |
|
260 | + */ |
|
255 | 261 | function __construct( $name ) { |
256 | 262 | $this->name = $name; |
257 | 263 | $this->value = $this->getConfig()->get( $this->name ); |
@@ -280,6 +286,9 @@ discard block |
||
280 | 286 | private $name; |
281 | 287 | private $value; |
282 | 288 | |
289 | + /** |
|
290 | + * @param string $name |
|
291 | + */ |
|
283 | 292 | function __construct( $name ) { |
284 | 293 | $this->name = $name; |
285 | 294 | $this->value = constant( $name ); |
@@ -197,7 +197,7 @@ discard block |
||
197 | 197 | * Get the default action for this special page, if none is given via URL/POST data. |
198 | 198 | * Subclasses should override this (or override loadAuth() so this is never called). |
199 | 199 | * @param string $subPage Subpage of the special page. |
200 | - * @return string an AuthManager::ACTION_* constant. |
|
200 | + * @return string|null an AuthManager::ACTION_* constant. |
|
201 | 201 | */ |
202 | 202 | protected function getDefaultAction( $subPage ) { |
203 | 203 | throw new BadMethodCallException( 'Subclass did not implement getDefaultAction' ); |
@@ -206,6 +206,7 @@ discard block |
||
206 | 206 | /** |
207 | 207 | * Return custom message key. |
208 | 208 | * Allows subclasses to customize messages. |
209 | + * @param string $defaultKey |
|
209 | 210 | * @return string |
210 | 211 | */ |
211 | 212 | protected function messageKey( $defaultKey ) { |
@@ -708,7 +709,7 @@ discard block |
||
708 | 709 | * Get an array value, or a default if it does not exist. |
709 | 710 | * @param array $array |
710 | 711 | * @param string $fieldName |
711 | - * @param mixed $default |
|
712 | + * @param integer $default |
|
712 | 713 | * @return mixed |
713 | 714 | */ |
714 | 715 | protected static function getField( array $array, $fieldName, $default = null ) { |
@@ -81,6 +81,9 @@ discard block |
||
81 | 81 | */ |
82 | 82 | abstract protected function logAuthResult( $success, $status = null ); |
83 | 83 | |
84 | + /** |
|
85 | + * @param string $name |
|
86 | + */ |
|
84 | 87 | public function __construct( $name ) { |
85 | 88 | global $wgUseMediaWikiUIEverywhere; |
86 | 89 | parent::__construct( $name ); |
@@ -342,7 +345,7 @@ discard block |
||
342 | 345 | * Show the success page. |
343 | 346 | * |
344 | 347 | * @param string $type Condition of return to; see `executeReturnTo` |
345 | - * @param string|Message $title Page's title |
|
348 | + * @param Message $title Page's title |
|
346 | 349 | * @param string $msgname |
347 | 350 | * @param string $injected_html |
348 | 351 | * @param StatusValue|null $extraMessages |
@@ -391,7 +394,6 @@ discard block |
||
391 | 394 | /** |
392 | 395 | * Replace some globals to make sure the fact that the user has just been logged in is |
393 | 396 | * reflected in the current request. |
394 | - * @param User $user |
|
395 | 397 | */ |
396 | 398 | protected function setSessionUserForCurrentRequest() { |
397 | 399 | global $wgUser, $wgLang; |
@@ -1322,6 +1324,9 @@ discard block |
||
1322 | 1324 | class LoginForm extends SpecialPage { |
1323 | 1325 | private $realLoginForm; |
1324 | 1326 | |
1327 | + /** |
|
1328 | + * @param DerivativeRequest $request |
|
1329 | + */ |
|
1325 | 1330 | public function __construct( $request = null ) { |
1326 | 1331 | global $wgDisableAuthManager; |
1327 | 1332 | if ( $wgDisableAuthManager ) { |
@@ -1418,6 +1423,10 @@ discard block |
||
1418 | 1423 | function getPageTitle( $subpage = false ) { |
1419 | 1424 | return call_user_func_array( [ $this->realLoginForm, __FUNCTION__ ], func_get_args() ); |
1420 | 1425 | } |
1426 | + |
|
1427 | + /** |
|
1428 | + * @param IContextSource $context |
|
1429 | + */ |
|
1421 | 1430 | public function setContext( $context ) { |
1422 | 1431 | return call_user_func_array( [ $this->realLoginForm, __FUNCTION__ ], func_get_args() ); |
1423 | 1432 | } |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | * @param string $name Name of the special page, as seen in links and URLs |
108 | 108 | * @param string $restriction User right required, e.g. "block" or "delete" |
109 | 109 | * @param bool $listed Whether the page is listed in Special:Specialpages |
110 | - * @param callable|bool $function Unused |
|
110 | + * @param boolean $function Unused |
|
111 | 111 | * @param string $file Unused |
112 | 112 | * @param bool $includable Whether the page can be included in normal pages |
113 | 113 | */ |
@@ -302,7 +302,7 @@ discard block |
||
302 | 302 | * Tells if the special page does something security-sensitive and needs extra defense against |
303 | 303 | * a stolen account (e.g. a reauthentication). What exactly that will mean is decided by the |
304 | 304 | * authentication framework. |
305 | - * @return bool|string False or the argument for AuthManager::securitySensitiveOperationStatus(). |
|
305 | + * @return boolean False or the argument for AuthManager::securitySensitiveOperationStatus(). |
|
306 | 306 | * Typically a special page needing elevated security would return its name here. |
307 | 307 | */ |
308 | 308 | protected function getLoginSecurityLevel() { |
@@ -85,7 +85,7 @@ |
||
85 | 85 | } |
86 | 86 | |
87 | 87 | /** |
88 | - * @return array |
|
88 | + * @return string[] |
|
89 | 89 | */ |
90 | 90 | function getOrderFields() { |
91 | 91 | return array( 'rd_namespace', 'rd_title', 'rd_from' ); |
@@ -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() { |
@@ -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', |