Commit 636bcebc authored by Stan Hu's avatar Stan Hu

Fix handling of non-existent groups

parent d26074c7
...@@ -19,15 +19,15 @@ module NamespacesHelper ...@@ -19,15 +19,15 @@ module NamespacesHelper
# group if one exists by that name to prevent duplicates. # group if one exists by that name to prevent duplicates.
unless extra_group.persisted? unless extra_group.persisted?
existing_group = Group.find_by(name: extra_group.name) existing_group = Group.find_by(name: extra_group.name)
extra_group = existing_group if existing_group extra_group = existing_group if existing_group&.persisted?
end end
if Ability.allowed?(current_user, :read_group, extra_group) if Ability.allowed?(current_user, :read_group, extra_group)
selected = extra_group.id if selected == :extra_group selected = extra_group.id if selected == :extra_group
groups |= [extra_group] groups |= [extra_group]
else
selected = :current_user
end end
selected ||= :current_user
end end
options = [] options = []
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment