Custom getters in Ruby on Rails

Question

I have a MailingList model that has_may :people

For most of my application, I only want to get people that are active

So @mailing_list.people should only return people that are active

In my model, I can't do

def people
  self.people.find_all{ |p| !p.activated_at.nil? }
end

because that keeps calling itself. What is the ruby/rails way to automatically filter the people. Another possible issue is that I think self.people returns an array of active record objects where self.people.find_all... will return an array. This will cause some of my code to break. It's easy fixes but is there a way to return active record objects? It would be nice to have the option.

Thanks!

Answer 1

This is a perfect example for a named scope:

class Person < ActiveRecord::Base
  named_scope :active, :conditions => 'activated_at is not null'
end

Then just call it:

# equivalent to Person.find(:all, :conditions => 'activated_at is not null')
@active_people = Person.active