ohm 1.0.0.rc2 → 1.0.0.rc3

data/lib/ohm.rb

@@ -1433,9 +1433,7 @@ module Ohm
       model.indices.each do |att|
         val = atts[att.to_s]
 
-        if val
-          model.key[:indices][att][val].srem(id)
-        end
+        model.key[:indices][att][val].srem(id)
       end
     end
 

data/test/filtering.rb

@@ -61,6 +61,19 @@ test "#except" do |john, jane|
   assert res.include?(jane)
 end
 
+test "indices bug related to a nil attribute" do |john, jane|
+  # First we create a record with a nil attribute
+  out = User.create(:status => nil, :lname => "Doe")
+
+  # Then, we update the old nil attribute to a different
+  # non-nil, value.
+  out.update(status: "inactive")
+
+  # At this point, the index for the nil attribute should
+  # have been cleared.
+  assert_equal 0, User.db.scard("User:indices:status:")
+end
+
 test "#union" do |john, jane|
   included = User.create(:status => "inactive", :lname => "Doe")
   excluded = User.create(:status => "super", :lname => "Doe")

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ohm
 version: !ruby/object:Gem::Version
-  version: 1.0.0.rc2
+  version: 1.0.0.rc3
   prerelease: 6
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-06 00:00:00.000000000 Z
+date: 2012-04-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nest
-  requirement: &2151942880 !ruby/object:Gem::Requirement
+  requirement: &70347702087860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -22,10 +22,10 @@ dependencies:
         version: '1.0'
   type: :runtime
   prerelease: false
-  version_requirements: *2151942880
+  version_requirements: *70347702087860
 - !ruby/object:Gem::Dependency
   name: scrivener
-  requirement: &2151941460 !ruby/object:Gem::Requirement
+  requirement: &70347702087160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -33,10 +33,10 @@ dependencies:
         version: 0.0.3
   type: :runtime
   prerelease: false
-  version_requirements: *2151941460
+  version_requirements: *70347702087160
 - !ruby/object:Gem::Dependency
   name: cutest
-  requirement: &2151940600 !ruby/object:Gem::Requirement
+  requirement: &70347702086580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -44,10 +44,10 @@ dependencies:
         version: '0.1'
   type: :development
   prerelease: false
-  version_requirements: *2151940600
+  version_requirements: *70347702086580
 - !ruby/object:Gem::Dependency
   name: batch
-  requirement: &2151936780 !ruby/object:Gem::Requirement
+  requirement: &70347702086060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -55,7 +55,7 @@ dependencies:
         version: 0.0.1
   type: :development
   prerelease: false
-  version_requirements: *2151936780
+  version_requirements: *70347702086060
 description: Ohm is a library that allows to store an object in Redis, a persistent
   key-value database. It includes an extensible list of validations and has very good
   performance.