[DATABASE] Update LocalUser table to not have a numerical id, add is_email_verified and fix getProfile accordingly

This commit is contained in:
Hugo Sales 2020-07-25 02:00:33 +00:00 committed by Hugo Sales
parent 7a7f7d3ae1
commit a85fc2d0bd
Signed by: someonewithpc
GPG Key ID: 7D0C7EAFC9D835A0

View File

@ -22,6 +22,7 @@ namespace App\Entity;
use App\Core\DB\DB; use App\Core\DB\DB;
use App\Core\UserRoles; use App\Core\UserRoles;
use App\Util\Common; use App\Util\Common;
use DateTime;
use DateTimeInterface; use DateTimeInterface;
use Symfony\Component\Security\Core\User\UserInterface; use Symfony\Component\Security\Core\User\UserInterface;
@ -43,11 +44,11 @@ class LocalUser implements UserInterface
{ {
// {{{ Autocode // {{{ Autocode
private int $id;
private ?string $nickname; private ?string $nickname;
private ?string $password; private ?string $password;
private ?string $outgoing_email; private ?string $outgoing_email;
private ?string $incoming_email; private ?string $incoming_email;
private ?bool $is_email_verified;
private ?string $language; private ?string $language;
private ?string $timezone; private ?string $timezone;
private ?string $sms_phone_number; private ?string $sms_phone_number;
@ -60,16 +61,6 @@ class LocalUser implements UserInterface
private \DateTimeInterface $created; private \DateTimeInterface $created;
private \DateTimeInterface $modified; private \DateTimeInterface $modified;
public function setId(int $id): self
{
$this->id = $id;
return $this;
}
public function getId(): int
{
return $this->id;
}
public function setNickname(?string $nickname): self public function setNickname(?string $nickname): self
{ {
$this->nickname = $nickname; $this->nickname = $nickname;
@ -110,6 +101,16 @@ class LocalUser implements UserInterface
return $this->incoming_email; return $this->incoming_email;
} }
public function setIsEmailVerified(?bool $is_email_verified): self
{
$this->is_email_verified = $is_email_verified;
return $this;
}
public function getIsEmailVerified(): ?bool
{
return $this->is_email_verified;
}
public function setLanguage(?string $language): self public function setLanguage(?string $language): self
{ {
$this->language = $language; $this->language = $language;
@ -222,17 +223,28 @@ class LocalUser implements UserInterface
// }}} Autocode // }}} Autocode
public function __construct(string $nickname, string $email, string $password)
{
$this->nickname = $nickname;
$this->outgoing_email = $email;
$this->incoming_email = $email;
$this->changePassword($password, true);
// TODO auto update created and modified
$this->created = new DateTime();
$this->modified = new DateTime();
}
public static function schemaDef(): array public static function schemaDef(): array
{ {
return [ return [
'name' => 'local_user', 'name' => 'local_user',
'description' => 'local users', 'description' => 'local users',
'fields' => [ 'fields' => [
'id' => ['type' => 'int', 'not null' => true, 'description' => 'foreign key to profile table'],
'nickname' => ['type' => 'varchar', 'length' => 64, 'description' => 'nickname or username, duped in profile'], 'nickname' => ['type' => 'varchar', 'length' => 64, 'description' => 'nickname or username, duped in profile'],
'password' => ['type' => 'varchar', 'length' => 191, 'description' => 'salted password, can be null for OpenID users'], 'password' => ['type' => 'varchar', 'length' => 191, 'description' => 'salted password, can be null for OpenID users'],
'outgoing_email' => ['type' => 'varchar', 'length' => 191, 'description' => 'email address for password recovery, notifications, etc.'], 'outgoing_email' => ['type' => 'varchar', 'length' => 191, 'description' => 'email address for password recovery, notifications, etc.'],
'incoming_email' => ['type' => 'varchar', 'length' => 191, 'description' => 'email address for post-by-email'], 'incoming_email' => ['type' => 'varchar', 'length' => 191, 'description' => 'email address for post-by-email'],
'is_email_verified' => ['type' => 'bool', 'default' => false, 'description' => 'Whether the user opened the comfirmation email'],
'language' => ['type' => 'varchar', 'length' => 50, 'description' => 'preferred language'], 'language' => ['type' => 'varchar', 'length' => 50, 'description' => 'preferred language'],
'timezone' => ['type' => 'varchar', 'length' => 50, 'description' => 'timezone'], 'timezone' => ['type' => 'varchar', 'length' => 50, 'description' => 'timezone'],
'sms_phone_number' => ['type' => 'varchar', 'length' => 64, 'description' => 'sms phone number'], 'sms_phone_number' => ['type' => 'varchar', 'length' => 64, 'description' => 'sms phone number'],
@ -245,19 +257,19 @@ class LocalUser implements UserInterface
'created' => ['type' => 'datetime', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was created'], 'created' => ['type' => 'datetime', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was created'],
'modified' => ['type' => 'timestamp', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was modified'], 'modified' => ['type' => 'timestamp', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was modified'],
], ],
'primary key' => ['id'], 'primary key' => ['nickname'],
'unique keys' => [ 'unique keys' => [
'user_nickname_key' => ['nickname'],
'user_outgoing_email_key' => ['outgoing_email'], 'user_outgoing_email_key' => ['outgoing_email'],
'user_incoming_email_key' => ['incoming_email'], 'user_incoming_email_key' => ['incoming_email'],
'user_sms_key' => ['sms_phone_number'], 'user_sms_key' => ['sms_phone_number'],
'user_uri_key' => ['uri'], 'user_uri_key' => ['uri'],
], ],
'foreign keys' => [ 'foreign keys' => [
'user_id_fkey' => ['profile', ['id' => 'id']], 'user_nickname_fkey' => ['profile', ['nickname' => 'nickname']],
'user_carrier_fkey' => ['sms_carrier', ['sms_carrier' => 'id']], 'user_carrier_fkey' => ['sms_carrier', ['sms_carrier' => 'id']],
], ],
'indexes' => [ 'indexes' => [
'user_nickname_idx' => ['nickname'],
'user_created_idx' => ['created'], 'user_created_idx' => ['created'],
'user_sms_email_idx' => ['sms_email'], 'user_sms_email_idx' => ['sms_email'],
], ],
@ -266,7 +278,7 @@ class LocalUser implements UserInterface
public function getProfile() public function getProfile()
{ {
return DB::find('profile', ['id' => $this->id]); return DB::findOneBy('profile', ['nickname' => $this->nickname]);
} }
/** /**
@ -280,8 +292,8 @@ class LocalUser implements UserInterface
/** /**
* Returns the password used to authenticate the user. * Returns the password used to authenticate the user.
* *
* Already in the autocode * Implemented in the auto code
* public function getPassword() */ */
/** /**
* Returns the salt that was originally used to encode the password. * Returns the salt that was originally used to encode the password.