couchrest-uniqueness-validation 0.2.0 → 0.2.1

data/VERSION

@@ -1 +1 @@
-0.2.0
+0.2.1

data/lib/couchrest-uniqueness-validation.rb

@@ -24,7 +24,7 @@ module CouchRest
       
       def unique?(target)
         value = target.validation_property_value(@field_name)
-        value.downcase! if @options[:downcase] && !value.blank?
+        value = value.downcase if @options[:downcase] && !value.blank?
         existing_docs = target.class.view(@options[:view].to_sym, :key => value, :limit => 1, :include_docs => false)['rows']
         
         # normal case when target.new_document? == true and

data/spec/couchrest-uniqueness-validation_spec.rb

@@ -92,11 +92,19 @@ describe CouchRest::Validation::UniquenessValidator do
       @some_doc.should be_valid
     end
 
-    it "works for nil values" do
+    it "should work for nil values" do
       @some_doc.another_prop = nil
       YetAnotherUniqueDoc.should_receive(:view).
         with(:by_another_prop, hash_including(:key => nil, :limit => 1, :include_docs => false))
       lambda{@some_doc.valid?}.should_not raise_error
     end
+
+    it "should not have a side effect of the field being left downcased afterwards" do
+      YetAnotherUniqueDoc.should_receive(:view).
+        with(:by_another_prop, hash_including(:key => @some_doc.another_prop.downcase, :limit => 1, :include_docs => false))
+      @some_doc.should be_valid
+      @some_doc.another_prop.should == 'Some Property Value'
+      @some_doc.another_prop.should_not == 'some property value'
+    end
   end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 2
-  - 0
-  version: 0.2.0
+  - 1
+  version: 0.2.1
 platform: ruby
 authors: 
 - alex