Download as
Browse Code »

Commit 252ec20a19d4019ca95061c946cc531308f0c150

Authored by Antonio Terceiro
1 parent d7430804
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

Ignore spam when constructing threads 

ActionItem2306
Showing 2 changed files with 15 additions and 2 deletions   Show diff stats
app/models/comment.rb
  Diff comments   View file @252ec20
... ... @@ -147,10 +147,14 @@ class Comment < ActiveRecord::Base
147 147 def self.as_thread
148 148 result = {}
149 149 root = []
150   - all.each do |c|
  150 + order(:id).each do |c|
151 151 c.replies = []
152 152 result[c.id] ||= c
153   - c.reply_of_id.nil? ? root << c : result[c.reply_of_id].replies << c
  153 + if result[c.reply_of_id]
  154 + result[c.reply_of_id].replies << c
  155 + else
  156 + root << c
  157 + end
154 158 end
155 159 root
156 160 end
... ...
test/unit/comment_test.rb
  Diff comments   View file @252ec20
... ... @@ -536,6 +536,15 @@ class CommentTest &lt; ActiveSupport::TestCase
536 536 assert_equal c, SpamNotification.marked_as_ham
537 537 end
538 538  
  539 + should 'ignore spam when constructing threads' do
  540 + original = create_comment
  541 + response = create_comment(:reply_of_id => original.id)
  542 + original.spam!
  543 +
  544 + assert_equivalent [response], Comment.without_spam.as_thread
  545 + end
  546 +
  547 +
539 548 should 'store User-Agent' do
540 549 c = Comment.new(:user_agent => 'foo')
541 550 assert_equal 'foo', c.user_agent
... ...