2009/7/11 Jeremy Evans <[email protected]>: > One way to do what you want is to have the add_*/remove_*/remove_all_* > methods take additional arguments, and pass those arguments down to > _add_*/_remove_*/_remove_all_*. Then you could just do: > > class Foo > def _add_bar(bar, values={}) > DB[:foos_bars].insert(values.merge > (:foo_id=>id, :bar_id=>bar.id)) > end > end > > I'm not opposed to a patch that does that, assuming it came with > decent specs.
Hi Jeremy, I've made an attempt at implementing your suggestion and have sent you a pull request on github as well. The patch in question is this one: http://github.com/clivecrous/sequel/commit/cb90ac5d9bb13aaffd51735df93ce9184fc761d9 Thanks, Clive --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "sequel-talk" group. 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/sequel-talk?hl=en -~----------~----~----~----~------~----~------~--~---
