FIX: Previous fix comitted the wrong thing and was broken :(

FIX: Previous fix comitted the wrong thing and was broken :frowning:

diff --git a/app/models/topic.rb b/app/models/topic.rb
index 8933848..e02c2bb 100644
--- a/app/models/topic.rb
+++ b/app/models/topic.rb
@@ -514,8 +514,8 @@ class Topic < ActiveRecord::Base
     if SiteSetting.digest_suppress_categories.present?
       remove_category_ids += SiteSetting.digest_suppress_categories.split("|").map(&:to_i)
     end
-    if SiteSetting.shared_drafts_category.present?
-      remove_category_ids << SiteSetting.shared_drafts_enabled?
+    if SiteSetting.shared_drafts_enabled?
+      remove_category_ids << SiteSetting.shared_drafts_category
     end
     if remove_category_ids.present?
       remove_category_ids.uniq!

GitHub sha: a1ec7113

What previous fix? SHA would have been great :+1:

1 Like

Very good point. This fixes 6da7a97eee3092374a1aac92ee3a6e3d33622626 which was broken.

1 Like