diff options
| author | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-02-04 16:32:41 +0100 |
|---|---|---|
| committer | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-02-04 16:32:41 +0100 |
| commit | a7c441aa17ba64eb702b583ac9198cdace599d2e (patch) | |
| tree | 814e26c0c59c0e171bb4207dbecec078c17d6d98 /app/models/ability.rb | |
| parent | c64c106091ee5526e9413b9929205134c0ce114f (diff) | |
| download | gitlab-ce-a7c441aa17ba64eb702b583ac9198cdace599d2e.tar.gz | |
Use `delete` instead of assignment operator when filtering build abilities
Diffstat (limited to 'app/models/ability.rb')
| -rw-r--r-- | app/models/ability.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/ability.rb b/app/models/ability.rb index 00f5f3a93b3..a866eadeebb 100644 --- a/app/models/ability.rb +++ b/app/models/ability.rb @@ -411,7 +411,7 @@ class Ability # If we can't read build we should also not have that # ability when looking at this in context of commit_status %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 rules end |