Allow to update a user saved in the database

Author: pH-7

src/Dal/UserDal.php

@@ -29,6 +29,35 @@ public static function create(UserEntity $userEntity): int|string
         return $id;
     }
 
+    public static function update(string $userUuid, UserEntity $userEntity): int|string
+    {
+        $userBean = R::findOne(self::TABLE_NAME, 'user_uuid = :userUuid', ['userUuid' => $userUuid]);
+
+        // If the user exists, update it
+        if ($userBean) {
+            $firstName = $userEntity->getFirstName();
+            $lastName = $userEntity->getLastName();
+            $phone = $userEntity->getPhone();
+
+            if ($firstName) {
+                $userBean->firstName = $firstName;
+            }
+
+            if ($lastName) {
+                $userBean->lastName = $lastName;
+            }
+
+            if ($phone) {
+                $userBean->phone = $phone;
+            }
+
+            // save the user
+            return R::store($userBean);
+        }
+
+        return 0;
+    }
+
     public static function get(string $userUuid): ?array
     {
         $bindings = ['userUuid' => $userUuid];

src/Entity/User.php

@@ -5,13 +5,13 @@ class User
 {
     private string $userUuid;
 
-    private string $firstName;
+    private ?string $firstName = null;
 
-    private string $lastName;
+    private ?string $lastName = null;
 
-    private string $email;
+    private ?string $email = null;
 
-    private string $phone;
+    private ?string $phone = null;
 
     private string $password;
 
@@ -36,7 +36,7 @@ public function setFirstName(string $firstName): self
         return $this;
     }
 
-    public function getFirstName(): string
+    public function getFirstName(): ?string
     {
         return $this->firstName;
     }
@@ -48,7 +48,7 @@ public function setLastName(string $lastName): self
         return $this;
     }
 
-    public function getLastName(): string
+    public function getLastName(): ?string
     {
         return $this->lastName;
     }
@@ -60,7 +60,7 @@ public function setEmail(string $email): self
         return $this;
     }
 
-    public function getEmail(): string
+    public function getEmail(): ?string
     {
         return $this->email;
     }
@@ -72,7 +72,7 @@ public function setPhone(string $phone): self
         return $this;
     }
 
-    public function getPhone(): string
+    public function getPhone(): ?string
     {
         return $this->phone;
     }

src/Service/User.php

@@ -15,7 +15,7 @@ class User
 {
     public const DATE_TIME_FORMAT = 'Y-m-d H:i:s';
 
-    public function create(mixed $data): object
+    public function create(mixed $data): array|object
     {
         $userValidation = new UserValidation($data);
         if ($userValidation->isCreationSchemaValid()) {
@@ -46,6 +46,37 @@ public function create(mixed $data): object
         throw new InvalidValidationException("Invalid user payload");
     }
 
+    public function update(mixed $postBody): array|object
+    {
+        $userValidation = new UserValidation($postBody);
+        if ($userValidation->isUpdateSchemaValid()) {
+            $userUuid = $postBody->userUuid;
+
+            $userEntity = new UserEntity();
+            if (!empty($postBody->first)) {
+                $userEntity->setFirstName($postBody->first);
+            }
+
+            if (!empty($postBody->last)) {
+                $userEntity->setLastName($postBody->last);
+            }
+
+            if (!empty($postBody->phone)) {
+                $userEntity->setPhone($postBody->phone);
+            }
+
+            $result = UserDal::update($userUuid, $userEntity);
+            if ($result) {
+                return $postBody;
+            }
+
+            // if invalid, give back an empty response
+            return [];
+        }
+
+        throw new InvalidValidationException("Invalid user payload");
+    }
+
     public function retrieveAll(): array
     {
         $users = UserDal::getAll();
@@ -73,16 +104,6 @@ public function retrieve(string $userUuid): array
         throw new InvalidValidationException("Invalid user UUID");
     }
 
-    public function update(mixed $postBody): object
-    {
-        $userValidation = new UserValidation($postBody);
-        if ($userValidation->isUpdateSchemaValid()) {
-            return $postBody;
-        }
-
-        throw new InvalidValidationException("Invalid user payload");
-    }
-
     public function remove(object $data): bool
     {
         $userValidation = new UserValidation($data);

src/Validation/UserValidation.php

@@ -17,8 +17,8 @@ public function isCreationSchemaValid(): bool
         $schemaValidation =
             v::attribute('first', v::stringType()->length(self::MINIMUM_NAME_LENGTH, self::MAXIMUM_NAME_LENGTH))
             ->attribute('last', v::stringType()->length(self::MINIMUM_NAME_LENGTH, self::MAXIMUM_NAME_LENGTH))
-            ->attribute('email', v::email(), mandatory: false)
-            ->attribute('phone', v::phone(), mandatory: false);
+            ->attribute('email', v::email())
+            ->attribute('phone', v::phone());
 
         return $schemaValidation->validate($this->data);
     }
@@ -30,7 +30,12 @@ public function isRemoveSchemaValid(): bool
 
     public function isUpdateSchemaValid(): bool
     {
-        // same schema for both creation and update
-        return $this->isCreationSchemaValid();
+        $schemaValidation =
+            v::attribute('userUuid', v::uuid())
+            ->attribute('first', v::stringType()->length(self::MINIMUM_NAME_LENGTH, self::MAXIMUM_NAME_LENGTH), mandatory: false)
+            ->attribute('last', v::stringType()->length(self::MINIMUM_NAME_LENGTH, self::MAXIMUM_NAME_LENGTH), mandatory: false)
+            ->attribute('phone', v::phone(), mandatory: false);
+
+        return $schemaValidation->validate($this->data);
     }
 }