Revert r113650 and reapply r113619 and r113649 with one modification: User::createNew...
[lhc/web/wiklou.git] / includes / specials / SpecialUserlogin.php
index c89bf7a..761f1a7 100644 (file)
@@ -49,34 +49,44 @@ class LoginForm extends SpecialPage {
        var $mSkipCookieCheck, $mReturnToQuery, $mToken, $mStickHTTPS;
        var $mType, $mReason, $mRealName;
        var $mAbortLoginErrorMsg = 'login-abort-generic';
+       private $mLoaded = false;
 
        /**
         * @var ExternalUser
         */
        private $mExtUser = null;
 
+       /**
+        * @ var WebRequest
+        */
+       private $mOverrideRequest = null;
+
        /**
         * @param WebRequest $request
         */
        public function __construct( $request = null ) {
                parent::__construct( 'Userlogin' );
 
-               if ( $request === null ) {
-                       global $wgRequest;
-                       $this->load( $wgRequest );
-               } else {
-                       $this->load( $request );
-               }
+               $this->mOverrideRequest = $request;
        }
 
        /**
         * Loader
-        *
-        * @param $request WebRequest object
         */
-       function load( $request ) {
+       function load() {
                global $wgAuth, $wgHiddenPrefs, $wgEnableEmail, $wgRedirectOnLogin;
 
+               if ( $this->mLoaded ) {
+                       return;
+               }
+               $this->mLoaded = true;
+
+               if ( $this->mOverrideRequest === null ) {
+                       $request = $this->getRequest();
+               } else {
+                       $request = $this->mOverrideRequest;
+               }
+
                $this->mType = $request->getText( 'type' );
                $this->mUsername = $request->getText( 'wpName' );
                $this->mPassword = $request->getText( 'wpPassword' );
@@ -115,23 +125,39 @@ class LoginForm extends SpecialPage {
                }
 
                if( !$wgAuth->validDomain( $this->mDomain ) ) {
-                       $this->mDomain = 'invaliddomain';
+                       if ( isset( $_SESSION['wsDomain'] ) ) {
+                               $this->mDomain = $_SESSION['wsDomain'];
+                       } else {
+                               $this->mDomain = 'invaliddomain';
+                       }
                }
                $wgAuth->setDomain( $this->mDomain );
 
-               # When switching accounts, it sucks to get automatically logged out
+               # 1. When switching accounts, it sucks to get automatically logged out
+               # 2. Do not return to PasswordReset after a successful password change
+               #    but goto Wiki start page (Main_Page) instead ( bug 33997 )
                $returnToTitle = Title::newFromText( $this->mReturnTo );
-               if( is_object( $returnToTitle ) && $returnToTitle->isSpecial( 'Userlogout' ) ) {
+               if( is_object( $returnToTitle ) && (
+                       $returnToTitle->isSpecial( 'Userlogout' )
+                       || $returnToTitle->isSpecial( 'PasswordReset' ) ) ) {
                        $this->mReturnTo = '';
                        $this->mReturnToQuery = '';
                }
        }
 
+       function getDescription() {
+               return $this->msg( $this->getUser()->isAllowed( 'createaccount' ) ?
+                       'userlogin' : 'userloginnocreate' )->text();
+       }
+
        public function execute( $par ) {
                if ( session_id() == '' ) {
                        wfSetupSession();
                }
 
+               $this->load();
+               $this->setHeaders();
+
                if ( $par == 'signup' ) { # Check for [[Special:Userlogin/signup]]
                        $this->mType = 'signup';
                }
@@ -141,11 +167,14 @@ class LoginForm extends SpecialPage {
                        return;
                } elseif( $this->mPosted ) {
                        if( $this->mCreateaccount ) {
-                               return $this->addNewAccount();
+                               $this->addNewAccount();
+                               return;
                        } elseif ( $this->mCreateaccountMail ) {
-                               return $this->addNewAccountMailPassword();
+                               $this->addNewAccountMailPassword();
+                               return;
                        } elseif ( ( 'submitlogin' == $this->mAction ) || $this->mLoginattempt ) {
-                               return $this->processLogin();
+                               $this->processLogin();
+                               return;
                        }
                }
                $this->mainLoginForm( '' );
@@ -155,10 +184,8 @@ class LoginForm extends SpecialPage {
         * @private
         */
        function addNewAccountMailPassword() {
-               global $wgOut;
-
                if ( $this->mEmail == '' ) {
-                       $this->mainLoginForm( wfMsgExt( 'noemail', array( 'parsemag', 'escape' ), $this->mUsername ) );
+                       $this->mainLoginForm( $this->msg( 'noemailcreate' )->escaped() );
                        return;
                }
 
@@ -176,42 +203,45 @@ class LoginForm extends SpecialPage {
                wfRunHooks( 'AddNewAccount', array( $u, true ) );
                $u->addNewUserLogEntry( true, $this->mReason );
 
-               $wgOut->setPageTitle( wfMsg( 'accmailtitle' ) );
+               $out = $this->getOutput();
+               $out->setPageTitle( $this->msg( 'accmailtitle' ) );
 
                if( !$result->isGood() ) {
-                       $this->mainLoginForm( wfMsg( 'mailerror', $result->getWikiText() ) );
+                       $this->mainLoginForm( $this->msg( 'mailerror', $result->getWikiText() )->text() );
                } else {
-                       $wgOut->addWikiMsg( 'accmailtext', $u->getName(), $u->getEmail() );
-                       $wgOut->returnToMain( false );
+                       $out->addWikiMsg( 'accmailtext', $u->getName(), $u->getEmail() );
+                       $out->returnToMain( false );
                }
        }
 
        /**
         * @private
+        * @return bool
         */
        function addNewAccount() {
-               global $wgUser, $wgEmailAuthentication, $wgOut;
+               global $wgUser, $wgEmailAuthentication, $wgLoginLanguageSelector;
 
                # Create the account and abort if there's a problem doing so
                $u = $this->addNewAccountInternal();
                if( $u == null ) {
-                       return;
+                       return false;
                }
 
                # If we showed up language selection links, and one was in use, be
                # smart (and sensible) and save that language as the user's preference
-               global $wgLoginLanguageSelector;
                if( $wgLoginLanguageSelector && $this->mLanguage ) {
                        $u->setOption( 'language', $this->mLanguage );
                }
 
+               $out = $this->getOutput();
+
                # Send out an email authentication message if needed
-               if( $wgEmailAuthentication && User::isValidEmailAddr( $u->getEmail() ) ) {
+               if( $wgEmailAuthentication && Sanitizer::validateEmail( $u->getEmail() ) ) {
                        $status = $u->sendConfirmationMail();
                        if( $status->isGood() ) {
-                               $wgOut->addWikiMsg( 'confirmemail_oncreate' );
+                               $out->addWikiMsg( 'confirmemail_oncreate' );
                        } else {
-                               $wgOut->addWikiText( $status->getWikiText( 'confirmemail_sendfailed' ) );
+                               $out->addWikiText( $status->getWikiText( 'confirmemail_sendfailed' ) );
                        }
                }
 
@@ -221,40 +251,42 @@ class LoginForm extends SpecialPage {
                # If not logged in, assume the new account as the current one and set
                # session cookies then show a "welcome" message or a "need cookies"
                # message as needed
-               if( $wgUser->isAnon() ) {
+               if( $this->getUser()->isAnon() ) {
+                       $u->setCookies();
                        $wgUser = $u;
-                       $wgUser->setCookies();
-                       wfRunHooks( 'AddNewAccount', array( $wgUser, false ) );
-                       $wgUser->addNewUserLogEntry();
+                       // This should set it for OutputPage and the Skin
+                       // which is needed or the personal links will be
+                       // wrong.
+                       $this->getContext()->setUser( $u );
+                       wfRunHooks( 'AddNewAccount', array( $u, false ) );
+                       $u->addNewUserLogEntry();
                        if( $this->hasSessionCookie() ) {
-                               return $this->successfulCreation();
+                               $this->successfulCreation();
                        } else {
-                               return $this->cookieRedirectCheck( 'new' );
+                               $this->cookieRedirectCheck( 'new' );
                        }
                } else {
                        # Confirm that the account was created
-                       $self = SpecialPage::getTitleFor( 'Userlogin' );
-                       $wgOut->setPageTitle( wfMsgHtml( 'accountcreated' ) );
-                       $wgOut->addWikiMsg( 'accountcreatedtext', $u->getName() );
-                       $wgOut->returnToMain( false, $self );
+                       $out->setPageTitle( $this->msg( 'accountcreated' ) );
+                       $out->addWikiMsg( 'accountcreatedtext', $u->getName() );
+                       $out->returnToMain( false, $this->getTitle() );
                        wfRunHooks( 'AddNewAccount', array( $u, false ) );
                        $u->addNewUserLogEntry( false, $this->mReason );
-                       return true;
                }
+               return true;
        }
 
        /**
         * @private
+        * @return bool|\User
         */
        function addNewAccountInternal() {
-               global $wgUser, $wgOut;
-               global $wgMemc, $wgAccountCreationThrottle;
-               global $wgAuth, $wgMinimalPasswordLength;
-               global $wgEmailConfirmToEdit;
+               global $wgAuth, $wgMemc, $wgAccountCreationThrottle,
+                       $wgMinimalPasswordLength, $wgEmailConfirmToEdit;
 
                // If the user passes an invalid domain, something is fishy
                if( !$wgAuth->validDomain( $this->mDomain ) ) {
-                       $this->mainLoginForm( wfMsg( 'wrongpassword' ) );
+                       $this->mainLoginForm( $this->msg( 'wrongpassword' )->text() );
                        return false;
                }
 
@@ -266,47 +298,46 @@ class LoginForm extends SpecialPage {
                if( 'local' != $this->mDomain && $this->mDomain != '' ) {
                        if( !$wgAuth->canCreateAccounts() && ( !$wgAuth->userExists( $this->mUsername )
                                || !$wgAuth->authenticate( $this->mUsername, $this->mPassword ) ) ) {
-                               $this->mainLoginForm( wfMsg( 'wrongpassword' ) );
+                               $this->mainLoginForm( $this->msg( 'wrongpassword' )->text() );
                                return false;
                        }
                }
 
                if ( wfReadOnly() ) {
-                       $wgOut->readOnlyPage();
-                       return false;
+                       throw new ReadOnlyError;
                }
 
                # Request forgery checks.
                if ( !self::getCreateaccountToken() ) {
                        self::setCreateaccountToken();
-                       $this->mainLoginForm( wfMsgExt( 'nocookiesfornew', array( 'parseinline' ) ) );
+                       $this->mainLoginForm( $this->msg( 'nocookiesfornew' )->parse() );
                        return false;
                }
 
                # The user didn't pass a createaccount token
                if ( !$this->mToken ) {
-                       $this->mainLoginForm( wfMsg( 'sessionfailure' ) );
+                       $this->mainLoginForm( $this->msg( 'sessionfailure' )->text() );
                        return false;
                }
 
                # Validate the createaccount token
                if ( $this->mToken !== self::getCreateaccountToken() ) {
-                       $this->mainLoginForm( wfMsg( 'sessionfailure' ) );
+                       $this->mainLoginForm( $this->msg( 'sessionfailure' )->text() );
                        return false;
                }
 
                # Check permissions
-               if ( !$wgUser->isAllowed( 'createaccount' ) ) {
-                       $wgOut->permissionRequired( 'createaccount' );
-                       return false;
-               } elseif ( $wgUser->isBlockedFromCreateAccount() ) {
-                       $this->userBlockedMessage( $wgUser->isBlockedFromCreateAccount() );
+               $currentUser = $this->getUser();
+               if ( !$currentUser->isAllowed( 'createaccount' ) ) {
+                       throw new PermissionsError( 'createaccount' );
+               } elseif ( $currentUser->isBlockedFromCreateAccount() ) {
+                       $this->userBlockedMessage( $currentUser->isBlockedFromCreateAccount() );
                        return false;
                }
 
-               $ip = wfGetIP();
-               if ( $wgUser->isDnsBlacklisted( $ip, true /* check $wgProxyWhitelist */ ) ) {
-                       $this->mainLoginForm( wfMsg( 'sorbs_create_account_reason' ) . ' (' . htmlspecialchars( $ip ) . ')' );
+               $ip = $this->getRequest()->getIP();
+               if ( $currentUser->isDnsBlacklisted( $ip, true /* check $wgProxyWhitelist */ ) ) {
+                       $this->mainLoginForm( $this->msg( 'sorbs_create_account_reason' )->text() . ' ' . $this->msg( 'parentheses', $ip )->escaped() );
                        return false;
                }
 
@@ -314,17 +345,17 @@ class LoginForm extends SpecialPage {
                $name = trim( $this->mUsername );
                $u = User::newFromName( $name, 'creatable' );
                if ( !is_object( $u ) ) {
-                       $this->mainLoginForm( wfMsg( 'noname' ) );
+                       $this->mainLoginForm( $this->msg( 'noname' )->text() );
                        return false;
                }
 
                if ( 0 != $u->idForName() ) {
-                       $this->mainLoginForm( wfMsg( 'userexists' ) );
+                       $this->mainLoginForm( $this->msg( 'userexists' )->text() );
                        return false;
                }
 
                if ( 0 != strcmp( $this->mPassword, $this->mRetype ) ) {
-                       $this->mainLoginForm( wfMsg( 'badretype' ) );
+                       $this->mainLoginForm( $this->msg( 'badretype' )->text() );
                        return false;
                }
 
@@ -339,7 +370,7 @@ class LoginForm extends SpecialPage {
                                        $message = $valid;
                                        $params = array( $wgMinimalPasswordLength );
                                }
-                               $this->mainLoginForm( wfMsgExt( $message, array( 'parsemag' ), $params ) );
+                               $this->mainLoginForm( $this->msg( $message, $params )->text() );
                                return false;
                        } else {
                                # do not force a password for account creation by email
@@ -351,12 +382,12 @@ class LoginForm extends SpecialPage {
                # if you need a confirmed email address to edit, then obviously you
                # need an email address.
                if ( $wgEmailConfirmToEdit && empty( $this->mEmail ) ) {
-                       $this->mainLoginForm( wfMsg( 'noemailtitle' ) );
+                       $this->mainLoginForm( $this->msg( 'noemailtitle' )->text() );
                        return false;
                }
 
-               if( !empty( $this->mEmail ) && !User::isValidEmailAddr( $this->mEmail ) ) {
-                       $this->mainLoginForm( wfMsg( 'invalidemailaddress' ) );
+               if( !empty( $this->mEmail ) && !Sanitizer::validateEmail( $this->mEmail ) ) {
+                       $this->mainLoginForm( $this->msg( 'invalidemailaddress' )->text() );
                        return false;
                }
 
@@ -373,21 +404,26 @@ class LoginForm extends SpecialPage {
                        return false;
                }
 
-               if ( $wgAccountCreationThrottle && $wgUser->isPingLimitable() ) {
-                       $key = wfMemcKey( 'acctcreate', 'ip', $ip );
-                       $value = $wgMemc->get( $key );
-                       if ( !$value ) {
-                               $wgMemc->set( $key, 0, 86400 );
-                       }
-                       if ( $value >= $wgAccountCreationThrottle ) {
-                               $this->throttleHit( $wgAccountCreationThrottle );
-                               return false;
+               // Hook point to check for exempt from account creation throttle
+               if ( !wfRunHooks( 'ExemptFromAccountCreationThrottle', array( $ip ) ) ) {
+                       wfDebug( "LoginForm::exemptFromAccountCreationThrottle: a hook allowed account creation w/o throttle\n" );
+               } else {
+                       if ( ( $wgAccountCreationThrottle && $currentUser->isPingLimitable() ) ) {
+                               $key = wfMemcKey( 'acctcreate', 'ip', $ip );
+                               $value = $wgMemc->get( $key );
+                               if ( !$value ) {
+                                       $wgMemc->set( $key, 0, 86400 );
+                               }
+                               if ( $value >= $wgAccountCreationThrottle ) {
+                                       $this->throttleHit( $wgAccountCreationThrottle );
+                                       return false;
+                               }
+                               $wgMemc->incr( $key );
                        }
-                       $wgMemc->incr( $key );
                }
 
                if( !$wgAuth->addUser( $u, $this->mPassword, $this->mEmail, $this->mRealName ) ) {
-                       $this->mainLoginForm( wfMsg( 'externaldberror' ) );
+                       $this->mainLoginForm( $this->msg( 'externaldberror' )->text() );
                        return false;
                }
 
@@ -443,9 +479,12 @@ class LoginForm extends SpecialPage {
         * This may create a local account as a side effect if the
         * authentication plugin allows transparent local account
         * creation.
+        * @return int
         */
        public function authenticateUserData() {
-               global $wgUser, $wgAuth, $wgMemc;
+               global $wgUser, $wgAuth;
+
+               $this->load();
 
                if ( $this->mUsername == '' ) {
                        return self::NO_NAME;
@@ -466,22 +505,9 @@ class LoginForm extends SpecialPage {
                        return self::NEED_TOKEN;
                }
 
-               global $wgPasswordAttemptThrottle;
-
-               $throttleCount = 0;
-               if ( is_array( $wgPasswordAttemptThrottle ) ) {
-                       $throttleKey = wfMemcKey( 'password-throttle', wfGetIP(), md5( $this->mUsername ) );
-                       $count = $wgPasswordAttemptThrottle['count'];
-                       $period = $wgPasswordAttemptThrottle['seconds'];
-
-                       $throttleCount = $wgMemc->get( $throttleKey );
-                       if ( !$throttleCount ) {
-                               $wgMemc->add( $throttleKey, 1, $period ); // start counter
-                       } elseif ( $throttleCount < $count ) {
-                               $wgMemc->incr( $throttleKey );
-                       } elseif ( $throttleCount >= $count ) {
-                               return self::THROTTLED;
-                       }
+               $throttleCount = self::incLoginThrottle( $this->mUsername );
+               if ( $throttleCount === true ) {
+                       return self::THROTTLED;
                }
 
                // Validate the login token
@@ -489,13 +515,13 @@ class LoginForm extends SpecialPage {
                        return self::WRONG_TOKEN;
                }
 
-               // Load $wgUser now, and check to see if we're logging in as the same
-               // name. This is necessary because loading $wgUser (say by calling
-               // getName()) calls the UserLoadFromSession hook, which potentially
-               // creates the user in the database. Until we load $wgUser, checking
-               // for user existence using User::newFromName($name)->getId() below
+               // Load the current user now, and check to see if we're logging in as
+               // the same name. This is necessary because loading the current user
+               // (say by calling getName()) calls the UserLoadFromSession hook, which
+               // potentially creates the user in the database. Until we load $wgUser,
+               // checking for user existence using User::newFromName($name)->getId() below
                // will effectively be using stale data.
-               if ( $wgUser->getName() === $this->mUsername ) {
+               if ( $this->getUser()->getName() === $this->mUsername ) {
                        wfDebug( __METHOD__ . ": already logged in as {$this->mUsername}\n" );
                        return self::SUCCESS;
                }
@@ -573,15 +599,19 @@ class LoginForm extends SpecialPage {
                } else {
                        $wgAuth->updateUser( $u );
                        $wgUser = $u;
+                       // This should set it for OutputPage and the Skin
+                       // which is needed or the personal links will be
+                       // wrong.
+                       $this->getContext()->setUser( $u );
 
                        // Please reset throttle for successful logins, thanks!
-                       if( $throttleCount ) {
-                               $wgMemc->delete( $throttleKey );
+                       if ( $throttleCount ) {
+                               self::clearLoginThrottle( $this->mUsername );
                        }
 
                        if ( $isAutoCreated ) {
                                // Must be run after $wgUser is set, for correct new user log
-                               wfRunHooks( 'AuthPluginAutoCreate', array( $wgUser ) );
+                               wfRunHooks( 'AuthPluginAutoCreate', array( $u ) );
                        }
 
                        $retval = self::SUCCESS;
@@ -590,6 +620,48 @@ class LoginForm extends SpecialPage {
                return $retval;
        }
 
+       /**
+        * Increment the login attempt throttle hit count for the (username,current IP)
+        * tuple unless the throttle was already reached.
+        * @param $username string The user name
+        * @return Bool|Integer The integer hit count or True if it is already at the limit
+        */
+       public static function incLoginThrottle( $username ) {
+               global $wgPasswordAttemptThrottle, $wgMemc, $wgRequest;
+               $username = trim( $username ); // sanity
+
+               $throttleCount = 0;
+               if ( is_array( $wgPasswordAttemptThrottle ) ) {
+                       $throttleKey = wfMemcKey( 'password-throttle', $wgRequest->getIP(), md5( $username ) );
+                       $count = $wgPasswordAttemptThrottle['count'];
+                       $period = $wgPasswordAttemptThrottle['seconds'];
+
+                       $throttleCount = $wgMemc->get( $throttleKey );
+                       if ( !$throttleCount ) {
+                               $wgMemc->add( $throttleKey, 1, $period ); // start counter
+                       } elseif ( $throttleCount < $count ) {
+                               $wgMemc->incr( $throttleKey );
+                       } elseif ( $throttleCount >= $count ) {
+                               return true;
+                       }
+               }
+
+               return $throttleCount;
+       }
+
+       /**
+        * Clear the login attempt throttle hit count for the (username,current IP) tuple.
+        * @param $username string The user name
+        * @return void
+        */
+       public static function clearLoginThrottle( $username ) {
+               global $wgMemc, $wgRequest;
+               $username = trim( $username ); // sanity
+
+               $throttleKey = wfMemcKey( 'password-throttle', $wgRequest->getIP(), md5( $username ) );
+               $wgMemc->delete( $throttleKey );
+       }
+
        /**
         * Attempt to automatically create a user on login. Only succeeds if there
         * is an external authentication method which allows it.
@@ -599,9 +671,9 @@ class LoginForm extends SpecialPage {
         * @return integer Status code
         */
        function attemptAutoCreate( $user ) {
-               global $wgAuth, $wgUser, $wgAutocreatePolicy;
+               global $wgAuth, $wgAutocreatePolicy;
 
-               if ( $wgUser->isBlockedFromCreateAccount() ) {
+               if ( $this->getUser()->isBlockedFromCreateAccount() ) {
                        wfDebug( __METHOD__ . ": user is blocked from account creation\n" );
                        return self::CREATE_BLOCKED;
                }
@@ -649,79 +721,83 @@ class LoginForm extends SpecialPage {
        }
 
        function processLogin() {
-               global $wgUser;
+               global $wgMemc, $wgLang;
 
                switch ( $this->authenticateUserData() ) {
                        case self::SUCCESS:
                                # We've verified now, update the real record
-                               if( (bool)$this->mRemember != (bool)$wgUser->getOption( 'rememberpassword' ) ) {
-                                       $wgUser->setOption( 'rememberpassword', $this->mRemember ? 1 : 0 );
-                                       $wgUser->saveSettings();
+                               $user = $this->getUser();
+                               if( (bool)$this->mRemember != (bool)$user->getOption( 'rememberpassword' ) ) {
+                                       $user->setOption( 'rememberpassword', $this->mRemember ? 1 : 0 );
+                                       $user->saveSettings();
                                } else {
-                                       $wgUser->invalidateCache( true );
+                                       $user->invalidateCache();
                                }
-                               $wgUser->setCookies();
+                               $user->setCookies();
                                self::clearLoginToken();
 
                                // Reset the throttle
-                               $key = wfMemcKey( 'password-throttle', wfGetIP(), md5( $this->mUsername ) );
-                               global $wgMemc;
+                               $request = $this->getRequest();
+                               $key = wfMemcKey( 'password-throttle', $request->getIP(), md5( $this->mUsername ) );
                                $wgMemc->delete( $key );
 
                                if( $this->hasSessionCookie() || $this->mSkipCookieCheck ) {
                                        /* Replace the language object to provide user interface in
                                         * correct language immediately on this first page load.
                                         */
-                                       global $wgLang, $wgRequest;
-                                       $code = $wgRequest->getVal( 'uselang', $wgUser->getOption( 'language' ) );
-                                       $wgLang = Language::factory( $code );
-                                       return $this->successfulLogin();
+                                       $code = $request->getVal( 'uselang', $user->getOption( 'language' ) );
+                                       $userLang = Language::factory( $code );
+                                       $wgLang = $userLang;
+                                       $this->getContext()->setLanguage( $userLang );
+                                       $this->successfulLogin();
                                } else {
-                                       return $this->cookieRedirectCheck( 'login' );
+                                       $this->cookieRedirectCheck( 'login' );
                                }
                                break;
 
                        case self::NEED_TOKEN:
-                               $this->mainLoginForm( wfMsgExt( 'nocookiesforlogin', array( 'parseinline' ) ) );
+                               $this->mainLoginForm( $this->msg( 'nocookiesforlogin' )->parse() );
                                break;
                        case self::WRONG_TOKEN:
-                               $this->mainLoginForm( wfMsg( 'sessionfailure' ) );
+                               $this->mainLoginForm( $this->msg( 'sessionfailure' )->text() );
                                break;
                        case self::NO_NAME:
                        case self::ILLEGAL:
-                               $this->mainLoginForm( wfMsg( 'noname' ) );
+                               $this->mainLoginForm( $this->msg( 'noname' )->text() );
                                break;
                        case self::WRONG_PLUGIN_PASS:
-                               $this->mainLoginForm( wfMsg( 'wrongpassword' ) );
+                               $this->mainLoginForm( $this->msg( 'wrongpassword' )->text() );
                                break;
                        case self::NOT_EXISTS:
-                               if( $wgUser->isAllowed( 'createaccount' ) ) {
-                                       $this->mainLoginForm( wfMsgExt( 'nosuchuser', 'parseinline', $this->mUsername ) );
+                               if( $this->getUser()->isAllowed( 'createaccount' ) ) {
+                                       $this->mainLoginForm( $this->msg( 'nosuchuser',
+                                               wfEscapeWikiText( $this->mUsername ) )->parse() );
                                } else {
-                                       $this->mainLoginForm( wfMsg( 'nosuchusershort', htmlspecialchars( $this->mUsername ) ) );
+                                       $this->mainLoginForm( $this->msg( 'nosuchusershort',
+                                               wfEscapeWikiText( $this->mUsername ) )->text() );
                                }
                                break;
                        case self::WRONG_PASS:
-                               $this->mainLoginForm( wfMsg( 'wrongpassword' ) );
+                               $this->mainLoginForm( $this->msg( 'wrongpassword' )->text() );
                                break;
                        case self::EMPTY_PASS:
-                               $this->mainLoginForm( wfMsg( 'wrongpasswordempty' ) );
+                               $this->mainLoginForm( $this->msg( 'wrongpasswordempty' )->text() );
                                break;
                        case self::RESET_PASS:
-                               $this->resetLoginForm( wfMsg( 'resetpass_announce' ) );
+                               $this->resetLoginForm( $this->msg( 'resetpass_announce' )->text() );
                                break;
                        case self::CREATE_BLOCKED:
-                               $this->userBlockedMessage( $wgUser->mBlock );
+                               $this->userBlockedMessage( $this->getUser()->mBlock );
                                break;
                        case self::THROTTLED:
-                               $this->mainLoginForm( wfMsg( 'login-throttled' ) );
+                               $this->mainLoginForm( $this->msg( 'login-throttled' )->text() );
                                break;
                        case self::USER_BLOCKED:
-                               $this->mainLoginForm( wfMsgExt( 'login-userblocked',
-                                       array( 'parsemag', 'escape' ), $this->mUsername ) );
+                               $this->mainLoginForm( $this->msg( 'login-userblocked',
+                                       $this->mUsername )->escaped() );
                                break;
                        case self::ABORTED:
-                               $this->mainLoginForm( wfMsg( $this->mAbortLoginErrorMsg  ) );
+                               $this->mainLoginForm( $this->msg( $this->mAbortLoginErrorMsg )->text() );
                                break;
                        default:
                                throw new MWException( 'Unhandled case value' );
@@ -729,9 +805,9 @@ class LoginForm extends SpecialPage {
        }
 
        function resetLoginForm( $error ) {
-               global $wgOut;
-               $wgOut->addHTML( Xml::element('p', array( 'class' => 'error' ), $error ) );
+               $this->getOutput()->addHTML( Xml::element('p', array( 'class' => 'error' ), $error ) );
                $reset = new SpecialChangePassword();
+               $reset->setContext( $this->getContext() );
                $reset->execute( null );
        }
 
@@ -741,28 +817,28 @@ class LoginForm extends SpecialPage {
         * @param $emailTitle String: message name of email title
         * @param $emailText String: message name of email text
         * @return Status object
-        * @private
         */
        function mailPasswordInternal( $u, $throttle = true, $emailTitle = 'passwordremindertitle', $emailText = 'passwordremindertext' ) {
-               global $wgServer, $wgScript, $wgUser, $wgNewPasswordExpiry;
+               global $wgServer, $wgScript, $wgNewPasswordExpiry;
 
                if ( $u->getEmail() == '' ) {
                        return Status::newFatal( 'noemail', $u->getName() );
                }
-               $ip = wfGetIP();
+               $ip = $this->getRequest()->getIP();
                if( !$ip ) {
                        return Status::newFatal( 'badipaddress' );
                }
 
-               wfRunHooks( 'User::mailPasswordInternal', array( &$wgUser, &$ip, &$u ) );
+               $currentUser = $this->getUser();
+               wfRunHooks( 'User::mailPasswordInternal', array( &$currentUser, &$ip, &$u ) );
 
                $np = $u->randomPassword();
                $u->setNewpassword( $np, $throttle );
                $u->saveSettings();
                $userLanguage = $u->getOption( 'language' );
-               $m = wfMsgExt( $emailText, array( 'parsemag', 'language' => $userLanguage ), $ip, $u->getName(), $np,
-                               $wgServer . $wgScript, round( $wgNewPasswordExpiry / 86400 ) );
-               $result = $u->sendMail( wfMsgExt( $emailTitle, array( 'parsemag', 'language' => $userLanguage ) ), $m );
+               $m = $this->msg( $emailText, $ip, $u->getName(), $np, $wgServer . $wgScript,
+                       round( $wgNewPasswordExpiry / 86400 ) )->inLanguage( $userLanguage )->text();
+               $result = $u->sendMail( $this->msg( $emailTitle )->inLanguage( $userLanguage )->text(), $m );
 
                return $result;
        }
@@ -779,11 +855,10 @@ class LoginForm extends SpecialPage {
         * @private
         */
        function successfulLogin() {
-               global $wgUser, $wgOut;
-
                # Run any hooks; display injected HTML if any, else redirect
+               $currentUser = $this->getUser();
                $injected_html = '';
-               wfRunHooks( 'UserLoginComplete', array( &$wgUser, &$injected_html ) );
+               wfRunHooks( 'UserLoginComplete', array( &$currentUser, &$injected_html ) );
 
                if( $injected_html !== '' ) {
                        $this->displaySuccessfulLogin( 'loginsuccess', $injected_html );
@@ -797,7 +872,7 @@ class LoginForm extends SpecialPage {
                        if( $wgSecureLogin && !$this->mStickHTTPS ) {
                                $redirectUrl = preg_replace( '/^https:/', 'http:', $redirectUrl );
                        }
-                       $wgOut->redirect( $redirectUrl );
+                       $this->getOutput()->redirect( $redirectUrl );
                }
        }
 
@@ -808,36 +883,41 @@ class LoginForm extends SpecialPage {
         * @private
         */
        function successfulCreation() {
-               global $wgUser;
                # Run any hooks; display injected HTML
+               $currentUser = $this->getUser();
                $injected_html = '';
                $welcome_creation_msg = 'welcomecreation';
-               
-               wfRunHooks( 'UserLoginComplete', array( &$wgUser, &$injected_html ) );
-               
-               //let any extensions change what message is shown
+
+               wfRunHooks( 'UserLoginComplete', array( &$currentUser, &$injected_html ) );
+
+               /**
+                * Let any extensions change what message is shown.
+                * @see https://www.mediawiki.org/wiki/Manual:Hooks/BeforeWelcomeCreation
+                * @since 1.18
+                */
                wfRunHooks( 'BeforeWelcomeCreation', array( &$welcome_creation_msg, &$injected_html ) );
-               
+
                $this->displaySuccessfulLogin( $welcome_creation_msg, $injected_html );
        }
 
        /**
         * Display a "login successful" page.
+        * @param $msgname string
+        * @param $injected_html string
         */
        private function displaySuccessfulLogin( $msgname, $injected_html ) {
-               global $wgOut, $wgUser;
-
-               $wgOut->setPageTitle( wfMsg( 'loginsuccesstitle' ) );
+               $out = $this->getOutput();
+               $out->setPageTitle( $this->msg( 'loginsuccesstitle' ) );
                if( $msgname ){
-                       $wgOut->addWikiMsg( $msgname, $wgUser->getName() );
+                       $out->addWikiMsg( $msgname, wfEscapeWikiText( $this->getUser()->getName() ) );
                }
-               
-               $wgOut->addHTML( $injected_html );
+
+               $out->addHTML( $injected_html );
 
                if ( !empty( $this->mReturnTo ) ) {
-                       $wgOut->returnToMain( null, $this->mReturnTo, $this->mReturnToQuery );
+                       $out->returnToMain( null, $this->mReturnTo, $this->mReturnToQuery );
                } else {
-                       $wgOut->returnToMain( null );
+                       $out->returnToMain( null );
                }
        }
 
@@ -849,8 +929,6 @@ class LoginForm extends SpecialPage {
         * @param $block Block the block causing this error
         */
        function userBlockedMessage( Block $block ) {
-               global $wgOut;
-
                # Let's be nice about this, it's likely that this feature will be used
                # for blocking large numbers of innocent people, e.g. range blocks on
                # schools. Don't blame it on the user. There's a small chance that it
@@ -859,56 +937,56 @@ class LoginForm extends SpecialPage {
                # evade it, but we'll leave that to their guilty conscience to figure
                # out.
 
-               $wgOut->setPageTitle( wfMsg( 'cantcreateaccounttitle' ) );
+               $out = $this->getOutput();
+               $out->setPageTitle( $this->msg( 'cantcreateaccounttitle' ) );
 
                $block_reason = $block->mReason;
                if ( strval( $block_reason ) === '' ) {
-                       $block_reason = wfMsg( 'blockednoreason' );
+                       $block_reason = $this->msg( 'blockednoreason' )->text();
                }
 
-               $wgOut->addWikiMsg(
+               $out->addWikiMsg(
                        'cantcreateaccount-text',
                        $block->getTarget(),
                        $block_reason,
-                       $block->getBlocker()->getName()
+                       $block->getByName()
                );
-               
-               $wgOut->returnToMain( false );
+
+               $out->returnToMain( false );
        }
 
        /**
         * @private
         */
        function mainLoginForm( $msg, $msgtype = 'error' ) {
-               global $wgUser, $wgOut, $wgHiddenPrefs;
                global $wgEnableEmail, $wgEnableUserEmail;
-               global $wgRequest, $wgLoginLanguageSelector;
+               global $wgHiddenPrefs, $wgLoginLanguageSelector;
                global $wgAuth, $wgEmailConfirmToEdit, $wgCookieExpiration;
                global $wgSecureLogin, $wgPasswordResetRoutes;
 
-               $titleObj = SpecialPage::getTitleFor( 'Userlogin' );
+               $titleObj = $this->getTitle();
+               $user = $this->getUser();
 
                if ( $this->mType == 'signup' ) {
                        // Block signup here if in readonly. Keeps user from
                        // going through the process (filling out data, etc)
                        // and being informed later.
-                       if ( wfReadOnly() ) {
-                               $wgOut->readOnlyPage();
-                               return;
-                       } elseif ( $wgUser->isBlockedFromCreateAccount() ) {
-                               $this->userBlockedMessage( $wgUser->isBlockedFromCreateAccount() );
-                               return;
-                       } elseif ( count( $permErrors = $titleObj->getUserPermissionsErrors( 'createaccount', $wgUser, true ) )>0 ) {
-                               $wgOut->showPermissionsErrorPage( $permErrors, 'createaccount' );
+                       $permErrors = $titleObj->getUserPermissionsErrors( 'createaccount', $user, true );
+                       if ( count( $permErrors ) ) {
+                               throw new PermissionsError( 'createaccount', $permErrors );
+                       } elseif ( $user->isBlockedFromCreateAccount() ) {
+                               $this->userBlockedMessage( $user->isBlockedFromCreateAccount() );
                                return;
+                       } elseif ( wfReadOnly() ) {
+                               throw new ReadOnlyError;
                        }
                }
 
                if ( $this->mUsername == '' ) {
-                       if ( $wgUser->isLoggedIn() ) {
-                               $this->mUsername = $wgUser->getName();
+                       if ( $user->isLoggedIn() ) {
+                               $this->mUsername = $user->getName();
                        } else {
-                               $this->mUsername = $wgRequest->getCookie( 'UserName' );
+                               $this->mUsername = $this->getRequest()->getCookie( 'UserName' );
                        }
                }
 
@@ -934,18 +1012,16 @@ class LoginForm extends SpecialPage {
                        $linkq .= $returnto;
                }
 
-               # Pass any language selection on to the mode switch link
-               if( $wgLoginLanguageSelector && $this->mLanguage ) {
-                       $linkq .= '&uselang=' . $this->mLanguage;
-               }
-
-               $link = '<a href="' . htmlspecialchars ( $titleObj->getLocalURL( $linkq ) ) . '">';
-               $link .= wfMsgHtml( $linkmsg . 'link' ); # Calling either 'gotaccountlink' or 'nologinlink'
-               $link .= '</a>';
-
                # Don't show a "create account" link if the user can't
-               if( $this->showCreateOrLoginLink( $wgUser ) ) {
-                       $template->set( 'link', wfMsgExt( $linkmsg, array( 'parseinline', 'replaceafter' ), $link ) );
+               if( $this->showCreateOrLoginLink( $user ) ) {
+                       # Pass any language selection on to the mode switch link
+                       if( $wgLoginLanguageSelector && $this->mLanguage ) {
+                               $linkq .= '&uselang=' . $this->mLanguage;
+                       }
+                       $link = Html::element( 'a', array( 'href' => $titleObj->getLocalURL( $linkq ) ),
+                               $this->msg( $linkmsg . 'link' )->text() ); # Calling either 'gotaccountlink' or 'nologinlink'
+
+                       $template->set( 'link', $this->msg( $linkmsg )->rawParams( $link )->parse() );
                } else {
                        $template->set( 'link', '' );
                }
@@ -966,7 +1042,7 @@ class LoginForm extends SpecialPage {
                $template->set( 'action', $titleObj->getLocalURL( $q ) );
                $template->set( 'message', $msg );
                $template->set( 'messagetype', $msgtype );
-               $template->set( 'createemail', $wgEnableEmail && $wgUser->isLoggedIn() );
+               $template->set( 'createemail', $wgEnableEmail && $user->isLoggedIn() );
                $template->set( 'userealname', !in_array( 'realname', $wgHiddenPrefs ) );
                $template->set( 'useemail', $wgEnableEmail );
                $template->set( 'emailrequired', $wgEmailConfirmToEdit );
@@ -974,8 +1050,8 @@ class LoginForm extends SpecialPage {
                $template->set( 'canreset', $wgAuth->allowPasswordChange() );
                $template->set( 'resetlink', $resetLink );
                $template->set( 'canremember', ( $wgCookieExpiration > 0 ) );
-               $template->set( 'usereason', $wgUser->isLoggedIn() );
-               $template->set( 'remember', $wgUser->getOption( 'rememberpassword' ) || $this->mRemember );
+               $template->set( 'usereason', $user->isLoggedIn() );
+               $template->set( 'remember', $user->getOption( 'rememberpassword' ) || $this->mRemember );
                $template->set( 'cansecurelogin', ( $wgSecureLogin === true ) );
                $template->set( 'stickHTTPS', $this->mStickHTTPS );
 
@@ -994,8 +1070,25 @@ class LoginForm extends SpecialPage {
                # Prepare language selection links as needed
                if( $wgLoginLanguageSelector ) {
                        $template->set( 'languages', $this->makeLanguageSelector() );
-                       if( $this->mLanguage )
+                       if( $this->mLanguage ) {
                                $template->set( 'uselang', $this->mLanguage );
+                       }
+               }
+
+               // Use loginend-https for HTTPS requests if it's not blank, loginend otherwise
+               // Ditto for signupend
+               $usingHTTPS = WebRequest::detectProtocol() == 'https';
+               $loginendHTTPS = $this->msg( 'loginend-https' );
+               $signupendHTTPS = $this->msg( 'signupend-https' );
+               if ( $usingHTTPS && !$loginendHTTPS->isBlank() ) {
+                       $template->set( 'loginend', $loginendHTTPS->parse() );
+               } else {
+                       $template->set( 'loginend', $this->msg( 'loginend' )->parse() );
+               }
+               if ( $usingHTTPS && !$signupendHTTPS->isBlank() ) {
+                       $template->set( 'signupend', $signupendHTTPS->parse() );
+               } else {
+                       $template->set( 'signupend', $this->msg( 'signupend' )->parse() );
                }
 
                // Give authentication and captcha plugins a chance to modify the form
@@ -1006,15 +1099,9 @@ class LoginForm extends SpecialPage {
                        wfRunHooks( 'UserLoginForm', array( &$template ) );
                }
 
-               // Changes the title depending on permissions for creating account
-               if ( $wgUser->isAllowed( 'createaccount' ) ) {
-                       $wgOut->setPageTitle( wfMsg( 'userlogin' ) );
-               } else {
-                       $wgOut->setPageTitle( wfMsg( 'userloginnocreate' ) );
-               }
-
-               $wgOut->disallowUserJs(); // just in case...
-               $wgOut->addTemplate( $template );
+               $out = $this->getOutput();
+               $out->disallowUserJs(); // just in case...
+               $out->addTemplate( $template );
        }
 
        /**
@@ -1042,14 +1129,16 @@ class LoginForm extends SpecialPage {
         * previous pass through the system.
         *
         * @private
+        * @return bool
         */
        function hasSessionCookie() {
-               global $wgDisableCookieCheck, $wgRequest;
-               return $wgDisableCookieCheck ? true : $wgRequest->checkSessionCookie();
+               global $wgDisableCookieCheck;
+               return $wgDisableCookieCheck ? true : $this->getRequest()->checkSessionCookie();
        }
 
        /**
         * Get the login token from the current session
+        * @return Mixed
         */
        public static function getLoginToken() {
                global $wgRequest;
@@ -1076,6 +1165,7 @@ class LoginForm extends SpecialPage {
 
        /**
         * Get the createaccount token from the current session
+        * @return Mixed
         */
        public static function getCreateaccountToken() {
                global $wgRequest;
@@ -1102,8 +1192,6 @@ class LoginForm extends SpecialPage {
         * @private
         */
        function cookieRedirectCheck( $type ) {
-               global $wgOut;
-
                $titleObj = SpecialPage::getTitleFor( 'Userlogin' );
                $query = array( 'wpCookieCheck' => $type );
                if ( $this->mReturnTo ) {
@@ -1111,7 +1199,7 @@ class LoginForm extends SpecialPage {
                }
                $check = $titleObj->getFullURL( $query );
 
-               return $wgOut->redirect( $check );
+               $this->getOutput()->redirect( $check );
        }
 
        /**
@@ -1120,15 +1208,15 @@ class LoginForm extends SpecialPage {
        function onCookieRedirectCheck( $type ) {
                if ( !$this->hasSessionCookie() ) {
                        if ( $type == 'new' ) {
-                               return $this->mainLoginForm( wfMsgExt( 'nocookiesnew', array( 'parseinline' ) ) );
+                               $this->mainLoginForm( $this->msg( 'nocookiesnew' )->parse() );
                        } elseif ( $type == 'login' ) {
-                               return $this->mainLoginForm( wfMsgExt( 'nocookieslogin', array( 'parseinline' ) ) );
+                               $this->mainLoginForm( $this->msg( 'nocookieslogin' )->parse() );
                        } else {
                                # shouldn't happen
-                               return $this->mainLoginForm( wfMsg( 'error' ) );
+                               $this->mainLoginForm( $this->msg( 'error' )->text() );
                        }
                } else {
-                       return $this->successfulLogin();
+                       $this->successfulLogin();
                }
        }
 
@@ -1136,7 +1224,7 @@ class LoginForm extends SpecialPage {
         * @private
         */
        function throttleHit( $limit ) {
-               $this->mainLoginForm( wfMsgExt( 'acct_creation_throttle_hit', array( 'parseinline' ), $limit ) );
+               $this->mainLoginForm( $this->msg( 'acct_creation_throttle_hit' )->numParams( $limit )->parse() );
        }
 
        /**
@@ -1146,9 +1234,7 @@ class LoginForm extends SpecialPage {
         * @return string
         */
        function makeLanguageSelector() {
-               global $wgLang;
-
-               $msg = wfMessage( 'loginlanguagelinks' )->inContentLanguage();
+               $msg = $this->msg( 'loginlanguagelinks' )->inContentLanguage();
                if( !$msg->isBlank() ) {
                        $langs = explode( "\n", $msg->text() );
                        $links = array();
@@ -1156,10 +1242,11 @@ class LoginForm extends SpecialPage {
                                $lang = trim( $lang, '* ' );
                                $parts = explode( '|', $lang );
                                if ( count( $parts ) >= 2 ) {
-                                       $links[] = $this->makeLanguageSelectorLink( $parts[0], $parts[1] );
+                                       $links[] = $this->makeLanguageSelectorLink( $parts[0], trim( $parts[1] ) );
                                }
                        }
-                       return count( $links ) > 0 ? wfMsgHtml( 'loginlanguagelabel', $wgLang->pipeList( $links ) ) : '';
+                       return count( $links ) > 0 ? $this->msg( 'loginlanguagelabel' )->rawParams(
+                               $this->getLanguage()->pipeList( $links ) )->escaped() : '';
                } else {
                        return '';
                }
@@ -1171,10 +1258,9 @@ class LoginForm extends SpecialPage {
         *
         * @param $text Link text
         * @param $lang Language code
+        * @return string
         */
        function makeLanguageSelectorLink( $text, $lang ) {
-               global $wgUser;
-               $self = SpecialPage::getTitleFor( 'Userlogin' );
                $attr = array( 'uselang' => $lang );
                if( $this->mType == 'signup' ) {
                        $attr['type'] = 'signup';
@@ -1182,9 +1268,8 @@ class LoginForm extends SpecialPage {
                if( $this->mReturnTo ) {
                        $attr['returnto'] = $this->mReturnTo;
                }
-               $skin = $wgUser->getSkin();
-               return $skin->linkKnown(
-                       $self,
+               return Linker::linkKnown(
+                       $this->getTitle(),
                        htmlspecialchars( $text ),
                        array(),
                        $attr