Как перевести оператор find_by_sql в named_scope? - PullRequest
Новая
       15

Как перевести оператор find_by_sql в named_scope?

0 голосов
/ 25 мая 2010

Как мне перевести следующее в 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

Вот что у меня есть сейчас, но, похоже, это не работает: 

  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, это то, что сработало; 

  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' }
      }

1 Ответ

0 голосов
/ 25 мая 2010

Проблема в том, что вы заменили выбор из users, comments на внутреннее объединение с comments. Чтобы правильно перевести SQL в named_scope, используйте это: 

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' } }

И вы не можете иметь значение по умолчанию для лямбда-параметров, поэтому вы не можете использовать |cutoff = 0|.

...