Add devunt to CREDITS as patch contributor
[lhc/web/wiklou.git] / includes / User.php
index c2af93c..6b7348a 100644 (file)
@@ -369,8 +369,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 String Username, validated by Title::newFromText()
-        * @param $validate String|Bool Validate username. Takes the same parameters as
+        * @param string $name Username, validated by Title::newFromText()
+        * @param string|Bool $validate Validate username. Takes the same parameters as
         *    User::getCanonicalName(), except that true is accepted as an alias
         *    for 'valid', for BC.
         *
@@ -399,7 +399,7 @@ class User {
        /**
         * Static factory method for creation from a given user ID.
         *
-        * @param $id Int Valid user ID
+        * @param int $id Valid user ID
         * @return User The corresponding User object
         */
        public static function newFromId( $id ) {
@@ -417,7 +417,7 @@ class User {
         *
         * If the code is invalid or has expired, returns NULL.
         *
-        * @param $code String Confirmation code
+        * @param string $code Confirmation code
         * @return User object, or null
         */
        public static function newFromConfirmationCode( $code ) {
@@ -437,8 +437,7 @@ 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.
+        * @param $request WebRequest object to use; $wgRequest will be used if omitted.
         * @return User object
         */
        public static function newFromSession( WebRequest $request = null ) {
@@ -458,8 +457,8 @@ class User {
         * 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
-        * @param $data Array Further data to load into the object (see User::loadFromRow for valid keys)
+        * @param array $row A row from the user table
+        * @param array $data Further data to load into the object (see User::loadFromRow for valid keys)
         * @return User
         */
        public static function newFromRow( $row, $data = null ) {
@@ -472,7 +471,7 @@ class User {
 
        /**
         * Get the username corresponding to a given user ID
-        * @param $id Int User ID
+        * @param int $id User ID
         * @return String|bool The corresponding username
         */
        public static function whoIs( $id ) {
@@ -482,7 +481,7 @@ class User {
        /**
         * Get the real name of a user given their user ID
         *
-        * @param $id Int User ID
+        * @param int $id User ID
         * @return String|bool The corresponding user's real name
         */
        public static function whoIsReal( $id ) {
@@ -491,7 +490,7 @@ class User {
 
        /**
         * Get database id given a user name
-        * @param $name String Username
+        * @param string $name Username
         * @return Int|Null The corresponding user's ID, or null if user is nonexistent
         */
        public static function idFromName( $name ) {
@@ -543,7 +542,7 @@ 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 String to match
+        * @param string $name to match
         * @return Bool
         */
        public static function isIP( $name ) {
@@ -554,11 +553,11 @@ class User {
         * Is the input a valid username?
         *
         * Checks if the input is a valid username, we don't want an empty string,
-        * an IP address, anything that containins slashes (would mess up subpages),
+        * an IP address, anything that contains slashes (would mess up subpages),
         * is longer than the maximum allowed username size or doesn't begin with
         * a capital letter.
         *
-        * @param $name String to match
+        * @param string $name to match
         * @return Bool
         */
        public static function isValidUserName( $name ) {
@@ -574,7 +573,6 @@ class User {
                        return false;
                }
 
-
                // Ensure that the name can't be misresolved as a different title,
                // such as with extra namespace keys at the start.
                $parsed = Title::newFromText( $name );
@@ -613,7 +611,7 @@ class User {
         * If an account already exists in this form, login will be blocked
         * by a failure to pass this function.
         *
-        * @param $name String to match
+        * @param string $name to match
         * @return Bool
         */
        public static function isUsableName( $name ) {
@@ -650,7 +648,7 @@ class User {
         * Additional blacklisting may be added here rather than in
         * isValidUserName() to avoid disrupting existing accounts.
         *
-        * @param $name String to match
+        * @param string $name to match
         * @return Bool
         */
        public static function isCreatableName( $name ) {
@@ -680,7 +678,7 @@ class User {
        /**
         * Is the input a valid password for this user?
         *
-        * @param $password String Desired password
+        * @param string $password Desired password
         * @return Bool
         */
        public function isValidPassword( $password ) {
@@ -691,7 +689,7 @@ class User {
        /**
         * Given unvalidated password input, return error message on failure.
         *
-        * @param $password String Desired password
+        * @param string $password Desired password
         * @return mixed: true on success, string or array of error message on failure
         */
        public function getPasswordValidity( $password ) {
@@ -751,7 +749,7 @@ class User {
         * to be liberal enough for wide use. Some invalid addresses will still
         * pass validation here.
         *
-        * @param $addr String E-mail address
+        * @param string $addr E-mail address
         * @return Bool
         * @deprecated since 1.18 call Sanitizer::isValidEmail() directly
         */
@@ -763,8 +761,8 @@ class User {
        /**
         * Given unvalidated user input, return a canonical username, or false if
         * the username is invalid.
-        * @param $name String User input
-        * @param $validate String|Bool type of validation to use:
+        * @param string $name User input
+        * @param string|Bool $validate type of validation to use:
         *                - false        No validation
         *                - 'valid'      Valid for batch processes
         *                - 'usable'     Valid for batch processes and login
@@ -823,7 +821,7 @@ class User {
        /**
         * Count the number of edits of a user
         *
-        * @param $uid Int User ID to check
+        * @param int $uid User ID to check
         * @return Int the user's edit count
         *
         * @deprecated since 1.21 in favour of User::getEditCount
@@ -857,7 +855,7 @@ class User {
         * @note This no longer clears uncached lazy-initialised properties;
         *       the constructor does that instead.
         *
-        * @param $name string
+        * @param $name string|bool
         */
        public function loadDefaults( $name = false ) {
                wfProfileIn( __METHOD__ );
@@ -893,11 +891,11 @@ class User {
        /**
         * Return whether an item has been loaded.
         *
-        * @param $item String: item to check. Current possibilities:
+        * @param string $item item to check. Current possibilities:
         *              - id
         *              - name
         *              - realname
-        * @param $all String: 'all' to check if the whole object has been loaded
+        * @param string $all '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
@@ -980,10 +978,13 @@ class User {
                }
 
                if ( $request->getSessionData( 'wsToken' ) ) {
-                       $passwordCorrect = $proposedUser->getToken( false ) === $request->getSessionData( 'wsToken' );
+                       $passwordCorrect = ( $proposedUser->getToken( false ) === $request->getSessionData( 'wsToken' ) );
                        $from = 'session';
                } elseif ( $request->getCookie( 'Token' ) ) {
-                       $passwordCorrect = $proposedUser->getToken( false ) === $request->getCookie( 'Token' );
+                       # Get the token from DB/cache and clean it up to remove garbage padding.
+                       # This deals with historical problems with bugs and the default column value.
+                       $token = rtrim( $proposedUser->getToken( false ) ); // correct token
+                       $passwordCorrect = ( strlen( $token ) && $token === $request->getCookie( 'Token' ) );
                        $from = 'cookie';
                } else {
                        # No session or persistent login cookie
@@ -1040,8 +1041,8 @@ class User {
        /**
         * Initialize this object from a row from the user table.
         *
-        * @param $row Array Row from the user table to load.
-        * @param $data Array Further user data to load into the object
+        * @param array $row Row from the user table to load.
+        * @param array $data Further user data to load into the object
         *
         *      user_groups             Array with groups out of the user_groups table
         *      user_properties         Array with properties out of the user_properties table
@@ -1154,7 +1155,7 @@ class User {
         *   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)
+        * @param string $event key in $wgAutopromoteOnce (each one has groups/criteria)
         *
         * @return array Array of groups the user has been promoted to.
         *
@@ -1193,7 +1194,7 @@ class User {
         * Clear various cached data stored in this object. The cache of the user table
         * data (i.e. self::$mCacheVars) is not cleared unless $reloadFrom is given.
         *
-        * @param $reloadFrom bool|String Reload user and user_groups table data from a
+        * @param bool|String $reloadFrom Reload user and user_groups table data from a
         *   given source. May be "name", "id", "defaults", "session", or false for
         *   no reload.
         */
@@ -1250,7 +1251,7 @@ class User {
        /**
         * Get a given default option value.
         *
-        * @param $opt String Name of option to retrieve
+        * @param string $opt Name of option to retrieve
         * @return String Default option value
         */
        public static function getDefaultOption( $opt ) {
@@ -1262,10 +1263,9 @@ class User {
                }
        }
 
-
        /**
         * Get blocking information
-        * @param $bFromSlave Bool Whether to check the slave database first. To
+        * @param bool $bFromSlave 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.
@@ -1339,8 +1339,8 @@ class User {
        /**
         * Whether the given IP is in a DNS blacklist.
         *
-        * @param $ip String IP to check
-        * @param $checkWhitelist Bool: whether to check the whitelist first
+        * @param string $ip IP to check
+        * @param bool $checkWhitelist whether to check the whitelist first
         * @return Bool True if blacklisted.
         */
        public function isDnsBlacklisted( $ip, $checkWhitelist = false ) {
@@ -1360,8 +1360,8 @@ class User {
        /**
         * Whether the given IP is in a given DNS blacklist.
         *
-        * @param $ip String IP to check
-        * @param $bases String|Array of Strings: URL of the DNS blacklist
+        * @param string $ip IP to check
+        * @param string|array $bases of Strings: URL of the DNS blacklist
         * @return Bool True if blacklisted.
         */
        public function inDnsBlacklist( $ip, $bases ) {
@@ -1461,7 +1461,7 @@ class User {
         * @note When using a shared cache like memcached, IP-address
         * last-hit counters will be shared across wikis.
         *
-        * @param $action String Action to enforce; 'edit' if unspecified
+        * @param string $action Action to enforce; 'edit' if unspecified
         * @return Bool True if a rate limiter was tripped
         */
        public function pingLimiter( $action = 'edit' ) {
@@ -1556,7 +1556,7 @@ class User {
        /**
         * Check if user is blocked
         *
-        * @param $bFromSlave Bool Whether to check the slave database instead of the master
+        * @param bool $bFromSlave Whether to check the slave database instead of the master
         * @return Bool True if blocked, false otherwise
         */
        public function isBlocked( $bFromSlave = true ) { // hacked from false due to horrible probs on site
@@ -1566,7 +1566,7 @@ class User {
        /**
         * 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
+        * @param bool $bFromSlave Whether to check the slave database instead of the master
         * @return Block|null
         */
        public function getBlock( $bFromSlave = true ) {
@@ -1578,7 +1578,7 @@ class User {
         * Check if user is blocked from editing a particular article
         *
         * @param $title Title to check
-        * @param $bFromSlave Bool whether to check the slave database instead of the master
+        * @param bool $bFromSlave whether to check the slave database instead of the master
         * @return Bool
         */
        function isBlockedFrom( $title, $bFromSlave = false ) {
@@ -1630,9 +1630,9 @@ class User {
        /**
         * Check if user is blocked on all wikis.
         * Do not use for actual edit permission checks!
-        * This is intented for quick UI checks.
+        * This is intended for quick UI checks.
         *
-        * @param $ip String IP address, uses current client if none given
+        * @param string $ip IP address, uses current client if none given
         * @return Bool True if blocked, false otherwise
         */
        public function isBlockedGlobally( $ip = '' ) {
@@ -1702,7 +1702,7 @@ class User {
 
        /**
         * Set the user and reload all fields according to a given ID
-        * @param $v Int User ID to reload
+        * @param int $v User ID to reload
         */
        public function setId( $v ) {
                $this->mId = $v;
@@ -1736,9 +1736,9 @@ class User {
         * address for an anonymous user to something other than the current
         * remote IP.
         *
-        * @note User::newFromName() has rougly the same function, when the named user
+        * @note User::newFromName() has roughly the same function, when the named user
         * does not exist.
-        * @param $str String New user name to set
+        * @param string $str New user name to set
         */
        public function setName( $str ) {
                $this->load();
@@ -1818,9 +1818,9 @@ class User {
         * Internal uncached check for new messages
         *
         * @see getNewtalk()
-        * @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
-        * @param $fromMaster Bool true to fetch from the master, false for a slave
+        * @param string $field 'user_ip' for anonymous users, 'user_id' otherwise
+        * @param string|Int $id User's IP address for anonymous users, User ID otherwise
+        * @param bool $fromMaster true to fetch from the master, false for a slave
         * @return Bool True if the user has new messages
         */
        protected function checkNewtalk( $field, $id, $fromMaster = false ) {
@@ -1836,8 +1836,8 @@ class User {
 
        /**
         * Add or update the new messages flag
-        * @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
+        * @param string $field 'user_ip' for anonymous users, 'user_id' otherwise
+        * @param string|Int $id User's IP address for anonymous users, User ID otherwise
         * @param $curRev Revision new, as yet unseen revision of the user talk page. Ignored if null.
         * @return Bool True if successful, false otherwise
         */
@@ -1862,8 +1862,8 @@ class User {
 
        /**
         * Clear the new messages flag for the given 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
+        * @param string $field 'user_ip' for anonymous users, 'user_id' otherwise
+        * @param string|Int $id User's IP address for anonymous users, User ID otherwise
         * @return Bool True if successful, false otherwise
         */
        protected function deleteNewtalk( $field, $id ) {
@@ -1882,7 +1882,7 @@ class User {
 
        /**
         * Update the 'You have new messages!' status.
-        * @param $val Bool Whether the user has new messages
+        * @param bool $val Whether the user has new messages
         * @param $curRev Revision new, as yet unseen revision of the user talk page. Ignored if null or !$val.
         */
        public function setNewtalk( $val, $curRev = null ) {
@@ -1978,7 +1978,7 @@ class User {
 
        /**
         * Validate the cache for this account.
-        * @param $timestamp String A timestamp in TS_MW format
+        * @param string $timestamp A timestamp in TS_MW format
         *
         * @return bool
         */
@@ -2007,7 +2007,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 String New password to set
+        * @param string $str New password to set
         * @throws PasswordError on failure
         *
         * @return bool
@@ -2046,7 +2046,9 @@ class User {
        /**
         * Set the password and reset the random token unconditionally.
         *
-        * @param $str String New password to set
+        * @param string|null $str New password to set or null to set an invalid
+        *        password hash meaning that the user will not be able to log in
+        *        through the web interface.
         */
        public function setInternalPassword( $str ) {
                $this->load();
@@ -2064,7 +2066,7 @@ class User {
 
        /**
         * Get the user's current token.
-        * @param $forceCreation Force the generation of a new token if the user doesn't have one (default=true for backwards compatibility)
+        * @param bool $forceCreation Force the generation of a new token if the user doesn't have one (default=true for backwards compatibility)
         * @return String Token
         */
        public function getToken( $forceCreation = true ) {
@@ -2079,7 +2081,7 @@ class User {
         * Set the random token (used for persistent authentication)
         * Called from loadDefaults() among other places.
         *
-        * @param $token String|bool If specified, set the token to this value
+        * @param string|bool $token If specified, set the token to this value
         */
        public function setToken( $token = false ) {
                $this->load();
@@ -2093,8 +2095,8 @@ class User {
        /**
         * Set the password for a password reminder or new account email
         *
-        * @param $str String New password to set
-        * @param $throttle Bool If true, reset the throttle timestamp to the present
+        * @param string $str New password to set
+        * @param bool $throttle If true, reset the throttle timestamp to the present
         */
        public function setNewpassword( $str, $throttle = true ) {
                $this->load();
@@ -2141,7 +2143,7 @@ class User {
 
        /**
         * Set the user's e-mail address
-        * @param $str String New e-mail address
+        * @param string $str New e-mail address
         */
        public function setEmail( $str ) {
                $this->load();
@@ -2157,7 +2159,7 @@ class User {
         * Set the user's e-mail address and a confirmation mail if needed.
         *
         * @since 1.20
-        * @param $str String New e-mail address
+        * @param string $str New e-mail address
         * @return Status
         */
        public function setEmailWithConfirmation( $str ) {
@@ -2203,7 +2205,7 @@ class User {
 
        /**
         * Set the user's real name
-        * @param $str String New real name
+        * @param string $str New real name
         */
        public function setRealName( $str ) {
                $this->load();
@@ -2213,9 +2215,9 @@ class User {
        /**
         * Get the user's current setting for a given option.
         *
-        * @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
+        * @param string $oname The option to check
+        * @param string $defaultOverride A default value returned if the option does not exist
+        * @param bool $ignoreHidden = whether to ignore the effects of $wgHiddenPrefs
         * @return String User's current value for the option
         * @see getBoolOption()
         * @see getIntOption()
@@ -2268,7 +2270,7 @@ class User {
        /**
         * Get the user's current setting for a given option, as a boolean value.
         *
-        * @param $oname String The option to check
+        * @param string $oname The option to check
         * @return Bool User's current value for the option
         * @see getOption()
         */
@@ -2279,8 +2281,8 @@ class User {
        /**
         * Get the user's current setting for a given option, as a boolean value.
         *
-        * @param $oname String The option to check
-        * @param $defaultOverride Int A default value returned if the option does not exist
+        * @param string $oname The option to check
+        * @param int $defaultOverride A default value returned if the option does not exist
         * @return Int User's current value for the option
         * @see getOption()
         */
@@ -2295,7 +2297,7 @@ class User {
        /**
         * Set the given option for a user.
         *
-        * @param $oname String The option to set
+        * @param string $oname The option to set
         * @param $val mixed New value to set
         */
        public function setOption( $oname, $val ) {
@@ -2317,6 +2319,7 @@ class User {
         * - 'registered' - preferences which are registered in core MediaWiki or
         *                  by extensions using the UserGetDefaultOptions hook.
         * - 'registered-multiselect' - as above, using the 'multiselect' type.
+        * - 'registered-checkmatrix' - as above, using the 'checkmatrix' type.
         * - 'userjs' - preferences with names starting with 'userjs-', intended to
         *              be used by user scripts.
         * - 'unused' - preferences about which MediaWiki doesn't know anything.
@@ -2333,6 +2336,7 @@ class User {
                return array(
                        'registered',
                        'registered-multiselect',
+                       'registered-checkmatrix',
                        'userjs',
                        'unused'
                );
@@ -2346,7 +2350,7 @@ class User {
         *
         * @see User::listOptionKinds
         * @param $context IContextSource
-        * @param $options array assoc. array with options keys to check as keys. Defaults to $this->mOptions.
+        * @param array $options assoc. array with options keys to check as keys. Defaults to $this->mOptions.
         * @return array the key => kind mapping data
         */
        public function getOptionKinds( IContextSource $context, $options = null ) {
@@ -2358,8 +2362,8 @@ class User {
                $prefs = Preferences::getPreferences( $this, $context );
                $mapping = array();
 
-               // Multiselect options are stored in the database with one key per
-               // option, each having a boolean value. Extract those keys.
+               // Multiselect and checkmatrix options are stored in the database with
+               // one key per option, each having a boolean value. Extract those keys.
                $multiselectOptions = array();
                foreach ( $prefs as $name => $info ) {
                        if ( ( isset( $info['type'] ) && $info['type'] == 'multiselect' ) ||
@@ -2374,6 +2378,23 @@ class User {
                                unset( $prefs[$name] );
                        }
                }
+               $checkmatrixOptions = array();
+               foreach ( $prefs as $name => $info ) {
+                       if ( ( isset( $info['type'] ) && $info['type'] == 'checkmatrix' ) ||
+                                       ( isset( $info['class'] ) && $info['class'] == 'HTMLCheckMatrix' ) ) {
+                               $columns = HTMLFormField::flattenOptions( $info['columns'] );
+                               $rows = HTMLFormField::flattenOptions( $info['rows'] );
+                               $prefix = isset( $info['prefix'] ) ? $info['prefix'] : $name;
+
+                               foreach ( $columns as $column ) {
+                                       foreach ( $rows as $row ) {
+                                               $checkmatrixOptions["$prefix-$column-$row"] = true;
+                                       }
+                               }
+
+                               unset( $prefs[$name] );
+                       }
+               }
 
                // $value is ignored
                foreach ( $options as $key => $value ) {
@@ -2381,6 +2402,8 @@ class User {
                                $mapping[$key] = 'registered';
                        } elseif( isset( $multiselectOptions[$key] ) ) {
                                $mapping[$key] = 'registered-multiselect';
+                       } elseif( isset( $checkmatrixOptions[$key] ) ) {
+                               $mapping[$key] = 'registered-checkmatrix';
                        } elseif ( substr( $key, 0, 7 ) === 'userjs-' ) {
                                $mapping[$key] = 'userjs';
                        } else {
@@ -2398,15 +2421,15 @@ class User {
         * Supported values are everything that can be reported by getOptionKinds()
         * and 'all', which forces a reset of *all* preferences and overrides everything else.
         *
-        * @param $resetKinds array|string which kinds of preferences to reset. Defaults to
-        *                                 array( 'registered', 'registered-multiselect', 'unused' )
-        *                                 for backwards-compatibility.
+        * @param array|string $resetKinds which kinds of preferences to reset. Defaults to
+        *             array( 'registered', 'registered-multiselect', 'registered-checkmatrix', 'unused' )
+        *             for backwards-compatibility.
         * @param $context IContextSource|null context source used when $resetKinds
-        *                                     does not contain 'all', passed to getOptionKinds().
-        *                                     Defaults to RequestContext::getMain() when null.
+        *             does not contain 'all', passed to getOptionKinds().
+        *             Defaults to RequestContext::getMain() when null.
         */
        public function resetOptions(
-               $resetKinds = array( 'registered', 'registered-multiselect', 'unused' ),
+               $resetKinds = array( 'registered', 'registered-multiselect', 'registered-checkmatrix', 'unused' ),
                IContextSource $context = null
        ) {
                $this->load();
@@ -2469,7 +2492,7 @@ class User {
         */
        public function getStubThreshold() {
                global $wgMaxArticleSize; # Maximum article size, in Kb
-               $threshold = intval( $this->getOption( 'stubthreshold' ) );
+               $threshold = $this->getIntOption( 'stubthreshold' );
                if ( $threshold > $wgMaxArticleSize * 1024 ) {
                        # If they have set an impossible value, disable the preference
                        # so we can use the parser cache again.
@@ -2507,7 +2530,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 Bool Whether to avoid the cache
+        * @param bool $recache Whether to avoid the cache
         * @return Array of String internal group names
         */
        public function getEffectiveGroups( $recache = false ) {
@@ -2530,7 +2553,7 @@ class User {
         * 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
+        * @param bool $recache Whether to avoid the cache
         * @return Array of String internal group names
         */
        public function getAutomaticGroups( $recache = false ) {
@@ -2612,7 +2635,7 @@ class User {
        /**
         * Add the user to the given group.
         * This takes immediate effect.
-        * @param $group String Name of the group to add
+        * @param string $group Name of the group to add
         */
        public function addGroup( $group ) {
                if( wfRunHooks( 'UserAddGroup', array( $this, &$group ) ) ) {
@@ -2637,7 +2660,7 @@ class User {
        /**
         * Remove the user from the given group.
         * This takes immediate effect.
-        * @param $group String Name of the group to remove
+        * @param string $group Name of the group to remove
         */
        public function removeGroup( $group ) {
                $this->load();
@@ -2900,7 +2923,7 @@ class User {
 
        /**
         * Set this user's options from an encoded string
-        * @param $str String Encoded options to import
+        * @param string $str Encoded options to import
         *
         * @deprecated in 1.19 due to removal of user_options from the user table
         */
@@ -2928,9 +2951,9 @@ 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;
+        * @param string $name Name of the cookie to set
+        * @param string $value Value to set
+        * @param int $exp Expiration time, as a UNIX time value;
         *                   if 0 or not specified, use the default $wgCookieExpiration
         * @param $secure Bool
         *  true: Force setting the secure attribute when setting the cookie
@@ -2943,7 +2966,7 @@ class User {
 
        /**
         * Clear a cookie on the user's client
-        * @param $name String Name of the cookie to clear
+        * @param string $name Name of the cookie to clear
         */
        protected function clearCookie( $name ) {
                $this->setCookie( $name, '', time() - 86400 );
@@ -2954,7 +2977,7 @@ class User {
         *
         * @param $request WebRequest object to use; $wgRequest will be used if null
         *        is passed.
-        * @param $secure Whether to force secure/insecure cookies or use default
+        * @param bool $secure Whether to force secure/insecure cookies or use default
         */
        public function setCookies( $request = null, $secure = null ) {
                if ( $request === null ) {
@@ -2962,7 +2985,9 @@ class User {
                }
 
                $this->load();
-               if ( 0 == $this->mId ) return;
+               if ( 0 == $this->mId ) {
+                       return;
+               }
                if ( !$this->mToken ) {
                        // When token is empty or NULL generate a new one and then save it to the database
                        // This allows a wiki to re-secure itself after a leak of it's user table or $wgSecretKey
@@ -3096,8 +3121,8 @@ 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 as user_* fields:
+        * @param string $name Username to add
+        * @param array $params 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
@@ -3112,6 +3137,7 @@ class User {
        public static function createNew( $name, $params = array() ) {
                $user = new User;
                $user->load();
+               $user->setToken(); // init token
                if ( isset( $params['options'] ) ) {
                        $user->mOptions = $params['options'] + (array)$user->mOptions;
                        unset( $params['options'] );
@@ -3173,6 +3199,9 @@ class User {
         */
        public function addToDatabase() {
                $this->load();
+               if ( !$this->mToken ) {
+                       $this->setToken(); // init token
+               }
 
                $this->mTouched = self::newTouchedTimestamp();
 
@@ -3277,7 +3306,7 @@ class User {
                // since it disables the parser cache, its value will always
                // be 0 when this function is called by parsercache.
 
-               $confstr =        $this->getOption( 'math' );
+               $confstr = $this->getOption( 'math' );
                $confstr .= '!' . $this->getStubThreshold();
                $confstr .= '!' . ( $this->getOption( 'numberheadings' ) ? '1' : '' );
                $confstr .= '!' . $wgLang->getCode();
@@ -3369,7 +3398,7 @@ class User {
 
        /**
         * Check to see if the given clear-text password is one of the accepted passwords
-        * @param $password String: user password.
+        * @param string $password user password.
         * @return Boolean: True if the given password is correct, otherwise False.
         */
        public function checkPassword( $password ) {
@@ -3436,7 +3465,7 @@ class User {
         * Alias for getEditToken.
         * @deprecated since 1.19, use getEditToken instead.
         *
-        * @param $salt String|Array of Strings Optional function-specific data for hashing
+        * @param string|array $salt 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
         */
@@ -3453,7 +3482,7 @@ class User {
         *
         * @since 1.19
         *
-        * @param $salt String|Array of Strings Optional function-specific data for hashing
+        * @param string|array $salt 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
         */
@@ -3480,11 +3509,10 @@ class User {
        /**
         * Generate a looking random token for various uses.
         *
-        * @param $salt String Optional salt value
         * @return String The new random token
-        * @deprecated since 1.20; Use MWCryptRand for secure purposes or wfRandomString for pesudo-randomness
+        * @deprecated since 1.20; Use MWCryptRand for secure purposes or wfRandomString for pseudo-randomness
         */
-       public static function generateToken( $salt = '' ) {
+       public static function generateToken() {
                return MWCryptRand::generateHex( 32 );
        }
 
@@ -3494,8 +3522,8 @@ class User {
         * user's own login session, not a form submission from a third-party
         * site.
         *
-        * @param $val String Input value to compare
-        * @param $salt String Optional function-specific data for hashing
+        * @param string $val Input value to compare
+        * @param string $salt Optional function-specific data for hashing
         * @param $request WebRequest object to use or null to use $wgRequest
         * @return Boolean: Whether the token matches
         */
@@ -3511,8 +3539,8 @@ class User {
         * Check given value against the token value stored in the session,
         * ignoring the suffix.
         *
-        * @param $val String Input value to compare
-        * @param $salt String Optional function-specific data for hashing
+        * @param string $val Input value to compare
+        * @param string $salt Optional function-specific data for hashing
         * @param $request WebRequest object to use or null to use $wgRequest
         * @return Boolean: Whether the token matches
         */
@@ -3525,7 +3553,7 @@ class User {
         * Generate a new e-mail confirmation token and send a confirmation/invalidation
         * mail to the user's given address.
         *
-        * @param $type String: message to send, either "created", "changed" or "set"
+        * @param string $type message to send, either "created", "changed" or "set"
         * @return Status object
         */
        public function sendConfirmationMail( $type = 'created' ) {
@@ -3559,10 +3587,10 @@ class User {
         * Send an e-mail to this user's account. Does not check for
         * confirmed status or validity.
         *
-        * @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
+        * @param string $subject Message subject
+        * @param string $body Message body
+        * @param string $from Optional From address; if unspecified, default $wgPasswordSender will be used
+        * @param string $replyto Reply-To address
         * @return Status
         */
        public function sendMail( $subject, $body, $from = null, $replyto = null ) {
@@ -3601,8 +3629,8 @@ class User {
        }
 
        /**
-       * Return a URL the user can use to confirm their email address.
-        * @param $token String Accepts the email confirmation token
+        * Return a URL the user can use to confirm their email address.
+        * @param string $token Accepts the email confirmation token
         * @return String New token URL
         */
        private function confirmationTokenUrl( $token ) {
@@ -3611,7 +3639,7 @@ class User {
 
        /**
         * Return a URL the user can use to invalidate their email address.
-        * @param $token String Accepts the email confirmation token
+        * @param string $token Accepts the email confirmation token
         * @return String New token URL
         */
        private function invalidationTokenUrl( $token ) {
@@ -3628,8 +3656,8 @@ class User {
         * also sometimes can get corrupted in some browsers/mailers
         * (bug 6957 with Gmail and Internet Explorer).
         *
-        * @param $page String Special page
-        * @param $token String Token
+        * @param string $page Special page
+        * @param string $token Token
         * @return String Formatted URL
         */
        protected function getTokenUrl( $page, $token ) {
@@ -3669,7 +3697,7 @@ class User {
 
        /**
         * Set the e-mail authentication timestamp.
-        * @param $timestamp String TS_MW timestamp
+        * @param string $timestamp TS_MW timestamp
         */
        function setEmailAuthenticationTimestamp( $timestamp ) {
                $this->load();
@@ -3746,8 +3774,9 @@ class User {
        /**
         * Get the timestamp of account creation.
         *
-        * @return String|Bool Timestamp of account creation, or false for
-        *     non-existent/anonymous user accounts.
+        * @return String|Bool|Null Timestamp of account creation, false for
+        *     non-existent/anonymous user accounts, or null if existing account
+        *     but information is not in database.
         */
        public function getRegistration() {
                if ( $this->isAnon() ) {
@@ -3782,7 +3811,7 @@ class User {
        /**
         * Get the permissions associated with a given list of groups
         *
-        * @param $groups Array of Strings List of internal group names
+        * @param array $groups of Strings List of internal group names
         * @return Array of Strings List of permission key names for given groups combined
         */
        public static function getGroupPermissions( $groups ) {
@@ -3809,7 +3838,7 @@ class User {
        /**
         * Get all the groups who have a given permission
         *
-        * @param $role String Role to check
+        * @param string $role Role to check
         * @return Array of Strings List of internal group names with the given permission
         */
        public static function getGroupsWithPermission( $role ) {
@@ -3826,8 +3855,8 @@ class User {
        /**
         * Check, if the given group has the given permission
         *
-        * @param $group String Group to check
-        * @param $role String Role to check
+        * @param string $group Group to check
+        * @param string $role Role to check
         * @return bool
         */
        public static function groupHasPermission( $group, $role ) {
@@ -3839,7 +3868,7 @@ class User {
        /**
         * Get the localized descriptive name for a group, if it exists
         *
-        * @param $group String Internal group name
+        * @param string $group Internal group name
         * @return String Localized descriptive group name
         */
        public static function getGroupName( $group ) {
@@ -3850,8 +3879,8 @@ class User {
        /**
         * Get the localized descriptive name for a member of a group, if it exists
         *
-        * @param $group String Internal group name
-        * @param $username String Username for gender (since 1.19)
+        * @param string $group Internal group name
+        * @param string $username Username for gender (since 1.19)
         * @return String Localized name for group member
         */
        public static function getGroupMember( $group, $username = '#' ) {
@@ -3904,7 +3933,7 @@ class User {
        /**
         * Get the title of a page describing a particular group
         *
-        * @param $group String Internal group name
+        * @param string $group Internal group name
         * @return Title|Bool Title of the page if it exists, false otherwise
         */
        public static function getGroupPage( $group ) {
@@ -3921,8 +3950,8 @@ class User {
         * Create a link to the group in HTML, if available;
         * else return the group name.
         *
-        * @param $group String Internal name of the group
-        * @param $text String The text of the link
+        * @param string $group Internal name of the group
+        * @param string $text The text of the link
         * @return String HTML link to the group
         */
        public static function makeGroupLinkHTML( $group, $text = '' ) {
@@ -3941,8 +3970,8 @@ class User {
         * Create a link to the group in Wikitext, if available;
         * else return the group name.
         *
-        * @param $group String Internal name of the group
-        * @param $text String The text of the link
+        * @param string $group Internal name of the group
+        * @param string $text The text of the link
         * @return String Wikilink to the group
         */
        public static function makeGroupLinkWiki( $group, $text = '' ) {
@@ -3961,7 +3990,7 @@ class User {
        /**
         * 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
+        * @param string $group 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),
@@ -4134,7 +4163,7 @@ class User {
        /**
         * Get the description of a given right
         *
-        * @param $right String Right to query
+        * @param string $right Right to query
         * @return String Localized description of the right
         */
        public static function getRightDescription( $right ) {
@@ -4146,8 +4175,8 @@ class User {
        /**
         * Make an old-style password hash
         *
-        * @param $password String Plain-text password
-        * @param $userId String User ID
+        * @param string $password Plain-text password
+        * @param string $userId User ID
         * @return String Password hash
         */
        public static function oldCrypt( $password, $userId ) {
@@ -4162,7 +4191,7 @@ class User {
        /**
         * Make a new-style password hash
         *
-        * @param $password String Plain-text password
+        * @param string $password Plain-text password
         * @param bool|string $salt Optional salt, may be random or the user ID.
 
         *                     If unspecified or false, will generate one automatically
@@ -4190,9 +4219,9 @@ 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 String Password hash
-        * @param $password String Plain-text password to compare
-        * @param $userId String|bool User ID for old-style password salt
+        * @param string $hash Password hash
+        * @param string $password Plain-text password to compare
+        * @param string|bool $userId User ID for old-style password salt
         *
         * @return Boolean
         */
@@ -4221,7 +4250,7 @@ class User {
         * Add a newuser log entry for this user.
         * Before 1.19 the return value was always true.
         *
-        * @param $action string|bool: account creation type.
+        * @param string|bool $action account creation type.
         *   - String, one of the following values:
         *     - 'create' for an anonymous user creating an account for himself.
         *       This will force the action's performer to be the created user itself,
@@ -4233,7 +4262,7 @@ class User {
         *     - false will be converted to 'create' if this object is the same as
         *       $wgUser and to 'create2' otherwise
         *
-        * @param $reason String: user supplied reason
+        * @param string $reason user supplied reason
         *
         * @return int|bool True if not $wgNewUserLog; otherwise ID of log item or 0 on failure
         */
@@ -4291,7 +4320,7 @@ class User {
        /**
         * Load the user options either from cache, the database or an array
         *
-        * @param $data Rows for the current user out of the user_properties table
+        * @param array $data Rows for the current user out of the user_properties table
         */
        protected function loadOptions( $data = null ) {
                global $wgContLang;