FEATURE: Show dismiss posts/topics buttons on category filtered lists

[cpradio]

FEATURE: Add Dismiss Posts/Topics button to Category filtered Unread View

[discoursebot]

You've signed the CLA, cpradio. Thank you! This pull request is ready for review.

[coding-horror]

Oh nice one cpradio!

On Oct 23, 2014, at 2:59 PM, discoursebot notifications@github.com wrote:

You've signed the CLA, cpradio. Thank you! This pull request is ready for review.

—
Reply to this email directly or view it on GitHub.

[cpradio]

Thank you :)

[eviltrout]

Thanks! I left a couple of small comments.

[cpradio]

Comments applied. Thanks!

[SamSaffron]

if params[:category_id]
+        topic_ids = TopicQuery.unread_filter(tq.joined_topic_user).listable_topics.where('category_id = ?', params[:category_id]).pluck(:id)
+      else
+        topic_ids = TopicQuery.unread_filter(tq.joined_topic_user).listable_topics.pluck(:id)
+      end

topics =  TopicQuery.unread_filter(tq.joined_topic_user).listable_topics
topics =  topics.where('category_id = ?', params[:category_id]) if params[:category_id]
topic_ids = topics.pluck(:id)

[SamSaffron]

this change looks good, lets apply my comment and get it merged

[cpradio]

Thanks @SamSaffron! As usual, you found a way to make my logic much cleaner. 👍
I'm slowly getting the hang of Ruby

[SamSaffron]

looks good, thanks!