#1089 confirmed
Nate Wiger

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?



Comments and changes to this ticket

  • Dan Kubb (dkubb)

    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.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

People watching this ticket