Come avvisare il commentatore?

Question

Se scrivo un commento sul vostro stato e poi commentate, non ricevo una notifica per farmi sapere che avete commentato.

Attualmente solo il proprietario dello stato riceve le notifiche. Come possiamo far sapere se anche chi ha commentato uno stato dovrebbe ricevere una notifica?

modelli

class Comment < ActiveRecord::Base 
    after_save :create_notification 
    has_many :notifications 
    def create_notification 
    author = valuation.user 
    notifications.create(
     comment:  self, 
     valuation: valuation, 
     user:   author,  
     read:   false 
    ) 
    end 
end 

class Notification < ActiveRecord::Base 
    belongs_to :comment 
    belongs_to :valuation 
    belongs_to :user 
end 

class Valuation < ActiveRecord::Base 
    belongs_to :user 
    has_many :notifications 
    has_many :comments 
end 

controllori

class CommentsController < ApplicationController 
    before_action :set_commentable, only: [:index, :new, :create] 
    before_action :set_comment, only: [:edit, :update, :destroy] 

    def create 
    @comment = @commentable.comments.new(comment_params) 
    if @comment.save 
     redirect_to @commentable, notice: "Comment created." 
    else 
     render :new 
    end 
    end 

    private 

    def set_commentable 
    @commentable = find_commentable 
    end 

    def set_comment 
    @comment = current_user.comments.find(params[:id]) 
    end 

    def find_commentable 
    if params[:goal_id] 
     Goal.find(params[:goal_id]) 
    elsif params[:habit_id] 
     Habit.find(params[:habit_id]) 
    elsif params[:valuation_id] 
     Valuation.find(params[:valuation_id]) 
    elsif params[:stat_id] 
     Stat.find(params[:stat_id]) 
    end 
    end 
end 

class NotificationsController < ApplicationController 
    def index 
    @notifications = current_user.notifications 
    @notifications.each do |notification| 
     notification.update_attribute(:read, true) 
    end 
    end 
end 

vista

#notifications/_notification.html.erb 
commented on <%= link_to "your value", notification_valuation_path(notification, notification.valuation_id) %> 

#comments/_form.html.erb 
<%= form_for [@commentable, @comment] do |f| %> 
    <%= f.text_area :content %> 
<% end %> 

Answer 1

Invece di creare una notifi per l'autore, è necessario recuperare l'elenco degli utenti che hanno commentato la valutazione. Non vedo uno belongs_to :user nel modello Comment, ma presumo che ce ne sia uno, in quanto è possibile eseguire uno current_user.comments in CommentsController.

Aggiungiamo un rapporto has_many in Valuation per recuperare tutti i commentatori per un dato di valutazione:

class Valuation < ActiveRecord::Base 
    belongs_to :user 
    has_many :notifications 
    has_many :comments 
    has_many :commentators, -> { distinct }, through: :comments, source: :user 
end 

Se si utilizza una versione di Rails < 4, è necessario sostituire -> { distinct } da uniq: true

Poi , puoi semplicemente utilizzare questa nuova relazione per creare notifiche per tutti i commentatori nel modello Comment:

class Comment < ActiveRecord::Base 
    after_save :create_notification 
    belongs_to :user 
    belongs_to :valuation 
    has_many :notifications 

    def create_notification 
    to_notify = [valuation.user] + valuation.commentators 
    to_notify = to_notify.uniq 
    to_notify.delete(user) # Do not send notification to current user 
    to_notify.each do |notification_user| 
     notifications.create(
     comment:  self, 
     valuation: valuation, 
     user:   notification_user,  
     read:   false 
    ) 
    end 
    end 
end