Add save! to raise exception if things go wrong

[leomayleomay]

No description provided.

[kirs]

AFAIK || syntax if prefered over or

[kaspth]

@kirs is right in the syntax. But overall this is a no go for me because we're not targeting Active Record as the Rails integration point, but Active Model.
A record invalid or form invalid error using the Active Form namespace would be fine for me.

Kasper

Den 25/11/2014 kl. 22.17 skrev Kir Shatrov [email protected]:

In lib/active_form/base.rb:

@@ -42,6 +42,10 @@ def save
end
end

• def save!

•  save or raise ActiveRecord::RecordInvalid.new(self)
  

  AFAIK || syntax if prefered over or

—
Reply to this email directly or view it on GitHub.

[kirs]

we're not targeting Active Record as the Rails integration point, but Active Model.
A record invalid or form invalid error using the Active Form namespace would be fine for me.

Good point, I totally agree.

[kirs]

I'll be happy to merge it right after #18