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.

