Bob wrote:
[...]
> For obvious reasons, this is horribly inefficient so I have refactored
> that code to:
> 
> def self.has_open_billing_window
>   Customer.find(:all, :joins => :billing_windows, :conditions =>
> "billing_windows.closed_on IS NULL")
> end
> 
> 
> This definitely seems cleaner but I don't like the fact that the
> conditions clause is specific to the database query language.  

The clause isn't specific to the DB query language.  It contains only 
standard SQL, and so should work perfectly in any SQL database.

Granted, it won't necessary work in a non-SQL database, but those are 
typically structured very differently from SQL databases anyway.

> Does
> anyone know a better way to accomplish this?

You don't need one.

> 
> Bob

Best,
--
Marnen Laibow-Koser
http://www.marnen.org
[email protected]
-- 
Posted via http://www.ruby-forum.com/.

-- 
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: 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/rubyonrails-talk?hl=en.

Reply via email to