diff options
| author | dmathieu <42@dmathieu.com> | 2011-05-26 08:09:38 +0200 |
|---|---|---|
| committer | dmathieu <42@dmathieu.com> | 2011-05-26 11:18:30 +0200 |
| commit | 16571f9c42e2717eeb8bf7c915911fa1ca714525 (patch) | |
| tree | 2e5767f931ace577a172b782e687a67837eec6ca | |
| parent | ee46bb73d44338b7dfb8976cc59d742b9394148a (diff) | |
| download | rails-16571f9c42e2717eeb8bf7c915911fa1ca714525.tar.gz rails-16571f9c42e2717eeb8bf7c915911fa1ca714525.tar.bz2 rails-16571f9c42e2717eeb8bf7c915911fa1ca714525.zip | |
don't raise an exception if the format isn't recognized
Fixed while traveling to heuruko
| -rw-r--r-- | actionpack/lib/action_controller/metal/instrumentation.rb | 2 | ||||
| -rw-r--r-- | actionpack/test/controller/mime_responds_test.rb | 6 |
2 files changed, 7 insertions, 1 deletions
diff --git a/actionpack/lib/action_controller/metal/instrumentation.rb b/actionpack/lib/action_controller/metal/instrumentation.rb index 16cbbce2fb..85d0f5f699 100644 --- a/actionpack/lib/action_controller/metal/instrumentation.rb +++ b/actionpack/lib/action_controller/metal/instrumentation.rb @@ -19,7 +19,7 @@ module ActionController :controller => self.class.name, :action => self.action_name, :params => request.filtered_parameters, - :format => request.format.ref, + :format => request.format.try(:ref), :method => request.method, :path => (request.fullpath rescue "unknown") } diff --git a/actionpack/test/controller/mime_responds_test.rb b/actionpack/test/controller/mime_responds_test.rb index 4a5e597500..26270571cf 100644 --- a/actionpack/test/controller/mime_responds_test.rb +++ b/actionpack/test/controller/mime_responds_test.rb @@ -498,6 +498,12 @@ class RespondToControllerTest < ActionController::TestCase assert_equal '<html><div id="iphone">Hello iPhone future from iPhone!</div></html>', @response.body assert_equal "text/html", @response.content_type end + + def test_invalid_format + get :using_defaults, :format => "invalidformat" + assert_equal " ", @response.body + assert_equal "text/html", @response.content_type + end end class RespondWithController < ActionController::Base |