Skip to content

Commit ad417aa

Browse files
committed
Reuse return from keys_in_common
1 parent 90cc14d commit ad417aa

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

lib/grape/validations/validators/exactly_one_of.rb

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,10 @@ module Grape
66
module Validations
77
class ExactlyOneOfValidator < MultipleParamsBase
88
def validate_params!(params)
9-
return if keys_in_common(params).length == 1
10-
raise Grape::Exceptions::Validation.new(params: all_keys, message: message(:exactly_one)) if keys_in_common(params).length.zero?
11-
raise Grape::Exceptions::Validation.new(params: keys_in_common(params), message: message(:mutual_exclusion))
9+
keys = keys_in_common(params)
10+
return if keys.length == 1
11+
raise Grape::Exceptions::Validation.new(params: all_keys, message: message(:exactly_one)) if keys.length.zero?
12+
raise Grape::Exceptions::Validation.new(params: keys, message: message(:mutual_exclusion))
1213
end
1314
end
1415
end

0 commit comments

Comments
 (0)