FIX: don't double add users to topic allowed users

In latest rails building the same relation twice results in an error
Previous versions would deduplicate
This commit is contained in:
Sam 2018-12-10 14:45:02 +11:00
parent a34bc92e1a
commit e17d0c0526
1 changed files with 9 additions and 3 deletions

View File

@ -190,7 +190,11 @@ class TopicCreator
add_users(topic, @opts[:target_usernames])
add_emails(topic, @opts[:target_emails])
add_groups(topic, @opts[:target_group_names])
topic.topic_allowed_users.build(user_id: @user.id)
if !@added_users.include?(user)
topic.topic_allowed_users.build(user_id: @user.id)
end
end
def save_topic(topic)
@ -228,8 +232,10 @@ class TopicCreator
display_name = email.split("@").first
if user = find_or_create_user(email, display_name)
@added_users << user
topic.topic_allowed_users.build(user_id: user.id)
if !@added_users.include?(user)
@added_users << user
topic.topic_allowed_users.build(user_id: user.id)
end
len += 1
end
end