remove Item

Author: mmelograno

src/SplitIO/Component/Cache/Item.php

@@ -34,8 +34,7 @@ public function __construct(array $options = array())
      *   If the $key string is not a legal value a \Psr\Cache\InvalidArgumentException
      *   MUST be thrown.
      *
-     * @return \SplitIO\Component\Cache\Item
-     *   The corresponding Cache Item.
+     * @return string
      */
     public function getItem($key)
     {

src/SplitIO/Component/Cache/Pool.php

@@ -36,7 +36,7 @@ public function isInSegment($segmentName, $key)
      */
     public function getChangeNumber($segmentName)
     {
-        $since = Di::getCache()->getItem(self::getCacheKeyForSinceParameter($segmentName))->get();
+        $since = Di::getCache()->getItem(self::getCacheKeyForSinceParameter($segmentName));
         // empty check for nullable value
         return (empty($since)) ? -1 : $since;
     }

src/SplitIO/Component/Cache/SegmentCache.php

@@ -37,7 +37,7 @@ private static function getSplitNameFromCacheKey($key)
      */
     public function getChangeNumber()
     {
-        $since = Di::getCache()->getItem(self::getCacheKeyForSinceParameter())->get();
+        $since = Di::getCache()->getItem(self::getCacheKeyForSinceParameter());
         // empty check for nullable value
         return (empty($since)) ? -1 : $since;
     }
@@ -50,7 +50,7 @@ public function getSplit($splitName)
     {
         $cache = Di::getCache();
         $cacheItem = $cache->getItem(self::getCacheKeyForSplit($splitName));
-        return $cacheItem->get();
+        return $cacheItem;
     }
 
     /**
@@ -63,7 +63,7 @@ public function getSplits($splitNames)
         $cacheItems = $cache->getItems(array_map('self::getCacheKeyForSplit', $splitNames));
         $toReturn = array();
         foreach ($cacheItems as $key => $value) {
-            $toReturn[self::getSplitNameFromCacheKey($key)] = $value->get();
+            $toReturn[self::getSplitNameFromCacheKey($key)] = $value;
         }
         return $toReturn;
     }
@@ -100,7 +100,7 @@ public function trafficTypeExists($trafficType)
     {
         $cache = Di::getCache();
 
-        $count = $cache->getItem(self::getCacheKeyForTrafficType($trafficType))->get();
+        $count = $cache->getItem(self::getCacheKeyForTrafficType($trafficType));
         // empty check for nullable value
         return (empty($count) || $count < 1) ? false : true;
     }

src/SplitIO/Component/Cache/SplitCache.php

@@ -14,7 +14,7 @@ public function getKeys($pattern = '*');
 
     /**
      * @param string $key
-     * @return \SplitIO\Component\Cache\Item
+     * @return string
      */
     public function getItem($key);
 

src/SplitIO/Component/Cache/Storage/Adapter/CacheStorageAdapterInterface.php

@@ -205,19 +205,11 @@ private function getRedisConfiguration($options)
 
     /**
      * @param string $key
-     * @return \SplitIO\Component\Cache\Item
+     * @return string
      */
     public function getItem($key)
     {
-        $item = new Item($key);
-
-        $redisItem = $this->client->get($key);
-
-        if ($redisItem !== null) {
-            $item->set($redisItem);
-        }
-
-        return $item;
+        return $this->client->get($key);
     }
 
 
@@ -231,11 +223,7 @@ public function getItem($key)
      *   If any of the keys in $keys are not a legal value a \Psr\Cache\InvalidArgumentException
      *   MUST be thrown.
      *
-     * @return array|\Traversable
-     *   A traversable collection of Cache Items keyed by the cache keys of
-     *   each item. A Cache item will be returned for each key, even if that
-     *   key is not found. However, if no keys are specified then an empty
-     *   traversable MUST be returned instead.
+     * @return array
      */
     public function getItems(array $keys = array())
     {
@@ -245,10 +233,7 @@ public function getItems(array $keys = array())
         }
         $values = $this->client->mget($keys);
         foreach ($keys as $index => $key) {
-            $toReturn[$key] = new Item($key);
-            if (!is_null($values[$index])) {
-                $toReturn[$key]->set($values[$index]);
-            }
+            $toReturn[$key] = $values[$index];
         }
         return $toReturn;
     }

src/SplitIO/Component/Cache/Storage/Adapter/PRedis.php

@@ -25,7 +25,7 @@ public function __construct(PRedis $cacheAdapter)
 
     /**
      * @param string $key
-     * @return \SplitIO\Component\Cache\Item
+     * @return string
      */
     public function getItem($key)
     {
@@ -35,7 +35,7 @@ public function getItem($key)
             Di::getLogger()->critical("An error occurred getting " . $key . " from redis.");
             Di::getLogger()->critical($e->getMessage());
             Di::getLogger()->critical($e->getTraceAsString());
-            return new Item($key);
+            return null;
         }
     }
 

src/SplitIO/Component/Cache/Storage/Adapter/SafeRedisWrapper.php

@@ -50,7 +50,7 @@ public function testRedisWithOnlyParameters()
         $predisClient->set('this_is_a_test_key', 'this-is-a-test-value');
 
         $value = $predis->getItem('this_is_a_test_key');
-        $this->assertEquals('this-is-a-test-value', $value->get());
+        $this->assertEquals('this-is-a-test-value', $value);
 
         $predisClient->del('this_is_a_test_key');
     }
@@ -76,7 +76,7 @@ public function testRedisWithParametersAndPrefix()
         $predisClient->set('test-redis-assertion.this_is_a_test_key', 'this-is-a-test-value');
 
         $value = $predis->getItem('this_is_a_test_key');
-        $this->assertEquals('this-is-a-test-value', $value->get());
+        $this->assertEquals('this-is-a-test-value', $value);
 
         $predisClient->del('test-redis-assertion.this_is_a_test_key');
     }
@@ -103,7 +103,7 @@ public function testRedisWithParametersPrefixAndSentinels()
         $predisClient->set('test-redis-assertion.this_is_a_test_key', 'this-is-a-test-value');
 
         $value = $predis->getItem('this_is_a_test_key');
-        $this->assertEquals('this-is-a-test-value', $value->get());
+        $this->assertEquals('this-is-a-test-value', $value);
 
         $predisClient->del('test-redis-assertion.this_is_a_test_key');
     }

tests/Suite/Adapter/RedisAdapterTest.php

@@ -16,7 +16,7 @@ public function __construct(PRedis $predis)
 
     /**
      * @param string $key
-     * @return \SplitIO\Component\Cache\Item
+     * @return string
      */
     public function getItem($key)
     {

tests/Suite/Redis/PRedisReadOnlyMock.php

@@ -40,7 +40,7 @@ public function testAllMethodsException()
         $refProperty->setValue($predisAdapter, $predisMock);
         $safeRedisWrapper = new SafeRedisWrapper($predisAdapter);
 
-        $this->assertEquals(false, $safeRedisWrapper->getItem("some")->get());
+        $this->assertEquals(false, $safeRedisWrapper->getItem("some"));
         $this->assertEquals(array(), $safeRedisWrapper->getItems(array("some")));
         $this->assertEquals(false, $safeRedisWrapper->isOnList("some", "another"));
         $this->assertEquals(array(), $safeRedisWrapper->getKeys("some"));