Refactorisation d'une recherche pour les enfants de parents créés après un certain temps

Question

J'essaie de trouver des commentaires créés après un datetime qui n'ont pas été créés par l'utilisateur actuel.

Au début, je l'ai fait ..

current_user.comments.find(:all, :conditions=>["created_at > ? AND user_id != ?", 
          current_user.last_checked_mail, current_user]) 

mais le problème ici est que parce qu'il commence par le modèle utilisateur, ses commentaires ne trouvant exclusivement faites par cet utilisateur.

Ainsi, au lieu, j'ai commencé de chercher tous les commentaires associés à l'utilisateur, et les enfants des commentaires tant que leurs user_id ne current_user

Comment.find(:all, :conditions => ["user_id = ?", current_user]).select { |c| 
    c.comments.find(:all, :conditions => ["user_id != ?", current_user]) 
} 

Mais il semble que dessine encore chaque commentaire lié à la current_user, et pas leurs enfants.

Mon modèle Commentaire:

belongs_to :commentable, :polymorphic => true 
has_many :comments, :as => :commentable 

Answer 1

Vous avez donc une relation parent dans votre modèle de commentaire? Utiliser ça!

class Comment < ActiveRecord::Base 

    belongs_to :user 
    belongs_to :parent, :class_name => "Comment" 

    def self.after(date) 
    where "created_at > ?", date 
    end 

    def self.replies_for(comments) 
    where :parent_id => comments.map(&:id) 
    end 

    def self.exclude_user(user_id) 
    where "user_id != ?", user_id 
    end 

end 

class User < ActiveRecord::Base 

    has_many :comments 

    def new_comments 
     comments.after(last_check_mail) 
    end 

    def new_responses 
     Comment.replies_for(new_comments).after(last_check_mail).exclude_user(id) 
    end 

end 

current_user.new_responses 

PS. Ceci est pour Rails 3.

Answer 2

Essayez:

Comment.find(:all, :conditions => ["created_at > ? AND user_id != ?", current_user.last_checked_mail, current_user]) 

Answer 3

Essayez ceci:

Comment.all(:conditions => ["user_id != ? AND parent_id = ? AND created_at > ?", 
      current_user.id, current_user.id, current_user.last_checked_mail]) 

meilleure solution est de créer un named_scope sur le modèle Comment.

class Comment 

    named_scope :other_comments, lambda {|u| { :conditions => 
       ["user_id != ? AND parent_id = ? AND created_at > ?", 
        u.id, u.id, u.last_checked_mail ] }} 
end 

Maintenant, vous pouvez obtenir d'autres commentaires comme:

Comment.other_comments(current_user) 

Modifier 1 réponse Mise à jour sur la base des associations polymorphes:

class Comment 
    def self.other_comments(u) 
    Comment.all(
     :joins => "JOIN comments A 
       ON A.commentable_type = 'User' AND 
         A.commentable_id = #{u.id} AND 
         comments.commentable_type = 'Comment' AND 
         comments.commentable_id = A.id", 

     :conditions => ["comments.user_id != ? AND comments.created_at > ?", 
      u.id, u.last_checked_mail] 
    ) 
    end 
end 

Maintenant vous pouvez obtenir d'autres commentaires comme:

Comment.other_comments(current_user)