Fixed CacheItem implementation and marked internal.

src/Porter/Cache/CacheItem.php

@@ -3,6 +3,9 @@
 
 use Psr\Cache\CacheItemInterface;
 
+/**
+ * @internal Only this library may create instances of this class.
+ */
 final class CacheItem implements CacheItemInterface
 {
     private $key;
@@ -11,7 +14,7 @@ final class CacheItem implements CacheItemInterface
 
     private $hit;
 
-    public function __construct($key, $value, $hit = false)
+    private function __construct($key, $value, $hit)
     {
         $this->key = "$key";
         $this->value = $value;
@@ -35,7 +38,9 @@ public function isHit()
 
     public function set($value)
     {
-        throw new CacheOperationProhibitedException('Cannot directly modify an item\'s value.');
+        $this->value = $value;
+
+        return $this;
     }
 
     public function expiresAt($expiration)

src/Porter/Cache/MemoryCache.php

@@ -11,7 +11,15 @@ class MemoryCache extends \ArrayObject implements CacheItemPoolInterface
 {
     public function getItem($key)
     {
-        return new CacheItem($key, $this->hasItem($key) ? $this[$key] : null, $this->hasItem($key));
+        return call_user_func(
+            \Closure::bind(
+                function () use ($key) {
+                    return new self($key, $this->hasItem($key) ? $this[$key] : null, $this->hasItem($key));
+                },
+                $this,
+                CacheItem::class
+            )
+        );
     }
 
     public function getItems(array $keys = [])

src/Porter/Connector/CachingConnector.php

@@ -3,7 +3,6 @@
 
 use Psr\Cache\CacheItemPoolInterface;
 use ScriptFUSION\Porter\Cache\CacheEnabler;
-use ScriptFUSION\Porter\Cache\CacheItem;
 use ScriptFUSION\Porter\Cache\MemoryCache;
 use ScriptFUSION\Porter\Options\EncapsulatedOptions;
 
@@ -34,7 +33,7 @@ public function fetch($source, EncapsulatedOptions $options = null)
 
         $data = $this->fetchFreshData($source, $options);
 
-        isset($hash) && $this->cache->save(new CacheItem($hash, $data));
+        isset($hash) && $this->cache->save($this->cache->getItem($hash)->set($data));
 
         return $data;
     }

test/Unit/Porter/Cache/MemoryCacheTest.php

@@ -76,14 +76,14 @@ public function testDeleteInvalidItem()
 
     public function testSave()
     {
-        $this->cache->save(new CacheItem('bar', 'baz'));
+        $this->cache->save($this->cache->getItem('bar')->set('baz'));
 
         self::assertSame('baz', $this->cache->getItem('bar')->get());
     }
 
     public function testSaveDeferred()
     {
-        $this->cache->saveDeferred(new CacheItem('bar', 'baz'));
+        $this->cache->save($this->cache->getItem('bar')->set('baz'));
 
         self::assertSame('baz', $this->cache->getItem('bar')->get());
     }