Localisation updates for core messages from translatewiki.net (2009-06-15 07:42 UTC)
[lhc/web/wiklou.git] / includes / User.php
index bc4b131..c227d29 100644 (file)
@@ -5,19 +5,19 @@
  */
 
 /**
- * \type{\int} Number of characters in user_token field.
+ * \int Number of characters in user_token field.
  * @ingroup Constants
  */
 define( 'USER_TOKEN_LENGTH', 32 );
 
 /**
- * \type{\int} Serialized record version.
+ * \int Serialized record version.
  * @ingroup Constants
  */
-define( 'MW_USER_VERSION', 6 );
+define( 'MW_USER_VERSION', 8 );
 
 /**
- * \type{\string} Some punctuation to prevent editing from broken text-mangling proxies.
+ * \string Some punctuation to prevent editing from broken text-mangling proxies.
  * @ingroup Constants
  */
 define( 'EDIT_TOKEN_SUFFIX', '+\\' );
@@ -91,6 +91,7 @@ class User {
                'diffonly',
                'showhiddencats',
                'noconvertlink',
+               'norollbackdiff',
        );
 
        /**
@@ -108,7 +109,6 @@ class User {
                'mNewpassword',
                'mNewpassTime',
                'mEmail',
-               'mOptions',
                'mTouched',
                'mToken',
                'mEmailAuthenticated',
@@ -118,6 +118,8 @@ class User {
                'mEditCount',
                // user_group table
                'mGroups',
+               // user_properties table
+               'mOptionOverrides',
        );
 
        /**
@@ -140,17 +142,23 @@ class User {
                'createtalk',
                'delete',
                'deletedhistory',
+               'deleterevision',
                'edit',
                'editinterface',
                'editusercssjs',
+               'hideuser',
                'import',
                'importupload',
                'ipblock-exempt',
                'markbotedits',
                'minoredit',
                'move',
+               'movefile',
+               'move-rootuserpages',
+               'move-subpages',
                'nominornewtalk',
                'noratelimit',
+               'override-export-depth',
                'patrol',
                'protect',
                'proxyunbannable',
@@ -160,33 +168,37 @@ class User {
                'reupload-shared',
                'rollback',
                'siteadmin',
+               'suppressionlog',
                'suppressredirect',
+               'suppressrevision',
                'trackback',
                'undelete',
                'unwatchedpages',
                'upload',
                'upload_by_url',
                'userrights',
+               'userrights-interwiki',
+               'writeapi',
        );
        /**
-        * \type{\string} Cached results of getAllRights()
+        * \string Cached results of getAllRights()
         */
        static $mAllRights = false;
 
        /** @name Cache variables */
        //@{
        var $mId, $mName, $mRealName, $mPassword, $mNewpassword, $mNewpassTime,
-               $mEmail, $mOptions, $mTouched, $mToken, $mEmailAuthenticated,
-               $mEmailToken, $mEmailTokenExpires, $mRegistration, $mGroups;
+               $mEmail, $mTouched, $mToken, $mEmailAuthenticated,
+               $mEmailToken, $mEmailTokenExpires, $mRegistration, $mGroups, $mOptionOverrides;
        //@}
 
        /**
-        * \type{\bool} Whether the cache variables have been loaded.
+        * \bool Whether the cache variables have been loaded.
         */
-       var $mDataLoaded;
+       var $mDataLoaded, $mAuthLoaded, $mOptionsLoaded;
 
        /**
-        * \type{\string} Initialization data source if mDataLoaded==false. May be one of:
+        * \string Initialization data source if mDataLoaded==false. May be one of:
         *  - 'defaults'   anonymous user initialised from class defaults
         *  - 'name'       initialise from mName
         *  - 'id'         initialise from mId
@@ -199,7 +211,8 @@ class User {
        /** @name Lazy-initialized variables, invalidated with clearInstanceCache */
        //@{
        var $mNewtalk, $mDatePreference, $mBlockedby, $mHash, $mSkin, $mRights,
