Include conditional for adding tag mods to connect channel (forem#1366)

* fix slug typo * fix specs * use blank? instead of nil?
anonymous-security-public · Dec 19, 2018 · 1586d9f · 1586d9f
1 parent c2a8554
commit 1586d9f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/app/labor/assign_tag_moderator.rb b/app/labor/assign_tag_moderator.rb
@@ -4,7 +4,9 @@ def self.add_tag_moderators(user_ids, tag_names)
       user = User.find(user_id)
       tag = Tag.find_by(name: tag_names[index])
       user.add_role(:tag_moderator, tag)
-      ChatChannel.find_by(slug: "tag-moderators").add_users(user)
+      if user.chat_channels.find_by(slug: "tag-moderators").blank?
+        ChatChannel.find_by(slug: "tag-moderators").add_users(user)
+      end
       NotifyMailer.tag_moderator_confirmation_email(user, tag.name).deliver unless tag.name == "go"
     end
   end

diff --git a/environment b/environment