> > > - Extend "destroy_if" to accept a Proc which returns either true or > false as well > > Oops, that's wrong, I mean "extend the allow_destroy option" of course.
Not too many opinions thus far. :-) It basically boils down to this: Should the reject_if option only be relevant for creates and updates or for deletes as well? I tend towards "yes", but a patch for this will change the behaviour of allow_destroy in conjunction with reject_if. Such an API change might cause headaches on existing applications. (A temporary warning if both options are set might be useful.) Opinions anyone? -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To view this discussion on the web visit https://groups.google.com/d/msg/rubyonrails-core/-/4byqu8qPUOcJ. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en.
