diff options
author | Aaron Patterson <aaron.patterson@gmail.com> | 2015-09-22 16:20:19 -0700 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2015-09-22 16:20:27 -0700 |
commit | a9f28600e901b11a9222e34bfae8642bfb753186 (patch) | |
tree | 68760792098d0df97e7fd6f052e08fbb602316be /actionpack/lib/action_controller | |
parent | 686897225e9d0372deac71a6dfdf881676bb3dea (diff) | |
download | rails-a9f28600e901b11a9222e34bfae8642bfb753186.tar.gz rails-a9f28600e901b11a9222e34bfae8642bfb753186.tar.bz2 rails-a9f28600e901b11a9222e34bfae8642bfb753186.zip |
don't deal with `nil` values
We can know whether or not there is a content type object, and just exit
early. There is no need to `try` so hard.
Diffstat (limited to 'actionpack/lib/action_controller')
-rw-r--r-- | actionpack/lib/action_controller/metal/params_wrapper.rb | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/actionpack/lib/action_controller/metal/params_wrapper.rb b/actionpack/lib/action_controller/metal/params_wrapper.rb index e680432127..c38fc40b81 100644 --- a/actionpack/lib/action_controller/metal/params_wrapper.rb +++ b/actionpack/lib/action_controller/metal/params_wrapper.rb @@ -276,7 +276,9 @@ module ActionController # Checks if we should perform parameters wrapping. def _wrapper_enabled? - ref = request.content_mime_type.try(:ref) + return false unless request.has_content_type? + + ref = request.content_mime_type.ref _wrapper_formats.include?(ref) && _wrapper_key && !request.request_parameters[_wrapper_key] end end |