]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/User.php
Merge pull request #6989 from nupplaphil/task/basePath_baseUrl_fix
[friendica.git] / src / Model / User.php
index 65c7d8c9738747167ee66270fbd2e650aa50abee..472ed09247846ea8cf39a0131ca8c68969fcb68b 100644 (file)
@@ -7,8 +7,8 @@ namespace Friendica\Model;
 
 use DivineOmega\PasswordExposed;
 use Exception;
-use Friendica\Core\Addon;
 use Friendica\Core\Config;
+use Friendica\Core\Hook;
 use Friendica\Core\L10n;
 use Friendica\Core\Logger;
 use Friendica\Core\PConfig;
@@ -23,20 +23,66 @@ use Friendica\Util\Network;
 use Friendica\Util\Strings;
 use LightOpenID;
 
-require_once 'boot.php';
-require_once 'include/dba.php';
-require_once 'include/enotify.php';
-require_once 'include/text.php';
 /**
  * @brief This class handles User related functions
  */
 class User
 {
+       /**
+        * Page/profile types
+        *
+        * PAGE_FLAGS_NORMAL is a typical personal profile account
+        * PAGE_FLAGS_SOAPBOX automatically approves all friend requests as Contact::SHARING, (readonly)
+        * PAGE_FLAGS_COMMUNITY automatically approves all friend requests as Contact::SHARING, but with
+        *      write access to wall and comments (no email and not included in page owner's ACL lists)
+        * PAGE_FLAGS_FREELOVE automatically approves all friend requests as full friends (Contact::FRIEND).
+        *
+        * @{
+        */
+       const PAGE_FLAGS_NORMAL    = 0;
+       const PAGE_FLAGS_SOAPBOX   = 1;
+       const PAGE_FLAGS_COMMUNITY = 2;
+       const PAGE_FLAGS_FREELOVE  = 3;
+       const PAGE_FLAGS_BLOG      = 4;
+       const PAGE_FLAGS_PRVGROUP  = 5;
+       /**
+        * @}
+        */
+
+       /**
+        * Account types
+        *
+        * ACCOUNT_TYPE_PERSON - the account belongs to a person
+        *      Associated page types: PAGE_FLAGS_NORMAL, PAGE_FLAGS_SOAPBOX, PAGE_FLAGS_FREELOVE
+        *
+        * ACCOUNT_TYPE_ORGANISATION - the account belongs to an organisation
+        *      Associated page type: PAGE_FLAGS_SOAPBOX
+        *
+        * ACCOUNT_TYPE_NEWS - the account is a news reflector
+        *      Associated page type: PAGE_FLAGS_SOAPBOX
+        *
+        * ACCOUNT_TYPE_COMMUNITY - the account is community forum
+        *      Associated page types: PAGE_COMMUNITY, PAGE_FLAGS_PRVGROUP
+        *
+        * ACCOUNT_TYPE_RELAY - the account is a relay
+        *      This will only be assigned to contacts, not to user accounts
+        * @{
+        */
+       const ACCOUNT_TYPE_PERSON =       0;
+       const ACCOUNT_TYPE_ORGANISATION = 1;
+       const ACCOUNT_TYPE_NEWS =         2;
+       const ACCOUNT_TYPE_COMMUNITY =    3;
+       const ACCOUNT_TYPE_RELAY =        4;
+       /**
+        * @}
+        */
+
        /**
         * Returns true if a user record exists with the provided id
         *
         * @param  integer $uid
         * @return boolean
+        * @throws Exception
         */
        public static function exists($uid)
        {
@@ -46,6 +92,7 @@ class User
        /**
         * @param  integer       $uid
         * @return array|boolean User record if it exists, false otherwise
+        * @throws Exception
         */
        public static function getById($uid)
        {
@@ -58,6 +105,7 @@ class User
         * @param string $url
         *
         * @return integer user id
+        * @throws Exception
         */
        public static function getIdForURL($url)
        {
@@ -74,6 +122,7 @@ class User
         *
         * @param int $uid
         * @return boolean|array
+        * @throws Exception
         */
        public static function getOwnerDataById($uid) {
                $r = DBA::fetchFirst("SELECT
@@ -85,7 +134,8 @@ class User
                        `user`.`spubkey`,
                        `user`.`page-flags`,
                        `user`.`account-type`,
-                       `user`.`prvnets`
+                       `user`.`prvnets`,
+                       `user`.`account_removed`
                        FROM `contact`
                        INNER JOIN `user`
                                ON `user`.`uid` = `contact`.`uid`
@@ -97,6 +147,19 @@ class User
                if (!DBA::isResult($r)) {
                        return false;
                }
+
+               if (empty($r['nickname'])) {
+                       return false;
+               }
+
+               // Check if the returned data is valid, otherwise fix it. See issue #6122
+               $url = System::baseUrl() . '/profile/' . $r['nickname'];
+               $addr = $r['nickname'] . '@' . substr(System::baseUrl(), strpos(System::baseUrl(), '://') + 3);
+
+               if (($addr != $r['addr']) || ($r['url'] != $url) || ($r['nurl'] != Strings::normaliseLink($r['url']))) {
+                       Contact::updateSelfFromUserID($uid);
+               }
+
                return $r;
        }
 
@@ -105,6 +168,7 @@ class User
         *
         * @param int $nick
         * @return boolean|array
+        * @throws Exception
         */
        public static function getOwnerDataByNick($nick)
        {
@@ -124,6 +188,7 @@ class User
         * @param string $network network name
         *
         * @return int group id
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function getDefaultGroup($uid, $network = '')
        {
@@ -182,7 +247,7 @@ class User
                if (strpos($user['password'], '$') === false) {
                        //Legacy hash that has not been replaced by a new hash yet
                        if (self::hashPasswordLegacy($password) === $user['password']) {
-                               self::updatePassword($user['uid'], $password);
+                               self::updatePasswordHashed($user['uid'], self::hashPassword($password));
 
                                return $user['uid'];
                        }
@@ -190,14 +255,14 @@ class User
                        //Legacy hash that has been double-hashed and not replaced by a new hash yet
                        //Warning: `legacy_password` is not necessary in sync with the content of `password`
                        if (password_verify(self::hashPasswordLegacy($password), $user['password'])) {
-                               self::updatePassword($user['uid'], $password);
+                               self::updatePasswordHashed($user['uid'], self::hashPassword($password));
 
                                return $user['uid'];
                        }
                } elseif (password_verify($password, $user['password'])) {
                        //New password hash
                        if (password_needs_rehash($user['password'], PASSWORD_DEFAULT)) {
-                               self::updatePassword($user['uid'], $password);
+                               self::updatePasswordHashed($user['uid'], self::hashPassword($password));
                        }
 
                        return $user['uid'];
@@ -270,7 +335,7 @@ class User
         */
        public static function generateNewPassword()
        {
-               return Strings::getRandomName(6) . mt_rand(100, 9999);
+               return ucfirst(Strings::getRandomName(8)) . mt_rand(1000, 9999);
        }
 
        /**
@@ -307,6 +372,7 @@ class User
         *
         * @param string $password
         * @return string
+        * @throws Exception
         */
        public static function hashPassword($password)
        {
@@ -323,9 +389,26 @@ class User
         * @param int    $uid
         * @param string $password
         * @return bool
+        * @throws Exception
         */
        public static function updatePassword($uid, $password)
        {
+               $password = trim($password);
+
+               if (empty($password)) {
+                       throw new Exception(L10n::t('Empty passwords are not allowed.'));
+               }
+
+               if (!Config::get('system', 'disable_password_exposed', false) && self::isPasswordExposed($password)) {
+                       throw new Exception(L10n::t('The new password has been exposed in a public data dump, please choose another.'));
+               }
+
+               $allowed_characters = '!"#$%&\'()*+,-./;<=>?@[\]^_`{|}~';
+
+               if (!preg_match('/^[a-z0-9' . preg_quote($allowed_characters, '/') . ']+$/i', $password)) {
+                       throw new Exception(L10n::t('The password can\'t contain accentuated letters, white spaces or colons (:)'));
+               }
+
                return self::updatePasswordHashed($uid, self::hashPassword($password));
        }
 
@@ -336,6 +419,7 @@ class User
         * @param int    $uid
         * @param string $pasword_hashed
         * @return bool
+        * @throws Exception
         */
        private static function updatePasswordHashed($uid, $pasword_hashed)
        {
@@ -357,6 +441,7 @@ class User
         *
         * @param string $nickname The nickname that should be checked
         * @return boolean True is the nickname is blocked on the node
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function isNicknameBlocked($nickname)
        {
@@ -390,17 +475,19 @@ class User
         * - Create self-contact
         * - Create profile image
         *
-        * @param array $data
-        * @return string
-        * @throw Exception
+        * @param  array $data
+        * @return array
+        * @throws \ErrorException
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
+        * @throws Exception
         */
        public static function create(array $data)
        {
-               $a = get_app();
+               $a = \get_app();
                $return = ['user' => null, 'password' => ''];
 
                $using_invites = Config::get('system', 'invitation_only');
-               $num_invites   = Config::get('system', 'number_invites');
 
                $invite_id  = !empty($data['invite_id'])  ? Strings::escapeTags(trim($data['invite_id']))  : '';
                $username   = !empty($data['username'])   ? Strings::escapeTags(trim($data['username']))   : '';
@@ -411,12 +498,12 @@ class User
                $password   = !empty($data['password'])   ? trim($data['password'])           : '';
                $password1  = !empty($data['password1'])  ? trim($data['password1'])          : '';
                $confirm    = !empty($data['confirm'])    ? trim($data['confirm'])            : '';
-               $blocked    = !empty($data['blocked'])    ? intval($data['blocked'])          : 0;
-               $verified   = !empty($data['verified'])   ? intval($data['verified'])         : 0;
+               $blocked    = !empty($data['blocked']);
+               $verified   = !empty($data['verified']);
                $language   = !empty($data['language'])   ? Strings::escapeTags(trim($data['language']))   : 'en';
 
-               $publish = !empty($data['profile_publish_reg']) && intval($data['profile_publish_reg']) ? 1 : 0;
-               $netpublish = strlen(Config::get('system', 'directory')) ? $publish : 0;
+               $publish = !empty($data['profile_publish_reg']);
+               $netpublish = $publish && Config::get('system', 'directory');
 
                if ($password1 != $confirm) {
                        throw new Exception(L10n::t('Passwords do not match. Password unchanged.'));
@@ -463,8 +550,6 @@ class User
                        $openid_url = '';
                }
 
-               $err = '';
-
                // collapse multiple spaces in name
                $username = preg_replace('/ +/', ' ', $username);
 
@@ -671,12 +756,12 @@ class User
                                }
 
                                if (!$photo_failure) {
-                                       DBA::update('photo', ['profile' => 1], ['resource-id' => $hash]);
+                                       Photo::update(['profile' => 1], ['resource-id' => $hash]);
                                }
                        }
                }
 
-               Addon::callHooks('register_account', $uid);
+               Hook::callAll('register_account', $uid);
 
                $return['user'] = $user;
                return $return;
@@ -690,6 +775,7 @@ class User
         * @param string $siteurl
         * @param string $password Plaintext password
         * @return NULL|boolean from notification() and email() inherited
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function sendRegisterPendingEmail($user, $sitename, $siteurl, $password)
        {
@@ -725,6 +811,7 @@ class User
         * @param string $siteurl
         * @param string $password Plaintext password
         * @return NULL|boolean from notification() and email() inherited
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function sendRegisterOpenEmail($user, $sitename, $siteurl, $password)
        {
@@ -761,7 +848,7 @@ class User
                        If you ever want to delete your account, you can do so at %3$s/removeme
 
                        Thank you and welcome to %2$s.',
-                       $user['email'], $sitename, $siteurl, $user['username'], $password
+                       $user['nickname'], $sitename, $siteurl, $user['username'], $password
                ));
 
                return notification([
@@ -777,42 +864,37 @@ class User
 
        /**
         * @param object $uid user to remove
-        * @return void
+        * @return bool
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function remove($uid)
        {
                if (!$uid) {
-                       return;
+                       return false;
                }
 
-               $a = get_app();
-
                Logger::log('Removing user: ' . $uid);
 
                $user = DBA::selectFirst('user', [], ['uid' => $uid]);
 
-               Addon::callHooks('remove_user', $user);
+               Hook::callAll('remove_user', $user);
 
                // save username (actually the nickname as it is guaranteed
                // unique), so it cannot be re-registered in the future.
                DBA::insert('userd', ['username' => $user['nickname']]);
 
                // The user and related data will be deleted in "cron_expire_and_remove_users" (cronjobs.php)
-               DBA::update('user', ['account_removed' => true, 'account_expires_on' => DateTimeFormat::utc(DateTimeFormat::utcNow() . " + 7 day")], ['uid' => $uid]);
-               Worker::add(PRIORITY_HIGH, "Notifier", "removeme", $uid);
+               DBA::update('user', ['account_removed' => true, 'account_expires_on' => DateTimeFormat::utc('now + 7 day')], ['uid' => $uid]);
+               Worker::add(PRIORITY_HIGH, 'Notifier', 'removeme', $uid);
 
                // Send an update to the directory
                $self = DBA::selectFirst('contact', ['url'], ['uid' => $uid, 'self' => true]);
-               Worker::add(PRIORITY_LOW, "Directory", $self['url']);
+               Worker::add(PRIORITY_LOW, 'Directory', $self['url']);
 
                // Remove the user relevant data
-               Worker::add(PRIORITY_LOW, "RemoveUser", $uid);
+               Worker::add(PRIORITY_NEGLIGIBLE, 'RemoveUser', $uid);
 
-               if ($uid == local_user()) {
-                       unset($_SESSION['authenticated']);
-                       unset($_SESSION['uid']);
-                       $a->internalRedirect();
-               }
+               return true;
        }
 
        /**
@@ -822,18 +904,19 @@ class User
         * @return array All identities for this user
         *
         * Example for a return:
-        *      [
-        *              [
-        *                      'uid' => 1,
-        *                      'username' => 'maxmuster',
-        *                      'nickname' => 'Max Mustermann'
-        *              ],
-        *              [
-        *                      'uid' => 2,
-        *                      'username' => 'johndoe',
-        *                      'nickname' => 'John Doe'
-        *              ]
-        *      ]
+        *    [
+        *        [
+        *            'uid' => 1,
+        *            'username' => 'maxmuster',
+        *            'nickname' => 'Max Mustermann'
+        *        ],
+        *        [
+        *            'uid' => 2,
+        *            'username' => 'johndoe',
+        *            'nickname' => 'John Doe'
+        *        ]
+        *    ]
+        * @throws Exception
         */
        public static function identities($uid)
        {