fixed other cases of comments not accounting for pingable state

Author: Oaphi

app/controllers/comments_controller.rb

@@ -273,8 +273,7 @@ def post_follow
 
   def pingable
     thread = params[:id] == '-1' ? CommentThread.new(post_id: params[:post]) : CommentThread.find(params[:id])
-    ids = helpers.get_pingable(thread)
-    users = User.where(id: ids)
+    users = User.where(id: thread.pingable)
     render json: users.to_h { |u| [u.username, u.id] }
   end
 
@@ -377,7 +376,7 @@ def check_if_target_post_locked
   # @param content [String] content to extract pings from
   # @return [Array<Integer>] list of pinged user ids
   def check_for_pings(thread, content)
-    pingable = helpers.get_pingable(thread)
+    pingable = thread.pingable
     matches = content.scan(/@#(\d+)/)
     matches.flatten.select { |m| pingable.include?(m.to_i) }.map(&:to_i)
   end

app/helpers/comments_helper.rb

@@ -149,34 +149,6 @@ def rate_limited_error_msg(user, post)
     I18n.t('comments.errors.rate_limited', count: comments_count)
   end
 
-  ##
-  # Get a list of user IDs who should be pingable in a specified comment thread. This combines the post author, answer
-  # authors, recent history event authors, recent comment authors on the post (in any thread), and all thread followers.
-  # @param thread [CommentThread]
-  # @return [Array<Integer>]
-  def get_pingable(thread)
-    post = thread.post
-
-    # post author +
-    # answer authors +
-    # last 500 history event users +
-    # last 500 comment authors +
-    # all thread followers
-    query = <<~END_SQL
-      SELECT posts.user_id FROM posts WHERE posts.id = #{post.id}
-      UNION DISTINCT
-      SELECT DISTINCT posts.user_id FROM posts WHERE posts.parent_id = #{post.id}
-      UNION DISTINCT
-      SELECT DISTINCT ph.user_id FROM post_histories ph WHERE ph.post_id = #{post.id}
-      UNION DISTINCT
-      SELECT DISTINCT comments.user_id FROM comments WHERE comments.post_id = #{post.id}
-      UNION DISTINCT
-      SELECT DISTINCT tf.user_id FROM thread_followers tf WHERE tf.comment_thread_id = #{thread.id || '-1'}
-    END_SQL
-
-    ActiveRecord::Base.connection.execute(query).to_a.flatten
-  end
-
   ##
   # Is the specified user comment rate limited for the specified post?
   # @param user [User] The user to check.

app/models/comment.rb

@@ -33,6 +33,12 @@ def content_length
     end
   end
 
+  def pings
+    pingable = thread.pingable
+    matches = content.scan(/@#(\d+)/)
+    matches.flatten.select { |m| pingable.include?(m.to_i) }.map(&:to_i)
+  end
+
   private
 
   def create_follower

app/models/comment_thread.rb

@@ -13,6 +13,10 @@ class CommentThread < ApplicationRecord
 
   after_create :create_follower
 
+  def self.post_followed?(post, user)
+    ThreadFollower.where(post: post, user: user).any?
+  end
+
   def read_only?
     locked? || archived? || deleted?
   end
@@ -26,8 +30,27 @@ def can_access?(user)
       post.can_access?(user)
   end
 
-  def self.post_followed?(post, user)
-    ThreadFollower.where(post: post, user: user).any?
+  # Gets a list of user IDs who should be pingable in the thread.
+  # @return [Array<Integer>]
+  def pingable
+    # post author +
+    # answer authors +
+    # last 500 history event users +
+    # last 500 comment authors +
+    # all thread followers
+    query = <<~END_SQL
+      SELECT posts.user_id FROM posts WHERE posts.id = #{post.id}
+      UNION DISTINCT
+      SELECT DISTINCT posts.user_id FROM posts WHERE posts.parent_id = #{post.id}
+      UNION DISTINCT
+      SELECT DISTINCT ph.user_id FROM post_histories ph WHERE ph.post_id = #{post.id}
+      UNION DISTINCT
+      SELECT DISTINCT comments.user_id FROM comments WHERE comments.post_id = #{post.id}
+      UNION DISTINCT
+      SELECT DISTINCT tf.user_id FROM thread_followers tf WHERE tf.comment_thread_id = #{id || '-1'}
+    END_SQL
+
+    ActiveRecord::Base.connection.execute(query).to_a.flatten
   end
 
   private

app/views/comment_threads/_expanded.html.erb

@@ -10,7 +10,7 @@
 
 <% max_shown_comments = 5 %>
 <% comment_id ||= defined?(comment_id) ? comment_id : nil %>
-<% pingable = get_pingable(thread) %>
+<% pingable = thread.pingable %>
 <% shown_comments_count = [max_shown_comments, thread.reply_count].min %>
 
 <%

app/views/flags/_flag.html.erb

@@ -16,7 +16,7 @@
       <%= render 'posts/type_agnostic', show_type_tag: true, show_category_tag: true, post: flag.post %>
     </div>
   <% elsif flag.post_type == 'Comment' %>
-    <%= render 'comments/comment', comment: flag.post, with_post_link: true %>
+    <%= render 'comments/comment', comment: flag.post, with_post_link: true, pingable: flag.post.comment_thread.pingable %>
   <% end %>
   <div class="widget--body">
     <p>

app/views/moderator/recent_comments.html.erb

@@ -9,8 +9,16 @@
   communities; consider using activity logs on user profiles instead.
 </p>
 
+<%
+  thread_pingables = @comments.map(&:comment_thread).to_set.to_h do |thread|
+    [thread, thread.pingable]
+  end
+%>
+
 <% @comments.each do |comment| %>
-  <%= render 'comments/comment', comment: comment, with_post_link: true %>
+  <%= render 'comments/comment', comment: comment,
+                                 pingable: thread_pingables[comment.comment_thread],
+                                 with_post_link: true %>
 <% end %>
 
 <div class="has-padding-top-4">