active_hash 0.10.0 → 1.0.0

data/CHANGELOG

@@ -1,3 +1,6 @@
+2013-06-24 (v1.0.0)
+  - save is a no-op on existing records, instead of raising an error (issue #63)
+
 2013-06-24 (v0.10.0)
   - added ActiveYaml::Aliases module so you can DRY up your repetitive yaml (brett-richardson)
 

data/lib/active_hash/base.rb

@@ -67,6 +67,12 @@ module ActiveHash
         end
       end
 
+      def exists?(record)
+        if record.id.present?
+          record_index[record.id.to_s].present?
+        end
+      end
+
       def insert(record)
         @records ||= []
         record.attributes[:id] ||= next_id
@@ -444,7 +450,9 @@ module ActiveHash
     end
 
     def save(*args)
-      self.class.insert(self)
+      unless self.class.exists?(self)
+        self.class.insert(self)
+      end
       true
     end
 

data/lib/active_hash/version.rb

@@ -1,5 +1,5 @@
 module ActiveHash
   module Gem
-    VERSION = "0.10.0"
+    VERSION = "1.0.0"
   end
 end

data/spec/active_hash/base_spec.rb

@@ -898,6 +898,16 @@ describe ActiveHash, "Base" do
       Country.dirty.should be_true
     end
 
+    it "it is a no-op if the object has already been added to the collection" do
+      Country.all.should be_empty
+      country = Country.new :id => 1, :name => "foo"
+      country.save
+      country.name = "bar"
+      country.save
+      country.save!
+      Country.all.should == [country]
+    end
+
   end
 
   describe ".create" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_hash
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 1.0.0
   prerelease: 
 platform: ruby
 authors:
@@ -152,7 +152,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 514643025697724371
+      hash: -1921449152937241746
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: