Merge pull request #1 from Nyholm/long_keys

Fix for long keys

Author: Tobias Bohman

composer.json

@@ -31,7 +31,7 @@
         "ext-memcached":        "*",
         "psr/cache":            "1.0.0",
         "cache/adapter-common": "^0.1",
-        "cache/taggable-cache": "^0.4",
+        "cache/taggable-cache": "^0.3",
         "cache/hierarchical-cache": "^0.1"
     },
     "require-dev":

src/MemcachedCachePool.php

@@ -40,7 +40,7 @@ public function __construct(\Memcached $cache)
 
     protected function fetchObjectFromCache($key)
     {
-        return $this->cache->get($this->getHierarchyKey($key));
+        return $this->cache->get($this->trimKey($this->getHierarchyKey($key)));
     }
 
     protected function clearAllObjectsFromCache()
@@ -51,8 +51,8 @@ protected function clearAllObjectsFromCache()
     protected function clearOneObjectFromCache($key)
     {
         $this->commit();
-        $key = $this->getHierarchyKey($key, $path);
-        $this->cache->increment($path, 1, 0);
+        $key = $this->trimKey($this->getHierarchyKey($key, $path));
+        $this->cache->increment($this->trimKey($path), 1, 0);
         $this->clearHierarchyKeyCache();
 
         if ($this->cache->delete($key)) {
@@ -69,13 +69,28 @@ protected function storeItemInCache($key, CacheItemInterface $item, $ttl)
             $ttl = 0;
         }
 
-        $key = $this->getHierarchyKey($key);
+        $key = $this->trimKey($this->getHierarchyKey($key));
 
         return $this->cache->set($key, $item, $ttl);
     }
 
     protected function getValueFormStore($key)
     {
-        return $this->cache->get($key);
+        return $this->cache->get($this->trimKey($key));
+    }
+
+    /**
+     * Calculate a key. If it is more than 250 chars we should hash the key.
+     *
+     * @param $key
+     * @param null $ref
+     */
+    private function trimKey($key)
+    {
+        if (strlen($key) < 250) {
+            return $key;
+        }
+        // This should maybe be logged
+        return sha1($key);
     }
 }