[Cache] Use generator in ArrayAdapter

symfony · Jan 25, 2016 · 367e784 · 367e784
1 parent 0d3bde8
commit 367e784
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 7 deletions.
diff --git a/src/Symfony/Component/Cache/Adapter/ArrayAdapter.php b/src/Symfony/Component/Cache/Adapter/ArrayAdapter.php
@@ -58,16 +58,11 @@ public function getItem($key)
      */
     public function getItems(array $keys = array())
     {
-        $f = $this->createCacheItem;
-        $items = array();
-        $now = time();
-
         foreach ($keys as $key) {
-            $isHit = isset($this->expiries[$this->validateKey($key)]) && ($this->expiries[$key] >= $now || !$this->deleteItem($key));
-            $items[$key] = $f($key, $isHit ? $this->values[$key] : null, $isHit);
+            $this->validateKey($key);
         }
 
-        return $items;
+        return $this->generateItems($keys);
     }
 
     /**
@@ -176,4 +171,15 @@ private function validateKey($key)
 
         return $key;
     }
+
+    private function generateItems(array $keys)
+    {
+        $f = $this->createCacheItem;
+
+        foreach ($keys as $key) {
+            $isHit = isset($this->expiries[$key]) && ($this->expiries[$key] >= time() || !$this->deleteItem($key));
+
+            yield $key => $f($key, $isHit ? $this->values[$key] : null, $isHit);
+        }
+    }
 }
diff --git a/src/Symfony/Component/Cache/Tests/Adapter/ArrayAdapterTest.php b/src/Symfony/Component/Cache/Tests/Adapter/ArrayAdapterTest.php
@@ -21,6 +21,7 @@ class ArrayAdapterTest extends CachePoolTest
 {
     protected $skippedTests = array(
         'testDeferredSaveWithoutCommit' => 'Assumes a shared cache which ArrayAdapter is not.',
+        'testSaveWithoutExpire' => 'Assumes a shared cache which ArrayAdapter is not.',
         'testDeferredExpired' => 'Failing for now, needs to be fixed.',
     );
 

diff --git a/src/Symfony/Component/Cache/Tests/Adapter/DoctrineAdapterTest.php b/src/Symfony/Component/Cache/Tests/Adapter/DoctrineAdapterTest.php
@@ -22,6 +22,7 @@ class DoctrineAdapterTest extends CachePoolTest
 {
     protected $skippedTests = array(
         'testDeferredSaveWithoutCommit' => 'Assumes a shared cache which ArrayCache is not.',
+        'testSaveWithoutExpire' => 'Assumes a shared cache which ArrayCache is not.',
         'testDeferredExpired' => 'Failing for now, needs to be fixed.',
     );
 

diff --git a/src/Symfony/Component/Cache/Tests/Adapter/ProxyAdapterTest.php b/src/Symfony/Component/Cache/Tests/Adapter/ProxyAdapterTest.php
@@ -22,6 +22,7 @@ class ProxyAdapterTest extends CachePoolTest
 {
     protected $skippedTests = array(
         'testDeferredSaveWithoutCommit' => 'Assumes a shared cache which ArrayAdapter is not.',
+        'testSaveWithoutExpire' => 'Assumes a shared cache which ArrayAdapter is not.',
         'testDeferredExpired' => 'Failing for now, needs to be fixed.',
     );