add a publish endpoint

config/routes.yaml

@@ -15,6 +15,13 @@ create_article:
     methods: [POST]
     controller: App\Application\Controller\ArticleController::create
 
+edit_article:
+    path: /articles/{id}
+    methods: [PATCH]
+    controller: App\Application\Controller\ArticleController::update
+    requirements:
+        id: '[\d]+'
+
 publish_article:
     path: /articles/{id}/publish
     methods: [PATCH]

config/services/bus.yaml

@@ -6,3 +6,4 @@ services:
                 - '@App\Domain\Command\Handler\ListArticles'
                 - '@App\Domain\Command\Handler\WriteArticle'
                 - '@App\Domain\Command\Handler\PublishArticle'
+                - '@App\Domain\Command\Handler\EditArticle'

config/services/command_handlers.yaml

@@ -18,4 +18,7 @@ services:
     App\Domain\Command\Handler\PublishArticle:
         arguments:
             - '@Doctrine\ORM\EntityManagerInterface'
-            - '@App\Application\Repository\ArticleRepository'
+
+    App\Domain\Command\Handler\EditArticle:
+        arguments:
+            - '@Doctrine\ORM\EntityManagerInterface'

src/Application/Controller/ArticleController.php

@@ -6,6 +6,7 @@
 use App\Application\JsonDefinition\Article as ArticleDefinition;
 use App\Domain\Command as Command;
 use App\Domain\Command\Bus\CommandBus;
+use App\Domain\Exception\ResourceNotFoundException;
 use Exception;
 use Symfony\Component\Form\FormFactoryInterface;
 use Symfony\Component\HttpFoundation\JsonResponse;
@@ -32,7 +33,6 @@ public function __construct(
         $this->formFactory = $formFactory;
     }
 
