Add support for Number grouping(commafy) based on CLDR number grouping patterns like...
[lhc/web/wiklou.git] / includes / User.php
index 6340815..7df864f 100644 (file)
@@ -1,6 +1,22 @@
 <?php
 /**
  * Implements the User class for the %MediaWiki software.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
+ *
  * @file
  */
 
@@ -71,7 +87,7 @@ class User {
                'mEmailTokenExpires',
                'mRegistration',
                'mEditCount',
-               // user_group table
+               // user_groups table
                'mGroups',
                // user_properties table
                'mOptionOverrides',
@@ -134,6 +150,7 @@ class User {
                'suppressredirect',
                'suppressrevision',
                'trackback',
+               'unblockself',
                'undelete',
                'unwatchedpages',
                'upload',
@@ -151,16 +168,24 @@ class User {
        //@{
        var $mId, $mName, $mRealName, $mPassword, $mNewpassword, $mNewpassTime,
                $mEmail, $mTouched, $mToken, $mEmailAuthenticated,
-               $mEmailToken, $mEmailTokenExpires, $mRegistration, $mGroups, $mOptionOverrides;
+               $mEmailToken, $mEmailTokenExpires, $mRegistration, $mGroups, $mOptionOverrides,
+               $mCookiePassword, $mEditCount, $mAllowUsertalk;
        //@}
 
        /**
         * Bool Whether the cache variables have been loaded.
         */
-       var $mDataLoaded, $mAuthLoaded, $mOptionsLoaded;
+       //@{
+       var $mOptionsLoaded;
+
+       /**
+        * Array with already loaded items or true if all items have been loaded.
+        */
+       private $mLoadedItems = array();
+       //@}
 
        /**
-        * String Initialization data source if mDataLoaded==false. May be one of:
+        * String Initialization data source if mLoadedItems!==true. May be one of:
         *  - 'defaults'   anonymous user initialised from class defaults
         *  - 'name'       initialise from mName
         *  - 'id'         initialise from mId
@@ -173,10 +198,25 @@ class User {
        /**
         * Lazy-initialized variables, invalidated with clearInstanceCache
         */
-       var $mNewtalk, $mDatePreference, $mBlockedby, $mHash, $mSkin, $mRights,
-               $mBlockreason, $mBlock, $mEffectiveGroups, $mBlockedGlobally,
+       var $mNewtalk, $mDatePreference, $mBlockedby, $mHash, $mRights,
+               $mBlockreason, $mEffectiveGroups, $mImplicitGroups, $mFormerGroups, $mBlockedGlobally,
                $mLocked, $mHideName, $mOptions;
 
+       /**
+        * @var WebRequest
+        */
+       private $mRequest;
+
+       /**
+        * @var Block
+        */
+       var $mBlock;
+
+       /**
+        * @var Block
+        */
+       private $mBlockedFromCreateAccount = false;
+
        static $idCacheByName = array();
 
        /**
@@ -193,17 +233,24 @@ class User {
                $this->clearInstanceCache( 'defaults' );
        }
 
+       /**
+        * @return String
+        */
+       function __toString(){
+               return $this->getName();
+       }
+
        /**
         * Load the user table data for this object from the source given by mFrom.
         */
-       function load() {
-               if ( $this->mDataLoaded ) {
+       public function load() {
+               if ( $this->mLoadedItems === true ) {
                        return;
                }
                wfProfileIn( __METHOD__ );
 
                # Set it now to avoid infinite recursion in accessors
-               $this->mDataLoaded = true;
+               $this->mLoadedItems = true;
 
                switch ( $this->mFrom ) {
                        case 'defaults':
@@ -234,9 +281,8 @@ class User {
        /**
         * Load user table data, given mId has already been set.
         * @return Bool false if the ID does not exist, true otherwise
-        * @private
         */
-       function loadFromId() {
+       public function loadFromId() {
                global $wgMemc;
                if ( $this->mId == 0 ) {
                        $this->loadDefaults();
@@ -272,7 +318,7 @@ class User {
        /**
         * Save user data to the shared cache
         */
-       function saveToCache() {
+       public function saveToCache() {
                $this->load();
                $this->loadGroups();
                $this->loadOptions();
@@ -290,7 +336,6 @@ class User {
                $wgMemc->set( $key, $data );
        }
 
-
        /** @name newFrom*() static factory methods */
        //@{
 
@@ -310,7 +355,7 @@ class User {
         *    username is not present in the database, the result will be a user object
         *    with a name, zero user ID and default settings.
         */
-       static function newFromName( $name, $validate = 'valid' ) {
+       public static function newFromName( $name, $validate = 'valid' ) {
                if ( $validate === true ) {
                        $validate = 'valid';
                }
@@ -322,6 +367,7 @@ class User {
                        $u = new User;
                        $u->mName = $name;
                        $u->mFrom = 'name';
+                       $u->setItemLoaded( 'name' );
                        return $u;
                }
        }
@@ -332,10 +378,11 @@ class User {
         * @param $id Int Valid user ID
         * @return User The corresponding User object
         */
-       static function newFromId( $id ) {
+       public static function newFromId( $id ) {
                $u = new User;
                $u->mId = $id;
                $u->mFrom = 'id';
+               $u->setItemLoaded( 'id' );
                return $u;
        }
 
@@ -349,7 +396,7 @@ class User {
         * @param $code String Confirmation code
         * @return User
         */
-       static function newFromConfirmationCode( $code ) {
+       public static function newFromConfirmationCode( $code ) {
                $dbr = wfGetDB( DB_SLAVE );
                $id = $dbr->selectField( 'user', 'user_id', array(
                        'user_email_token' => md5( $code ),
@@ -366,21 +413,31 @@ class User {
         * Create a new user object using data from session or cookies. If the
         * login credentials are invalid, the result is an anonymous user.
         *
+        * @param $request WebRequest object to use; $wgRequest will be used if
+        *        ommited.
         * @return User
         */
-       static function newFromSession() {
+       public static function newFromSession( WebRequest $request = null ) {
                $user = new User;
                $user->mFrom = 'session';
+               $user->mRequest = $request;
                return $user;
        }
 
        /**
         * Create a new user object from a user row.
-        * The row should have all fields from the user table in it.
+        * The row should have the following fields from the user table in it:
+        * - either user_name or user_id to load further data if needed (or both)
+        * - user_real_name
+        * - all other fields (email, password, etc.)
+        * It is useless to provide the remaining fields if either user_id,
+        * user_name and user_real_name are not provided because the whole row
+        * will be loaded once more from the database when accessing them.
+        *
         * @param $row Array A row from the user table
         * @return User
         */
-       static function newFromRow( $row ) {
+       public static function newFromRow( $row ) {
                $user = new User;
                $user->loadFromRow( $row );
                return $user;
@@ -388,7 +445,6 @@ class User {
 
        //@}
 
-
        /**
         * Get the username corresponding to a given user ID
         * @param $id Int User ID
@@ -405,7 +461,7 @@ class User {
         * @param $id Int User ID
         * @return String The corresponding user's real name
         */
-       static function whoIsReal( $id ) {
+       public static function whoIsReal( $id ) {
                $dbr = wfGetDB( DB_SLAVE );
                return $dbr->selectField( 'user', 'user_real_name', array( 'user_id' => $id ), __METHOD__ );
        }
@@ -415,7 +471,7 @@ class User {
         * @param $name String Username
         * @return Int|Null The corresponding user's ID, or null if user is nonexistent
         */
-       static function idFromName( $name ) {
+       public static function idFromName( $name ) {
                $nt = Title::makeTitleSafe( NS_USER, $name );
                if( is_null( $nt ) ) {
                        # Illegal name
@@ -467,7 +523,7 @@ class User {
         * @param $name String to match
         * @return Bool
         */
-       static function isIP( $name ) {
+       public static function isIP( $name ) {
                return preg_match('/^\d{1,3}\.\d{1,3}\.\d{1,3}\.(?:xxx|\d{1,3})$/',$name) || IP::isIPv6($name);
        }
 
@@ -482,7 +538,7 @@ class User {
         * @param $name String to match
         * @return Bool
         */
-       static function isValidUserName( $name ) {
+       public static function isValidUserName( $name ) {
                global $wgContLang, $wgMaxNameChars;
 
                if ( $name == ''
@@ -536,7 +592,7 @@ class User {
         * @param $name String to match
         * @return Bool
         */
-       static function isUsableName( $name ) {
+       public static function isUsableName( $name ) {
                global $wgReservedUsernames;
                // Must be a valid username, obviously ;)
                if ( !self::isValidUserName( $name ) ) {
@@ -573,7 +629,7 @@ class User {
         * @param $name String to match
         * @return Bool
         */
-       static function isCreatableName( $name ) {
+       public static function isCreatableName( $name ) {
                global $wgInvalidUsernameCharacters;
 
                // Ensure that the username isn't longer than 235 bytes, so that
@@ -603,7 +659,7 @@ class User {
         * @param $password String Desired password
         * @return Bool
         */
-       function isValidPassword( $password ) {
+       public function isValidPassword( $password ) {
                //simple boolean wrapper for getPasswordValidity
                return $this->getPasswordValidity( $password ) === true;
        }
@@ -614,7 +670,7 @@ class User {
         * @param $password String Desired password
         * @return mixed: true on success, string or array of error message on failure
         */
-       function getPasswordValidity( $password ) {
+       public function getPasswordValidity( $password ) {
                global $wgMinimalPasswordLength, $wgContLang;
 
                static $blockedLogins = array(
@@ -673,29 +729,10 @@ class User {
         *
         * @param $addr String E-mail address
         * @return Bool
+        * @deprecated since 1.18 call Sanitizer::isValidEmail() directly
         */
        public static function isValidEmailAddr( $addr ) {
-               $result = null;
-               if( !wfRunHooks( 'isValidEmailAddr', array( $addr, &$result ) ) ) {
-                       return $result;
-               }
-
-               // Please note strings below are enclosed in brackets [], this make the
-               // hyphen "-" a range indicator. Hence it is double backslashed below.
-               // See bug 26948
-               $rfc5322_atext   = "a-z0-9!#$%&'*+\\-\/=?^_`{|}~" ;
-               $rfc1034_ldh_str = "a-z0-9\\-" ;
-
-               $HTML5_email_regexp = "/
-               ^                      # start of string
-               [$rfc5322_atext\\.]+    # user part which is liberal :p
-               @                      # 'apostrophe'
-               [$rfc1034_ldh_str]+       # First domain part
-               (\\.[$rfc1034_ldh_str]+)*  # Following part prefixed with a dot
-               $                      # End of string
-               /ix" ; // case Insensitive, eXtended
-
-               return (bool) preg_match( $HTML5_email_regexp, $addr );
+               return Sanitizer::validateEmail( $addr );
        }
 
        /**
@@ -707,8 +744,10 @@ class User {
         *                - 'valid'      Valid for batch processes
         *                - 'usable'     Valid for batch processes and login
         *                - 'creatable'  Valid for batch processes, login and account creation
+        *
+        * @return bool|string
         */
-       static function getCanonicalName( $name, $validate = 'valid' ) {
+       public static function getCanonicalName( $name, $validate = 'valid' ) {
                # Force usernames to capital
                global $wgContLang;
                $name = $wgContLang->ucfirst( $name );
@@ -762,7 +801,7 @@ class User {
         * @param $uid Int User ID to check
         * @return Int the user's edit count
         */
-       static function edits( $uid ) {
+       public static function edits( $uid ) {
                wfProfileIn( __METHOD__ );
                $dbr = wfGetDB( DB_SLAVE );
                // check if the user_editcount field has been initialized
@@ -798,7 +837,7 @@ class User {
         *
         * @return String new random password
         */
-       static function randomPassword() {
+       public static function randomPassword() {
                global $wgMinimalPasswordLength;
                $pwchars = 'ABCDEFGHJKLMNPQRSTUVWXYZabcdefghjkmnpqrstuvwxyz';
                $l = strlen( $pwchars ) - 1;
@@ -807,7 +846,7 @@ class User {
                $digit = mt_rand( 0, $pwlength - 1 );
                $np = '';
                for ( $i = 0; $i < $pwlength; $i++ ) {
-                       $np .= $i == $digit ? chr( mt_rand( 48, 57 ) ) : $pwchars{ mt_rand( 0, $l ) };
+                       $np .= $i == $digit ? chr( mt_rand( 48, 57 ) ) : $pwchars[ mt_rand( 0, $l ) ];
                }
                return $np;
        }
@@ -817,13 +856,12 @@ class User {
         *
         * @note This no longer clears uncached lazy-initialised properties;
         *       the constructor does that instead.
-        * @private
+        *
+        * @param $name string
         */
-       function loadDefaults( $name = false ) {
+       public function loadDefaults( $name = false ) {
                wfProfileIn( __METHOD__ );
 
-               global $wgRequest;
-
                $this->mId = 0;
                $this->mName = $name;
                $this->mRealName = '';
@@ -833,8 +871,9 @@ class User {
                $this->mOptionOverrides = null;
                $this->mOptionsLoaded = false;
 
-               if( $wgRequest->getCookie( 'LoggedOut' ) !== null ) {
-                       $this->mTouched = wfTimestamp( TS_MW, $wgRequest->getCookie( 'LoggedOut' ) );
+               $loggedOut = $this->getRequest()->getCookie( 'LoggedOut' );
+               if( $loggedOut !== null ) {
+                       $this->mTouched = wfTimestamp( TS_MW, $loggedOut );
                } else {
                        $this->mTouched = '0'; # Allow any pages to be cached
                }
@@ -851,13 +890,41 @@ class User {
                wfProfileOut( __METHOD__ );
        }
 
+       /**
+        * Return whether an item has been loaded.
+        *
+        * @param $item String: item to check. Current possibilities:
+        *              - id
+        *              - name
+        *              - realname
+        * @param $all String: 'all' to check if the whole object has been loaded
+        *        or any other string to check if only the item is available (e.g.
+        *        for optimisation)
+        * @return Boolean
+        */
+       public function isItemLoaded( $item, $all = 'all' ) {
+               return ( $this->mLoadedItems === true && $all === 'all' ) ||
+                       ( isset( $this->mLoadedItems[$item] ) && $this->mLoadedItems[$item] === true );
+       }
+
+       /**
+        * Set that an item has been loaded
+        *
+        * @param $item String
+        */
+       private function setItemLoaded( $item ) {
+               if ( is_array( $this->mLoadedItems ) ) {
+                       $this->mLoadedItems[$item] = true;
+               }
+       }
+
        /**
         * Load user data from the session or login cookie. If there are no valid
         * credentials, initialises the user as an anonymous user.
         * @return Bool True if the user is logged in, false otherwise.
         */
        private function loadFromSession() {
-               global $wgRequest, $wgExternalAuthType, $wgAutocreatePolicy;
+               global $wgExternalAuthType, $wgAutocreatePolicy;
 
                $result = null;
                wfRunHooks( 'UserLoadFromSession', array( $this, &$result ) );
@@ -873,8 +940,10 @@ class User {
                        }
                }
 
-               $cookieId = $wgRequest->getCookie( 'UserID' );
-               $sessId = $wgRequest->getSessionData( 'wsUserID' );
+               $request = $this->getRequest();
+
+               $cookieId = $request->getCookie( 'UserID' );
+               $sessId = $request->getSessionData( 'wsUserID' );
 
                if ( $cookieId !== null ) {
                        $sId = intval( $cookieId );
@@ -884,42 +953,43 @@ class User {
                                        cookie user ID ($sId) don't match!" );
                                return false;
                        }
-                       $wgRequest->setSessionData( 'wsUserID', $sId );
-               } else if ( $sessId !== null && $sessId != 0 ) {
+                       $request->setSessionData( 'wsUserID', $sId );
+               } elseif ( $sessId !== null && $sessId != 0 ) {
                        $sId = $sessId;
                } else {
                        $this->loadDefaults();
                        return false;
                }
 
-               if ( $wgRequest->getSessionData( 'wsUserName' ) !== null ) {
-                       $sName = $wgRequest->getSessionData( 'wsUserName' );
-               } else if ( $wgRequest->getCookie( 'UserName' ) !== null ) {
-                       $sName = $wgRequest->getCookie( 'UserName' );
-                       $wgRequest->setSessionData( 'wsUserName', $sName );
+               if ( $request->getSessionData( 'wsUserName' ) !== null ) {
+                       $sName = $request->getSessionData( 'wsUserName' );
+               } elseif ( $request->getCookie( 'UserName' ) !== null ) {
+                       $sName = $request->getCookie( 'UserName' );
+                       $request->setSessionData( 'wsUserName', $sName );
                } else {
                        $this->loadDefaults();
                        return false;
                }
 
-               $this->mId = $sId;
-               if ( !$this->loadFromId() ) {
-                       # Not a valid ID, loadFromId has switched the object to anon for us
+               $proposedUser = User::newFromId( $sId );
+               if ( !$proposedUser->isLoggedIn() ) {
+                       # Not a valid ID
+                       $this->loadDefaults();
                        return false;
                }
 
                global $wgBlockDisablesLogin;
-               if( $wgBlockDisablesLogin && $this->isBlocked() ) {
+               if( $wgBlockDisablesLogin && $proposedUser->isBlocked() ) {
                        # User blocked and we've disabled blocked user logins
                        $this->loadDefaults();
                        return false;
                }
 
-               if ( $wgRequest->getSessionData( 'wsToken' ) !== null ) {
-                       $passwordCorrect = $this->mToken == $wgRequest->getSessionData( 'wsToken' );
+               if ( $request->getSessionData( 'wsToken' ) !== null ) {
+                       $passwordCorrect = $proposedUser->getToken() === $request->getSessionData( 'wsToken' );
                        $from = 'session';
-               } else if ( $wgRequest->getCookie( 'Token' ) !== null ) {
-                       $passwordCorrect = $this->mToken == $wgRequest->getCookie( 'Token' );
+               } elseif ( $request->getCookie( 'Token' ) !== null ) {
+                       $passwordCorrect = $proposedUser->getToken() === $request->getCookie( 'Token' );
                        $from = 'cookie';
                } else {
                        # No session or persistent login cookie
@@ -927,8 +997,9 @@ class User {
                        return false;
                }
 
-               if ( ( $sName == $this->mName ) && $passwordCorrect ) {
-                       $wgRequest->setSessionData( 'wsToken', $this->mToken );
+               if ( ( $sName === $proposedUser->getName() ) && $passwordCorrect ) {
+                       $this->loadFromUserObject( $proposedUser );
+                       $request->setSessionData( 'wsToken', $this->mToken );
                        wfDebug( "User: logged in from $from\n" );
                        return true;
                } else {
@@ -941,12 +1012,11 @@ class User {
 
        /**
         * Load user and user_group data from the database.
-        * $this::mId must be set, this is how the user is identified.
+        * $this->mId must be set, this is how the user is identified.
         *
         * @return Bool True if the user exists, false if the user is anonymous
-        * @private
         */
-       function loadFromDatabase() {
+       public function loadFromDatabase() {
                # Paranoia
                $this->mId = intval( $this->mId );
 
@@ -980,33 +1050,77 @@ class User {
         *
         * @param $row Array Row from the user table to load.
         */
-       function loadFromRow( $row ) {
-               $this->mDataLoaded = true;
+       public function loadFromRow( $row ) {
+               $all = true;
+
+               if ( isset( $row->user_name ) ) {
+                       $this->mName = $row->user_name;
+                       $this->mFrom = 'name';
+                       $this->setItemLoaded( 'name' );
+               } else {
+                       $all = false;
+               }
+
+               if ( isset( $row->user_real_name ) ) {
+                       $this->mRealName = $row->user_real_name;
+                       $this->setItemLoaded( 'realname' );
+               } else {
+                       $all = false;
+               }
 
                if ( isset( $row->user_id ) ) {
                        $this->mId = intval( $row->user_id );
+                       $this->mFrom = 'id';
+                       $this->setItemLoaded( 'id' );
+               } else {
+                       $all = false;
+               }
+
+               if ( isset( $row->user_editcount ) ) {
+                       $this->mEditCount = $row->user_editcount;
+               } else {
+                       $all = false;
+               }
+
+               if ( isset( $row->user_password ) ) {
+                       $this->mPassword = $row->user_password;
+                       $this->mNewpassword = $row->user_newpassword;
+                       $this->mNewpassTime = wfTimestampOrNull( TS_MW, $row->user_newpass_time );
+                       $this->mEmail = $row->user_email;
+                       $this->decodeOptions( $row->user_options );
+                       $this->mTouched = wfTimestamp(TS_MW,$row->user_touched);
+                       $this->mToken = $row->user_token;
+                       $this->mEmailAuthenticated = wfTimestampOrNull( TS_MW, $row->user_email_authenticated );
+                       $this->mEmailToken = $row->user_email_token;
+                       $this->mEmailTokenExpires = wfTimestampOrNull( TS_MW, $row->user_email_token_expires );
+                       $this->mRegistration = wfTimestampOrNull( TS_MW, $row->user_registration );
+               } else {
+                       $all = false;
+               }
+
+               if ( $all ) {
+                       $this->mLoadedItems = true;
+               }
+       }
+
+       /**
+        * Load the data for this user object from another user object.
+        *
+        * @param $user User
+        */
+       protected function loadFromUserObject( $user ) {
+               $user->load();
+               $user->loadGroups();
+               $user->loadOptions();
+               foreach ( self::$mCacheVars as $var ) {
+                       $this->$var = $user->$var;
                }
-               $this->mName = $row->user_name;
-               $this->mRealName = $row->user_real_name;
-               $this->mPassword = $row->user_password;
-               $this->mNewpassword = $row->user_newpassword;
-               $this->mNewpassTime = wfTimestampOrNull( TS_MW, $row->user_newpass_time );
-               $this->mEmail = $row->user_email;
-               $this->decodeOptions( $row->user_options );
-               $this->mTouched = wfTimestamp(TS_MW,$row->user_touched);
-               $this->mToken = $row->user_token;
-               $this->mEmailAuthenticated = wfTimestampOrNull( TS_MW, $row->user_email_authenticated );
-               $this->mEmailToken = $row->user_email_token;
-               $this->mEmailTokenExpires = wfTimestampOrNull( TS_MW, $row->user_email_token_expires );
-               $this->mRegistration = wfTimestampOrNull( TS_MW, $row->user_registration );
-               $this->mEditCount = $row->user_editcount;
        }
 
        /**
         * Load the groups from the database if they aren't already loaded.
-        * @private
         */
-       function loadGroups() {
+       private function loadGroups() {
                if ( is_null( $this->mGroups ) ) {
                        $dbr = wfGetDB( DB_MASTER );
                        $res = $dbr->select( 'user_groups',
@@ -1020,24 +1134,63 @@ class User {
                }
        }
 
+       /**
+        * Add the user to the group if he/she meets given criteria.
+        *
+        * Contrary to autopromotion by \ref $wgAutopromote, the group will be
+        *   possible to remove manually via Special:UserRights. In such case it
+        *   will not be re-added automatically. The user will also not lose the
+        *   group if they no longer meet the criteria.
+        *
+        * @param $event String key in $wgAutopromoteOnce (each one has groups/criteria)
+        *
+        * @return array Array of groups the user has been promoted to.
+        *
+        * @see $wgAutopromoteOnce
+        */
+       public function addAutopromoteOnceGroups( $event ) {
+               global $wgAutopromoteOnceLogInRC;
+
+               $toPromote = array();
+               if ( $this->getId() ) {
+                       $toPromote = Autopromote::getAutopromoteOnceGroups( $this, $event );
+                       if ( count( $toPromote ) ) {
+                               $oldGroups = $this->getGroups(); // previous groups
+                               foreach ( $toPromote as $group ) {
+                                       $this->addGroup( $group );
+                               }
+                               $newGroups = array_merge( $oldGroups, $toPromote ); // all groups
+
+                               $log = new LogPage( 'rights', $wgAutopromoteOnceLogInRC /* in RC? */ );
+                               $log->addEntry( 'autopromote',
+                                       $this->getUserPage(),
+                                       '', // no comment
+                                       // These group names are "list to texted"-ed in class LogPage.
+                                       array( implode( ', ', $oldGroups ), implode( ', ', $newGroups ) )
+                               );
+                       }
+               }
+               return $toPromote;
+       }
+
        /**
         * Clear various cached data stored in this object.
-        * @param $reloadFrom String Reload user and user_groups table data from a
+        * @param $reloadFrom bool|String Reload user and user_groups table data from a
         *   given source. May be "name", "id", "defaults", "session", or false for
         *   no reload.
         */
-       function clearInstanceCache( $reloadFrom = false ) {
+       public function clearInstanceCache( $reloadFrom = false ) {
                $this->mNewtalk = -1;
                $this->mDatePreference = null;
                $this->mBlockedby = -1; # Unset
                $this->mHash = false;
-               $this->mSkin = null;
                $this->mRights = null;
                $this->mEffectiveGroups = null;
+               $this->mImplicitGroups = null;
                $this->mOptions = null;
 
                if ( $reloadFrom ) {
-                       $this->mDataLoaded = false;
+                       $this->mLoadedItems = array();
                        $this->mFrom = $reloadFrom;
                }
        }
@@ -1048,17 +1201,11 @@ class User {
         *
         * @return Array of String options
         */
-       static function getDefaultOptions() {
-               global $wgNamespacesToBeSearchedDefault;
-               /**
-                * Site defaults will override the global/language defaults
-                */
-               global $wgDefaultUserOptions, $wgContLang, $wgDefaultSkin;
-               $defOpt = $wgDefaultUserOptions + $wgContLang->getDefaultUserOptionOverrides();
-
-               /**
-                * default language setting
-                */
+       public static function getDefaultOptions() {
+               global $wgNamespacesToBeSearchedDefault, $wgDefaultUserOptions, $wgContLang, $wgDefaultSkin;
+
+               $defOpt = $wgDefaultUserOptions;
+               # default language setting
                $variant = $wgContLang->getDefaultVariant();
                $defOpt['variant'] = $variant;
                $defOpt['language'] = $variant;
@@ -1067,6 +1214,8 @@ class User {
                }
                $defOpt['skin'] = $wgDefaultSkin;
 
+               wfRunHooks( 'UserGetDefaultOptions', array( &$defOpt ) );
+
                return $defOpt;
        }
 
@@ -1088,13 +1237,12 @@ class User {
 
        /**
         * Get blocking information
-        * @private
         * @param $bFromSlave Bool Whether to check the slave database first. To
         *                    improve performance, non-critical checks are done
         *                    against slaves. Check when actually saving should be
         *                    done against master.
         */
-       function getBlockedStatus( $bFromSlave = true ) {
+       private function getBlockedStatus( $bFromSlave = true ) {
                global $wgProxyWhitelist, $wgUser;
 
                if ( -1 != $this->mBlockedby ) {
@@ -1115,50 +1263,32 @@ class User {
                $this->mHideName = 0;
                $this->mAllowUsertalk = 0;
 
-               # Check if we are looking at an IP or a logged-in user
-               if ( $this->isIP( $this->getName() ) ) {
-                       $ip = $this->getName();
+               # We only need to worry about passing the IP address to the Block generator if the
+               # user is not immune to autoblocks/hardblocks, and they are the current user so we
+               # know which IP address they're actually coming from
+               if ( !$this->isAllowed( 'ipblock-exempt' ) && $this->getID() == $wgUser->getID() ) {
+                       $ip = $this->getRequest()->getIP();
                } else {
-                       # Check if we are looking at the current user
-                       # If we don't, and the user is logged in, we don't know about
-                       # his IP / autoblock status, so ignore autoblock of current user's IP
-                       if ( $this->getID() != $wgUser->getID() ) {
-                               $ip = '';
-                       } else {
-                               # Get IP of current user
-                               $ip = wfGetIP();
-                       }
-               }
-
-               if ( $this->isAllowed( 'ipblock-exempt' ) ) {
-                       # Exempt from all types of IP-block
-                       $ip = '';
+                       $ip = null;
                }
 
                # User/IP blocking
-               $this->mBlock = new Block();
-               $this->mBlock->fromMaster( !$bFromSlave );
-               if ( $this->mBlock->load( $ip , $this->mId ) ) {
+               $this->mBlock = Block::newFromTarget( $this->getName(), $ip, !$bFromSlave );
+               if ( $this->mBlock instanceof Block ) {
                        wfDebug( __METHOD__ . ": Found block.\n" );
-                       $this->mBlockedby = $this->mBlock->mBy;
-                       if( $this->mBlockedby == 0 )
-                               $this->mBlockedby = $this->mBlock->mByName;
+                       $this->mBlockedby = $this->mBlock->getByName();
                        $this->mBlockreason = $this->mBlock->mReason;
                        $this->mHideName = $this->mBlock->mHideName;
-                       $this->mAllowUsertalk = $this->mBlock->mAllowUsertalk;
+                       $this->mAllowUsertalk = !$this->mBlock->prevents( 'editownusertalk' );
                        if ( $this->isLoggedIn() && $wgUser->getID() == $this->getID() ) {
                                $this->spreadBlock();
                        }
-               } else {
-                       // Bug 13611: don't remove mBlock here, to allow account creation blocks to
-                       // apply to users. Note that the existence of $this->mBlock is not used to
-                       // check for edit blocks, $this->mBlockedby is instead.
                }
 
                # Proxy blocking
-               if ( !$this->isAllowed( 'proxyunbannable' ) && !in_array( $ip, $wgProxyWhitelist ) ) {
+               if ( $ip !== null && !$this->isAllowed( 'proxyunbannable' ) && !in_array( $ip, $wgProxyWhitelist ) ) {
                        # Local list
-                       if ( wfIsLocallyBlockedProxy( $ip ) ) {
+                       if ( self::isLocallyBlockedProxy( $ip ) ) {
                                $this->mBlockedby = wfMsg( 'proxyblocker' );
                                $this->mBlockreason = wfMsg( 'proxyblockreason' );
                        }
@@ -1185,7 +1315,7 @@ class User {
         * @param $checkWhitelist Bool: whether to check the whitelist first
         * @return Bool True if blacklisted.
         */
-       function isDnsBlacklisted( $ip, $checkWhitelist = false ) {
+       public function isDnsBlacklisted( $ip, $checkWhitelist = false ) {
                global $wgEnableSorbs, $wgEnableDnsBlacklist,
                        $wgSorbsUrl, $wgDnsBlacklistUrls, $wgProxyWhitelist;
 
@@ -1206,18 +1336,28 @@ class User {
         * @param $bases String|Array of Strings: URL of the DNS blacklist
         * @return Bool True if blacklisted.
         */
-       function inDnsBlacklist( $ip, $bases ) {
+       public function inDnsBlacklist( $ip, $bases ) {
                wfProfileIn( __METHOD__ );
 
                $found = false;
-               // FIXME: IPv6 ???  (http://bugs.php.net/bug.php?id=33170)
+               // @todo FIXME: IPv6 ???  (http://bugs.php.net/bug.php?id=33170)
                if( IP::isIPv4( $ip ) ) {
                        # Reverse IP, bug 21255
                        $ipReversed = implode( '.', array_reverse( explode( '.', $ip ) ) );
 
                        foreach( (array)$bases as $base ) {
                                # Make hostname
-                               $host = "$ipReversed.$base";
+                               # If we have an access key, use that too (ProjectHoneypot, etc.)
+                               if( is_array( $base ) ) {
+                                       if( count( $base ) >= 2 ) {
+                                               # Access key is 1, base URL is 0
+                                               $host = "{$base[1]}.$ipReversed.{$base[0]}";
+                                       } else {
+                                               $host = "$ipReversed.{$base[0]}";
+                                       }
+                               } else {
+                                       $host = "$ipReversed.$base";
+                               }
 
                                # Send query
                                $ipList = gethostbynamel( $host );
@@ -1236,19 +1376,48 @@ class User {
                return $found;
        }
 
+       /**
+        * Check if an IP address is in the local proxy list
+        *
+        * @param $ip string
+        *
+        * @return bool
+        */
+       public static function isLocallyBlockedProxy( $ip ) {
+               global $wgProxyList;
+
+               if ( !$wgProxyList ) {
+                       return false;
+               }
+               wfProfileIn( __METHOD__ );
+
+               if ( !is_array( $wgProxyList ) ) {
+                       # Load from the specified file
+                       $wgProxyList = array_map( 'trim', file( $wgProxyList ) );
+               }
+
+               if ( !is_array( $wgProxyList ) ) {
+                       $ret = false;
+               } elseif ( array_search( $ip, $wgProxyList ) !== false ) {
+                       $ret = true;
+               } elseif ( array_key_exists( $ip, $wgProxyList ) ) {
+                       # Old-style flipped proxy list
+                       $ret = true;
+               } else {
+                       $ret = false;
+               }
+               wfProfileOut( __METHOD__ );
+               return $ret;
+       }
+
        /**
         * Is this user subject to rate limiting?
         *
         * @return Bool True if rate limited
         */
        public function isPingLimitable() {
-               global $wgRateLimitsExcludedGroups;
                global $wgRateLimitsExcludedIPs;
-               if( array_intersect( $this->getEffectiveGroups(), $wgRateLimitsExcludedGroups ) ) {
-                       // Deprecated, but kept for backwards-compatibility config
-                       return false;
-               }
-               if( in_array( wfGetIP(), $wgRateLimitsExcludedIPs ) ) {
+               if( in_array( $this->getRequest()->getIP(), $wgRateLimitsExcludedIPs ) ) {
                        // No other good way currently to disable rate limits
                        // for specific IPs. :P
                        // But this is a crappy hack and should die.
@@ -1267,7 +1436,7 @@ class User {
         * @param $action String Action to enforce; 'edit' if unspecified
         * @return Bool True if a rate limiter was tripped
         */
-       function pingLimiter( $action = 'edit' ) {
+       public function pingLimiter( $action = 'edit' ) {
                # Call the 'PingLimiter' hook
                $result = false;
                if( !wfRunHooks( 'PingLimiter', array( &$this, $action, $result ) ) ) {
@@ -1289,7 +1458,7 @@ class User {
                $limits = $wgRateLimits[$action];
                $keys = array();
                $id = $this->getId();
-               $ip = wfGetIP();
+               $ip = $this->getRequest()->getIP();
                $userLimit = false;
 
                if( isset( $limits['anon'] ) && $id == 0 ) {
@@ -1337,7 +1506,9 @@ class User {
                                if( $count > $max ) {
                                        wfDebug( __METHOD__ . ": tripped! $key at $count $summary\n" );
                                        if( $wgRateLimitLog ) {
-                                               @error_log( wfTimestamp( TS_MW ) . ' ' . wfWikiID() . ': ' . $this->getName() . " tripped $key at $count $summary\n", 3, $wgRateLimitLog );
+                                               wfSuppressWarnings();
+                                               file_put_contents( $wgRateLimitLog, wfTimestamp( TS_MW ) . ' ' . wfWikiID() . ': ' . $this->getName() . " tripped $key at $count $summary\n", FILE_APPEND );
+                                               wfRestoreWarnings();
                                        }
                                        $triggered = true;
                                } else {
@@ -1360,9 +1531,19 @@ class User {
         * @param $bFromSlave Bool Whether to check the slave database instead of the master
         * @return Bool True if blocked, false otherwise
         */
-       function isBlocked( $bFromSlave = true ) { // hacked from false due to horrible probs on site
+       public function isBlocked( $bFromSlave = true ) { // hacked from false due to horrible probs on site
+               return $this->getBlock( $bFromSlave ) instanceof Block && $this->getBlock()->prevents( 'edit' );
+       }
+
+       /**
+        * Get the block affecting the user, or null if the user is not blocked
+        *
+        * @param $bFromSlave Bool Whether to check the slave database instead of the master
+        * @return Block|null
+        */
+       public function getBlock( $bFromSlave = true ){
                $this->getBlockedStatus( $bFromSlave );
-               return $this->mBlockedby !== 0;
+               return $this->mBlock instanceof Block ? $this->mBlock : null;
        }
 
        /**
@@ -1395,7 +1576,7 @@ class User {
         * If user is blocked, return the name of the user who placed the block
         * @return String name of blocker
         */
-       function blockedBy() {
+       public function blockedBy() {
                $this->getBlockedStatus();
                return $this->mBlockedby;
        }
@@ -1404,7 +1585,7 @@ class User {
         * If user is blocked, return the specified reason for the block
         * @return String Blocking reason
         */
-       function blockedFor() {
+       public function blockedFor() {
                $this->getBlockedStatus();
                return $this->mBlockreason;
        }
@@ -1413,9 +1594,9 @@ class User {
         * If user is blocked, return the ID for the block
         * @return Int Block ID
         */
-       function getBlockId() {
+       public function getBlockId() {
                $this->getBlockedStatus();
-               return ( $this->mBlock ? $this->mBlock->mId : false );
+               return ( $this->mBlock ? $this->mBlock->getId() : false );
        }
 
        /**
@@ -1426,15 +1607,15 @@ class User {
         * @param $ip String IP address, uses current client if none given
         * @return Bool True if blocked, false otherwise
         */
-       function isBlockedGlobally( $ip = '' ) {
+       public function isBlockedGlobally( $ip = '' ) {
                if( $this->mBlockedGlobally !== null ) {
                        return $this->mBlockedGlobally;
                }
                // User is already an IP?
                if( IP::isIPAddress( $this->getName() ) ) {
                        $ip = $this->getName();
-               } else if( !$ip ) {
-                       $ip = wfGetIP();
+               } elseif( !$ip ) {
+                       $ip = $this->getRequest()->getIP();
                }
                $blocked = false;
                wfRunHooks( 'UserIsBlockedGlobally', array( &$this, $ip, &$blocked ) );
@@ -1447,7 +1628,7 @@ class User {
         *
         * @return Bool True if locked, false otherwise
         */
-       function isLocked() {
+       public function isLocked() {
                if( $this->mLocked !== null ) {
                        return $this->mLocked;
                }
@@ -1462,7 +1643,7 @@ class User {
         *
         * @return Bool True if hidden, false otherwise
         */
-       function isHidden() {
+       public function isHidden() {
                if( $this->mHideName !== null ) {
                        return $this->mHideName;
                }
@@ -1479,12 +1660,12 @@ class User {
         * Get the user's ID.
         * @return Int The user's ID; 0 if the user is anonymous or nonexistent
         */
-       function getId() {
-               if( $this->mId === null and $this->mName !== null
-               and User::isIP( $this->mName ) ) {
+       public function getId() {
+               if( $this->mId === null && $this->mName !== null
+               && User::isIP( $this->mName ) ) {
                        // Special case, we know the user is anonymous
                        return 0;
-               } elseif( $this->mId === null ) {
+               } elseif( !$this->isItemLoaded( 'id' ) ) {
                        // Don't load if this was initialized from an ID
                        $this->load();
                }
@@ -1495,7 +1676,7 @@ class User {
         * Set the user and reload all fields according to a given ID
         * @param $v Int User ID to reload
         */
-       function setId( $v ) {
+       public function setId( $v ) {
                $this->mId = $v;
                $this->clearInstanceCache( 'id' );
        }
@@ -1504,15 +1685,15 @@ class User {
         * Get the user name, or the IP of an anonymous user
         * @return String User's name or IP address
         */
-       function getName() {
-               if ( !$this->mDataLoaded && $this->mFrom == 'name' ) {
+       public function getName() {
+               if ( $this->isItemLoaded( 'name', 'only' ) ) {
                        # Special case optimisation
                        return $this->mName;
                } else {
                        $this->load();
                        if ( $this->mName === false ) {
                                # Clean up IPs
-                               $this->mName = IP::sanitizeIP( wfGetIP() );
+                               $this->mName = IP::sanitizeIP( $this->getRequest()->getIP() );
                        }
                        return $this->mName;
                }
@@ -1531,7 +1712,7 @@ class User {
         * does not exist.
         * @param $str String New user name to set
         */
-       function setName( $str ) {
+       public function setName( $str ) {
                $this->load();
                $this->mName = $str;
        }
@@ -1540,7 +1721,7 @@ class User {
         * Get the user's name escaped by underscores.
         * @return String Username escaped by underscores.
         */
-       function getTitleKey() {
+       public function getTitleKey() {
                return str_replace( ' ', '_', $this->getName() );
        }
 
@@ -1548,7 +1729,7 @@ class User {
         * Check if the user has new messages.
         * @return Bool True if the user has new messages
         */
-       function getNewtalk() {
+       public function getNewtalk() {
                $this->load();
 
                # Load the newtalk status if it is unloaded (mNewtalk=-1)
@@ -1581,7 +1762,7 @@ class User {
         * Return the talk page(s) this user has new messages on.
         * @return Array of String page URLs
         */
-       function getNewMessageLinks() {
+       public function getNewMessageLinks() {
                $talks = array();
                if( !wfRunHooks( 'UserRetrieveNewTalks', array( &$this, &$talks ) ) )
                        return $talks;
@@ -1601,9 +1782,8 @@ class User {
         * @param $id String|Int User's IP address for anonymous users, User ID otherwise
         * @param $fromMaster Bool true to fetch from the master, false for a slave
         * @return Bool True if the user has new messages
-        * @private
         */
-       function checkNewtalk( $field, $id, $fromMaster = false ) {
+       protected function checkNewtalk( $field, $id, $fromMaster = false ) {
                if ( $fromMaster ) {
                        $db = wfGetDB( DB_MASTER );
                } else {
@@ -1619,9 +1799,8 @@ class User {
         * @param $field String 'user_ip' for anonymous users, 'user_id' otherwise
         * @param $id String|Int User's IP address for anonymous users, User ID otherwise
         * @return Bool True if successful, false otherwise
-        * @private
         */
-       function updateNewtalk( $field, $id ) {
+       protected function updateNewtalk( $field, $id ) {
                $dbw = wfGetDB( DB_MASTER );
                $dbw->insert( 'user_newtalk',
                        array( $field => $id ),
@@ -1641,9 +1820,8 @@ class User {
         * @param $field String 'user_ip' for anonymous users, 'user_id' otherwise
         * @param $id String|Int User's IP address for anonymous users, User ID otherwise
         * @return Bool True if successful, false otherwise
-        * @private
         */
-       function deleteNewtalk( $field, $id ) {
+       protected function deleteNewtalk( $field, $id ) {
                $dbw = wfGetDB( DB_MASTER );
                $dbw->delete( 'user_newtalk',
                        array( $field => $id ),
@@ -1661,7 +1839,7 @@ class User {
         * Update the 'You have new messages!' status.
         * @param $val Bool Whether the user has new messages
         */
-       function setNewtalk( $val ) {
+       public function setNewtalk( $val ) {
                if( wfReadOnly() ) {
                        return;
                }
@@ -1725,7 +1903,7 @@ class User {
         * Updates user_touched field, and removes account data from memcached
         * for reload on the next hit.
         */
-       function invalidateCache() {
+       public function invalidateCache() {
                if( wfReadOnly() ) {
                        return;
                }
@@ -1746,8 +1924,10 @@ class User {
        /**
         * Validate the cache for this account.
         * @param $timestamp String A timestamp in TS_MW format
+        *
+        * @return bool
         */
-       function validateCache( $timestamp ) {
+       public function validateCache( $timestamp ) {
                $this->load();
                return ( $timestamp >= $this->mTouched );
        }
@@ -1756,7 +1936,7 @@ class User {
         * Get the user touched timestamp
         * @return String timestamp
         */
-       function getTouched() {
+       public function getTouched() {
                $this->load();
                return $this->mTouched;
        }
@@ -1774,8 +1954,10 @@ class User {
         *
         * @param $str String New password to set
         * @throws PasswordError on failure
+        *
+        * @return bool
         */
-       function setPassword( $str ) {
+       public function setPassword( $str ) {
                global $wgAuth;
 
                if( $str !== null ) {
@@ -1811,7 +1993,7 @@ class User {
         *
         * @param $str String New password to set
         */
-       function setInternalPassword( $str ) {
+       public function setInternalPassword( $str ) {
                $this->load();
                $this->setToken();
 
@@ -1829,7 +2011,7 @@ class User {
         * Get the user's current token.
         * @return String Token
         */
-       function getToken() {
+       public function getToken() {
                $this->load();
                return $this->mToken;
        }
@@ -1838,10 +2020,9 @@ class User {
         * Set the random token (used for persistent authentication)
         * Called from loadDefaults() among other places.
         *
-        * @param $token String If specified, set the token to this value
-        * @private
+        * @param $token String|bool If specified, set the token to this value
         */
-       function setToken( $token = false ) {
+       public function setToken( $token = false ) {
                global $wgSecretKey, $wgProxyKey;
                $this->load();
                if ( !$token ) {
@@ -1862,9 +2043,8 @@ class User {
         * Set the cookie password
         *
         * @param $str String New cookie password
-        * @private
         */
-       function setCookiePassword( $str ) {
+       private function setCookiePassword( $str ) {
                $this->load();
                $this->mCookiePassword = md5( $str );
        }
@@ -1875,7 +2055,7 @@ class User {
         * @param $str String New password to set
         * @param $throttle Bool If true, reset the throttle timestamp to the present
         */
-       function setNewpassword( $str, $throttle = true ) {
+       public function setNewpassword( $str, $throttle = true ) {
                $this->load();
                $this->mNewpassword = self::crypt( $str );
                if ( $throttle ) {
@@ -1888,7 +2068,7 @@ class User {
         * $wgPasswordReminderResendTime hours?
         * @return Bool
         */
-       function isPasswordReminderThrottled() {
+       public function isPasswordReminderThrottled() {
                global $wgPasswordReminderResendTime;
                $this->load();
                if ( !$this->mNewpassTime || !$wgPasswordReminderResendTime ) {
@@ -1902,7 +2082,7 @@ class User {
         * Get the user's e-mail address
         * @return String User's email address
         */
-       function getEmail() {
+       public function getEmail() {
                $this->load();
                wfRunHooks( 'UserGetEmail', array( $this, &$this->mEmail ) );
                return $this->mEmail;
@@ -1912,7 +2092,7 @@ class User {
         * Get the timestamp of the user's e-mail authentication
         * @return String TS_MW timestamp
         */
-       function getEmailAuthenticationTimestamp() {
+       public function getEmailAuthenticationTimestamp() {
                $this->load();
                wfRunHooks( 'UserGetEmailAuthenticationTimestamp', array( $this, &$this->mEmailAuthenticated ) );
                return $this->mEmailAuthenticated;
@@ -1922,7 +2102,7 @@ class User {
         * Set the user's e-mail address
         * @param $str String New e-mail address
         */
-       function setEmail( $str ) {
+       public function setEmail( $str ) {
                $this->load();
                $this->mEmail = $str;
                wfRunHooks( 'UserSetEmail', array( $this, &$this->mEmail ) );
@@ -1932,8 +2112,11 @@ class User {
         * Get the user's real name
         * @return String User's real name
         */
-       function getRealName() {
-               $this->load();
+       public function getRealName() {
+               if ( !$this->isItemLoaded( 'realname' ) ) {
+                       $this->load();
+               }
+
                return $this->mRealName;
        }
 
@@ -1941,7 +2124,7 @@ class User {
         * Set the user's real name
         * @param $str String New real name
         */
-       function setRealName( $str ) {
+       public function setRealName( $str ) {
                $this->load();
                $this->mRealName = $str;
        }
@@ -1951,11 +2134,13 @@ class User {
         *
         * @param $oname String The option to check
         * @param $defaultOverride String A default value returned if the option does not exist
+        * @param $ignoreHidden Bool = whether to ignore the effects of $wgHiddenPrefs
         * @return String User's current value for the option
         * @see getBoolOption()
         * @see getIntOption()
         */
-       function getOption( $oname, $defaultOverride = null ) {
+       public function getOption( $oname, $defaultOverride = null, $ignoreHidden = false ) {
+               global $wgHiddenPrefs;
                $this->loadOptions();
 
                if ( is_null( $this->mOptions ) ) {
@@ -1965,6 +2150,15 @@ class User {
                        $this->mOptions = User::getDefaultOptions();
                }
 
+               # We want 'disabled' preferences to always behave as the default value for
+               # users, even if they have set the option explicitly in their settings (ie they
+               # set it, and then it was disabled removing their ability to change it).  But
+               # we don't want to erase the preferences in the database in case the preference
+               # is re-enabled again.  So don't touch $mOptions, just override the returned value
+               if( in_array( $oname, $wgHiddenPrefs ) && !$ignoreHidden ){
+                       return self::getDefaultOption( $oname );
+               }
+
                if ( array_key_exists( $oname, $this->mOptions ) ) {
                        return $this->mOptions[$oname];
                } else {
@@ -1978,8 +2172,23 @@ class User {
         * @return array
         */
        public function getOptions() {
+               global $wgHiddenPrefs;
                $this->loadOptions();
-               return $this->mOptions;
+               $options = $this->mOptions;
+
+               # We want 'disabled' preferences to always behave as the default value for
+               # users, even if they have set the option explicitly in their settings (ie they
+               # set it, and then it was disabled removing their ability to change it).  But
+               # we don't want to erase the preferences in the database in case the preference
+               # is re-enabled again.  So don't touch $mOptions, just override the returned value
+               foreach( $wgHiddenPrefs as $pref ){
+                       $default = self::getDefaultOption( $pref );
+                       if( $default !== null ){
+                               $options[$pref] = $default;
+                       }
+               }
+
+               return $options;
        }
 
        /**
@@ -1989,11 +2198,10 @@ class User {
         * @return Bool User's current value for the option
         * @see getOption()
         */
-       function getBoolOption( $oname ) {
+       public function getBoolOption( $oname ) {
                return (bool)$this->getOption( $oname );
        }
 
-
        /**
         * Get the user's current setting for a given option, as a boolean value.
         *
@@ -2002,7 +2210,7 @@ class User {
         * @return Int User's current value for the option
         * @see getOption()
         */
-       function getIntOption( $oname, $defaultOverride=0 ) {
+       public function getIntOption( $oname, $defaultOverride=0 ) {
                $val = $this->getOption( $oname );
                if( $val == '' ) {
                        $val = $defaultOverride;
@@ -2016,15 +2224,10 @@ class User {
         * @param $oname String The option to set
         * @param $val mixed New value to set
         */
-       function setOption( $oname, $val ) {
+       public function setOption( $oname, $val ) {
                $this->load();
                $this->loadOptions();
 
-               if ( $oname == 'skin' ) {
-                       # Clear cached skin, so the new one displays immediately in Special:Preferences
-                       $this->mSkin = null;
-               }
-
                // Explicitly NULL values should refer to defaults
                global $wgDefaultUserOptions;
                if( is_null( $val ) && isset( $wgDefaultUserOptions[$oname] ) ) {
@@ -2037,7 +2240,7 @@ class User {
        /**
         * Reset all options to the site defaults
         */
-       function resetOptions() {
+       public function resetOptions() {
                $this->mOptions = self::getDefaultOptions();
        }
 
@@ -2045,7 +2248,7 @@ class User {
         * Get the user's preferred date format.
         * @return String User's preferred date format
         */
-       function getDatePreference() {
+       public function getDatePreference() {
                // Important migration for old data rows
                if ( is_null( $this->mDatePreference ) ) {
                        global $wgLang;
@@ -2061,8 +2264,10 @@ class User {
 
        /**
         * Get the user preferred stub threshold
+        *
+        * @return int
         */
-       function getStubThreshold() {
+       public function getStubThreshold() {
                global $wgMaxArticleSize; # Maximum article size, in Kb
                $threshold = intval( $this->getOption( 'stubthreshold' ) );
                if ( $threshold > $wgMaxArticleSize * 1024 ) {
@@ -2075,16 +2280,29 @@ class User {
 
        /**
         * Get the permissions this user has.
+        * @param $ns int If numeric, get permissions for this namespace
         * @return Array of String permission names
         */
-       function getRights() {
+       public function getRights( $ns = null ) {
+               $key = is_null( $ns ) ? '*' : intval( $ns );
+
                if ( is_null( $this->mRights ) ) {
-                       $this->mRights = self::getGroupPermissions( $this->getEffectiveGroups() );
-                       wfRunHooks( 'UserGetRights', array( $this, &$this->mRights ) );
+                       $this->mRights = array();
+               }
+
+               if ( !isset( $this->mRights[$key] ) ) {
+                       $this->mRights[$key] = self::getGroupPermissions( $this->getEffectiveGroups(), $ns );
+                       wfRunHooks( 'UserGetRights', array( $this, &$this->mRights[$key], $ns ) );
                        // Force reindexation of rights when a hook has unset one of them
-                       $this->mRights = array_values( $this->mRights );
+                       $this->mRights[$key] = array_values( $this->mRights[$key] );
                }
-               return $this->mRights;
+               if ( is_null( $ns ) ) {
+                       return $this->mRights[$key];
+               } else {
+                       // Merge non namespace specific rights
+                       return array_merge( $this->mRights[$key], $this->getRights() );
+               }
+
        }
 
        /**
@@ -2092,7 +2310,7 @@ class User {
         * The implicit * and user groups are not included.
         * @return Array of String internal group names
         */
-       function getGroups() {
+       public function getGroups() {
                $this->load();
                return $this->mGroups;
        }
@@ -2104,32 +2322,78 @@ class User {
         * @param $recache Bool Whether to avoid the cache
         * @return Array of String internal group names
         */
-       function getEffectiveGroups( $recache = false ) {
+       public function getEffectiveGroups( $recache = false ) {
                if ( $recache || is_null( $this->mEffectiveGroups ) ) {
                        wfProfileIn( __METHOD__ );
-                       $this->mEffectiveGroups = $this->getGroups();
-                       $this->mEffectiveGroups[] = '*';
-                       if( $this->getId() ) {
-                               $this->mEffectiveGroups[] = 'user';
+                       $this->mEffectiveGroups = array_unique( array_merge(
+                               $this->getGroups(), // explicit groups
+                               $this->getAutomaticGroups( $recache ) // implicit groups
+                       ) );
+                       # Hook for additional groups
+                       wfRunHooks( 'UserEffectiveGroups', array( &$this, &$this->mEffectiveGroups ) );
+                       wfProfileOut( __METHOD__ );
+               }
+               return $this->mEffectiveGroups;
+       }
+
+       /**
+        * Get the list of implicit group memberships this user has.
+        * This includes 'user' if logged in, '*' for all accounts,
+        * and autopromoted groups
+        * @param $recache Bool Whether to avoid the cache
+        * @return Array of String internal group names
+        */
+       public function getAutomaticGroups( $recache = false ) {
+               if ( $recache || is_null( $this->mImplicitGroups ) ) {
+                       wfProfileIn( __METHOD__ );
+                       $this->mImplicitGroups = array( '*' );
+                       if ( $this->getId() ) {
+                               $this->mImplicitGroups[] = 'user';
 
-                               $this->mEffectiveGroups = array_unique( array_merge(
-                                       $this->mEffectiveGroups,
+                               $this->mImplicitGroups = array_unique( array_merge(
+                                       $this->mImplicitGroups,
                                        Autopromote::getAutopromoteGroups( $this )
                                ) );
-
-                               # Hook for additional groups
-                               wfRunHooks( 'UserEffectiveGroups', array( &$this, &$this->mEffectiveGroups ) );
+                       }
+                       if ( $recache ) {
+                               # Assure data consistency with rights/groups,
+                               # as getEffectiveGroups() depends on this function
+                               $this->mEffectiveGroups = null;
                        }
                        wfProfileOut( __METHOD__ );
                }
-               return $this->mEffectiveGroups;
+               return $this->mImplicitGroups;
+       }
+
+       /**
+        * Returns the groups the user has belonged to.
+        *
+        * The user may still belong to the returned groups. Compare with getGroups().
+        *
+        * The function will not return groups the user had belonged to before MW 1.17
+        *
+        * @return array Names of the groups the user has belonged to.
+        */
+       public function getFormerGroups() {
+               if( is_null( $this->mFormerGroups ) ) {
+                       $dbr = wfGetDB( DB_MASTER );
+                       $res = $dbr->select( 'user_former_groups',
+                               array( 'ufg_group' ),
+                               array( 'ufg_user' => $this->mId ),
+                               __METHOD__ );
+                       $this->mFormerGroups = array();
+                       foreach( $res as $row ) {
+                               $this->mFormerGroups[] = $row->ufg_group;
+                       }
+               }
+               return $this->mFormerGroups;
        }
 
        /**
         * Get the user's edit count.
         * @return Int
         */
-       function getEditCount() {
+       public function getEditCount() {
                if( $this->getId() ) {
                        if ( !isset( $this->mEditCount ) ) {
                                /* Populate the count, if it has not been populated yet */
@@ -2147,21 +2411,22 @@ class User {
         * This takes immediate effect.
         * @param $group String Name of the group to add
         */
-       function addGroup( $group ) {
-               $dbw = wfGetDB( DB_MASTER );
-               if( $this->getId() ) {
-                       $dbw->insert( 'user_groups',
-                               array(
-                                       'ug_user'  => $this->getID(),
-                                       'ug_group' => $group,
-                               ),
-                               __METHOD__,
-                               array( 'IGNORE' ) );
+       public function addGroup( $group ) {
+               if( wfRunHooks( 'UserAddGroup', array( $this, &$group ) ) ) {
+                       $dbw = wfGetDB( DB_MASTER );
+                       if( $this->getId() ) {
+                               $dbw->insert( 'user_groups',
+                                       array(
+                                               'ug_user'  => $this->getID(),
+                                               'ug_group' => $group,
+                                       ),
+                                       __METHOD__,
+                                       array( 'IGNORE' ) );
+                       }
                }
-
                $this->loadGroups();
                $this->mGroups[] = $group;
-               $this->mRights = User::getGroupPermissions( $this->getEffectiveGroups( true ) );
+               $this->mRights = null;
 
                $this->invalidateCache();
        }
@@ -2171,18 +2436,27 @@ class User {
         * This takes immediate effect.
         * @param $group String Name of the group to remove
         */
-       function removeGroup( $group ) {
+       public function removeGroup( $group ) {
                $this->load();
-               $dbw = wfGetDB( DB_MASTER );
-               $dbw->delete( 'user_groups',
-                       array(
-                               'ug_user'  => $this->getID(),
-                               'ug_group' => $group,
-                       ), __METHOD__ );
-
+               if( wfRunHooks( 'UserRemoveGroup', array( $this, &$group ) ) ) {
+                       $dbw = wfGetDB( DB_MASTER );
+                       $dbw->delete( 'user_groups',
+                               array(
+                                       'ug_user'  => $this->getID(),
+                                       'ug_group' => $group,
+                               ), __METHOD__ );
+                       // Remember that the user was in this group
+                       $dbw->insert( 'user_former_groups',
+                               array(
+                                       'ufg_user'  => $this->getID(),
+                                       'ufg_group' => $group,
+                               ),
+                               __METHOD__,
+                               array( 'IGNORE' ) );
+               }
                $this->loadGroups();
                $this->mGroups = array_diff( $this->mGroups, array( $group ) );
-               $this->mRights = User::getGroupPermissions( $this->getEffectiveGroups( true ) );
+               $this->mRights = null;
 
                $this->invalidateCache();
        }
@@ -2191,7 +2465,7 @@ class User {
         * Get whether the user is logged in
         * @return Bool
         */
-       function isLoggedIn() {
+       public function isLoggedIn() {
                return $this->getID() != 0;
        }
 
@@ -2199,16 +2473,50 @@ class User {
         * Get whether the user is anonymous
         * @return Bool
         */
-       function isAnon() {
+       public function isAnon() {
                return !$this->isLoggedIn();
        }
 
        /**
         * Check if user is allowed to access a feature / make an action
-        * @param $action String action to be checked
-        * @return Boolean: True if action is allowed, else false
+        *
+        * @internal param \String $varargs permissions to test
+        * @return Boolean: True if user is allowed to perform *any* of the given actions
+        *
+        * @return bool
+        */
+       public function isAllowedAny( /*...*/ ){
+               $permissions = func_get_args();
+               foreach( $permissions as $permission ){
+                       if( $this->isAllowed( $permission ) ){
+                               return true;
+                       }
+               }
+               return false;
+       }
+
+       /**
+        *
+        * @internal param $varargs string
+        * @return bool True if the user is allowed to perform *all* of the given actions
         */
-       function isAllowed( $action = '' ) {
+       public function isAllowedAll( /*...*/ ){
+               $permissions = func_get_args();
+               foreach( $permissions as $permission ){
+                       if( !$this->isAllowed( $permission ) ){
+                               return false;
+                       }
+               }
+               return true;
+       }
+
+       /**
+        * Internal mechanics of testing a permission
+        * @param $action String
+        * @param $ns int|null Namespace optional
+        * @return bool
+        */
+       public function isAllowed( $action = '', $ns = null ) {
                if ( $action === '' ) {
                        return true; // In the spirit of DWIM
                }
@@ -2220,7 +2528,7 @@ class User {
                }
                # Use strict parameter to avoid matching numeric 0 accidentally inserted
                # by misconfiguration: 0 == 'foo'
-               return in_array( $action, $this->getRights(), true );
+               return in_array( $action, $this->getRights( $ns ), true );
        }
 
        /**
@@ -2229,7 +2537,7 @@ class User {
         */
        public function useRCPatrol() {
                global $wgUseRCPatrol;
-               return( $wgUseRCPatrol && ( $this->isAllowed( 'patrol' ) || $this->isAllowed( 'patrolmarks' ) ) );
+               return $wgUseRCPatrol && $this->isAllowedAny( 'patrol', 'patrolmarks' );
        }
 
        /**
@@ -2238,50 +2546,31 @@ class User {
         */
        public function useNPPatrol() {
                global $wgUseRCPatrol, $wgUseNPPatrol;
-               return( ( $wgUseRCPatrol || $wgUseNPPatrol ) && ( $this->isAllowed( 'patrol' ) || $this->isAllowed( 'patrolmarks' ) ) );
+               return( ( $wgUseRCPatrol || $wgUseNPPatrol ) && ( $this->isAllowedAny( 'patrol', 'patrolmarks' ) ) );
        }
 
        /**
-        * Get the current skin, loading it if required, and setting a title
-        * @param $t Title: the title to use in the skin
-        * @return Skin The current skin
-        * @todo: FIXME : need to check the old failback system [AV]
-        */
-       function getSkin( $t = null ) {
-               if( !$this->mSkin ) {
-                       global $wgOut;
-                       $this->mSkin = $this->createSkinObject();
-                       $this->mSkin->setTitle( $wgOut->getTitle() );
-               }
-               if ( $t && ( !$this->mSkin->getTitle() || !$t->equals( $this->mSkin->getTitle() ) ) ) {
-                       $skin = $this->createSkinObject();
-                       $skin->setTitle( $t );
-                       return $skin;
+        * Get the WebRequest object to use with this object
+        *
+        * @return WebRequest
+        */
+       public function getRequest() {
+               if ( $this->mRequest ) {
+                       return $this->mRequest;
                } else {
-                       return $this->mSkin;
-               }
-       }
-
-       // Creates a Skin object, for getSkin()
-       private function createSkinObject() {
-               wfProfileIn( __METHOD__ );
-
-               global $wgHiddenPrefs;
-               if( !in_array( 'skin', $wgHiddenPrefs ) ) {
                        global $wgRequest;
-                       # get the user skin
-                       $userSkin = $this->getOption( 'skin' );
-                       $userSkin = $wgRequest->getVal( 'useskin', $userSkin );
-               } else {
-                       # if we're not allowing users to override, then use the default
-                       global $wgDefaultSkin;
-                       $userSkin = $wgDefaultSkin;
+                       return $wgRequest;
                }
+       }
 
-               $skin = Skin::newFromKey( $userSkin );
-               wfProfileOut( __METHOD__ );
-
-               return $skin;
+       /**
+        * Get the current skin, loading it if required
+        * @return Skin The current skin
+        * @todo FIXME: Need to check the old failback system [AV]
+        * @deprecated since 1.18 Use ->getSkin() in the most relevant outputting context you have
+        */
+       public function getSkin() {
+               return RequestContext::getMain()->getSkin();
        }
 
        /**
@@ -2289,7 +2578,7 @@ class User {
         * @param $title Title of the article to look at
         * @return Bool
         */
-       function isWatched( $title ) {
+       public function isWatched( $title ) {
                $wl = WatchedItem::fromUserTitle( $this, $title );
                return $wl->isWatched();
        }
@@ -2298,7 +2587,7 @@ class User {
         * Watch an article.
         * @param $title Title of the article to look at
         */
-       function addWatch( $title ) {
+       public function addWatch( $title ) {
                $wl = WatchedItem::fromUserTitle( $this, $title );
                $wl->addWatch();
                $this->invalidateCache();
@@ -2308,7 +2597,7 @@ class User {
         * Stop watching an article.
         * @param $title Title of the article to look at
         */
-       function removeWatch( $title ) {
+       public function removeWatch( $title ) {
                $wl = WatchedItem::fromUserTitle( $this, $title );
                $wl->removeWatch();
                $this->invalidateCache();
@@ -2320,8 +2609,8 @@ class User {
         * the next change of the page if it's watched etc.
         * @param $title Title of the article to look at
         */
-       function clearNotification( &$title ) {
-               global $wgUser, $wgUseEnotif, $wgShowUpdatedMarker;
+       public function clearNotification( &$title ) {
+               global $wgUseEnotif, $wgShowUpdatedMarker;
 
                # Do nothing if the database is locked to writes
                if( wfReadOnly() ) {
@@ -2349,13 +2638,11 @@ class User {
                // and when it does have to be executed, it can be on a slave
                // If this is the user's newtalk page, we always update the timestamp
                if( $title->getNamespace() == NS_USER_TALK &&
-                       $title->getText() == $wgUser->getName() )
+                       $title->getText() == $this->getName() )
                {
                        $watched = true;
-               } elseif ( $this->getId() == $wgUser->getId() ) {
-                       $watched = $title->userIsWatching();
                } else {
-                       $watched = true;
+                       $watched = $this->isWatched( $title );
                }
 
                // If the page is watched by the user (or may be watched), update the timestamp on any
@@ -2378,22 +2665,21 @@ class User {
         * Resets all of the given user's page-change notification timestamps.
         * If e-notif e-mails are on, they will receive notification mails on
         * the next change of any watched page.
-        *
-        * @param $currentUser Int User ID
         */
-       function clearAllNotifications( $currentUser ) {
+       public function clearAllNotifications() {
                global $wgUseEnotif, $wgShowUpdatedMarker;
                if ( !$wgUseEnotif && !$wgShowUpdatedMarker ) {
                        $this->setNewtalk( false );
                        return;
                }
-               if( $currentUser != 0 )  {
+               $id = $this->getId();
+               if( $id != 0 )  {
                        $dbw = wfGetDB( DB_MASTER );
                        $dbw->update( 'watchlist',
                                array( /* SET */
                                        'wl_notificationtimestamp' => null
                                ), array( /* WHERE */
-                                       'wl_user' => $currentUser
+                                       'wl_user' => $id
                                ), __METHOD__
                        );
                #       We also need to clear here the "you have new message" notification for the own user_talk page
@@ -2404,9 +2690,8 @@ class User {
        /**
         * Set this user's options from an encoded string
         * @param $str String Encoded options to import
-        * @private
         */
-       function decodeOptions( $str ) {
+       public function decodeOptions( $str ) {
                if( !$str )
                        return;
 
@@ -2435,8 +2720,7 @@ class User {
         *                   if 0 or not specified, use the default $wgCookieExpiration
         */
        protected function setCookie( $name, $value, $exp = 0 ) {
-               global $wgRequest;
-               $wgRequest->response()->setcookie( $name, $value, $exp );
+               $this->getRequest()->response()->setcookie( $name, $value, $exp );
        }
 
        /**
@@ -2449,9 +2733,14 @@ class User {
 
        /**
         * Set the default cookies for this session on the user's client.
+        *
+        * @param $request WebRequest object to use; $wgRequest will be used if null
+        *        is passed.
         */
-       function setCookies() {
-               global $wgRequest;
+       public function setCookies( $request = null ) {
+               if ( $request === null ) {
+                       $request = $this->getRequest();
+               }
 
                $this->load();
                if ( 0 == $this->mId ) return;
@@ -2473,7 +2762,7 @@ class User {
                wfRunHooks( 'UserSetCookies', array( $this, &$session, &$cookies ) );
 
                foreach ( $session as $name => $value ) {
-                       $wgRequest->setSessionData( $name, $value );
+                       $request->setSessionData( $name, $value );
                }
                foreach ( $cookies as $name => $value ) {
                        if ( $value === false ) {
@@ -2487,7 +2776,7 @@ class User {
        /**
         * Log this user out.
         */
-       function logout() {
+       public function logout() {
                if( wfRunHooks( 'UserLogout', array( &$this ) ) ) {
                        $this->doLogout();
                }
@@ -2495,15 +2784,12 @@ class User {
 
        /**
         * Clear the user's cookies and session, and reset the instance cache.
-        * @private
         * @see logout()
         */
-       function doLogout() {
-               global $wgRequest;
-
+       public function doLogout() {
                $this->clearInstanceCache( 'defaults' );
 
-               $wgRequest->setSessionData( 'wsUserID', 0 );
+               $this->getRequest()->setSessionData( 'wsUserID', 0 );
 
                $this->clearCookie( 'UserID' );
                $this->clearCookie( 'Token' );
@@ -2516,7 +2802,7 @@ class User {
         * Save this user's settings into the database.
         * @todo Only rarely do all these fields need to be set!
         */
-       function saveSettings() {
+       public function saveSettings() {
                $this->load();
                if ( wfReadOnly() ) { return; }
                if ( 0 == $this->mId ) { return; }
@@ -2554,7 +2840,7 @@ class User {
         * If only this user's username is known, and it exists, return the user ID.
         * @return Int
         */
-       function idForName() {
+       public function idForName() {
                $s = trim( $this->getName() );
                if ( $s === '' ) return 0;
 
@@ -2570,9 +2856,9 @@ class User {
         * Add a user to the database, return the user object
         *
         * @param $name String Username to add
-        * @param $params Array of Strings Non-default parameters to save to the database:
-        *   - password             The user's password. Password logins will be disabled if this is omitted.
-        *   - newpassword          A temporary password mailed to the user
+        * @param $params Array of Strings Non-default parameters to save to the database as user_* fields:
+        *   - password             The user's password hash. Password logins will be disabled if this is omitted.
+        *   - newpassword          Hash for a temporary password that has been mailed to the user
         *   - email                The user's email address
         *   - email_authenticated  The email authentication timestamp
         *   - real_name            The user's real name
@@ -2582,7 +2868,7 @@ class User {
         *
         * @return User object, or null if the username already exists
         */
-       static function createNew( $name, $params = array() ) {
+       public static function createNew( $name, $params = array() ) {
                $user = new User;
                $user->load();
                if ( isset( $params['options'] ) ) {
@@ -2621,7 +2907,7 @@ class User {
        /**
         * Add this existing user object to the database
         */
-       function addToDatabase() {
+       public function addToDatabase() {
                $this->load();
                $dbw = wfGetDB( DB_MASTER );
                $seqVal = $dbw->nextSequenceValue( 'user_user_id_seq' );
@@ -2653,19 +2939,19 @@ class User {
         * If this (non-anonymous) user is blocked, block any IP address
         * they've successfully logged in from.
         */
-       function spreadBlock() {
+       public function spreadBlock() {
                wfDebug( __METHOD__ . "()\n" );
                $this->load();
                if ( $this->mId == 0 ) {
                        return;
                }
 
-               $userblock = Block::newFromDB( '', $this->mId );
+               $userblock = Block::newFromTarget( $this->getName() );
                if ( !$userblock ) {
                        return;
                }
 
-               $userblock->doAutoblock( wfGetIP() );
+               $userblock->doAutoblock( $this->getRequest()->getIP() );
        }
 
        /**
@@ -2679,10 +2965,10 @@ class User {
         * which will give them a chance to modify this key based on their own
         * settings.
         *
-        * @deprecated @since 1.17 use the ParserOptions object to get the relevant options
+        * @deprecated since 1.17 use the ParserOptions object to get the relevant options
         * @return String Page rendering hash
         */
-       function getPageRenderingHash() {
+       public function getPageRenderingHash() {
                global $wgUseDynamicDates, $wgRenderHashAppend, $wgLang, $wgContLang;
                if( $this->mHash ){
                        return $this->mHash;
@@ -2722,20 +3008,32 @@ class User {
 
        /**
         * Get whether the user is explicitly blocked from account creation.
-        * @return Bool
+        * @return Bool|Block
         */
-       function isBlockedFromCreateAccount() {
+       public function isBlockedFromCreateAccount() {
                $this->getBlockedStatus();
-               return $this->mBlock && $this->mBlock->mCreateAccount;
+               if( $this->mBlock && $this->mBlock->prevents( 'createaccount' ) ){
+                       return $this->mBlock;
+               }
+
+               # bug 13611: if the IP address the user is trying to create an account from is
+               # blocked with createaccount disabled, prevent new account creation there even
+               # when the user is logged in
+               if( $this->mBlockedFromCreateAccount === false ){
+                       $this->mBlockedFromCreateAccount = Block::newFromTarget( null, $this->getRequest()->getIP() );
+               }
+               return $this->mBlockedFromCreateAccount instanceof Block && $this->mBlockedFromCreateAccount->prevents( 'createaccount' )
+                       ? $this->mBlockedFromCreateAccount
+                       : false;
        }
 
        /**
         * Get whether the user is blocked from using Special:Emailuser.
         * @return Bool
         */
-       function isBlockedFromEmailuser() {
+       public function isBlockedFromEmailuser() {
                $this->getBlockedStatus();
-               return $this->mBlock && $this->mBlock->mBlockEmail;
+               return $this->mBlock && $this->mBlock->prevents( 'sendemail' );
        }
 
        /**
@@ -2751,7 +3049,7 @@ class User {
         *
         * @return Title: User's personal page title
         */
-       function getUserPage() {
+       public function getUserPage() {
                return Title::makeTitle( NS_USER, $this->getName() );
        }
 
@@ -2760,33 +3058,17 @@ class User {
         *
         * @return Title: User's talk page title
         */
-       function getTalkPage() {
+       public function getTalkPage() {
                $title = $this->getUserPage();
                return $title->getTalkPage();
        }
 
-       /**
-        * Get the maximum valid user ID.
-        * @return Integer: User ID
-        * @static
-        */
-       function getMaxID() {
-               static $res; // cache
-
-               if ( isset( $res ) ) {
-                       return $res;
-               } else {
-                       $dbr = wfGetDB( DB_SLAVE );
-                       return $res = $dbr->selectField( 'user', 'max(user_id)', false, __METHOD__ );
-               }
-       }
-
        /**
         * Determine whether the user is a newbie. Newbies are either
         * anonymous IPs, or the most recently created accounts.
         * @return Bool
         */
-       function isNewbie() {
+       public function isNewbie() {
                return !$this->isAllowed( 'autoconfirmed' );
        }
 
@@ -2795,8 +3077,8 @@ class User {
         * @param $password String: user password.
         * @return Boolean: True if the given password is correct, otherwise False.
         */
-       function checkPassword( $password ) {
-               global $wgAuth;
+       public function checkPassword( $password ) {
+               global $wgAuth, $wgLegacyEncoding;
                $this->load();
 
                // Even though we stop people from creating passwords that
@@ -2819,11 +3101,13 @@ class User {
                }
                if ( self::comparePasswords( $this->mPassword, $password, $this->mId ) ) {
                        return true;
-               } elseif ( function_exists( 'iconv' ) ) {
+               } elseif ( $wgLegacyEncoding ) {
                        # Some wikis were converted from ISO 8859-1 to UTF-8, the passwords can't be converted
                        # Check for this with iconv
                        $cp1252Password = iconv( 'UTF-8', 'WINDOWS-1252//TRANSLIT', $password );
-                       if ( self::comparePasswords( $this->mPassword, $cp1252Password, $this->mId ) ) {
+                       if ( $cp1252Password != $password &&
+                               self::comparePasswords( $this->mPassword, $cp1252Password, $this->mId ) )
+                       {
                                return true;
                        }
                }
@@ -2833,10 +3117,15 @@ class User {
        /**
         * Check if the given clear-text password matches the temporary password
         * sent by e-mail for password reset operations.
+        *
+        * @param $plaintext string
+        *
         * @return Boolean: True if matches, false otherwise
         */
-       function checkTemporaryPassword( $plaintext ) {
+       public function checkTemporaryPassword( $plaintext ) {
                global $wgNewPasswordExpiry;
+
+               $this->load();
                if( self::comparePasswords( $this->mNewpassword, $plaintext, $this->getId() ) ) {
                        if ( is_null( $this->mNewpassTime ) ) {
                                return true;
@@ -2855,18 +3144,21 @@ class User {
         * submission.
         *
         * @param $salt String|Array of Strings Optional function-specific data for hashing
+        * @param $request WebRequest object to use or null to use $wgRequest
         * @return String The new edit token
         */
-       function editToken( $salt = '' ) {
-               global $wgRequest;
+       public function editToken( $salt = '', $request = null ) {
+               if ( $request == null ) {
+                       $request = $this->getRequest();
+               }
 
                if ( $this->isAnon() ) {
                        return EDIT_TOKEN_SUFFIX;
                } else {
-                       $token = $wgRequest->getSessionData( 'wsEditToken' );
+                       $token = $request->getSessionData( 'wsEditToken' );
                        if ( $token === null ) {
                                $token = self::generateToken();
-                               $wgRequest->setSessionData( 'wsEditToken', $token );
+                               $request->setSessionData( 'wsEditToken', $token );
                        }
                        if( is_array( $salt ) ) {
                                $salt = implode( '|', $salt );
@@ -2894,10 +3186,11 @@ class User {
         *
         * @param $val String Input value to compare
         * @param $salt String Optional function-specific data for hashing
+        * @param $request WebRequest object to use or null to use $wgRequest
         * @return Boolean: Whether the token matches
         */
-       function matchEditToken( $val, $salt = '' ) {
-               $sessionToken = $this->editToken( $salt );
+       public function matchEditToken( $val, $salt = '', $request = null ) {
+               $sessionToken = $this->editToken( $salt, $request );
                if ( $val != $sessionToken ) {
                        wfDebug( "User::matchEditToken: broken session data\n" );
                }
@@ -2910,10 +3203,11 @@ class User {
         *
         * @param $val String Input value to compare
         * @param $salt String Optional function-specific data for hashing
+        * @param $request WebRequest object to use or null to use $wgRequest
         * @return Boolean: Whether the token matches
         */
-       function matchEditTokenNoSuffix( $val, $salt = '' ) {
-               $sessionToken = $this->editToken( $salt );
+       public function matchEditTokenNoSuffix( $val, $salt = '', $request = null ) {
+               $sessionToken = $this->editToken( $salt, $request );
                return substr( $sessionToken, 0, 32 ) == substr( $val, 0, 32 );
        }
 
@@ -2924,7 +3218,7 @@ class User {
         * @param $type String: message to send, either "created", "changed" or "set"
         * @return Status object
         */
-       function sendConfirmationMail( $type = 'created' ) {
+       public function sendConfirmationMail( $type = 'created' ) {
                global $wgLang;
                $expiration = null; // gets passed-by-ref and defined in next line.
                $token = $this->confirmationToken( $expiration );
@@ -2942,7 +3236,7 @@ class User {
 
                return $this->sendMail( wfMsg( 'confirmemail_subject' ),
                        wfMsg( $message,
-                               wfGetIP(),
+                               $this->getRequest()->getIP(),
                                $this->getName(),
                                $url,
                                $wgLang->timeanddate( $expiration, false ),
@@ -2961,7 +3255,7 @@ class User {
         * @param $replyto String Reply-To address
         * @return Status
         */
-       function sendMail( $subject, $body, $from = null, $replyto = null ) {
+       public function sendMail( $subject, $body, $from = null, $replyto = null ) {
                if( is_null( $from ) ) {
                        global $wgPasswordSender, $wgPasswordSenderName;
                        $sender = new MailAddress( $wgPasswordSender, $wgPasswordSenderName );
@@ -2980,11 +3274,10 @@ class User {
         * @note Call saveSettings() after calling this function to commit
         * this change to the database.
         *
-        * @param[out] &$expiration \mixed Accepts the expiration time
+        * @param &$expiration \mixed Accepts the expiration time
         * @return String New token
-        * @private
         */
-       function confirmationToken( &$expiration ) {
+       private function confirmationToken( &$expiration ) {
                global $wgUserEmailConfirmationTokenExpiry;
                $now = time();
                $expires = $now + $wgUserEmailConfirmationTokenExpiry;
@@ -3001,9 +3294,8 @@ class User {
        * Return a URL the user can use to confirm their email address.
         * @param $token String Accepts the email confirmation token
         * @return String New token URL
-        * @private
         */
-       function confirmationTokenUrl( $token ) {
+       private function confirmationTokenUrl( $token ) {
                return $this->getTokenUrl( 'ConfirmEmail', $token );
        }
 
@@ -3011,15 +3303,14 @@ class User {
         * Return a URL the user can use to invalidate their email address.
         * @param $token String Accepts the email confirmation token
         * @return String New token URL
-        * @private
         */
-       function invalidationTokenUrl( $token ) {
+       private function invalidationTokenUrl( $token ) {
                return $this->getTokenUrl( 'Invalidateemail', $token );
        }
 
        /**
         * Internal function to format the e-mail validation/invalidation URLs.
-        * This uses $wgArticlePath directly as a quickie hack to use the
+        * This uses a quickie hack to use the
         * hardcoded English names of the Special: pages, for ASCII safety.
         *
         * @note Since these URLs get dropped directly into emails, using the
@@ -3032,20 +3323,19 @@ class User {
         * @return String Formatted URL
         */
        protected function getTokenUrl( $page, $token ) {
-               global $wgArticlePath;
-               return wfExpandUrl(
-                       str_replace(
-                               '$1',
-                               "Special:$page/$token",
-                               $wgArticlePath ) );
+               // Hack to bypass localization of 'Special:'
+               $title = Title::makeTitle( NS_MAIN, "Special:$page/$token" );
+               return $title->getCanonicalUrl();
        }
 
        /**
         * Mark the e-mail address confirmed.
         *
         * @note Call saveSettings() after calling this function to commit the change.
+        *
+        * @return true
         */
-       function confirmEmail() {
+       public function confirmEmail() {
                $this->setEmailAuthenticationTimestamp( wfTimestampNow() );
                wfRunHooks( 'ConfirmEmailComplete', array( $this ) );
                return true;
@@ -3056,6 +3346,7 @@ class User {
         * address if it was already confirmed.
         *
         * @note Call saveSettings() after calling this function to commit the change.
+        * @return true
         */
        function invalidateEmail() {
                $this->load();
@@ -3081,7 +3372,7 @@ class User {
         * site configuration?
         * @return Bool
         */
-       function canSendEmail() {
+       public function canSendEmail() {
                global $wgEnableEmail, $wgEnableUserEmail;
                if( !$wgEnableEmail || !$wgEnableUserEmail || !$this->isAllowed( 'sendemail' ) ) {
                        return false;
@@ -3096,7 +3387,7 @@ class User {
         * site configuration?
         * @return Bool
         */
-       function canReceiveEmail() {
+       public function canReceiveEmail() {
                return $this->isEmailConfirmed() && !$this->getOption( 'disablemail' );
        }
 
@@ -3110,17 +3401,20 @@ class User {
         *
         * @return Bool
         */
-       function isEmailConfirmed() {
+       public function isEmailConfirmed() {
                global $wgEmailAuthentication;
                $this->load();
                $confirmed = true;
                if( wfRunHooks( 'EmailConfirmed', array( &$this, &$confirmed ) ) ) {
-                       if( $this->isAnon() )
+                       if( $this->isAnon() ) {
                                return false;
-                       if( !self::isValidEmailAddr( $this->mEmail ) )
+                       }
+                       if( !Sanitizer::validateEmail( $this->mEmail ) ) {
                                return false;
-                       if( $wgEmailAuthentication && !$this->getEmailAuthenticationTimestamp() )
+                       }
+                       if( $wgEmailAuthentication && !$this->getEmailAuthenticationTimestamp() ) {
                                return false;
+                       }
                        return true;
                } else {
                        return $confirmed;
@@ -3131,7 +3425,7 @@ class User {
         * Check whether there is an outstanding request for e-mail confirmation.
         * @return Bool
         */
-       function isEmailConfirmationPending() {
+       public function isEmailConfirmationPending() {
                global $wgEmailAuthentication;
                return $wgEmailAuthentication &&
                        !$this->isEmailConfirmed() &&
@@ -3146,9 +3440,11 @@ class User {
         *     non-existent/anonymous user accounts.
         */
        public function getRegistration() {
-               return $this->getId() > 0
-                       ? $this->mRegistration
-                       : false;
+               if ( $this->isAnon() ) {
+                       return false;
+               }
+               $this->load();
+               return $this->mRegistration;
        }
 
        /**
@@ -3177,40 +3473,70 @@ class User {
         * Get the permissions associated with a given list of groups
         *
         * @param $groups Array of Strings List of internal group names
+        * @param $ns int
+        *
         * @return Array of Strings List of permission key names for given groups combined
         */
-       static function getGroupPermissions( $groups ) {
+       public static function getGroupPermissions( array $groups, $ns = null ) {
                global $wgGroupPermissions, $wgRevokePermissions;
                $rights = array();
-               // grant every granted permission first
+
+               // Grant every granted permission first
                foreach( $groups as $group ) {
                        if( isset( $wgGroupPermissions[$group] ) ) {
-                               $rights = array_merge( $rights,
-                                       // array_filter removes empty items
-                                       array_keys( array_filter( $wgGroupPermissions[$group] ) ) );
+                               $rights = array_merge( $rights, self::extractRights(
+                                       $wgGroupPermissions[$group], $ns ) );
                        }
                }
-               // now revoke the revoked permissions
+
+               // Revoke the revoked permissions
                foreach( $groups as $group ) {
                        if( isset( $wgRevokePermissions[$group] ) ) {
-                               $rights = array_diff( $rights,
-                                       array_keys( array_filter( $wgRevokePermissions[$group] ) ) );
+                               $rights = array_diff( $rights, self::extractRights(
+                                       $wgRevokePermissions[$group], $ns ) );
                        }
                }
                return array_unique( $rights );
        }
 
+       /**
+        * Helper for User::getGroupPermissions
+        * @param $list array
+        * @param $ns int
+        * @return array
+        */
+       private static function extractRights( $list, $ns ) {
+               $rights = array();
+               foreach( $list as $right => $value ) {
+                       if ( is_array( $value ) ) {
+                               # This is a list of namespaces where the permission applies
+                               if ( !is_null( $ns ) && !empty( $value[$ns] ) ) {
+                                       $rights[] = $right;
+                               }
+                       } else {
+                               # This is a boolean indicating that the permission applies
+                               if ( $value ) {
+                                       $rights[] = $right;
+                               }
+                       }
+               }
+               return $rights;
+       }
+
        /**
         * Get all the groups who have a given permission
         *
         * @param $role String Role to check
+        * @param $ns int
+        *
+        *
         * @return Array of Strings List of internal group names with the given permission
         */
-       static function getGroupsWithPermission( $role ) {
+       public static function getGroupsWithPermission( $role, $ns = null ) {
                global $wgGroupPermissions;
                $allowedGroups = array();
                foreach ( $wgGroupPermissions as $group => $rights ) {
-                       if ( isset( $rights[$role] ) && $rights[$role] ) {
+                       if ( in_array( $role, self::getGroupPermissions( array( $group ), $ns ), true ) ) {
                                $allowedGroups[] = $group;
                        }
                }
@@ -3223,7 +3549,7 @@ class User {
         * @param $group String Internal group name
         * @return String Localized descriptive group name
         */
-       static function getGroupName( $group ) {
+       public static function getGroupName( $group ) {
                $msg = wfMessage( "group-$group" );
                return $msg->isBlank() ? $group : $msg->text();
        }
@@ -3234,7 +3560,7 @@ class User {
         * @param $group String Internal group name
         * @return String Localized name for group member
         */
-       static function getGroupMember( $group ) {
+       public static function getGroupMember( $group ) {
                $msg = wfMessage( "group-$group-member" );
                return $msg->isBlank() ? $group : $msg->text();
        }
@@ -3245,7 +3571,7 @@ class User {
         * are not included, as they are defined automatically, not in the database.
         * @return Array of internal group names
         */
-       static function getAllGroups() {
+       public static function getAllGroups() {
                global $wgGroupPermissions, $wgRevokePermissions;
                return array_diff(
                        array_merge( array_keys( $wgGroupPermissions ), array_keys( $wgRevokePermissions ) ),
@@ -3257,7 +3583,7 @@ class User {
         * Get a list of all available permissions.
         * @return Array of permission names
         */
-       static function getAllRights() {
+       public static function getAllRights() {
                if ( self::$mAllRights === false ) {
                        global $wgAvailableRights;
                        if ( count( $wgAvailableRights ) ) {
@@ -3287,7 +3613,7 @@ class User {
         * @param $group String Internal group name
         * @return Title|Bool Title of the page if it exists, false otherwise
         */
-       static function getGroupPage( $group ) {
+       public static function getGroupPage( $group ) {
                $msg = wfMessage( 'grouppage-' . $group )->inContentLanguage();
                if( $msg->exists() ) {
                        $title = Title::newFromText( $msg->text() );
@@ -3305,15 +3631,13 @@ class User {
         * @param $text String The text of the link
         * @return String HTML link to the group
         */
-       static function makeGroupLinkHTML( $group, $text = '' ) {
+       public static function makeGroupLinkHTML( $group, $text = '' ) {
                if( $text == '' ) {
                        $text = self::getGroupName( $group );
                }
                $title = self::getGroupPage( $group );
                if( $title ) {
-                       global $wgUser;
-                       $sk = $wgUser->getSkin();
-                       return $sk->link( $title, htmlspecialchars( $text ) );
+                       return Linker::link( $title, htmlspecialchars( $text ) );
                } else {
                        return $text;
                }
@@ -3327,7 +3651,7 @@ class User {
         * @param $text String The text of the link
         * @return String Wikilink to the group
         */
-       static function makeGroupLinkWiki( $group, $text = '' ) {
+       public static function makeGroupLinkWiki( $group, $text = '' ) {
                if( $text == '' ) {
                        $text = self::getGroupName( $group );
                }
@@ -3349,7 +3673,7 @@ class User {
         *     'add-self' => array( addablegroups to self),
         *     'remove-self' => array( removable groups from self) )
         */
-       static function changeableByGroup( $group ) {
+       public static function changeableByGroup( $group ) {
                global $wgAddGroups, $wgRemoveGroups, $wgGroupsAddToSelf, $wgGroupsRemoveFromSelf;
 
                $groups = array( 'add' => array(), 'remove' => array(), 'add-self' => array(), 'remove-self' => array() );
@@ -3413,7 +3737,7 @@ class User {
         *  'add-self' => array( addablegroups to self),
         *  'remove-self' => array( removable groups from self) )
         */
-       function changeableGroups() {
+       public function changeableGroups() {
                if( $this->isAllowed( 'userrights' ) ) {
                        // This group gives the right to modify everything (reverse-
                        // compatibility with old "userrights lets you change
@@ -3453,7 +3777,7 @@ class User {
         * Increment the user's edit-count field.
         * Will have no effect for anonymous users.
         */
-       function incEditCount() {
+       public function incEditCount() {
                if( !$this->isAnon() ) {
                        $dbw = wfGetDB( DB_MASTER );
                        $dbw->update( 'user',
@@ -3499,12 +3823,10 @@ class User {
         * @param $right String Right to query
         * @return String Localized description of the right
         */
-       static function getRightDescription( $right ) {
+       public static function getRightDescription( $right ) {
                $key = "right-$right";
-               $name = wfMsg( $key );
-               return $name == '' || wfEmptyMsg( $key, $name )
-                       ? $right
-                       : $name;
+               $msg = wfMessage( $key );
+               return $msg->isBlank() ? $right : $msg->text();
        }
 
        /**
@@ -3514,7 +3836,7 @@ class User {
         * @param $userId String User ID
         * @return String Password hash
         */
-       static function oldCrypt( $password, $userId ) {
+       public static function oldCrypt( $password, $userId ) {
                global $wgPasswordSalt;
                if ( $wgPasswordSalt ) {
                        return md5( $userId . '-' . md5( $password ) );
@@ -3527,11 +3849,12 @@ class User {
         * Make a new-style password hash
         *
         * @param $password String Plain-text password
-        * @param $salt String Optional salt, may be random or the user ID.
+        * @param bool|string $salt Optional salt, may be random or the user ID.
+
         *                     If unspecified or false, will generate one automatically
         * @return String Password hash
         */
-       static function crypt( $password, $salt = false ) {
+       public static function crypt( $password, $salt = false ) {
                global $wgPasswordSalt;
 
                $hash = '';
@@ -3555,10 +3878,11 @@ class User {
         *
         * @param $hash String Password hash
         * @param $password String Plain-text password to compare
-        * @param $userId String User ID for old-style password salt
-        * @return Boolean:
+        * @param $userId String|bool User ID for old-style password salt
+        *
+        * @return Boolean
         */
-       static function comparePasswords( $hash, $password, $userId = false ) {
+       public static function comparePasswords( $hash, $password, $userId = false ) {
                $type = substr( $hash, 0, 3 );
 
                $result = false;
@@ -3584,6 +3908,8 @@ class User {
         *
         * @param $byEmail Boolean: account made by email?
         * @param $reason String: user supplied reason
+        *
+        * @return true
         */
        public function addNewUserLogEntry( $byEmail = false, $reason = '' ) {
                global $wgUser, $wgContLang, $wgNewUserLog;
@@ -3617,10 +3943,12 @@ class User {
        /**
         * Add an autocreate newuser log entry for this user
         * Used by things like CentralAuth and perhaps other authplugins.
+        *
+        * @return true
         */
        public function addNewUserLogEntryAutoCreate() {
-               global $wgNewUserLog, $wgLogAutocreatedAccounts;
-               if( !$wgNewUserLog || !$wgLogAutocreatedAccounts ) {
+               global $wgNewUserLog;
+               if( !$wgNewUserLog ) {
                        return true; // disabled
                }
                $log = new LogPage( 'newusers', false );
@@ -3628,6 +3956,9 @@ class User {
                return true;
        }
 
+       /**
+        * @todo document
+        */
        protected function loadOptions() {
                $this->load();
                if ( $this->mOptionsLoaded || !$this->getId() )
@@ -3664,6 +3995,9 @@ class User {
                wfRunHooks( 'UserLoadOptions', array( $this, &$this->mOptions ) );
        }
 
+       /**
+        * @todo document
+        */
        protected function saveOptions() {
                global $wgAllowPrefChange;
 
@@ -3678,8 +4012,9 @@ class User {
 
                // Allow hooks to abort, for instance to save to a global profile.
                // Reset options to default state before saving.
-               if( !wfRunHooks( 'UserSaveOptions', array( $this, &$saveOptions ) ) )
+               if( !wfRunHooks( 'UserSaveOptions', array( $this, &$saveOptions ) ) ) {
                        return;
+               }
 
                foreach( $saveOptions as $key => $value ) {
                        # Don't bother storing default values
@@ -3727,7 +4062,7 @@ class User {
         * actually just returns array() unconditionally at the moment.  May as
         * well keep it around for when the browser bugs get fixed, though.
         *
-        * FIXME : This does not belong here; put it in Html or Linker or somewhere
+        * @todo FIXME: This does not belong here; put it in Html or Linker or somewhere
         *
         * @return array Array of HTML attributes suitable for feeding to
         *   Html::element(), directly or indirectly.  (Don't feed to Xml::*()!
@@ -3744,7 +4079,7 @@ class User {
                # Note that the pattern requirement will always be satisfied if the
                # input is empty, so we need required in all cases.
                #
-               # FIXME (bug 23769): This needs to not claim the password is required
+               # @todo FIXME: Bug 23769: This needs to not claim the password is required
                # if e-mail confirmation is being used.  Since HTML5 input validation
                # is b0rked anyway in some browsers, just return nothing.  When it's
                # re-enabled, fix this code to not output required for e-mail
@@ -3768,8 +4103,4 @@ class User {
 
                return $ret;
        }
-
-
-
-
 }