@@ -11,7 +11,11 @@ class SpamReport < ActiveRecord::Base if comment.matching_spam_reports.empty? create!(:comment => comment, :hit_count => 1) else - comment.matching_spam_reports.each { |r| r.increment! :hit_count } + comment.matching_spam_reports.each do |r| + r.increment :hit_count + r.confirmed_at = nil + r.save! + end end end app/models/spam_report.rb @@ -28,14 +28,16 @@ describe SpamReport do report.matching_comments.should_not include(good) end - it "should increment hit count when reporting comment spam that already exists" do + it "should increment hit count when reporting comment spam that already exists and reset confirmation" do SpamReport.delete_all # delete so comment doesn't match an older report bad = Factory(:comment, :name => 'badbadbad') bad_2 = Factory(:comment, :name => 'badbadbad') report = SpamReport.report_comment(bad) report.reload.hit_count.should == 1 + report.update_attribute(:confirmed_at, Time.now) SpamReport.report_comment(bad_2) report.reload.hit_count.should == 2 + report.confirmed_at.should be_nil end it "should remove matching comments when confirming" do spec/models/spam_report_spec.rb 23145fc1bd2a3843207fd69b2b56c27cd32ed23e Ryan Bates ryan@railscasts.com http://github.com/ryanb/railscasts/commit/2ad8da3946ae03634e4816615acb0996ac6f21ce 2ad8da3946ae03634e4816615acb0996ac6f21ce 2009-10-28T16:04:18-07:00 2009-10-28T16:04:18-07:00 unconfirm spam report if a spam is added after it's confirmed 7bdb4b9e6508af0189b53d8b50be2d93e18a068e Ryan Bates ryan@railscasts.com