Commit a7c441aa authored by Kamil Trzcinski's avatar Kamil Trzcinski

Use `delete` instead of assignment operator when filtering build abilities

parent c64c1060
...@@ -411,7 +411,7 @@ class Ability ...@@ -411,7 +411,7 @@ class Ability
# If we can't read build we should also not have that # If we can't read build we should also not have that
# ability when looking at this in context of commit_status # ability when looking at this in context of commit_status
%w(read create update admin).each do |rule| %w(read create update admin).each do |rule|
rules -= [:"#{rule}_commit_status"] unless rules.include?(:"#{rule}_build") rules.delete(:"#{rule}_commit_status") unless rules.include?(:"#{rule}_build")
end end
rules rules
end end
......
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