[DoctrineBridge] Fixed an issue with DoctrineParserCache

symfony · Jan 1, 2014 · 16728f7 · 16728f7
1 parent e0402ba
commit 16728f7
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/src/Symfony/Bridge/Doctrine/ExpressionLanguage/DoctrineParserCache.php b/src/Symfony/Bridge/Doctrine/ExpressionLanguage/DoctrineParserCache.php
@@ -35,7 +35,11 @@ public function __construct(Cache $cache)
      */
     public function fetch($key)
     {
-        return $this->cache->fetch($key);
+        if (false === $value = $this->cache->fetch($key)) {
+            return null;
+        }
+
+        return $value;
     }
 
     /**

diff --git a/src/Symfony/Bridge/Doctrine/Tests/ExpressionLanguage/DoctrineParserCacheTest.php b/src/Symfony/Bridge/Doctrine/Tests/ExpressionLanguage/DoctrineParserCacheTest.php
@@ -29,6 +29,19 @@ public function testFetch()
         $this->assertEquals('bar', $result);
     }
 
+    public function testFetchUnexisting()
+    {
+        $doctrineCacheMock = $this->getMock('Doctrine\Common\Cache\Cache');
+        $parserCache = new DoctrineParserCache($doctrineCacheMock);
+
+        $doctrineCacheMock
+            ->expects($this->once())
+            ->method('fetch')
+            ->will($this->returnValue(false));
+
+        $this->assertNull($parserCache->fetch(''));
+    }
+
     public function testSave()
     {
         $doctrineCacheMock = $this->getMock('Doctrine\Common\Cache\Cache');