@@ -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',  |