Comment traduire l'instruction find_by_sql en named_scope?

Question

Comment est-ce que je traduis ce qui suit en un named_scope?

def self.commentors(cutoff=0) 
    return User.find_by_sql("select users.*, count(*) as total_comments from users, comments 
     where (users.id = comments.user_id) and (comments.public_comment = 1) and (comments.aasm_state = 'posted') and (comments.talkboard_user_id is null) 
     group by users.id having total_comments > #{cutoff} order by total_comments desc") 
    end 

Voici ce que j'ai en ce moment, mais il ne semble pas fonctionner:

named_scope :commentors, lambda { |count=0| 
    { :select => "users.*, count(*) as total_comments", 
     :joins => :comments, 
     :conditions => { :comments => { :public_comment => 1, :aasm_state => 'posted', :talkboard_user_id => nil} }, 
     :group => "users.id", 
     :having => "total_comments > #{count.to_i}", 
     :order => "total_comments desc" 
    } 
    } 

Ulimately, c'est ce qui a fonctionné;

named_scope :commentors, lambda { |*args| 
     { :select => 'users.*, count(*) as total_comments', 
     :joins => :comments, 
     :conditions => { :comments => { :public_comment => 1, :aasm_state => 'posted', :talkboard_user_id => nil} }, 
     :group => 'users.id', 
     :having => ['total_comments > ?', args.first || 0], 
     :order => 'total_comments desc' } 
     } 

Answer 1

Le problème que vous avez est parce que vous avez remplacé la sélection de users, comments avec une jointure avec comments. Pour traduire correctement le SQL à un named_scope, utilisez ceci:

named_scope :commentors, lambda { |cutoff| 
    { :select => 'users.*, count(*) as total_comments', 
     :conditions => { :comments => { :public_comment => 1, :aasm_state => 'posted', :talkboard_user_id => nil } }, 
     :from => 'users, comments', 
     :group => 'users.id', 
     :having => ['total_comments > ?', cutoff], 
     :order => 'total_comments desc' } } 

Et vous ne pouvez pas avoir une valeur par défaut pour les paramètres lambda, de sorte que vous ne pouvez pas utiliser |cutoff = 0|.