On Tue, Apr 19, 2011 at 09:41:21AM +0200, Arthur Taylor wrote: > Update - That got merged (thanks!). So only the one question. > > Arthur > > On 18/04/11 14:37, Arthur Taylor wrote: > >Hi all, > > > >I've been playing around with Arel this week (actually outside of > >Rails) and found an issue with the clone implementation. I've > >added a patch and a testcase here: > > > >https://github.com/codders/arel/commit/660f706491ac012cb133554cffeaa6b9ae6046e9 > > > > > >and made a pull request against the rails/arel repo. > > > >I've also been playing around with support for ordering on > >Expressions (e.g. ORDER BY COUNT(DISTINCT blah) DESC): > > > >https://github.com/codders/arel/commit/10a9fe64b87579592307ffbc3b18bf69c5bcf15c > > > > > >and even more experimentally, with adding placeholders to allow a > >query to be evaluated multiple times in different dynamic > >contexts: > > > >https://github.com/codders/arel/commit/4b9ae3de9edee084303e6f29145b0e4ce5e04481
There are no tests for this, so I'm not sure what it actually does. Is "placeholder" in the SQL BNF? > >So... a) How can I get my bugfix merged - who should I talk to, > >what else do you need? > >b) What's the best forum for discussing / getting involved in Arel > >development. For now, we should discuss here. I'll open a mailing list soon, and we can move there eventually. :-) -- Aaron Patterson http://tenderlovemaking.com/
pgpoRtNpVAIHs.pgp
Description: PGP signature
