Improve model destroy (#91)

Author: santib

.reek.yml

@@ -26,6 +26,7 @@ detectors:
     exclude:
       - 'YAAF::Form#promote_legacy_errors'
       - 'YAAF::Form#save_models'
+      - 'YAAF::Form#validate_models'
   InstanceVariableAssumption:
     enabled: false
   IrresponsibleModule:
@@ -46,6 +47,7 @@ detectors:
     enabled: true
     exclude:
       - 'YAAF::Form#save_models'
+      - 'YAAF::Form#validate_models'
   MissingSafeMethod:
     enabled: false
     exclude: []

lib/yaaf/form.rb

@@ -62,7 +62,11 @@ def save_models(options)
     end
 
     def validate_models
-      models.each { |model| promote_errors(model) if model.invalid? }
+      models.each do |model|
+        next if model.respond_to?(:marked_for_destruction?) && model.marked_for_destruction?
+
+        promote_errors(model) if model.invalid?
+      end
     end
 
     def promote_errors(model)