migrates to JsonSerializable

Lemon-Framework · Jan 24, 2023 · 9ed5154 · 9ed5154
1 parent bf02389
commit 9ed5154
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 17 deletions.
diff --git a/src/Lemon/Contracts/Http/Jsonable.php b/src/Lemon/Contracts/Http/Jsonable.php
diff --git a/src/Lemon/Http/ResponseFactory.php b/src/Lemon/Http/ResponseFactory.php
@@ -4,6 +4,7 @@
 
 namespace Lemon\Http;
 
+use JsonSerializable;
 use Lemon\Contracts\Http\Jsonable;
 use Lemon\Contracts\Http\ResponseFactory as ResponseFactoryContract;
 use Lemon\Contracts\Templating\Factory as Templating;
@@ -55,8 +56,8 @@ public function resolve(mixed $data): Response
             return $data;
         }
 
-        if ($data instanceof Jsonable) {
-            return new JsonResponse($data->toJson());
+        if ($data instanceof JsonSerializable) {
+            return new JsonResponse($data);
         }
 
         if ($data instanceof Template) {

diff --git a/tests/Http/ResponseFactoryTest.php b/tests/Http/ResponseFactoryTest.php
@@ -4,6 +4,7 @@
 
 namespace Lemon\Tests\Http;
 
+use JsonSerializable;
 use Lemon\Config\Config;
 use Lemon\Contracts\Http\Jsonable;
 use Lemon\Contracts\Templating\Compiler;
@@ -137,14 +138,14 @@ public function getExtension(): string
     }
 }
 
-class SimpleJson implements Jsonable
+class SimpleJson implements JsonSerializable
 {
     public function __construct(
         private array $json
     ) {
     }
 
-    public function toJson(): array
+    public function jsonSerialize(): mixed
     {
         return $this->json;
     }