Permalink
Browse files

MemCacheStore#decrement should use data instance variable not local v…

…ariable [#521 state:resolved]
  • Loading branch information...
josh committed Jul 16, 2008
1 parent 73ade4f commit 7ae2105d57d3c08bebde44bd72093dd43c48d613
Showing with 15 additions and 15 deletions.
  1. +15 −15 activesupport/lib/active_support/cache/mem_cache_store.rb
@@ -29,8 +29,8 @@ def read(key, options = nil)
nil
end
- # Set key = value. Pass :unless_exist => true if you don't
- # want to update the cache if the key is already set.
+ # Set key = value. Pass :unless_exist => true if you don't
+ # want to update the cache if the key is already set.
def write(key, value, options = nil)
super
method = options && options[:unless_exist] ? :add : :set
@@ -56,33 +56,33 @@ def exist?(key, options = nil)
!read(key, options).nil?
end
- def increment(key, amount = 1)
+ def increment(key, amount = 1)
log("incrementing", key, amount)
-
- response = @data.incr(key, amount)
+
+ response = @data.incr(key, amount)
response == Response::NOT_FOUND ? nil : response
- rescue MemCache::MemCacheError
+ rescue MemCache::MemCacheError
nil
end
def decrement(key, amount = 1)
log("decrement", key, amount)
-
- response = data.decr(key, amount)
+
+ response = @data.decr(key, amount)
response == Response::NOT_FOUND ? nil : response
- rescue MemCache::MemCacheError
+ rescue MemCache::MemCacheError
nil
- end
-
+ end
+
def delete_matched(matcher, options = nil)
super
raise "Not supported by Memcache"
- end
-
+ end
+
def clear
@data.flush_all
- end
-
+ end
+
def stats
@data.stats
end

0 comments on commit 7ae2105

Please sign in to comment.