On Aug 26, 6:34 pm, Mike Luu <[email protected]> wrote: > On Aug 26, 2010, at 3:41 PM, Jeremy Evans wrote: > > > On Aug 24, 9:44 pm, Jeremy Evans <[email protected]> wrote: > >> Try the patch athttp://pastie.org/1114166.txt. I haven't had a > >> chance to run it through the full test suite, but it certainly appears > >> to work. I plan to push it to Github tomorrow after running it > >> through the full test suite, but if you could give me any feedback on > >> it, I'd appreciate it. > > > I didn't get any feedback on this, but it appears to work in my > > testing, so I committed it: > > >http://github.com/jeremyevans/sequel/commit/183bf388b171f43740dc91125... > >http://github.com/jeremyevans/sequel/commit/975f5f70f6c95ceea02a17092... > > > Thanks, > > Jeremy > > Jeremy, > > Sorry I didn't get back to you any sooner... > Thank for your work on this issue! We appreciate how responsive you always > are. > > Regarding the original problem, my use case also involved aliasing the > :join_table option of a many_to_many association. I have fix for that last > bit here: > > http://gist.github.com/552592 > > Other than that, your patch fixed the problem.
Mike, Thanks for the patch, I applied it with minor changes: http://github.com/jeremyevans/sequel/commit/c28bfb285b73d73f5c5a49083cbea7b749ef006b Jeremy -- 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.
