changes for pull request review

dotkernel · Nov 1, 2023 · 98f417f · 98f417f
1 parent 41e245f
commit 98f417f
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/src/User/src/Entity/User.php b/src/User/src/Entity/User.php
@@ -53,10 +53,10 @@ class User extends AbstractEntity implements UserEntityInterface
     #[ORM\Column(name: "password", type: "string", length: 191)]
     protected string $password;
 
-    #[ORM\Column("status", type: "string", length: 20)]
+    #[ORM\Column(name: "status", type: "string", length: 20)]
     protected string $status = self::STATUS_PENDING;
 
-    #[ORM\Column("isDeleted", type: "boolean")]
+    #[ORM\Column(name: "isDeleted", type: "boolean")]
     protected bool $isDeleted = false;
 
     #[ORM\Column(name: "hash", type: "string", length: 64, unique: true)]

diff --git a/src/User/src/Entity/UserResetPasswordEntity.php b/src/User/src/Entity/UserResetPasswordEntity.php
@@ -11,7 +11,9 @@
 use Doctrine\ORM\Mapping as ORM;
 use Throwable;
 
-#[ORM\Entity, ORM\Table(name: "user_reset_password"), ORM\HasLifecycleCallbacks]
+#[ORM\Entity]
+#[ORM\Table(name: "user_reset_password")]
+#[ORM\HasLifecycleCallbacks]
 class UserResetPasswordEntity extends AbstractEntity
 {
     public const STATUS_COMPLETED = 'completed';
@@ -21,7 +23,7 @@ class UserResetPasswordEntity extends AbstractEntity
         self::STATUS_REQUESTED,
     ];
 
-    #[ORM\ManyToOne(targetEntity: "User", cascade: ['persist', 'remove'], inversedBy: "resetPasswords")]
+    #[ORM\ManyToOne(targetEntity: User::class, cascade: ['persist', 'remove'], inversedBy: "resetPasswords")]
     #[ORM\JoinColumn(name: "userUuid", referencedColumnName: "uuid")]
     protected User $user;
 

diff --git a/src/User/src/Entity/UserRole.php b/src/User/src/Entity/UserRole.php
@@ -14,12 +14,10 @@
 #[ORM\HasLifecycleCallbacks]
 class UserRole extends AbstractEntity implements RoleInterface
 {
-    public const ROLE_GUEST    = 'guest';
-    public const ROLE_API_USER = 'api_user';
-    public const ROLE_USER     = 'user';
-    public const ROLES         = [
+    public const ROLE_GUEST = 'guest';
+    public const ROLE_USER  = 'user';
+    public const ROLES      = [
         self::ROLE_GUEST,
-        self::ROLE_API_USER,
         self::ROLE_USER,
     ];