Thank you for the suggestion. All callbacks now are only skipped if an exception is raised and we are removing the special meaning of false, so I think this behavior correct.
On Tue, Aug 4, 2015 at 4:48 PM Bruno Macabeus <bruno.macab...@gmail.com> wrote: > As stated in the documentation > <http://guides.rubyonrails.org/association_basics.html#association-callbacks>: > *If a before_add callback throws an exception, the object does not get > added to the collection* > > This way to prevent that save is very "ugly". > It would be better to keep the standard of the other callback (like > before_create): if it returns false, not saved. > Others users would also like it that way > <http://stackoverflow.com/questions/7273792/silently-skip-add-with-before-add-association-callback-instead-of-raising-an-exc> > . > > -- > You received this message because you are subscribed to the Google Groups > "Ruby on Rails: Core" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to rubyonrails-core+unsubscr...@googlegroups.com. > To post to this group, send email to rubyonrails-core@googlegroups.com. > Visit this group at http://groups.google.com/group/rubyonrails-core. > For more options, visit https://groups.google.com/d/optout. > -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-core+unsubscr...@googlegroups.com. To post to this group, send email to rubyonrails-core@googlegroups.com. Visit this group at http://groups.google.com/group/rubyonrails-core. For more options, visit https://groups.google.com/d/optout.