-    // @todo add swager doc here
     public function list(): JsonResponse
     {
         try {
@@ -57,27 +57,24 @@ public function list(): JsonResponse
         return new JsonResponse($definitions);
     }
 
-    // @todo add swager doc here
     public function read(string $slug): JsonResponse
     {
         try {
             $article = $this->bus->executeCommand(new Command\ReadArticle($slug));
         }
+        catch (ResourceNotFoundException $exception) {
+            return new JsonResponse(null, Response::HTTP_NOT_FOUND);
+        }
         catch (Exception $exception) {
             return new JsonResponse(
                 ["error" => $exception->getMessage()],
                 Response::HTTP_INTERNAL_SERVER_ERROR
             );
         }
 
-        if (null === $article) {
-            return new JsonResponse(null, Response::HTTP_NOT_FOUND);
-        }
-
         return new JsonResponse(new ArticleDefinition($article));
     }
 
-    // @todo add swager doc here
     public function create(Request $request): JsonResponse
     {
         $createArticle = new Command\WriteArticle();
@@ -87,7 +84,6 @@ public function create(Request $request): JsonResponse
 
         if (!$form->isSubmitted() || !$form->isValid()) {
             return new JsonResponse(
-                // @TODO improve this to have clearer error output
                 $form->getErrors(true)->__toString(),
                 Response::HTTP_BAD_REQUEST
             );
@@ -108,11 +104,48 @@ public function create(Request $request): JsonResponse
         );
     }
 
-    // @todo add swager doc here
+    public function update(Request $request, int $id): JsonResponse
+    {
+        try {
+            $article = $this->bus->executeCommand(new Command\ReadArticle($id));
+            $editArticle = new Command\EditArticle($article);
+
+            $form = $this->formFactory->create(ArticleType::class, $editArticle);
+            $form->submit($request->request->all(), false);
+
+            if (!$form->isSubmitted() || !$form->isValid()) {
+                return new JsonResponse(
+                    $form->getErrors(true)->__toString(),
+                    Response::HTTP_BAD_REQUEST
+                );
+            }
+
+            $editedArticle = $this->bus->executeCommand($editArticle);
+        }
+        catch (ResourceNotFoundException $exception) {
+            return new JsonResponse(null, Response::HTTP_NOT_FOUND);
+        }
+        catch (Exception $exception) {
+            return new JsonResponse(
+                ["error" => $exception->getMessage()],
+                Response::HTTP_INTERNAL_SERVER_ERROR
+            );
+        }
+
+        return new JsonResponse(
+            new ArticleDefinition($editedArticle),
+            Response::HTTP_CREATED
+        );
+    }
+
     public function publish(int $id): JsonResponse
     {
         try {
-            $article = $this->bus->executeCommand(new Command\PublishArticle($id));
+            $article = $this->bus->executeCommand(new Command\ReadArticle($id));
+            $this->bus->executeCommand(new Command\PublishArticle($article));
+        }
+        catch (ResourceNotFoundException $exception) {
+            return new JsonResponse(null, Response::HTTP_NOT_FOUND);
         }
         catch (Exception $exception) {
             return new JsonResponse(

src/Application/Controller/Endpoint.php

@@ -12,4 +12,6 @@ public function list(): JsonResponse;
     public function read(string $slug): JsonResponse;
 
     public function create(Request $request): JsonResponse;
+
+    public function update(Request $request, int $id): JsonResponse;
 }

src/Domain/Command/EditArticle.php

@@ -0,0 +1,55 @@
+<?php
+
+namespace App\Domain\Command;
+
+use App\Domain\Model\Article;
+
+class EditArticle
+{
+    /**
+     * @var Article
+     */
+    private $article;
+
+    /**
+     * @var string
+     */
+    private $title;
+
+    /**
+     * @var string
+     */
+    private $content;
+
+    public function __construct(Article $article)
+    {
+        $this->article = $article;
+        $this->title = $article->getTitle();
+        $this->content = $article->getContent();
+    }
+
+    public function getArticle(): Article
+    {
+        return $this->article;
+    }
+
+    public function getTitle(): string
+    {
+        return $this->title;
+    }
+
+    public function setTitle(string $title)
+    {
+        $this->title = $title;
+    }
+
+    public function getContent(): string
+    {
+        return $this->content;
+    }
+
+    public function setContent(string $content)
+    {
+        $this->content = $content;
+    }
+}

src/Domain/Command/Handler/EditArticle.php

@@ -0,0 +1,35 @@
+<?php
+
+namespace App\Domain\Command\Handler;
+
+use App\Domain\Model\Article;
+use App\Domain\Command\EditArticle as EditArticleCommand;
+use Doctrine\Common\Persistence\ObjectManager;
+
+class EditArticle
+{
+    /**
+     * @var ObjectManager
+     */
+    private $manager;
+
+    public function __construct(
+        ObjectManager $manager
+    ) {
+        $this->manager = $manager;
+    }
+
+    public function __invoke(EditArticleCommand $command): Article
+    {
+        $article = $command->getArticle();
+        $article->edit(
+            $command->getTitle(),
+            $command->getContent()
+        );
+
+        $this->manager->persist($article);
+        $this->manager->flush();
+
+        return $article;
+    }
+}

src/Domain/Command/Handler/PublishArticle.php

@@ -5,7 +5,6 @@
 use App\Domain\Model\Article;
 use App\Domain\Command\PublishArticle as PublishArticleCommand;
 use App\Domain\Exception\ResourceNotFoundException;
-use App\Domain\Repository\ArticleRepository;
 use Doctrine\Common\Persistence\ObjectManager;
 
 class PublishArticle
@@ -15,30 +14,18 @@ class PublishArticle
      */
     private $manager;
 
-    /**
-     * @var ArticleRepository
-     */
-    private $repository;
-
     public function __construct(
-        ObjectManager $manager,
-        ArticleRepository $repository
+        ObjectManager $manager
     ) {
         $this->manager = $manager;
-        $this->repository = $repository;
     }
 
     /**
      * @throws ResourceNotFoundException
      */
     public function __invoke(PublishArticleCommand $command): Article
     {
-        $article = $this->repository->find($command->getId());
-
-        if (null === $article) {
-            throw new ResourceNotFoundException();
-        }
-
+        $article = $command->getArticle();
         $article->publish();
 
         $this->manager->persist($article);

src/Domain/Command/Handler/ReadArticle.php

@@ -2,8 +2,9 @@
 
 namespace App\Domain\Command\Handler;
 
-use App\Domain\Model\Article;
 use App\Domain\Command\ReadArticle as ReadArticleCommand;
+use App\Domain\Exception\ResourceNotFoundException;
+use App\Domain\Model\Article;
 use App\Domain\Repository\ArticleRepository;
 
 class ReadArticle
@@ -18,8 +19,22 @@ public function __construct(ArticleRepository $repository)
         $this->repository = $repository;
     }
 
-    public function __invoke(ReadArticleCommand $command): ?Article
+    /**
+     * @throws ResourceNotFoundException
+     */
+    public function __invoke(ReadArticleCommand $command): Article
     {
-        return $this->repository->findOneBySlug($command->getSlug());
+        $id = $command->getId();
+
+        $article = is_numeric($id)
+            ? $this->repository->findOneById($id)
+            : $this->repository->findOneBySlug($id)
+        ;
+
+        if ($article === null) {
+            throw new ResourceNotFoundException();
+        }
+
+        return $article;
     }
 }

src/Domain/Command/PublishArticle.php

@@ -2,20 +2,22 @@
 
 namespace App\Domain\Command;
 
+use App\Domain\Model\Article;
+
 class PublishArticle
 {
     /**
-     * @var int
+     * @var Article
      */
-    private $id;
+    private $article;
 
-    public function __construct(int $id)
+    public function __construct(Article $article)
     {
-        $this->id = $id;
+        $this->article = $article;
     }
 
-    public function getId(): int
+    public function getArticle(): Article
     {
-        return $this->id;
+        return $this->article;
     }
 }

src/Domain/Command/ReadArticle.php

@@ -5,17 +5,23 @@
 class ReadArticle
 {
     /**
-     * @var string
+     * @var int|string
      */
     private $slug;
 
-    public function __construct(string $slug)
+    /**
+     * @param int|string
+     */
+    public function __construct($id)
     {
-        $this->slug = $slug;
+        $this->id = $id;
     }
 
-    public function getSlug(): string
+    /**
+     * @return int|string
+     */
+    public function getId()
     {
-        return $this->slug;
+        return $this->id;
     }
 }

src/Domain/Model/Article.php

@@ -51,6 +51,14 @@ public static function write(string $title, string $content, string $slug): self
         return new self($title, $content, $slug);
     }
 
+    public function edit(string $title, string $content): self
+    {
+        $this->title = $title;
+        $this->content = $content;
+
+        return $this;
+    }
+
     public function publish(): self
     {
         $this->draft = false;