improve best filter so it properly excludes all admin messages from count as well

Author: SamSaffron

lib/topic_view.rb

@@ -25,6 +25,7 @@ def initialize(topic_id, user=nil, options={})
     @filtered_posts = @topic.posts
     @filtered_posts = @filtered_posts.with_deleted if user.try(:staff?)
     @filtered_posts = @filtered_posts.best_of if options[:best_of].present?
+    @filtered_posts = @filtered_posts.where('posts.post_type <> ?', Post.types[:moderator_action]) if options[:best].present?
 
     if options[:username_filters].present?
       usernames = options[:username_filters].map{|u| u.downcase}
@@ -177,7 +178,6 @@ def filter_best(max)
     @index_offset = 0
     @posts = @filtered_posts.order('percent_rank asc, sort_order asc').where("post_number > 1")
     @posts = @posts.includes(:reply_to_user).includes(:topic).joins(:user).limit(max)
-    @posts = @posts.where('posts.post_type <> ?', Post.types[:moderator_action])
     @posts = @posts.to_a
     @posts.sort!{|a,b| a.post_number <=> b.post_number}
     @posts

spec/components/topic_view_spec.rb

@@ -35,6 +35,7 @@
       # should not get the status post
       best = TopicView.new(topic.id, nil, best: 99)
       best.posts.count.should == 2
+      best.filtered_posts_count.should == 3
 
     end