move serialization responsibility to cache objects

src/Contracts/CacheObject.php

@@ -14,4 +14,9 @@ interface CacheObject
     public function key(): CacheKey;
 
     public function ttl(): CarbonInterval;
+
+    /**
+     * @return CacheValueModifier<TValue>
+     */
+    public function modifier(): CacheValueModifier;
 }

src/Contracts/CacheValueModifier.php

@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+namespace EriBloo\CacheObjects\Contracts;
+
+/**
+ * @template T
+ */
+interface CacheValueModifier
+{
+    /**
+     * @return T
+     */
+    public function onLoad(string $value): mixed;
+
+    /**
+     * @param T $value
+     */
+    public function onSave(mixed $value): string;
+}

src/Drivers/LaravelDriver.php

@@ -16,9 +16,11 @@ public function __construct(
 
     public function set(mixed $value, CacheObject $cacheObject): string
     {
-        $key = $this->prepareKey($cacheObject);
-        $value = $this->prepareValue($value);
-        $ttl = $this->prepareTtl($cacheObject);
+        $key = (string) $cacheObject->key();
+        $value = $cacheObject->modifier()
+            ->onSave($value);
+        $ttl = (int) $cacheObject->ttl()
+->totalSeconds;
 
         if ($ttl <= 0) {
             $this->repository->forever($key, $value);
@@ -31,49 +33,19 @@ public function set(mixed $value, CacheObject $cacheObject): string
 
     public function get(CacheObject $cacheObject): mixed
     {
-        $key = $this->prepareKey($cacheObject);
+        $key = (string) $cacheObject->key();
         $value = $this->repository->get($key);
 
         if ($value === null) {
             return null;
         }
 
-        return $this->prepareReturn($value);
+        return $cacheObject->modifier()
+            ->onLoad($value);
     }
 
     public function delete(CacheObject $cacheObject): bool
     {
-        return $this->repository->forget($this->prepareKey($cacheObject));
-    }
-
-    /**
-     * @template TValue
-     *
-     * @param CacheObject<TValue> $cacheObject
-     */
-    private function prepareKey(CacheObject $cacheObject): string
-    {
-        return (string) $cacheObject->key();
-    }
-
-    private function prepareValue(mixed $value): string
-    {
-        return serialize($value);
-    }
-
-    private function prepareReturn(string $value): mixed
-    {
-        return unserialize($value);
-    }
-
-    /**
-     * @template TValue
-     *
-     * @param CacheObject<TValue> $cacheObject
-     */
-    private function prepareTtl(CacheObject $cacheObject): int
-    {
-        return (int) $cacheObject->ttl()
-            ->totalSeconds;
+        return $this->repository->forget((string) $cacheObject->key());
     }
 }

src/ValueObjects/HashedKey.php → src/ValueObjects/Keys/HashedKey.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace EriBloo\CacheObjects\ValueObjects;
+namespace EriBloo\CacheObjects\ValueObjects\Keys;
 
 use EriBloo\CacheObjects\Contracts\CacheKey;
 use InvalidArgumentException;

src/ValueObjects/StringKey.php → src/ValueObjects/Keys/StringKey.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace EriBloo\CacheObjects\ValueObjects;
+namespace EriBloo\CacheObjects\ValueObjects\Keys;
 
 use EriBloo\CacheObjects\Contracts\CacheKey;
 

src/ValueObjects/Values/SerializeModifier.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace EriBloo\CacheObjects\ValueObjects\Values;
+
+use EriBloo\CacheObjects\Contracts\CacheValueModifier;
+
+/**
+ * @template TValue
+ *
+ * @implements CacheValueModifier<TValue>
+ */
+final readonly class SerializeModifier implements CacheValueModifier
+{
+    /**
+     * @param class-string[]|bool $allowedClasses
+     */
+    public function __construct(
+        private array|bool $allowedClasses = true,
+    ) {}
+
+    public function onLoad(string $value): mixed
+    {
+        return unserialize($value, [
+            'allowed_classes' => $this->allowedClasses,
+        ]);
+    }
+
+    public function onSave(mixed $value): string
+    {
+        return serialize($value);
+    }
+}

tests/Fixtures/BasicCacheObject.php

@@ -7,7 +7,9 @@
 use Carbon\CarbonInterval;
 use EriBloo\CacheObjects\Concerns\CacheObjectActions;
 use EriBloo\CacheObjects\Contracts\CacheObject;
-use EriBloo\CacheObjects\ValueObjects\StringKey;
+use EriBloo\CacheObjects\Contracts\CacheValueModifier;
+use EriBloo\CacheObjects\ValueObjects\Keys\StringKey;
+use EriBloo\CacheObjects\ValueObjects\Values\SerializeModifier;
 
 /**
  * @implements CacheObject<string>
@@ -30,4 +32,9 @@ public function ttl(): CarbonInterval
     {
         return CarbonInterval::seconds(0);
     }
+
+    public function modifier(): CacheValueModifier
+    {
+        return new SerializeModifier(false);
+    }
 }

tests/Fixtures/HashedCacheObject.php

@@ -7,8 +7,10 @@
 use Carbon\CarbonInterval;
 use EriBloo\CacheObjects\Concerns\CacheObjectActions;
 use EriBloo\CacheObjects\Contracts\CacheObject;
-use EriBloo\CacheObjects\ValueObjects\HashedKey;
-use EriBloo\CacheObjects\ValueObjects\StringKey;
+use EriBloo\CacheObjects\Contracts\CacheValueModifier;
+use EriBloo\CacheObjects\ValueObjects\Keys\HashedKey;
+use EriBloo\CacheObjects\ValueObjects\Keys\StringKey;
+use EriBloo\CacheObjects\ValueObjects\Values\SerializeModifier;
 
 /**
  * @implements CacheObject<string>
@@ -26,4 +28,9 @@ public function ttl(): CarbonInterval
     {
         return CarbonInterval::seconds(0);
     }
+
+    public function modifier(): CacheValueModifier
+    {
+        return new SerializeModifier(false);
+    }
 }