cached_model 1.2.0 → 1.2.1

data/Rakefile

@@ -8,7 +8,7 @@ $VERBOSE = nil
 
 spec = Gem::Specification.new do |s|
   s.name = 'cached_model'
-  s.version = '1.2.0'
+  s.version = '1.2.1'
   s.summary = 'An ActiveRecord::Base model that caches records'
   s.authors = 'Robert Cottrell, Eric Hodel'
   s.email = 'bob@robotcoop.com'

data/lib/cached_model.rb

@@ -168,7 +168,7 @@ class CachedModel < ActiveRecord::Base
 
     # Fetch the record from the DB
     records = super
-    records.first.cache_store # only one
+    records.first.cache_store unless records.empty? # only one
     return records
   end
 

data/test/test_cached_model.rb

@@ -55,6 +55,7 @@ class ActiveRecord::Base
   end
 
   def self.find_by_sql(query)
+    return [] if query =~ /concrete\.id = -1/
     return [new]
   end
 
@@ -325,6 +326,14 @@ class TestCachedModel < Test::Unit::TestCase
     assert_equal record, Cache.cache[record.cache_key_memcache]
   end
 
+  def test_class_find_by_sql_no_record
+    q = "SELECT * FROM concrete WHERE (concrete.id = -1) LIMIT 1"
+
+    records = Concrete.find_by_sql q
+
+    assert_equal [], records
+  end
+
   def test_class_ttl
     assert_equal 900, CachedModel.ttl
     CachedModel.ttl = 800

metadata

@@ -3,8 +3,8 @@ rubygems_version: 0.8.99
 specification_version: 1
 name: cached_model
 version: !ruby/object:Gem::Version 
-  version: 1.2.0
-date: 2006-08-10 00:00:00 -07:00
+  version: 1.2.1
+date: 2006-08-18 00:00:00 -07:00
 summary: An ActiveRecord::Base model that caches records
 require_paths: 
 - lib