diff options
author | Z.J. van de Weg <git@zjvandeweg.nl> | 2016-11-12 11:18:33 +0100 |
---|---|---|
committer | Z.J. van de Weg <git@zjvandeweg.nl> | 2016-11-12 11:20:06 +0100 |
commit | 9869c480973a179a51aec79fcd5373e882195b1b (patch) | |
tree | b94b5a46df44f6df9661552d021487bf49d0d07e /app/models/project_feature.rb | |
parent | 6eeff67c6e03233d4480a55d05d4e0f1a88aef4c (diff) | |
download | gitlab-ce-9869c480973a179a51aec79fcd5373e882195b1b.tar.gz |
Remove unneeded nil checkszj-remove-unneeded-checks
The library we're using, default_value_for, is configured to interpret nil
as the default value, being ENABLED. Given the property can't be nil
this check is unneeded.
Diffstat (limited to 'app/models/project_feature.rb')
-rw-r--r-- | app/models/project_feature.rb | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/app/models/project_feature.rb b/app/models/project_feature.rb index 34fd5a57b5e..5c53c8f1ee5 100644 --- a/app/models/project_feature.rb +++ b/app/models/project_feature.rb @@ -49,20 +49,14 @@ class ProjectFeature < ActiveRecord::Base end def builds_enabled? - return true unless builds_access_level - builds_access_level > DISABLED end def wiki_enabled? - return true unless wiki_access_level - wiki_access_level > DISABLED end def merge_requests_enabled? - return true unless merge_requests_access_level - merge_requests_access_level > DISABLED end |