Refactor edit user

Author: luka-lta

src/Api/LinkCollection/Service/LinkCollectionService.php

@@ -54,7 +54,7 @@ public function getAllLinks(ServerRequestInterface $request): ApiResult
     {
         $mustRef = $request->getQueryParams()['mustRef'] ?? false;
 
-        $links = $this->repository->getAll($mustRef);
+        $links = $this->repository->getAll();
 
         if ($links->count() === 0) {
             return ApiResult::from(
@@ -63,7 +63,12 @@ public function getAllLinks(ServerRequestInterface $request): ApiResult
             );
         }
 
-        return ApiResult::from(JsonResult::from('Links fetched successfully', ['links' => $links->toArray()]));
+        return ApiResult::from(
+            JsonResult::from('Links fetched successfully', [
+                    'links' => $links->toArray($mustRef)
+                ]
+            )
+        );
     }
 
     public function createLink(ServerRequestInterface $request): ApiResult
@@ -122,7 +127,7 @@ public function disableLink(array $params): ApiResult
 
     public function editLink(ServerRequestInterface $request): ApiResult
     {
-        $linkId = (int) $request->getAttribute('linkId');
+        $linkId = (int)$request->getAttribute('linkId');
 
         $linkItem = $this->repository->getById(LinkId::fromInt($linkId));
 
@@ -157,7 +162,7 @@ private function checkUserHasChanges(LinkItem $linkItem, array $parsedBody): voi
         }
 
         if (isset($parsedBody['isActive'])) {
-            $linkMetaData->setIsActive((bool) $parsedBody['isActive']);
+            $linkMetaData->setIsActive((bool)$parsedBody['isActive']);
         }
 
         if (array_key_exists('iconName', $parsedBody)) {

src/Api/User/Create/CreateUserAction.php src/Api/User/Action/CreateUserAction.phpsrc/Api/User/Create/CreateUserAction.php renamed to src/Api/User/Action/CreateUserAction.php

@@ -1,21 +1,17 @@
 <?php
 
-namespace LukaLtaApi\Api\User\Create;
+namespace LukaLtaApi\Api\User\Action;
 
 use LukaLtaApi\Api\ApiAction;
 use LukaLtaApi\Api\RequestValidator;
-use LukaLtaApi\Api\User\Create\Service\CreateUserService;
-use LukaLtaApi\Value\Result\ApiResult;
-use LukaLtaApi\Value\Result\JsonResult;
-use LukaLtaApi\Value\User\UserEmail;
-use LukaLtaApi\Value\User\UserPassword;
+use LukaLtaApi\Api\User\Service\UserService;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;
 
 class CreateUserAction extends ApiAction
 {
     public function __construct(
-        private readonly CreateUserService $service,
+        private readonly UserService $service,
         private readonly RequestValidator $validator,
     ) {
     }
@@ -29,11 +25,6 @@ protected function execute(ServerRequestInterface $request, ResponseInterface $r
 
         $this->validator->validate($request, $rules);
 
-        $this->service->createUser(
-            UserEmail::from($request->getParsedBody()['email']),
-            $request->getParsedBody()['password']
-        );
-
-        return ApiResult::from(JsonResult::from('User created'))->getResponse($response);
+        return $this->service->createUser($request->getParsedBody())->getResponse($response);
     }
 }

src/Api/User/Action/GetAllUsersAction.php

@@ -0,0 +1,23 @@
+<?php
+
+declare(strict_types=1);
+
+namespace LukaLtaApi\Api\User\Action;
+
+use LukaLtaApi\Api\ApiAction;
+use LukaLtaApi\Api\User\Service\UserService;
+use Psr\Http\Message\ResponseInterface;
+use Psr\Http\Message\ServerRequestInterface;
+
+class GetAllUsersAction extends ApiAction
+{
+    public function __construct(
+        private readonly UserService $service
+    ) {
+    }
+
+    protected function execute(ServerRequestInterface $request, ResponseInterface $response): ResponseInterface
+    {
+        return $this->service->getAllUsers()->getResponse($response);
+    }
+}

src/Api/User/Action/GetAvatarAction.php

@@ -0,0 +1,23 @@
+<?php
+
+declare(strict_types=1);
+
+namespace LukaLtaApi\Api\User\Action;
+
+use LukaLtaApi\Api\ApiAction;
+use LukaLtaApi\Api\User\Service\UserService;
+use Psr\Http\Message\ResponseInterface;
+use Psr\Http\Message\ServerRequestInterface;
+
+class GetAvatarAction extends ApiAction
+{
+    public function __construct(
+        private readonly UserService $service,
+    ) {
+    }
+
+    protected function execute(ServerRequestInterface $request, ResponseInterface $response): ResponseInterface
+    {
+        return $this->service->getAvatar($request, $response);
+    }
+}

src/Api/User/Action/UpdateUserAction.php

@@ -0,0 +1,30 @@
+<?php
+
+namespace LukaLtaApi\Api\User\Action;
+
+use LukaLtaApi\Api\ApiAction;
+use LukaLtaApi\Api\RequestValidator;
+use LukaLtaApi\Api\User\Service\UserService;
+use Psr\Http\Message\ResponseInterface;
+use Psr\Http\Message\ServerRequestInterface;
+
+class UpdateUserAction extends ApiAction
+{
+    public function __construct(
+        private readonly UserService $service,
+        private readonly RequestValidator $validator,
+    ) {
+    }
+
+    protected function execute(ServerRequestInterface $request, ResponseInterface $response): ResponseInterface
+    {
+        $rules = [
+            'email' => ['required' => true, 'location' => RequestValidator::LOCATION_BODY],
+            'avatarUrl' => ['required' => false, 'location' => RequestValidator::LOCATION_BODY],
+        ];
+
+        $this->validator->validate($request, $rules);
+
+        return $this->service->updateUser($request)->getResponse($response);
+    }
+}