comment pings should have 'host' correctly set even if RequestContext is not populated

Author: Oaphi

app/helpers/comments_helper.rb

@@ -47,7 +47,7 @@ def pinged_users(content)
   # @param content [String] content to convert ping-strings for
   # @param pingable [Array<Integer>, nil] A list of user IDs. Any user ID not present will be displayed as 'unpingable'.
   # @return [ActiveSupport::SafeBuffer]
-  def render_pings(content, pingable: nil)
+  def render_pings(content, pingable: nil, host: nil)
     users = pinged_users(content)
 
     content.gsub(/@#(\d+)/) do |ping|
@@ -57,8 +57,10 @@ def render_pings(content, pingable: nil)
       else
         was_pung = pingable.present? && pingable.include?(user.id)
         classes = "ping #{'me' if user.same_as?(current_user)} #{'unpingable' unless was_pung}"
-        user_link user, class: classes, dir: 'ltr',
-                  title: was_pung ? '' : I18n.t('comments.warnings.unrelated_user_not_pinged')
+        user_link(user, { host: host },
+                  class: classes,
+                  dir: 'ltr',
+                  title: was_pung ? '' : I18n.t('comments.warnings.unrelated_user_not_pinged'))
       end
     end.html_safe
   end

app/views/comments/_comment.html.erb

@@ -77,7 +77,8 @@
     </div>
   </div>
   <div class="comment--body" dir="ltr">
-    <%= render_pings(raw(sanitize(render_comment_helpers(render_markdown(comment.content)), scrubber: CommentScrubber.new)),
-                     pingable: pingable) %>
+    <% rendered = render_comment_helpers(render_markdown(comment.content)) %>
+    <% sanitized = sanitize(rendered, scrubber: CommentScrubber.new) %>
+    <%= render_pings(raw(sanitized), pingable: pingable, host: comment.community.host) %>
   </div>
 </div>