freeform 2.0.0 → 2.0.1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZmM3NmYwN2Q0MzBjODk2MTRlNzA3NjU0YWRjMWI1YTY3MjMxNjI1Yw==
+    YWE2NTA4YmE3ZWY3ZjNjMDkzM2VmOTA4NzU5ZjI4Mjc3ZGQ0NTFkOA==
   data.tar.gz: !binary |-
-    YjVmOTEzMDZmNTNiNmQ2MGIwZTllMGRjMmI5NWU4ODFiYWFhMzhjMw==
+    Y2U1NjU4ZWJkZDIzOGMwODkwMDY5YTY3ZmFmOTRkOTdjMTRlZTA3Yg==
 SHA512:
   metadata.gz: !binary |-
-    YjU2ZDBlNTA2ZjNkZDdjZjgyMzY4NTg0NWRjZjM0ZTI0NjUwNWI5ZjZmZWMy
-    ZjUxNWFhZWQ2ZjZmODMyYzdmYjI4ZjJkNGIzNmEyYjhiNzNkMzMyY2E3NDMx
-    ZDgyOWI2M2ZlNjZiMTQxNDBkYzdjZWVmOTg0NzE3YzlhMDVlMWY=
+    MmJiNTNlMDUyM2U1ZThlZDJhODFkYzNjOTVlNmQ5MzAzYjViZmE3ZWRkZGIz
+    Zjc1Y2ZmNTY0MDY3NjhmYWZjNWM4MGRhNDM0Y2M2YTVhOTk5ZWFmNGE0NTgw
+    MDJjMmNjOWRiN2I0ZTlmYWFjZWY1ZDhmNmMxZDYzMGY3OGVjOWY=
   data.tar.gz: !binary |-
-    NTZhNjZjYmEzZWQxOWI4YTZmMjlhMjg5MTQ4OGJiYWY1MzQyZmE3NDVlYjA0
-    M2I4MGM2NDhkZGRkODdiYjZjMTU0YWRhMzRhNGQ5OTEyNDU4OTZkNTI0ZDJm
-    OTNiMTc0ZmU1YzZhMTE2ODBiZTIxOTQ4NTk2NWY2ZTA5YjA2NGQ=
+    YjhhZWZhZGJhYjA4NjYwZTljM2ExNDE5NGJjYzAxNzJkZjhmM2RjMTdjZmYw
+    NDgyMmVhNjFjNjE0Y2ZiN2ZkNzFlOGU0NTZhOTViMTNhMjQ2OTYyM2JiMTlj
+    MmIwNzU1MjlmYTEwZGUwNWJhZDczMTcxMzFhZjZiZmEwZmIzOWM=

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    freeform (2.0.0rc1)
+    freeform (2.0.0)
       activemodel
       formtastic
       rails (>= 3.2.0)

data/lib/freeform/form.rb

@@ -35,11 +35,10 @@ module FreeForm
       initialize_child_models
     end
 
-    def save
+    def save(*args)
       if valid?
         before_save
-        models.each { |m| m.destroy if m.marked_for_destruction? }
-        models.each { |m| m.save unless m.marked_for_destruction? }
+        persist_models
         after_save
         true
       else
@@ -47,7 +46,7 @@ module FreeForm
       end
     end
 
-    def save!
+    def save!(*args)
       if valid?
         save
       else
@@ -75,5 +74,11 @@ module FreeForm
         send("initialize_#{c}")
       end
     end
+
+    def persist_models
+      models.each { |m| m.destroy if m.marked_for_destruction? }
+      # We skip validation for underlying models, since we've already run our validation.
+      models.each { |m| m.save(:validate => false) unless m.marked_for_destruction? }
+    end
   end
 end

data/lib/freeform/version.rb

@@ -1,3 +1,3 @@
 module Freeform
-  VERSION = "2.0.0"
+  VERSION = "2.0.1"
 end

data/spec/dummy/app/models/task.rb

@@ -4,4 +4,5 @@ class Task < ActiveRecord::Base
   
   validates :project, :presence => true
   validates :start_date, :presence => true
+  validates_associated :milestones
 end

data/spec/form/persistence_spec.rb

@@ -185,7 +185,7 @@ describe FreeForm::Form do
       end
 
       describe "#save" do
-        it "should return true on 'save'" do
+        it "should return true on 'save'", :failing => true do
           form.save.should be_true
         end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: freeform
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - brycesenz