0
@@ -2,7 +2,7 @@ class Comment < Content
0
   validates_presence_of :author, :author_ip, :article_id
0
   before_create  { |comment| comment.site_id = comment.article.site_id }
0
   before_create  :check_comment_expiration
0
-  before_save    :increment_counter_cache
0
+  before_save    :update_counter_cache
0
   before_destroy :decrement_counter_cache
0
   belongs_to :article
0
   attr_protected :approved
0
@@ -30,12 +30,12 @@ class Comment < Content
0
       raise Article::CommentNotAllowed unless article.comments_allowed?
0
     end
0
 
0
-    def increment_counter_cache
0
+    def update_counter_cache
0
       Article.increment_counter 'comments_count', article_id if approved? && @old_approved == :false
0
-      decrement_counter_cache if !approved? && @old_approved == :true
0
+      Article.decrement_counter 'comments_count', article_id if !approved? && @old_approved == :true
0
     end
0
     
0
     def decrement_counter_cache
0
       Article.decrement_counter 'comments_count', article_id if approved?
0
     end
0
-end
0
\ No newline at end of file
0
+end

0
@@ -52,6 +52,14 @@ class CommentTest < Test::Unit::TestCase
0
     end
0
   end
0
   
0
+  def test_should_decrement_comment_count_upon_unapproval
0
+    assert_difference contents(:welcome), :comments_count, -1 do
0
+      contents(:welcome_comment).approved = false
0
+      assert contents(:welcome_comment).save
0
+      contents(:welcome).reload
0
+    end
0
+  end
0
+  
0
   def test_should_not_decrement_unapproved_comment_count
0
     assert_no_difference contents(:welcome), :comments_count do
0
       contents(:unwelcome_comment).destroy