diff options
| author | Robert Speicher <robert@gitlab.com> | 2018-09-13 17:53:05 +0300 |
|---|---|---|
| committer | Robert Speicher <robert@gitlab.com> | 2018-09-13 17:53:05 +0300 |
| commit | c7d1eef671dbf598814a6c2ff1f81b924583ae8a (patch) | |
| tree | 23988df1d29e933943858a66821ba7f493976b3e /app/validators | |
| parent | ff5d8b635f234241441009e41af7b61f5804b2c2 (diff) | |
| parent | 2039c8280db1646845c33d6c5a74e5f23ca6f4de (diff) | |
Merge branch 'rubocop-code-reuse' into 'master'
Add RuboCop cops to enforce code reusing rules
See merge request gitlab-org/gitlab-ce!21391
Diffstat (limited to 'app/validators')
| -rw-r--r-- | app/validators/variable_duplicates_validator.rb | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/app/validators/variable_duplicates_validator.rb b/app/validators/variable_duplicates_validator.rb index 90193e85f2a..d36a56e81b9 100644 --- a/app/validators/variable_duplicates_validator.rb +++ b/app/validators/variable_duplicates_validator.rb @@ -21,6 +21,7 @@ class VariableDuplicatesValidator < ActiveModel::EachValidator private + # rubocop: disable CodeReuse/ActiveRecord def validate_duplicates(record, attribute, values) duplicates = values.reject(&:marked_for_destruction?).group_by(&:key).select { |_, v| v.many? }.map(&:first) if duplicates.any? @@ -29,4 +30,5 @@ class VariableDuplicatesValidator < ActiveModel::EachValidator record.errors.add(attribute, error_message) end end + # rubocop: enable CodeReuse/ActiveRecord end |