Merge pull request #27 from neonstalwart/now

put `now` in scope for the usage in `get`
persvr · May 15, 2012 · 0108492 · 0108492
2 parents b9a3040 + 051daba
commit 0108492
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/store/cache.js b/store/cache.js
@@ -1,8 +1,7 @@
 /**
  * This is a wrapper store that can add caching to a store
  */
-var table = {},
-	when = require("promised-io/promise").when;
+var when = require("promised-io/promise").when;
 
 exports.Cache = function(store, cacheStore, options){
 	options = options || {};
@@ -11,9 +10,10 @@ exports.Cache = function(store, cacheStore, options){
 	var cleanupInterval = options.cleanupInterval || 1000;
 	var lastAccess = {};
 	var nextCheck = new Date().getTime();
+	var now;
 	cleanup();
 	function cleanup(){
-		var now = new Date().getTime();
+		now = new Date().getTime();
 		if(now > nextCheck){
 			nextCheck = now + cleanupInterval;
 			return when(cacheStore.query("expires<$1", {parameters:[now]}), function(results){
@@ -29,7 +29,7 @@ exports.Cache = function(store, cacheStore, options){
 			lastAccess[id] = now++;
 			if(!cached){
 				if(store){
-					cacheStore.put(cached = store.get[id], {id:id});
+					cacheStore.put(cached = store.get(id), {id:id});
 				}
 			}
 			return cached;
@@ -71,4 +71,4 @@ exports.Cache = function(store, cacheStore, options){
 			cacheStore["delete"](id);
 		}
 	};
-};
+};