Fix $lifeTime condition in Cache::doSave (#18)

Author: davidkrmela

src/Cache.php

@@ -108,7 +108,7 @@ protected function doSave($id, $data, $lifeTime = 0)
 	protected function doSaveDependingOnFiles($id, $data, array $files, $lifeTime = 0)
 	{
 		$dp = [NCache::TAGS => ['doctrine'], NCache::FILES => $files];
-		if ($lifeTime !== 0) {
+		if ($lifeTime > 0) {
 			$dp[NCache::EXPIRE] = time() + $lifeTime;
 		}
 

tests/KdybyTests/DoctrineCache/CacheTest.phpt

@@ -42,6 +42,18 @@ class CacheTest extends \Tester\TestCase
 		Assert::false($cache->fetch('nonexistent-key'));
 	}
 
+	public function testZeroAndNullLifetime()
+	{
+		$storage = new MemoryStorage();
+		$cache = new NetteCacheAdapter($storage, 'ns');
+
+		$cache->save('foo1', 'data', 0);
+		$cache->save('foo2', 'data', NULL);
+
+		Assert::true($cache->contains('foo1'));
+		Assert::true($cache->contains('foo2'));
+	}
+
 }
 
 (new CacheTest())->run();