This project is archived and is in readonly mode.
Bounded params in find(...:from)
Add the ability to pass in bounded parameters in the :from part of a find call. This is useful for selecting from a db function or stored procedure which needs parameters from input which is possibly tainted.
-
Dan BarryMy tests look worse and worse every time I see them, but I can't think of a way to make them clearer without writing (and testing in every supported db) a db function that simply returns all of the rows from a table. Should I do that or can anyone think of a better way to test this?
-
John Barnette- Title changed from [PATCH] bounded params in find(...:from) to Bounded params in find(...:from)
-
Dan Barry
Oops, I deleted my github fork while I still had an outstanding patch. Ignore the link in my comment and instead scroll down to the link to the patch on the bottom right.
-
Kyle Hargraves
+1, tests pass, and I've been using this in production for a while. It makes it much cleaner to call database functions from within AR without having to go through find_by_sql.
-
Pratik- State changed from new to invalid
The patch doesn't apply cleanly anymire.
I think it'll be a good idea to discuss this patch/feature in the core mailing list.
Thanks.