feat: add exceptions (#2)

* feat: add exceptions * fix: add strict types * chore: apply changes * fix: array result * chore: apply changes * chore: apply changes --------- Signed-off-by: Lyrisbee <Lyrisbee@users.noreply.github.com>
storipress · Dec 12, 2023 · d803442 · d803442
1 parent f5f73bf
commit d803442
Show file tree

Hide file tree

Showing 20 changed files with 289 additions and 102 deletions.
diff --git a/src/Exceptions/BadRequestException.php b/src/Exceptions/BadRequestException.php
@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Storipress\WordPress\Exceptions;
+
+use Storipress\WordPress\Objects\WordPressError;
+
+class BadRequestException extends WordPressException
+{
+    public function __construct(WordPressError $error)
+    {
+        parent::__construct($error, 400);
+    }
+}
diff --git a/src/Exceptions/Exception.php → src/Exceptions/CannotCreateException.php b/src/Exceptions/Exception.php → src/Exceptions/CannotCreateException.php
@@ -4,9 +4,7 @@
 
 namespace Storipress\WordPress\Exceptions;
 
-use Exception as BaseException;
-
-abstract class Exception extends BaseException
+class CannotCreateException extends WordPressException
 {
     //
 }
diff --git a/src/Exceptions/CannotUpdateException.php b/src/Exceptions/CannotUpdateException.php
@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Storipress\WordPress\Exceptions;
+
+class CannotUpdateException extends WordPressException
+{
+    //
+}
diff --git a/src/Exceptions/DuplicateTermSlugException.php b/src/Exceptions/DuplicateTermSlugException.php
@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Storipress\WordPress\Exceptions;
+
+class DuplicateTermSlugException extends WordPressException
+{
+    //
+}
diff --git a/src/Exceptions/ForbiddenException.php b/src/Exceptions/ForbiddenException.php
@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Storipress\WordPress\Exceptions;
+
+use Storipress\WordPress\Objects\WordPressError;
+
+class ForbiddenException extends WordPressException
+{
+    public function __construct(WordPressError $error)
+    {
+        parent::__construct($error, 403);
+    }
+}
diff --git a/src/Exceptions/NotFoundException.php b/src/Exceptions/NotFoundException.php
@@ -4,7 +4,12 @@
 
 namespace Storipress\WordPress\Exceptions;
 
-class NotFoundException extends HttpException
+use Storipress\WordPress\Objects\WordPressError;
+
+class NotFoundException extends WordPressException
 {
-    //
+    public function __construct(WordPressError $error)
+    {
+        parent::__construct($error, 404);
+    }
 }
diff --git a/src/Exceptions/HttpUnknownError.php → src/Exceptions/TermExistsException.php b/src/Exceptions/HttpUnknownError.php → src/Exceptions/TermExistsException.php
@@ -4,7 +4,7 @@
 
 namespace Storipress\WordPress\Exceptions;
 
-class HttpUnknownError extends HttpException
+class TermExistsException extends WordPressException
 {
     //
 }
diff --git a/src/Exceptions/UnauthorizedException.php b/src/Exceptions/UnauthorizedException.php
@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Storipress\WordPress\Exceptions;
+
+use Storipress\WordPress\Objects\WordPressError;
+
+class UnauthorizedException extends WordPressException
+{
+    public function __construct(WordPressError $error)
+    {
+        parent::__construct($error, 401);
+    }
+}
diff --git a/src/Exceptions/UnexpectedValueException.php b/src/Exceptions/UnexpectedValueException.php
@@ -4,7 +4,12 @@
 
 namespace Storipress\WordPress\Exceptions;
 
-class UnexpectedValueException extends Exception
+use Storipress\WordPress\Objects\WordPressError;
+
+class UnexpectedValueException extends WordPressException
 {
-    //
+    public function __construct(WordPressError $error)
+    {
+        parent::__construct($error, 500);
+    }
 }
diff --git a/src/Exceptions/HttpException.php → src/Exceptions/UnknownException.php b/src/Exceptions/HttpException.php → src/Exceptions/UnknownException.php
@@ -4,7 +4,7 @@
 
 namespace Storipress\WordPress\Exceptions;
 
-abstract class HttpException extends Exception
+class UnknownException extends WordPressException
 {
     //
 }
diff --git a/src/Exceptions/WordPressException.php b/src/Exceptions/WordPressException.php
@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Storipress\WordPress\Exceptions;
+
+use Exception as BaseException;
+use Storipress\WordPress\Objects\WordPressError;
+
+abstract class WordPressException extends BaseException
+{
+    /**
+     * Duplicate term id
+     */
+    public ?int $term_id;
+
+    public function __construct(public WordPressError $error, int $code)
+    {
+        $this->term_id = $error->data->term_id ?? null;
+
+        parent::__construct($error->message, $code);
+    }
+
+    public function getTermId(): ?int
+    {
+        return $this->term_id;
+    }
+}
diff --git a/src/Objects/WordPressError.php b/src/Objects/WordPressError.php
@@ -0,0 +1,16 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Storipress\WordPress\Objects;
+
+use stdClass;
+
+class WordPressError extends WordPressObject
+{
+    public string $code;
+
+    public string $message;
+
+    public stdClass $data;
+}
diff --git a/src/Requests/Category.php b/src/Requests/Category.php
@@ -4,8 +4,7 @@
 
 namespace Storipress\WordPress\Requests;
 
-use Storipress\WordPress\Exceptions\HttpException;
-use Storipress\WordPress\Exceptions\UnexpectedValueException;
+use Storipress\WordPress\Exceptions\WordPressException;
 use Storipress\WordPress\Objects\Category as CategoryObject;
 
 class Category extends Request
@@ -15,15 +14,14 @@ class Category extends Request
      *
      * @return array<int, CategoryObject>
      *
-     * @throws HttpException
-     * @throws UnexpectedValueException
+     * @throws WordPressException
      */
     public function list(): array
     {
         $data = $this->request('get', '/categories');
 
         if (!is_array($data)) {
-            throw new UnexpectedValueException();
+            throw $this->unexpectedValueException();
         }
 
         return array_map(
@@ -37,15 +35,14 @@ public function list(): array
      *
      * @param  array<string, mixed>  $arguments
      *
-     * @throws HttpException
-     * @throws UnexpectedValueException
+     * @throws WordPressException
      */
     public function create(array $arguments): CategoryObject
     {
         $data = $this->request('post', '/categories', $arguments);
 
         if (is_array($data)) {
-            throw new UnexpectedValueException();
+            throw $this->unexpectedValueException();
         }
 
         return CategoryObject::from($data);
@@ -54,8 +51,7 @@ public function create(array $arguments): CategoryObject
     /**
      * https://developer.wordpress.org/rest-api/reference/categories/#retrieve-a-category
      *
-     * @throws HttpException
-     * @throws UnexpectedValueException
+     * @throws WordPressException
      */
     public function retrieve(int $categoryId, string $context = 'view'): CategoryObject
     {
@@ -66,7 +62,7 @@ public function retrieve(int $categoryId, string $context = 'view'): CategoryObj
         ]);
 
         if (is_array($data)) {
-            throw new UnexpectedValueException();
+            throw $this->unexpectedValueException();
         }
 
         return CategoryObject::from($data);
@@ -77,8 +73,7 @@ public function retrieve(int $categoryId, string $context = 'view'): CategoryObj
      *
      * @param  array<string, mixed>  $arguments
      *
-     * @throws HttpException
-     * @throws UnexpectedValueException
+     * @throws WordPressException
      */
     public function update(int $categoryId, array $arguments): CategoryObject
     {
@@ -87,7 +82,7 @@ public function update(int $categoryId, array $arguments): CategoryObject
         $data = $this->request('patch', $uri, $arguments);
 
         if (is_array($data)) {
-            throw new UnexpectedValueException();
+            throw $this->unexpectedValueException();
         }
 
         return CategoryObject::from($data);
@@ -96,8 +91,7 @@ public function update(int $categoryId, array $arguments): CategoryObject
     /**
      * https://developer.wordpress.org/rest-api/reference/categories/#delete-a-category
      *
-     * @throws HttpException
-     * @throws UnexpectedValueException
+     * @throws WordPressException
      */
     public function delete(int $categoryId): bool
     {

diff --git a/src/Requests/GeneralRequest.php b/src/Requests/GeneralRequest.php
@@ -5,6 +5,7 @@
 namespace Storipress\WordPress\Requests;
 
 use stdClass;
+use Storipress\WordPress\Exceptions\WordPressException;
 
 class GeneralRequest extends Request
 {
@@ -13,8 +14,7 @@ class GeneralRequest extends Request
      * @param  array<mixed>  $arguments
      * @return stdClass|array<int, stdClass>
      *
-     * @throws \Storipress\WordPress\Exceptions\HttpException
-     * @throws \Storipress\WordPress\Exceptions\UnexpectedValueException
+     * @throws WordPressException
      */
     public function get(string $path, array $arguments): stdClass|array
     {
@@ -26,8 +26,7 @@ public function get(string $path, array $arguments): stdClass|array
      * @param  array<mixed>  $arguments
      * @return stdClass|array<int, stdClass>
      *
-     * @throws \Storipress\WordPress\Exceptions\HttpException
-     * @throws \Storipress\WordPress\Exceptions\UnexpectedValueException
+     * @throws WordPressException
      */
     public function post(string $path, array $arguments): stdClass|array
     {
@@ -39,8 +38,7 @@ public function post(string $path, array $arguments): stdClass|array
      * @param  array<mixed>  $arguments
      * @return stdClass|array<int, stdClass>
      *
-     * @throws \Storipress\WordPress\Exceptions\HttpException
-     * @throws \Storipress\WordPress\Exceptions\UnexpectedValueException
+     * @throws WordPressException
      */
     public function patch(string $path, array $arguments): stdClass|array
     {
@@ -51,8 +49,7 @@ public function patch(string $path, array $arguments): stdClass|array
      * @param  non-empty-string  $path
      * @param  array<mixed>  $arguments
      *
-     * @throws \Storipress\WordPress\Exceptions\HttpException
-     * @throws \Storipress\WordPress\Exceptions\UnexpectedValueException
+     * @throws WordPressException
      */
     public function delete(string $path, array $arguments): bool
     {