-               $mBlockreason, $mBlock, $mEffectiveGroups;
+               $mBlockreason, $mBlock, $mEffectiveGroups, $mBlockedGlobally, 
+               $mLocked, $mHideName, $mOptions;
        //@}
 
        /**
@@ -246,6 +259,7 @@ class User {
                                break;
                        case 'session':
                                $this->loadFromSession();
+                               wfRunHooks( 'UserLoadAfterLoadFromSession', array( $this ) );
                                break;
                        default:
                                throw new MWException( "Unrecognised value for User->mFrom: \"{$this->mFrom}\"" );
@@ -255,7 +269,7 @@ class User {
 
        /**
         * Load user table data, given mId has already been set.
-        * @return \type{\bool} false if the ID does not exist, true otherwise
+        * @return \bool false if the ID does not exist, true otherwise
         * @private
         */
        function loadFromId() {
@@ -297,6 +311,7 @@ class User {
        function saveToCache() {
                $this->load();
                $this->loadGroups();
+               $this->loadOptions();
                if ( $this->isAnon() ) {
                        // Anonymous users are uncached
                        return;
@@ -321,8 +336,8 @@ class User {
         * This is slightly less efficient than newFromId(), so use newFromId() if
         * you have both an ID and a name handy.
         *
-        * @param $name \type{\string} Username, validated by Title::newFromText()
-        * @param $validate \type{\mixed} Validate username. Takes the same parameters as
+        * @param $name \string Username, validated by Title::newFromText()
+        * @param $validate \mixed Validate username. Takes the same parameters as
         *    User::getCanonicalName(), except that true is accepted as an alias
         *    for 'valid', for BC.
         *
@@ -349,7 +364,7 @@ class User {
        /**
         * Static factory method for creation from a given user ID.
         *
-        * @param $id \type{\int} Valid user ID
+        * @param $id \int Valid user ID
         * @return \type{User} The corresponding User object
         */
        static function newFromId( $id ) {
@@ -366,7 +381,7 @@ class User {
         *
         * If the code is invalid or has expired, returns NULL.
         *
-        * @param $code \type{\string} Confirmation code
+        * @param $code \string Confirmation code
         * @return \type{User}
         */
        static function newFromConfirmationCode( $code ) {
@@ -411,8 +426,8 @@ class User {
 
        /**
         * Get the username corresponding to a given user ID
-        * @param $id \type{\int} %User ID
-        * @return \type{\string} The corresponding username
+        * @param $id \int User ID
+        * @return \string The corresponding username
         */
        static function whoIs( $id ) {
                $dbr = wfGetDB( DB_SLAVE );
@@ -422,8 +437,8 @@ class User {
        /**
         * Get the real name of a user given their user ID
         *
-        * @param $id \type{\int} %User ID
-        * @return \type{\string} The corresponding user's real name
+        * @param $id \int User ID
+        * @return \string The corresponding user's real name
         */
        static function whoIsReal( $id ) {
                $dbr = wfGetDB( DB_SLAVE );
@@ -432,12 +447,11 @@ class User {
 
        /**
         * Get database id given a user name
-        * @param $name \type{\string} Username
-        * @return \twotypes{\int,\null} The corresponding user's ID, or null if user is nonexistent
-        * @static
+        * @param $name \string Username
+        * @return \types{\int,\null} The corresponding user's ID, or null if user is nonexistent
         */
        static function idFromName( $name ) {
-               $nt = Title::newFromText( $name );
+               $nt = Title::makeTitleSafe( NS_USER, $name );
                if( is_null( $nt ) ) {
                        # Illegal name
                        return null;
@@ -465,8 +479,8 @@ class User {
         * addresses like this, if we allowed accounts like this to be created
         * new users could get the old edits of these anonymous users.
         *
-        * @param $name \type{\string} String to match
-        * @return \type{\bool} True or false
+        * @param $name \string String to match
+        * @return \bool True or false
         */
        static function isIP( $name ) {
                return preg_match('/^\d{1,3}\.\d{1,3}\.\d{1,3}\.(?:xxx|\d{1,3})$/',$name) || IP::isIPv6($name);
@@ -480,8 +494,8 @@ class User {
         * is longer than the maximum allowed username size or doesn't begin with
         * a capital letter.
         *
-        * @param $name \type{\string} String to match
-        * @return \type{\bool} True or false
+        * @param $name \string String to match
+        * @return \bool True or false
         */
        static function isValidUserName( $name ) {
                global $wgContLang, $wgMaxNameChars;
@@ -534,8 +548,8 @@ class User {
         * If an account already exists in this form, login will be blocked
         * by a failure to pass this function.
         *
-        * @param $name \type{\string} String to match
-        * @return \type{\bool} True or false
+        * @param $name \string String to match
+        * @return \bool True or false
         */
        static function isUsableName( $name ) {
                global $wgReservedUsernames;
@@ -572,22 +586,23 @@ class User {
         * rather than in isValidUserName() to avoid disrupting
         * existing accounts.
         *
-        * @param $name \type{\string} String to match
-        * @return \type{\bool} True or false
+        * @param $name \string String to match
+        * @return \bool True or false
         */
        static function isCreatableName( $name ) {
+               global $wgInvalidUsernameCharacters;
                return
                        self::isUsableName( $name ) &&
 
                        // Registration-time character blacklisting...
-                       strpos( $name, '@' ) === false;
+                       !preg_match( '/[' . preg_quote( $wgInvalidUsernameCharacters, '/' ) . ']/', $name );
        }
 
        /**
         * Is the input a valid password for this user?
         *
-        * @param $password \type{\string} Desired password
-        * @return \type{\bool} True or false
+        * @param $password \string Desired password
+        * @return \bool True or false
         */
        function isValidPassword( $password ) {
                global $wgMinimalPasswordLength, $wgContLang;
@@ -612,8 +627,8 @@ class User {
         *
         * @todo Check for RFC 2822 compilance (bug 959)
         *
-        * @param $addr \type{\string} E-mail address
-        * @return \type{\bool} True or false
+        * @param $addr \string E-mail address
+        * @return \bool True or false
         */
        public static function isValidEmailAddr( $addr ) {
                $result = null;
@@ -627,8 +642,8 @@ class User {
        /**
         * Given unvalidated user input, return a canonical username, or false if
         * the username is invalid.
-        * @param $name \type{\string} User input
-        * @param $validate \twotypes{\string,\bool} Type of validation to use:
+        * @param $name \string User input
+        * @param $validate \types{\string,\bool} Type of validation to use:
         *                - false        No validation
         *                - 'valid'      Valid for batch processes
         *                - 'usable'     Valid for batch processes and login
@@ -646,7 +661,9 @@ class User {
                        return false;
 
                # Clean up name according to title rules
-               $t = Title::newFromText( $name );
+               $t = ($validate === 'valid') ? 
+                       Title::newFromText( $name ) : Title::makeTitle( NS_USER, $name );
+               # Check for invalid titles
                if( is_null( $t ) ) {
                        return false;
                }
@@ -684,8 +701,8 @@ class User {
         * Count the number of edits of a user
         * @todo It should not be static and some day should be merged as proper member function / deprecated -- domas
         *
-        * @param $uid \type{\int} %User ID to check
-        * @return \type{\int} The user's edit count
+        * @param $uid \int User ID to check
+        * @return \int The user's edit count
         */
        static function edits( $uid ) {
                wfProfileIn( __METHOD__ );
@@ -721,7 +738,7 @@ class User {
         * Return a random password. Sourced from mt_rand, so it's not particularly secure.
         * @todo hash random numbers to improve security, like generateToken()
         *
-        * @return \type{\string} New random password
+        * @return \string New random password
         */
        static function randomPassword() {
                global $wgMinimalPasswordLength;
@@ -755,7 +772,8 @@ class User {
                $this->mPassword = $this->mNewpassword = '';
                $this->mNewpassTime = null;
                $this->mEmail = '';
-               $this->mOptions = null; # Defer init
+               $this->mOptionOverrides = null;
+               $this->mOptionsLoaded = false;
 
                if ( isset( $_COOKIE[$wgCookiePrefix.'LoggedOut'] ) ) {
                        $this->mTouched = wfTimestamp( TS_MW, $_COOKIE[$wgCookiePrefix.'LoggedOut'] );
@@ -786,7 +804,7 @@ class User {
        /**
         * Load user data from the session or login cookie. If there are no valid
         * credentials, initialises the user as an anonymous user.
-        * @return \type{\bool} True if the user is logged in, false otherwise.
+        * @return \bool True if the user is logged in, false otherwise.
         */
        private function loadFromSession() {
                global $wgMemc, $wgCookiePrefix;
@@ -797,20 +815,27 @@ class User {
                        return $result;
                }
 
-               if ( isset( $_SESSION['wsUserID'] ) ) {
-                       if ( 0 != $_SESSION['wsUserID'] ) {
+               if ( isset( $_COOKIE["{$wgCookiePrefix}UserID"] ) ) {
+                       $sId = intval( $_COOKIE["{$wgCookiePrefix}UserID"] );
+                       if( isset( $_SESSION['wsUserID'] ) && $sId != $_SESSION['wsUserID'] ) {
+                               $this->loadDefaults(); // Possible collision!
+                               wfDebugLog( 'loginSessions', "Session user ID ({$_SESSION['wsUserID']}) and 
+                                       cookie user ID ($sId) don't match!" );
+                               return false;
+                       }
+                       $_SESSION['wsUserID'] = $sId;
+               } else if ( isset( $_SESSION['wsUserID'] ) ) {
+                       if ( $_SESSION['wsUserID'] != 0 ) {
                                $sId = $_SESSION['wsUserID'];
                        } else {
                                $this->loadDefaults();
                                return false;
                        }
-               } else if ( isset( $_COOKIE["{$wgCookiePrefix}UserID"] ) ) {
-                       $sId = intval( $_COOKIE["{$wgCookiePrefix}UserID"] );
-                       $_SESSION['wsUserID'] = $sId;
                } else {
                        $this->loadDefaults();
                        return false;
                }
+
                if ( isset( $_SESSION['wsUserName'] ) ) {
                        $sName = $_SESSION['wsUserName'];
                } else if ( isset( $_COOKIE["{$wgCookiePrefix}UserName"] ) ) {
@@ -856,7 +881,7 @@ class User {
         * Load user and user_group data from the database.
         * $this::mId must be set, this is how the user is identified.
         *
-        * @return \type{\bool} True if the user exists, false if the user is anonymous
+        * @return \bool True if the user exists, false if the user is anonymous
         * @private
         */
        function loadFromDatabase() {
@@ -872,6 +897,8 @@ class User {
                $dbr = wfGetDB( DB_MASTER );
                $s = $dbr->selectRow( 'user', '*', array( 'user_id' => $this->mId ), __METHOD__ );
 
+               wfRunHooks( 'UserLoadFromDatabase', array( $this, &$s ) );
+
                if ( $s !== false ) {
                        # Initialise user table data
                        $this->loadFromRow( $s );
@@ -895,7 +922,7 @@ class User {
                $this->mDataLoaded = true;
 
                if ( isset( $row->user_id ) ) {
-                       $this->mId = $row->user_id;
+                       $this->mId = intval( $row->user_id );
                }
                $this->mName = $row->user_name;
                $this->mRealName = $row->user_real_name;
@@ -933,7 +960,7 @@ class User {
 
        /**
         * Clear various cached data stored in this object.
-        * @param $reloadFrom \type{\string} Reload user and user_groups table data from a
+        * @param $reloadFrom \string Reload user and user_groups table data from a
         *   given source. May be "name", "id", "defaults", "session", or false for
         *   no reload.
         */
@@ -945,6 +972,7 @@ class User {
                $this->mSkin = null;
                $this->mRights = null;
                $this->mEffectiveGroups = null;
+               $this->mOptions = null;
 
                if ( $reloadFrom ) {
                        $this->mDataLoaded = false;
@@ -963,7 +991,7 @@ class User {
                /**
                 * Site defaults will override the global/language defaults
                 */
-               global $wgDefaultUserOptions, $wgContLang;
+               global $wgDefaultUserOptions, $wgContLang, $wgDefaultSkin;
                $defOpt = $wgDefaultUserOptions + $wgContLang->getDefaultUserOptionOverrides();
 
                /**
@@ -972,25 +1000,26 @@ class User {
                $variant = $wgContLang->getPreferredVariant( false );
                $defOpt['variant'] = $variant;
                $defOpt['language'] = $variant;
-
-               foreach( $wgNamespacesToBeSearchedDefault as $nsnum => $val ) {
-                       $defOpt['searchNs'.$nsnum] = $val;
+               foreach(  SearchEngine::searchableNamespaces() as $nsnum => $nsname ) {
+                       $defOpt['searchNs'.$nsnum] = !empty($wgNamespacesToBeSearchedDefault[$nsnum]);
                }
+               $defOpt['skin'] = $wgDefaultSkin;
+               
                return $defOpt;
        }
 
        /**
         * Get a given default option value.
         *
-        * @param $opt \type{\string} Name of option to retrieve
-        * @return \type{\string} Default option value
+        * @param $opt \string Name of option to retrieve
+        * @return \string Default option value
         */
        public static function getDefaultOption( $opt ) {
                $defOpts = self::getDefaultOptions();
                if( isset( $defOpts[$opt] ) ) {
                        return $defOpts[$opt];
                } else {
-                       return '';
+                       return null;
                }
        }
 
@@ -999,9 +1028,14 @@ class User {
         * @return \type{\arrayof{\string}} Array of user toggle names
         */
        static function getToggles() {
-               global $wgContLang;
+               global $wgContLang, $wgUseRCPatrol;
                $extraToggles = array();
                wfRunHooks( 'UserToggles', array( &$extraToggles ) );
+               if( $wgUseRCPatrol ) {
+                       $extraToggles[] = 'hidepatrolled';
+                       $extraToggles[] = 'newpageshidepatrolled';
+                       $extraToggles[] = 'watchlisthidepatrolled';
+               }
                return array_merge( self::$mToggles, $extraToggles, $wgContLang->getExtraUserToggles() );
        }
 
@@ -1009,7 +1043,7 @@ class User {
        /**
         * Get blocking information
         * @private
-        * @param $bFromSlave \type{\bool} Whether to check the slave database first. To
+        * @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.
@@ -1034,6 +1068,7 @@ class User {
                
                $this->mBlockedby = 0;
                $this->mHideName = 0;
+               $this->mAllowUsertalk = 0;
                $ip = wfGetIP();
 
                if ($this->isAllowed( 'ipblock-exempt' ) ) {
@@ -1049,12 +1084,14 @@ class User {
                        $this->mBlockedby = $this->mBlock->mBy;
                        $this->mBlockreason = $this->mBlock->mReason;
                        $this->mHideName = $this->mBlock->mHideName;
+                       $this->mAllowUsertalk = $this->mBlock->mAllowUsertalk;
                        if ( $this->isLoggedIn() ) {
                                $this->spreadBlock();
                        }
                } else {
-                       $this->mBlock = null;
-                       wfDebug( __METHOD__.": No block.\n" );
+                       // 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
@@ -1083,8 +1120,8 @@ class User {
        /**
         * Whether the given IP is in the SORBS blacklist.
         *
-        * @param $ip \type{\string} IP to check
-        * @return \type{\bool} True if blacklisted
+        * @param $ip \string IP to check
+        * @return \bool True if blacklisted.
         */
        function inSorbsBlacklist( $ip ) {
                global $wgEnableSorbs, $wgSorbsUrl;
@@ -1096,28 +1133,24 @@ class User {
        /**
         * Whether the given IP is in a given DNS blacklist.
         *
-        * @param $ip \type{\string} IP to check
-        * @param $base \type{\string} URL of the DNS blacklist
-        * @return \type{\bool} True if blacklisted
+        * @param $ip \string IP to check
+        * @param $base \string URL of the DNS blacklist
+        * @return \bool True if blacklisted.
         */
        function inDnsBlacklist( $ip, $base ) {
                wfProfileIn( __METHOD__ );
 
                $found = false;
                $host = '';
-               // FIXME: IPv6 ???
-               $m = array();
-               if ( preg_match( '/^(\d{1,3})\.(\d{1,3})\.(\d{1,3})\.(\d{1,3})$/', $ip, $m ) ) {
+               // FIXME: IPv6 ???  (http://bugs.php.net/bug.php?id=33170)
+               if( IP::isIPv4($ip) ) {
                        # Make hostname
-                       for ( $i=4; $i>=1; $i-- ) {
-                               $host .= $m[$i] . '.';
-                       }
-                       $host .= $base;
+                       $host = "$ip.$base";
 
                        # Send query
                        $ipList = gethostbynamel( $host );
 
-                       if ( $ipList ) {
+                       if( $ipList ) {
                                wfDebug( "Hostname $host is {$ipList[0]}, it's a proxy says $base!\n" );
                                $found = true;
                        } else {
@@ -1132,7 +1165,7 @@ class User {
        /**
         * Is this user subject to rate limiting?
         *
-        * @return \type{\bool} True if rate limited
+        * @return \bool True if rate limited
         */
        public function isPingLimitable() {
                global $wgRateLimitsExcludedGroups;
@@ -1150,8 +1183,8 @@ class User {
         * @note When using a shared cache like memcached, IP-address
         * last-hit counters will be shared across wikis.
         *
-        * @param $action \type{\string} Action to enforce; 'edit' if unspecified
-        * @return \type{\bool} True if a rate limiter was tripped
+        * @param $action \string Action to enforce; 'edit' if unspecified
+        * @return \bool True if a rate limiter was tripped
         */
        function pingLimiter( $action='edit' ) {
 
@@ -1243,8 +1276,8 @@ class User {
        /**
         * Check if user is blocked
         * 
-        * @param $bFromSlave \type{\bool} Whether to check the slave database instead of the master
-        * @return \type{\bool} True if blocked, false otherwise
+        * @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
                wfDebug( "User::isBlocked: enter\n" );
@@ -1255,9 +1288,9 @@ class User {
        /**
         * Check if user is blocked from editing a particular article
         * 
-        * @param $title      \type{\string} Title to check
-        * @param $bFromSlave \type{\bool}   Whether to check the slave database instead of the master
-        * @return \type{\bool} True if blocked, false otherwise
+        * @param $title      \string Title to check
+        * @param $bFromSlave \bool   Whether to check the slave database instead of the master
+        * @return \bool True if blocked, false otherwise
         */
        function isBlockedFrom( $title, $bFromSlave = false ) {
                global $wgBlockAllowsUTEdit;
@@ -1266,8 +1299,9 @@ class User {
 
                wfDebug( __METHOD__.": asking isBlocked()\n" );
                $blocked = $this->isBlocked( $bFromSlave );
+               $allowUsertalk = ($wgBlockAllowsUTEdit ? $this->mAllowUsertalk : false);
                # If a user's name is suppressed, they cannot make edits anywhere
-               if ( !$this->mHideName && $wgBlockAllowsUTEdit && $title->getText() === $this->getName() &&
+               if ( !$this->mHideName && $allowUsertalk && $title->getText() === $this->getName() &&
                  $title->getNamespace() == NS_USER_TALK ) {
                        $blocked = false;
                        wfDebug( __METHOD__.": self-talk page, ignoring any blocks\n" );
@@ -1278,7 +1312,7 @@ class User {
 
        /**
         * If user is blocked, return the name of the user who placed the block
-        * @return \type{\string} name of blocker
+        * @return \string name of blocker
         */
        function blockedBy() {
                $this->getBlockedStatus();
@@ -1287,16 +1321,82 @@ class User {
 
        /**
         * If user is blocked, return the specified reason for the block
-        * @return \type{\string} Blocking reason
+        * @return \string Blocking reason
         */
        function blockedFor() {
                $this->getBlockedStatus();
                return $this->mBlockreason;
        }
+       
+       /**
+        * If user is blocked, return the ID for the block
+        * @return \int Block ID
+        */
+       function getBlockId() {
+               $this->getBlockedStatus();
+               return ($this->mBlock ? $this->mBlock->mId : false);
+       }
+       
+       /**
+        * Check if user is blocked on all wikis.
+        * Do not use for actual edit permission checks!
+        * This is intented for quick UI checks.
+        * 
+        * @param $ip \type{\string} IP address, uses current client if none given
+        * @return \type{\bool} True if blocked, false otherwise
+        */
+       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();
+               }
+               $blocked = false;
+               wfRunHooks( 'UserIsBlockedGlobally', array( &$this, $ip, &$blocked ) );
+               $this->mBlockedGlobally = (bool)$blocked;
+               return $this->mBlockedGlobally;
+       }
+       
+       /**
+        * Check if user account is locked
+        * 
+        * @return \type{\bool} True if locked, false otherwise
+        */
+       function isLocked() {
+               if( $this->mLocked !== null ) {
+                       return $this->mLocked;
+               }
+               global $wgAuth;
+               $authUser = $wgAuth->getUserInstance( $this );
+               $this->mLocked = (bool)$authUser->isLocked();
+               return $this->mLocked;
+       }
+       
+       /**
+        * Check if user account is hidden
+        * 
+        * @return \type{\bool} True if hidden, false otherwise
+        */
+       function isHidden() {
+               if( $this->mHideName !== null ) {
+                       return $this->mHideName;
+               }
+               $this->getBlockedStatus();
+               if( !$this->mHideName ) {
+                       global $wgAuth;
+                       $authUser = $wgAuth->getUserInstance( $this );
+                       $this->mHideName = (bool)$authUser->isHidden();
+               }
+               return $this->mHideName;
+       }
 
        /**
         * Get the user's ID.
-        * @return \type{\int} The user's ID; 0 if the user is anonymous or nonexistent
+        * @return \int The user's ID; 0 if the user is anonymous or nonexistent
         */
        function getId() {
                if( $this->mId === null and $this->mName !== null
@@ -1312,7 +1412,7 @@ class User {
 
        /**
         * Set the user and reload all fields according to a given ID
-        * @param $v \type{\int} %User ID to reload
+        * @param $v \int User ID to reload
         */
        function setId( $v ) {
                $this->mId = $v;
@@ -1321,7 +1421,7 @@ class User {
 
        /**
         * Get the user name, or the IP of an anonymous user
-        * @return \type{\string} User's name or IP address
+        * @return \string User's name or IP address
         */
        function getName() {
                if ( !$this->mDataLoaded && $this->mFrom == 'name' ) {
@@ -1348,7 +1448,7 @@ class User {
         *
         * @note User::newFromName() has rougly the same function, when the named user
         * does not exist.
-        * @param $str \type{\string} New user name to set
+        * @param $str \string New user name to set
         */
        function setName( $str ) {
                $this->load();
@@ -1357,7 +1457,7 @@ class User {
 
        /**
         * Get the user's name escaped by underscores.
-        * @return \type{\string} Username escaped by underscores
+        * @return \string Username escaped by underscores.
         */
        function getTitleKey() {
                return str_replace( ' ', '_', $this->getName() );
@@ -1365,7 +1465,7 @@ class User {
 
        /**
         * Check if the user has new messages.
-        * @return \type{\bool} True if the user has new messages
+        * @return \bool True if the user has new messages
         */
        function getNewtalk() {
                $this->load();
@@ -1417,10 +1517,10 @@ class User {
         * Internal uncached check for new messages
         *
         * @see getNewtalk()
-        * @param $field \type{\string} 'user_ip' for anonymous users, 'user_id' otherwise
-        * @param $id \twotypes{\string,\int} User's IP address for anonymous users, %User ID otherwise
-        * @param $fromMaster \type{\bool} true to fetch from the master, false for a slave
-        * @return \type{\bool} True if the user has new messages
+        * @param $field \string 'user_ip' for anonymous users, 'user_id' otherwise
+        * @param $id \types{\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 ) {
@@ -1436,9 +1536,9 @@ class User {
 
        /**
         * Add or update the new messages flag
-        * @param $field \type{\string} 'user_ip' for anonymous users, 'user_id' otherwise
-        * @param $id \twotypes{string,\int} User's IP address for anonymous users, %User ID otherwise
-        * @return \type{\bool} True if successful, false otherwise
+        * @param $field \string 'user_ip' for anonymous users, 'user_id' otherwise
+        * @param $id \types{\string,\int} User's IP address for anonymous users, User ID otherwise
+        * @return \bool True if successful, false otherwise
         * @private
         */
        function updateNewtalk( $field, $id ) {
@@ -1458,9 +1558,9 @@ class User {
 
        /**
         * Clear the new messages flag for the given user
-        * @param $field \type{\string} 'user_ip' for anonymous users, 'user_id' otherwise
-        * @param $id \twotypes{\string,\int} User's IP address for anonymous users, %User ID otherwise
-        * @return \type{\bool} True if successful, false otherwise
+        * @param $field \string 'user_ip' for anonymous users, 'user_id' otherwise
+        * @param $id \types{\string,\int} User's IP address for anonymous users, User ID otherwise
+        * @return \bool True if successful, false otherwise
         * @private
         */
        function deleteNewtalk( $field, $id ) {
@@ -1479,7 +1579,7 @@ class User {
 
        /**
         * Update the 'You have new messages!' status.
-        * @param $val \type{\bool} Whether the user has new messages
+        * @param $val \bool Whether the user has new messages
         */
        function setNewtalk( $val ) {
                if( wfReadOnly() ) {
@@ -1518,7 +1618,7 @@ class User {
        /**
         * Generate a current or new-future timestamp to be stored in the
         * user_touched field when we update things.
-        * @return \type{\string} Timestamp in TS_MW format
+        * @return \string Timestamp in TS_MW format
         */
        private static function newTouchedTimestamp() {
                global $wgClockSkewFudge;
@@ -1533,6 +1633,7 @@ class User {
         * Called implicitly from invalidateCache() and saveSettings().
         */
        private function clearSharedCache() {
+               $this->load();
                if( $this->mId ) {
                        global $wgMemc;
                        $wgMemc->delete( wfMemcKey( 'user', 'id', $this->mId ) );
@@ -1561,7 +1662,7 @@ class User {
 
        /**
         * Validate the cache for this account.
-        * @param $timestamp \type{\string} A timestamp in TS_MW format
+        * @param $timestamp \string A timestamp in TS_MW format
         */
        function validateCache( $timestamp ) {
                $this->load();
@@ -1587,7 +1688,7 @@ class User {
         * wipes it, so the account cannot be logged in until
         * a new password is set, for instance via e-mail.
         *
-        * @param $str \type{\string} New password to set
+        * @param $str \string New password to set
         * @throws PasswordError on failure
         */
        function setPassword( $str ) {
@@ -1617,7 +1718,7 @@ class User {
        /**
         * Set the password and reset the random token unconditionally.
         *
-        * @param $str \type{\string} New password to set
+        * @param $str \string New password to set
         */
        function setInternalPassword( $str ) {
                $this->load();
@@ -1635,7 +1736,7 @@ class User {
        
        /**
         * Get the user's current token.
-        * @return \type{\string} Token
+        * @return \string Token
         */
        function getToken() {
                $this->load();
@@ -1646,7 +1747,7 @@ class User {
         * Set the random token (used for persistent authentication)
         * Called from loadDefaults() among other places.
         *
-        * @param $token \type{\string} If specified, set the token to this value
+        * @param $token \string If specified, set the token to this value
         * @private
         */
        function setToken( $token = false ) {
@@ -1669,7 +1770,7 @@ class User {
        /**
         * Set the cookie password
         *
-        * @param $str \type{\string} New cookie password
+        * @param $str \string New cookie password
         * @private
         */
        function setCookiePassword( $str ) {
@@ -1680,8 +1781,8 @@ class User {
        /**
         * Set the password for a password reminder or new account email
         *
-        * @param $str \type{\string} New password to set
-        * @param $throttle \type{\bool} If true, reset the throttle timestamp to the present
+        * @param $str \string New password to set
+        * @param $throttle \bool If true, reset the throttle timestamp to the present
         */
        function setNewpassword( $str, $throttle = true ) {
                $this->load();
@@ -1694,7 +1795,7 @@ class User {
        /**
         * Has password reminder email been sent within the last 
         * $wgPasswordReminderResendTime hours?
-        * @return \type{\bool} True or false
+        * @return \bool True or false
         */
        function isPasswordReminderThrottled() {
                global $wgPasswordReminderResendTime;
@@ -1708,7 +1809,7 @@ class User {
 
        /**
         * Get the user's e-mail address
-        * @return \type{\string} User's e-mail address
+        * @return \string User's email address
         */
        function getEmail() {
                $this->load();
@@ -1718,7 +1819,7 @@ class User {
 
        /**
         * Get the timestamp of the user's e-mail authentication
-        * @return \type{\string} TS_MW timestamp
+        * @return \string TS_MW timestamp
         */
        function getEmailAuthenticationTimestamp() {
                $this->load();
@@ -1728,7 +1829,7 @@ class User {
 
        /**
         * Set the user's e-mail address
-        * @param $str \type{\string} New e-mail address
+        * @param $str \string New e-mail address
         */
        function setEmail( $str ) {
                $this->load();
@@ -1738,7 +1839,7 @@ class User {
 
        /**
         * Get the user's real name
-        * @return \type{\string} User's real name
+        * @return \string User's real name
         */
        function getRealName() {
                $this->load();
@@ -1747,7 +1848,7 @@ class User {
 
        /**
         * Set the user's real name
-        * @param $str \type{\string} New real name
+        * @param $str \string New real name
         */
        function setRealName( $str ) {
                $this->load();
@@ -1757,14 +1858,14 @@ class User {
        /**
         * Get the user's current setting for a given option.
         *
-        * @param $oname \type{\string} The option to check
-        * @param $defaultOverride \type{\string} A default value returned if the option does not exist
-        * @return \type{\string} User's current value for the option
+        * @param $oname \string The option to check
+        * @param $defaultOverride \string A default value returned if the option does not exist
+        * @return \string User's current value for the option
         * @see getBoolOption()
         * @see getIntOption()
         */
-       function getOption( $oname, $defaultOverride = '' ) {
-               $this->load();
+       function getOption( $oname, $defaultOverride = null ) {
+               $this->loadOptions();
 
                if ( is_null( $this->mOptions ) ) {
                        if($defaultOverride != '') {
@@ -1774,7 +1875,7 @@ class User {
                }
 
                if ( array_key_exists( $oname, $this->mOptions ) ) {
-                       return trim( $this->mOptions[$oname] );
+                       return $this->mOptions[$oname];
                } else {
                        return $defaultOverride;
                }
@@ -1783,8 +1884,8 @@ class User {
        /**
         * Get the user's current setting for a given option, as a boolean value.
         *
-        * @param $oname \type{\string} The option to check
-        * @return \type{\bool} User's current value for the option
+        * @param $oname \string The option to check
+        * @return \bool User's current value for the option
         * @see getOption()
         */
        function getBoolOption( $oname ) {
@@ -1795,9 +1896,9 @@ class User {
        /**
         * Get the user's current setting for a given option, as a boolean value.
         *
-        * @param $oname \type{\string} The option to check
-        * @param $defaultOverride \type{\int} A default value returned if the option does not exist
-        * @return \type{\int} User's current value for the option
+        * @param $oname \string The option to check
+        * @param $defaultOverride \int A default value returned if the option does not exist
+        * @return \int User's current value for the option
         * @see getOption()
         */
        function getIntOption( $oname, $defaultOverride=0 ) {
@@ -1811,36 +1912,37 @@ class User {
        /**
         * Set the given option for a user.
         *
-        * @param $oname \type{\string} The option to set
-        * @param $val \type{\mixed} New value to set
+        * @param $oname \string The option to set
+        * @param $val \mixed New value to set
         */
        function setOption( $oname, $val ) {
                $this->load();
-               if ( is_null( $this->mOptions ) ) {
-                       $this->mOptions = User::getDefaultOptions();
-               }
+               $this->loadOptions();
+               
                if ( $oname == 'skin' ) {
                        # Clear cached skin, so the new one displays immediately in Special:Preferences
                        unset( $this->mSkin );
                }
-               // Filter out any newlines that may have passed through input validation.
-               // Newlines are used to separate items in the options blob.
-               if( $val ) {
-                       $val = str_replace( "\r\n", "\n", $val );
-                       $val = str_replace( "\r", "\n", $val );
-                       $val = str_replace( "\n", " ", $val );
-               }
+               
                // Explicitly NULL values should refer to defaults
                global $wgDefaultUserOptions;
                if( is_null($val) && isset($wgDefaultUserOptions[$oname]) ) {
                        $val = $wgDefaultUserOptions[$oname];
                }
+               
                $this->mOptions[$oname] = $val;
        }
+       
+       /**
+        * Reset all options to the site defaults
+        */     
+       function resetOptions() {
+               $this->mOptions = User::getDefaultOptions();
+       }
 
        /**
         * Get the user's preferred date format.
-        * @return \type{\string} User's preferred date format
+        * @return \string User's preferred date format
         */
        function getDatePreference() {
                // Important migration for old data rows
@@ -1884,8 +1986,7 @@ class User {
         * Get the list of implicit group memberships this user has.
         * This includes all explicit groups, plus 'user' if logged in,
         * '*' for all accounts and autopromoted groups
-        *
-        * @param $recache \type{\bool} Whether to avoid the cache
+        * @param $recache \bool Whether to avoid the cache
         * @return \type{\arrayof{\string}} Array of internal group names
         */
        function getEffectiveGroups( $recache = false ) {
@@ -1909,10 +2010,10 @@ class User {
 
        /**
         * Get the user's edit count.
-        * @return \type{\int} User's edit count
+        * @return \int User'e edit count
         */
        function getEditCount() {
-               if ($this->mId) {
+               if ($this->getId()) {
                        if ( !isset( $this->mEditCount ) ) {
                                /* Populate the count, if it has not been populated yet */
                                $this->mEditCount = User::edits($this->mId);
@@ -1927,7 +2028,7 @@ class User {
        /**
         * Add the user to the given group.
         * This takes immediate effect.
-        * @param $group \type{\string} Name of the group to add
+        * @param $group \string Name of the group to add
         */
        function addGroup( $group ) {
                $dbw = wfGetDB( DB_MASTER );
@@ -1951,7 +2052,7 @@ class User {
        /**
         * Remove the user from the given group.
         * This takes immediate effect.
-        * @param $group \type{\string} Name of the group to remove
+        * @param $group \string Name of the group to remove
         */
        function removeGroup( $group ) {
                $this->load();
@@ -1973,7 +2074,7 @@ class User {
 
        /**
         * Get whether the user is logged in
-        * @return \type{\bool} True or false
+        * @return \bool True or false
         */
        function isLoggedIn() {
                return $this->getID() != 0;
@@ -1981,7 +2082,7 @@ class User {
 
        /**
         * Get whether the user is anonymous
-        * @return \type{\bool} True or false
+        * @return \bool True or false
         */
        function isAnon() {
                return !$this->isLoggedIn();
@@ -1989,7 +2090,7 @@ class User {
 
        /**
         * Get whether the user is a bot
-        * @return \type{\bool} True or false
+        * @return \bool True or false
         * @deprecated
         */
        function isBot() {
@@ -1999,20 +2100,26 @@ class User {
 
        /**
         * Check if user is allowed to access a feature / make an action
-        * @param $action \type{\string} action to be checked
-        * @return \type{\bool} True if action is allowed, else false
+        * @param $action \string action to be checked
+        * @return \bool True if action is allowed, else false
         */
-       function isAllowed($action='') {
+       function isAllowed( $action = '' ) {
                if ( $action === '' )
-                       // In the spirit of DWIM
-                       return true;
-
-               return in_array( $action, $this->getRights() );
+                       return true; // In the spirit of DWIM
+               # Patrolling may not be enabled
+               if( $action === 'patrol' || $action === 'autopatrol' ) {
+                       global $wgUseRCPatrol, $wgUseNPPatrol;
+                       if( !$wgUseRCPatrol && !$wgUseNPPatrol )
+                               return false;
+               }
+               # Use strict parameter to avoid matching numeric 0 accidentally inserted 
+               # by misconfiguration: 0 == 'foo'
+               return in_array( $action, $this->getRights(), true );
        }
 
        /**
        * Check whether to enable recent changes patrol features for this user
-       * @return \type{\bool} True or false
+       * @return \bool True or false
        */
        public function useRCPatrol() {
                global $wgUseRCPatrol;
@@ -2021,7 +2128,7 @@ class User {
 
        /**
        * Check whether to enable new pages patrol features for this user
-       * @return \type{\bool} True or false
+       * @return \bool True or false
        */
        public function useNPPatrol() {
                global $wgUseRCPatrol, $wgUseNPPatrol;
@@ -2029,29 +2136,44 @@ class User {
        }
 
        /**
-        * Get the current skin, loading it if required
-        * @return \type{Skin} Current skin
+        * 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() {
-               global $wgRequest;
+       function &getSkin( $t = null ) {
                if ( ! isset( $this->mSkin ) ) {
                        wfProfileIn( __METHOD__ );
 
-                       # get the user skin
-                       $userSkin = $this->getOption( 'skin' );
-                       $userSkin = $wgRequest->getVal('useskin', $userSkin);
-
+                       global $wgHiddenPrefs;
+                       if( !in_array( 'skin', $wgHiddenPrefs ) ) {
+                               # get the user skin
+                               global $wgRequest;
+                               $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;
+                       }
+                       
                        $this->mSkin =& Skin::newFromKey( $userSkin );
                        wfProfileOut( __METHOD__ );
                }
+               if( $t || !$this->mSkin->getTitle() ) {
+                       if ( !$t ) {
+                               global $wgOut;
+                               $t = $wgOut->getTitle();
+                       }
+                       $this->mSkin->setTitle( $t );
+               }
                return $this->mSkin;
        }
 
        /**
         * Check the watched status of an article.
         * @param $title \type{Title} Title of the article to look at
-        * @return \type{\bool} True if article is watched
+        * @return \bool True if article is watched
         */
        function isWatched( $title ) {
                $wl = WatchedItem::fromUserTitle( $this, $title );
@@ -2143,7 +2265,7 @@ class User {
         * If e-notif e-mails are on, they will receive notification mails on
         * the next change of any watched page.
         *
-        * @param $currentUser \type{\int} %User ID
+        * @param $currentUser \int User ID
         */
        function clearAllNotifications( $currentUser ) {
                global $wgUseEnotif, $wgShowUpdatedMarker;
@@ -2165,36 +2287,25 @@ class User {
                }
        }
 
-       /**
-        * Encode this user's options as a string
-        * @return \type{\string} Encoded options
-        * @private
-        */
-       function encodeOptions() {
-               $this->load();
-               if ( is_null( $this->mOptions ) ) {
-                       $this->mOptions = User::getDefaultOptions();
-               }
-               $a = array();
-               foreach ( $this->mOptions as $oname => $oval ) {
-                       array_push( $a, $oname.'='.$oval );
-               }
-               $s = implode( "\n", $a );
-               return $s;
-       }
-
        /**
         * Set this user's options from an encoded string
-        * @param $str \type{\string} Encoded options to import
+        * @param $str \string Encoded options to import
         * @private
         */
        function decodeOptions( $str ) {
+               if (!$str)
+                       return;
+                       
+               $this->mOptionsLoaded = true;
+               $this->mOptionOverrides = array();
+               
                $this->mOptions = array();
                $a = explode( "\n", $str );
                foreach ( $a as $s ) {
                        $m = array();
                        if ( preg_match( "/^(.[^=]*)=(.*)$/", $s, $m ) ) {
                                $this->mOptions[$m[1]] = $m[2];
+                               $this->mOptionOverrides[$m[1]] = $m[2];
                        }
                }
        }
@@ -2202,6 +2313,10 @@ class User {
        /**
         * Set a cookie on the user's client. Wrapper for 
         * WebResponse::setCookie
+        * @param $name \string Name of the cookie to set
+        * @param $value \string Value to set
+        * @param $exp \int Expiration time, as a UNIX time value; 
+        *                   if 0 or not specified, use the default $wgCookieExpiration
         */
        protected function setCookie( $name, $value, $exp=0 ) {
                global $wgRequest;
@@ -2210,7 +2325,7 @@ class User {
        
        /**
         * Clear a cookie on the user's client
-        * @param $name \type{\string} Name of the cookie to clear
+        * @param $name \string Name of the cookie to clear
         */
        protected function clearCookie( $name ) {
                $this->setCookie( $name, '', time() - 86400 );
@@ -2238,7 +2353,10 @@ class User {
                }
                
                wfRunHooks( 'UserSetCookies', array( $this, &$session, &$cookies ) );
-               $_SESSION = $session + $_SESSION;
+               #check for null, since the hook could cause a null value 
+               if ( !is_null( $session ) && isset( $_SESSION ) ){
+                       $_SESSION = $session + $_SESSION;
+               }
                foreach ( $cookies as $name => $value ) {
                        if ( $value === false ) {
                                $this->clearCookie( $name );
@@ -2252,7 +2370,6 @@ class User {
         * Log this user out.
         */
        function logout() {
-               global $wgUser;
                if( wfRunHooks( 'UserLogout', array(&$this) ) ) {
                        $this->doLogout();
                }
@@ -2296,7 +2413,7 @@ class User {
                                'user_real_name' => $this->mRealName,
                                'user_email' => $this->mEmail,
                                'user_email_authenticated' => $dbw->timestampOrNull( $this->mEmailAuthenticated ),
-                               'user_options' => $this->encodeOptions(),
+                               'user_options' => '',
                                'user_touched' => $dbw->timestamp($this->mTouched),
                                'user_token' => $this->mToken,
                                'user_email_token' => $this->mEmailToken,
@@ -2305,8 +2422,12 @@ class User {
                                'user_id' => $this->mId
                        ), __METHOD__
                );
+               
+               $this->saveOptions();
+               
                wfRunHooks( 'UserSaveSettings', array( $this ) );
                $this->clearSharedCache();
+               $this->getUserPage()->invalidateCache();
        }
 
        /**
@@ -2327,7 +2448,7 @@ class User {
        /**
         * Add a user to the database, return the user object
         *
-        * @param $name \type{\string} Username to add
+        * @param $name \string Username to add
         * @param $params \type{\arrayof{\string}} 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
@@ -2358,7 +2479,7 @@ class User {
                        'user_email' => $user->mEmail,
                        'user_email_authenticated' => $dbw->timestampOrNull( $user->mEmailAuthenticated ),
                        'user_real_name' => $user->mRealName,
-                       'user_options' => $user->encodeOptions(),
+                       'user_options' => '',
                        'user_token' => $user->mToken,
                        'user_registration' => $dbw->timestamp( $user->mRegistration ),
                        'user_editcount' => 0,
@@ -2392,7 +2513,7 @@ class User {
                                'user_email' => $this->mEmail,
                                'user_email_authenticated' => $dbw->timestampOrNull( $this->mEmailAuthenticated ),
                                'user_real_name' => $this->mRealName,
-                               'user_options' => $this->encodeOptions(),
+                               'user_options' => '',
                                'user_token' => $this->mToken,
                                'user_registration' => $dbw->timestamp( $this->mRegistration ),
                                'user_editcount' => 0,
@@ -2402,6 +2523,8 @@ class User {
 
                // Clear instance cache other than user table data, which is already accurate
                $this->clearInstanceCache();
+               
+               $this->saveOptions();
        }
 
        /**
@@ -2435,10 +2558,10 @@ class User {
         * which will give them a chance to modify this key based on their own
         * settings.
         *
-        * @return \type{\string} Page rendering hash
+        * @return \string Page rendering hash
         */
        function getPageRenderingHash() {
-               global $wgContLang, $wgUseDynamicDates, $wgLang;
+               global $wgUseDynamicDates, $wgRenderHashAppend, $wgLang, $wgContLang;
                if( $this->mHash ){
                        return $this->mHash;
                }
@@ -2458,6 +2581,8 @@ class User {
                $extra = $wgContLang->getExtraHashOptions();
                $confstr .= $extra;
 
+               $confstr .= $wgRenderHashAppend;
+
                // Give a chance for extensions to modify the hash, if they have
                // extra options or other effects on the parser cache.
                wfRunHooks( 'PageRenderingHash', array( &$confstr ) );
@@ -2470,7 +2595,7 @@ class User {
 
        /**
         * Get whether the user is explicitly blocked from account creation.
-        * @return \type{\bool} True if blocked
+        * @return \bool True if blocked
         */
        function isBlockedFromCreateAccount() {
                $this->getBlockedStatus();
@@ -2479,7 +2604,7 @@ class User {
 
        /**
         * Get whether the user is blocked from using Special:Emailuser.
-        * @return \type{\bool} True if blocked
+        * @return \bool True if blocked
         */
        function isBlockedFromEmailuser() {
                $this->getBlockedStatus();
@@ -2488,7 +2613,7 @@ class User {
 
        /**
         * Get whether the user is allowed to create an account.
-        * @return \type{\bool} True if allowed
+        * @return \bool True if allowed
         */
        function isAllowedToCreateAccount() {
                return $this->isAllowed( 'createaccount' ) && !$this->isBlockedFromCreateAccount();
@@ -2522,7 +2647,7 @@ class User {
 
        /**
         * Get the maximum valid user ID.
-        * @return \type{\int} %User ID
+        * @return \int User ID
         * @static
         */
        function getMaxID() {
@@ -2539,7 +2664,7 @@ class User {
        /**
         * Determine whether the user is a newbie. Newbies are either
         * anonymous IPs, or the most recently created accounts.
-        * @return \type{\bool} True if the user is a newbie
+        * @return \bool True if the user is a newbie
         */
        function isNewbie() {
                return !$this->isAllowed( 'autoconfirmed' );
@@ -2549,7 +2674,7 @@ class User {
         * Is the user active? We check to see if they've made at least
         * X number of edits in the last Y days.
         * 
-        * @return \type{\bool} True if the user is active, false if not.
+        * @return \bool True if the user is active, false if not.
         */
        public function isActiveEditor() {
                global $wgActiveUserEditCount, $wgActiveUserDays;
@@ -2573,8 +2698,8 @@ class User {
 
        /**
         * Check to see if the given clear-text password is one of the accepted passwords
-        * @param $password \type{\string} user password.
-        * @return \type{\bool} True if the given password is correct, otherwise False.
+        * @param $password \string user password.
+        * @return \bool True if the given password is correct, otherwise False.
         */
        function checkPassword( $password ) {
                global $wgAuth;
@@ -2614,10 +2739,17 @@ class User {
        /**
         * Check if the given clear-text password matches the temporary password
         * sent by e-mail for password reset operations.
-        * @return \type{\bool} True if matches, false otherwise
+        * @return \bool True if matches, false otherwise
         */
        function checkTemporaryPassword( $plaintext ) {
-               return self::comparePasswords( $this->mNewpassword, $plaintext, $this->getId() );
+               global $wgNewPasswordExpiry;
+               if( self::comparePasswords( $this->mNewpassword, $plaintext, $this->getId() ) ) {
+                       $this->load();
+                       $expiry = wfTimestamp( TS_UNIX, $this->mNewpassTime ) + $wgNewPasswordExpiry;
+                       return ( time() < $expiry );
+               } else {
+                       return false;
+               }
        }
 
        /**
@@ -2626,8 +2758,8 @@ class User {
         * login credentials aren't being hijacked with a foreign form
         * submission.
         *
-        * @param $salt \twotypes{\string,\arrayof{\string}} Optional function-specific data for hashing
-        * @return \type{\string} The new edit token
+        * @param $salt \types{\string,\arrayof{\string}} Optional function-specific data for hashing
+        * @return \string The new edit token
         */
        function editToken( $salt = '' ) {
                if ( $this->isAnon() ) {
@@ -2649,8 +2781,8 @@ class User {
        /**
         * Generate a looking random token for various uses.
         *
-        * @param $salt \type{\string} Optional salt value
-        * @return \type{\string} The new random token
+        * @param $salt \string Optional salt value
+        * @return \string The new random token
         */
        function generateToken( $salt = '' ) {
                $token = dechex( mt_rand() ) . dechex( mt_rand() );
@@ -2663,9 +2795,9 @@ class User {
         * user's own login session, not a form submission from a third-party
         * site.
         *
-        * @param $val \type{\string} Input value to compare
-        * @param $salt \type{\string} Optional function-specific data for hashing
-        * @return \type{\bool} Whether the token matches
+        * @param $val \string Input value to compare
+        * @param $salt \string Optional function-specific data for hashing
+        * @return \bool Whether the token matches
         */
        function matchEditToken( $val, $salt = '' ) {
                $sessionToken = $this->editToken( $salt );
@@ -2679,9 +2811,9 @@ class User {
         * Check given value against the token value stored in the session,
         * ignoring the suffix.
         *
-        * @param $val \type{\string} Input value to compare
-        * @param $salt \type{\string} Optional function-specific data for hashing
-        * @return \type{\bool} Whether the token matches
+        * @param $val \string Input value to compare
+        * @param $salt \string Optional function-specific data for hashing
+        * @return \bool Whether the token matches
         */
        function matchEditTokenNoSuffix( $val, $salt = '' ) {
                $sessionToken = $this->editToken( $salt );
@@ -2692,7 +2824,7 @@ class User {
         * Generate a new e-mail confirmation token and send a confirmation/invalidation
         * mail to the user's given address.
         *
-        * @return \twotypes{\bool,WikiError} True on success, a WikiError object on failure.
+        * @return \types{\bool,\type{WikiError}} True on success, a WikiError object on failure.
         */
        function sendConfirmationMail() {
                global $wgLang;
@@ -2708,18 +2840,20 @@ class User {
                                $this->getName(),
                                $url,
                                $wgLang->timeanddate( $expiration, false ),
-                               $invalidateURL ) );
+                               $invalidateURL,
+                               $wgLang->date( $expiration, false ),
+                               $wgLang->time( $expiration, false ) ) );
        }
 
        /**
         * Send an e-mail to this user's account. Does not check for
         * confirmed status or validity.
         *
-        * @param $subject \type{\string} Message subject
-        * @param $body \type{\string} Message body
-        * @param $from \type{\string} Optional From address; if unspecified, default $wgPasswordSender will be used
-        * @param $replyto \type{\string} Reply-to address
-        * @return \twotypes{\bool,WikiError} True on success, a WikiError object on failure
+        * @param $subject \string Message subject
+        * @param $body \string Message body
+        * @param $from \string Optional From address; if unspecified, default $wgPasswordSender will be used
+        * @param $replyto \string Reply-To address
+        * @return \types{\bool,\type{WikiError}} True on success, a WikiError object on failure
         */
        function sendMail( $subject, $body, $from = null, $replyto = null ) {
                if( is_null( $from ) ) {
@@ -2739,8 +2873,8 @@ class User {
         * @note Call saveSettings() after calling this function to commit
         * this change to the database.
         *
-        * @param[out] &$expiration \type{\mixed} Accepts the expiration time
-        * @return \type{\string} New token
+        * @param[out] &$expiration \mixed Accepts the expiration time
+        * @return \string New token
         * @private
         */
        function confirmationToken( &$expiration ) {
@@ -2757,8 +2891,8 @@ class User {
 
        /**
        * Return a URL the user can use to confirm their email address.
-        * @param $token \type{\string} Accepts the email confirmation token
-        * @return \type{\string} New token URL
+        * @param $token \string Accepts the email confirmation token
+        * @return \string New token URL
         * @private
         */
        function confirmationTokenUrl( $token ) {
@@ -2766,9 +2900,8 @@ class User {
        }
        /**
         * Return a URL the user can use to invalidate their email address.
-        *
-        * @param $token \type{\string} Accepts the email confirmation token
-        * @return \type{\string} New token URL
+        * @param $token \string Accepts the email confirmation token
+        * @return \string New token URL
         * @private
         */
        function invalidationTokenUrl( $token ) {
@@ -2785,9 +2918,9 @@ class User {
         * also sometimes can get corrupted in some browsers/mailers
         * (bug 6957 with Gmail and Internet Explorer).
         *
-        * @param $page \type{\string} Special page
-        * @param $token \type{\string} Token
-        * @return \type{\string} Formatted URL
+        * @param $page \string Special page
+        * @param $token \string Token
+        * @return \string Formatted URL
         */
        protected function getTokenUrl( $page, $token ) {
                global $wgArticlePath;
@@ -2824,7 +2957,7 @@ class User {
 
        /**
         * Set the e-mail authentication timestamp.
-        * @param $timestamp \type{\string} TS_MW timestamp
+        * @param $timestamp \string TS_MW timestamp
         */
        function setEmailAuthenticationTimestamp( $timestamp ) {
                $this->load();
@@ -2835,9 +2968,13 @@ class User {
        /**
         * Is this user allowed to send e-mails within limits of current
         * site configuration?
-        * @return \type{\bool} True if allowed
+        * @return \bool True if allowed
         */
        function canSendEmail() {
+               global $wgEnableEmail, $wgEnableUserEmail;
+               if( !$wgEnableEmail || !$wgEnableUserEmail ) {
+                       return false;
+               }
                $canSend = $this->isEmailConfirmed();
                wfRunHooks( 'UserCanSendEmail', array( &$this, &$canSend ) );
                return $canSend;
@@ -2846,7 +2983,7 @@ class User {
        /**
         * Is this user allowed to receive e-mails within limits of current
         * site configuration?
-        * @return \type{\bool} True if allowed
+        * @return \bool True if allowed
         */
        function canReceiveEmail() {
                return $this->isEmailConfirmed() && !$this->getOption( 'disablemail' );
@@ -2860,7 +2997,7 @@ class User {
         * confirmed their address by returning a code or using a password
         * sent to the address from the wiki.
         *
-        * @return \type{\bool} True if confirmed
+        * @return \bool True if confirmed
         */
        function isEmailConfirmed() {
                global $wgEmailAuthentication;
@@ -2881,7 +3018,7 @@ class User {
 
        /**
         * Check whether there is an outstanding request for e-mail confirmation.
-        * @return \type{\bool} True if pending
+        * @return \bool True if pending
         */
        function isEmailConfirmationPending() {
                global $wgEmailAuthentication;
@@ -2894,14 +3031,32 @@ class User {
        /**
         * Get the timestamp of account creation.
         *
-        * @return \twotypes{\string,\bool} string Timestamp of account creation, or false for
+        * @return \types{\string,\bool} string Timestamp of account creation, or false for
         *                                non-existent/anonymous user accounts.
         */
        public function getRegistration() {
-               return $this->mId > 0
+               return $this->getId() > 0
                        ? $this->mRegistration
                        : false;
        }
+       
+       /**
+        * Get the timestamp of the first edit
+        *
+        * @return \types{\string,\bool} string Timestamp of first edit, or false for
+        *                                non-existent/anonymous user accounts.
+        */
+       public function getFirstEditTimestamp() {
+               if( $this->getId() == 0 ) return false; // anons
+               $dbr = wfGetDB( DB_SLAVE );
+               $time = $dbr->selectField( 'revision', 'rev_timestamp',
+                       array( 'rev_user' => $this->getId() ),
+                       __METHOD__,
+                       array( 'ORDER BY' => 'rev_timestamp ASC' )
+               );
+               if( !$time ) return false; // no edits
+               return wfTimestamp( TS_MW, $time );
+       }       
 
        /**
         * Get the permissions associated with a given list of groups
@@ -2915,16 +3070,17 @@ class User {
                foreach( $groups as $group ) {
                        if( isset( $wgGroupPermissions[$group] ) ) {
                                $rights = array_merge( $rights,
+                                       // array_filter removes empty items
                                        array_keys( array_filter( $wgGroupPermissions[$group] ) ) );
                        }
                }
-               return $rights;
+               return array_unique($rights);
        }
        
        /**
         * Get all the groups who have a given permission
         * 
-        * @param $role \type{\string} Role to check
+        * @param $role \string Role to check
         * @return \type{\arrayof{\string}} List of internal group names with the given permission
         */
        static function getGroupsWithPermission( $role ) {
@@ -2941,8 +3097,8 @@ class User {
        /**
         * Get the localized descriptive name for a group, if it exists
         *
-        * @param $group \type{\string} Internal group name
-        * @return \type{\string} Localized descriptive group name
+        * @param $group \string Internal group name
+        * @return \string Localized descriptive group name
         */
        static function getGroupName( $group ) {
                global $wgMessageCache;
@@ -2957,8 +3113,8 @@ class User {
        /**
         * Get the localized descriptive name for a member of a group, if it exists
         *
-        * @param $group \type{\string} Internal group name
-        * @return \type{\string} Localized name for group member
+        * @param $group \string Internal group name
+        * @return \string Localized name for group member
         */
        static function getGroupMember( $group ) {
                global $wgMessageCache;
@@ -3015,8 +3171,8 @@ class User {
        /**
         * Get the title of a page describing a particular group
         *
-        * @param $group \type{\string} Internal group name
-        * @return \twotypes{Title,\bool} Title of the page if it exists, false otherwise
+        * @param $group \string Internal group name
+        * @return \types{\type{Title},\bool} Title of the page if it exists, false otherwise
         */
        static function getGroupPage( $group ) {
                global $wgMessageCache;
@@ -3034,9 +3190,9 @@ class User {
         * Create a link to the group in HTML, if available; 
         * else return the group name.
         *
-        * @param $group \type{\string} Internal name of the group
-        * @param $text \type{\string} The text of the link
-        * @return \type{\string} HTML link to the group
+        * @param $group \string Internal name of the group
+        * @param $text \string The text of the link
+        * @return \string HTML link to the group
         */
        static function makeGroupLinkHTML( $group, $text = '' ) {
                if( $text == '' ) {
@@ -3046,7 +3202,7 @@ class User {
                if( $title ) {
                        global $wgUser;
                        $sk = $wgUser->getSkin();
-                       return $sk->makeLinkObj( $title, htmlspecialchars( $text ) );
+                       return $sk->link( $title, htmlspecialchars( $text ) );
                } else {
                        return $text;
                }
@@ -3056,9 +3212,9 @@ class User {
         * Create a link to the group in Wikitext, if available; 
         * else return the group name.
         *
-        * @param $group \type{\string} Internal name of the group
-        * @param $text \type{\string} The text of the link
-        * @return \type{\string} Wikilink to the group
+        * @param $group \string Internal name of the group
+        * @param $text \string The text of the link
+        * @return \string Wikilink to the group
         */
        static function makeGroupLinkWiki( $group, $text = '' ) {
                if( $text == '' ) {
@@ -3072,6 +3228,114 @@ class User {
                        return $text;
                }
        }
+       
+       /**
+        * Returns an array of the groups that a particular group can add/remove.
+        *
+        * @param $group String: the group to check for whether it can add/remove
+        * @return Array array( 'add' => array( addablegroups ),
+        *  'remove' => array( removablegroups ),
+        *  'add-self' => array( addablegroups to self),
+        *  'remove-self' => array( removable groups from self) )
+        */
+       static function changeableByGroup( $group ) {
+               global $wgAddGroups, $wgRemoveGroups, $wgGroupsAddToSelf, $wgGroupsRemoveFromSelf;
+
+               $groups = array( 'add' => array(), 'remove' => array(), 'add-self' => array(), 'remove-self' => array() );
+               if( empty($wgAddGroups[$group]) ) {
+                       // Don't add anything to $groups
+               } elseif( $wgAddGroups[$group] === true ) {
+                       // You get everything
+                       $groups['add'] = self::getAllGroups();
+               } elseif( is_array($wgAddGroups[$group]) ) {
+                       $groups['add'] = $wgAddGroups[$group];
+               }
+
+               // Same thing for remove
+               if( empty($wgRemoveGroups[$group]) ) {
+               } elseif($wgRemoveGroups[$group] === true ) {
+                       $groups['remove'] = self::getAllGroups();
+               } elseif( is_array($wgRemoveGroups[$group]) ) {
+                       $groups['remove'] = $wgRemoveGroups[$group];
+               }
+               
+               // Re-map numeric keys of AddToSelf/RemoveFromSelf to the 'user' key for backwards compatibility
+               if( empty($wgGroupsAddToSelf['user']) || $wgGroupsAddToSelf['user'] !== true ) {
+                       foreach($wgGroupsAddToSelf as $key => $value) {
+                               if( is_int($key) ) {
+                                       $wgGroupsAddToSelf['user'][] = $value;
+                               }
+                       }
+               }
+               
+               if( empty($wgGroupsRemoveFromSelf['user']) || $wgGroupsRemoveFromSelf['user'] !== true ) {
+                       foreach($wgGroupsRemoveFromSelf as $key => $value) {
+                               if( is_int($key) ) {
+                                       $wgGroupsRemoveFromSelf['user'][] = $value;
+                               }
+                       }
+               }
+               
+               // Now figure out what groups the user can add to him/herself
+               if( empty($wgGroupsAddToSelf[$group]) ) {
+               } elseif( $wgGroupsAddToSelf[$group] === true ) {
+                       // No idea WHY this would be used, but it's there
+                       $groups['add-self'] = User::getAllGroups();
+               } elseif( is_array($wgGroupsAddToSelf[$group]) ) {
+                       $groups['add-self'] = $wgGroupsAddToSelf[$group];
+               }
+               
+               if( empty($wgGroupsRemoveFromSelf[$group]) ) {
+               } elseif( $wgGroupsRemoveFromSelf[$group] === true ) {
+                       $groups['remove-self'] = User::getAllGroups();
+               } elseif( is_array($wgGroupsRemoveFromSelf[$group]) ) {
+                       $groups['remove-self'] = $wgGroupsRemoveFromSelf[$group];
+               }
+               
+               return $groups;
+       }
+       
+       /**
+        * Returns an array of groups that this user can add and remove
+        * @return Array array( 'add' => array( addablegroups ),
+        *  'remove' => array( removablegroups ),
+        *  'add-self' => array( addablegroups to self),
+        *  'remove-self' => array( removable groups from self) )
+        */
+       function changeableGroups() {
+               if( $this->isAllowed( 'userrights' ) ) {
+                       // This group gives the right to modify everything (reverse-
+                       // compatibility with old "userrights lets you change
+                       // everything")
+                       // Using array_merge to make the groups reindexed
+                       $all = array_merge( User::getAllGroups() );
+                       return array(
+                               'add' => $all,
+                               'remove' => $all,
+                               'add-self' => array(),
+                               'remove-self' => array()
+                       );
+               }
+
+               // Okay, it's not so simple, we will have to go through the arrays
+               $groups = array(
+                               'add' => array(),
+                               'remove' => array(),
+                               'add-self' => array(),
+                               'remove-self' => array() );
+               $addergroups = $this->getEffectiveGroups();
+
+               foreach ($addergroups as $addergroup) {
+                       $groups = array_merge_recursive(
+                               $groups, $this->changeableByGroup($addergroup)
+                       );
+                       $groups['add']    = array_unique( $groups['add'] );
+                       $groups['remove'] = array_unique( $groups['remove'] );
+                       $groups['add-self'] = array_unique( $groups['add-self'] );
+                       $groups['remove-self'] = array_unique( $groups['remove-self'] );
+               }
+               return $groups;
+       }
 
        /**
         * Increment the user's edit-count field.
@@ -3120,8 +3384,8 @@ class User {
        /**
         * Get the description of a given right
         *
-        * @param $right \type{\string} Right to query
-        * @return \type{\string} Localized description of the right
+        * @param $right \string Right to query
+        * @return \string Localized description of the right
         */
        static function getRightDescription( $right ) {
                global $wgMessageCache;
@@ -3136,9 +3400,9 @@ class User {
        /**
         * Make an old-style password hash
         *
-        * @param $password \type{\string} Plain-text password
-        * @param $userId \type{\string} %User ID
-        * @return \type{\string} Password hash
+        * @param $password \string Plain-text password
+        * @param $userId \string User ID
+        * @return \string Password hash
         */
        static function oldCrypt( $password, $userId ) {
                global $wgPasswordSalt;
@@ -3152,21 +3416,26 @@ class User {
        /**
         * Make a new-style password hash
         *
-        * @param $password \type{\string} Plain-text password
-        * @param $salt \type{\string} Optional salt, may be random or the user ID. 
+        * @param $password \string Plain-text password
+        * @param $salt \string Optional salt, may be random or the user ID. 
         *                     If unspecified or false, will generate one automatically
-        * @return \type{\string} Password hash
+        * @return \string Password hash
         */
        static function crypt( $password, $salt = false ) {
                global $wgPasswordSalt;
 
-               if($wgPasswordSalt) {
+               $hash = '';
+               if( !wfRunHooks( 'UserCryptPassword', array( &$password, &$salt, &$wgPasswordSalt, &$hash ) ) ) {
+                       return $hash;
+               }
+               
+               if( $wgPasswordSalt ) {
                        if ( $salt === false ) {
                                $salt = substr( wfGenerateToken(), 0, 8 );
                        }
                        return ':B:' . $salt . ':' . md5( $salt . '-' . md5( $password ) );
                } else {
-                       return ':A:' . md5( $password);
+                       return ':A:' . md5( $password );
                }
        }
 
@@ -3174,14 +3443,20 @@ class User {
         * Compare a password hash with a plain-text password. Requires the user
         * ID if there's a chance that the hash is an old-style hash.
         *
-        * @param $hash \type{\string} Password hash
-        * @param $password \type{\string} Plain-text password to compare
-        * @param $userId \type{\string} %User ID for old-style password salt
-        * @return \type{\bool} True if matches, false otherwise
+        * @param $hash \string Password hash
+        * @param $password \string Plain-text password to compare
+        * @param $userId \string User ID for old-style password salt
+        * @return \bool
         */
        static function comparePasswords( $hash, $password, $userId = false ) {
                $m = false;
                $type = substr( $hash, 0, 3 );
+               
+               $result = false;
+               if( !wfRunHooks( 'UserComparePasswords', array( &$hash, &$password, &$userId, &$result ) ) ) {
+                       return $result;
+               }
+               
                if ( $type == ':A:' ) {
                        # Unsalted
                        return md5( $password ) === substr( $hash, 3 );
@@ -3197,7 +3472,7 @@ class User {
        
        /**
         * Add a newuser log entry for this user
-        * @param bool $byEmail, account made by email?
+        * @param $byEmail Boolean: account made by email?
         */
        public function addNewUserLogEntry( $byEmail = false ) {
                global $wgUser, $wgContLang, $wgNewUserLog;
@@ -3221,7 +3496,7 @@ class User {
         * Add an autocreate newuser log entry for this user
         * Used by things like CentralAuth and perhaps other authplugins.
         */
-       public static function addNewUserLogEntryAutoCreate() {
+       public function addNewUserLogEntryAutoCreate() {
                global $wgNewUserLog;
                if( empty($wgNewUserLog) ) {
                        return true; // disabled
@@ -3230,5 +3505,72 @@ class User {
                $log->addEntry( 'autocreate', $this->getUserPage(), '', array( $this->getId() ) );
                return true;
        }
+       
+       protected function loadOptions() {
+               $this->load();
+               if ($this->mOptionsLoaded || !$this->getId() )
+                       return;
+       
+               $this->mOptions = self::getDefaultOptions();
+               
+               // Maybe load from the object
+               
+               if ( !is_null($this->mOptionOverrides) ) {
+                       wfDebug( "Loading options for user ".$this->getId()." from override cache.\n" ); 
+                       foreach( $this->mOptionOverrides as $key => $value ) {
+                               $this->mOptions[$key] = $value;
+                       }
+               } else {
+                       wfDebug( "Loading options for user ".$this->getId()." from database.\n" );
+                       // Load from database
+                       $dbr = wfGetDB( DB_SLAVE );
+                       
+                       $res = $dbr->select( 'user_properties',
+                                                                       '*',
+                                                                       array('up_user' => $this->getId()),
+                                                                       __METHOD__
+                                                               );
+                       
+                       while( $row = $dbr->fetchObject( $res ) ) {
+                               $this->mOptionOverrides[$row->up_property] = $row->up_value;
+                               $this->mOptions[$row->up_property] = $row->up_value;
+                       }
+               }
+               
+               $this->mOptionsLoaded = true;
+               
+               wfRunHooks( 'UserLoadOptions', array( $this, &$this->mOptions ) );
+       }
+       
+       protected function saveOptions() {
+               $this->loadOptions();
+               $dbw = wfGetDB( DB_MASTER );
+               
+               $insert_rows = array();
+               
+               $saveOptions = $this->mOptions;
+               
+               // 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) ) )
+                       return;
+               
+               foreach( $saveOptions as $key => $value ) {
+                       if ( ( is_null(self::getDefaultOption($key)) &&
+                                       !( $value === false || is_null($value) ) ) ||
+                                       $value != self::getDefaultOption( $key ) ) {
+                               $insert_rows[] = array(
+                                               'up_user' => $this->getId(),
+                                               'up_property' => $key,
+                                               'up_value' => $value,
+                                       );
+                       }
+               }
+               
+               $dbw->begin();
+               $dbw->delete( 'user_properties', array( 'up_user' => $this->getId() ), __METHOD__ );
+               $dbw->insert( 'user_properties', $insert_rows, __METHOD__ );
+               $dbw->commit();
+       }
 
 }