Spec and patch for expanded find_by_sql
Reported by Nate Wiger | October 9th, 2009 @ 08:37 AM
So I've decided to take a whack at getting find_by_sql working in a more AR-like fashion, since it appears to be a stopping block for several of us per earlier tickets.
Before getting too far down the wrong path, can you take a look at this spec in my dm-more branch?
http://github.com/nateware/dm-more/commit/e975899a490ef06c6ee559016...
Thanks,
Nate
Comments and changes to this ticket
-
Dan Kubb (dkubb) November 6th, 2009 @ 01:50 AM
- State changed from “new” to “confirmed”
Nate, sorry this took me so long to respond to. I must've missed this when reviewing tickets.
The behavior you specified looks to be correct. In general the goal with dm-ar-finders, and find_by_sql specifically, is that we get as close behavior as we can to AR finder methods. Of course due to differences in how DM maps properties to columns, I can understand cases where we need to differ, but in general I will always support patches that bring the behavior closer to AR's.
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
Dan Kubb (dkubb)