On Tuesday, August 7, 2018 at 12:15:22 PM UTC-7, Alexander Popov wrote:
>
> OK.
>
> So, if I want to:
>
>    - Get result of transaction if everything is OK
>    - Get result of `after_rollback` block if there is `Sequel::Rollback`
>    - And get raised any other exception 
>
> I should use local variables like:
>
> def foo
>   result = nil
>   DB.transaction do
>     DB.after_rollback { result = validation_errors }
>     raise Sequel::Rollback unless entity.valid?
>     result = entity.save
>   end
>   result
> end
>
> Right? There is no more compact way?
>

That should work if you want to return the entity if the transaction 
commits, the validation errors if it is not, and any exceptions to be 
raised.

Your particular example could be made simpler, relying on the fact that 
Sequel::Model uses transactions when saving:

def foo
  entity.save(raise_on_failure: false) || validation_errors
end

However, if you are doing other things in your transaction block, you may 
need your more elaborate construction.  Note that in your code, validations 
are run twice (once in valid? and once in save).

Thanks,
Jeremy

-- 
You received this message because you are subscribed to the Google Groups 
"sequel-talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sequel-talk.
For more options, visit https://groups.google.com/d/optout.

Reply via email to