summaryrefslogtreecommitdiff
path: root/lib/api/triggers.rb
diff options
context:
space:
mode:
authorRobert Schilling <rschilling@student.tugraz.at>2017-02-24 11:24:40 +0100
committerRobert Schilling <rschilling@student.tugraz.at>2017-02-24 11:24:40 +0100
commitbff14cedf320d593520ee3de9cfd753399dd2de9 (patch)
tree143c593077e28825fc32378f3a46cba8c691792d /lib/api/triggers.rb
parentc5b29ed6f36779dbb96f4cdc7b1b0bce8bb8dc5e (diff)
downloadgitlab-ce-bff14cedf320d593520ee3de9cfd753399dd2de9.tar.gz
Simplyfy variables validation in triggers APIapi-simplify-variables-validation
Diffstat (limited to 'lib/api/triggers.rb')
-rw-r--r--lib/api/triggers.rb11
1 files changed, 3 insertions, 8 deletions
diff --git a/lib/api/triggers.rb b/lib/api/triggers.rb
index 87a717ba751..ea0ad852633 100644
--- a/lib/api/triggers.rb
+++ b/lib/api/triggers.rb
@@ -21,14 +21,9 @@ module API
unauthorized! unless trigger.project == project
# validate variables
- variables = params[:variables]
- if variables
- unless variables.all? { |key, value| key.is_a?(String) && value.is_a?(String) }
- render_api_error!('variables needs to be a map of key-valued strings', 400)
- end
-
- # convert variables from Mash to Hash
- variables = variables.to_h
+ variables = params[:variables].to_h
+ unless variables.all? { |key, value| key.is_a?(String) && value.is_a?(String) }
+ render_api_error!('variables needs to be a map of key-valued strings', 400)
end
# create request and trigger builds