aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
diff options
context:
space:
mode:
authorAaron Patterson <aaron.patterson@gmail.com>2013-01-18 17:27:19 -0800
committerAaron Patterson <aaron.patterson@gmail.com>2013-01-18 17:27:19 -0800
commitf209b176c3a39af11f25e7a77fd856b5ad55c996 (patch)
tree6ad667e6bfa66a2d82ed4b56a08f98c398df7618 /actionpack
parente636f555310d679c26dcac7b28c8caa6f62263e6 (diff)
parentbebb02fd1c5ea0e8da9fb61eefcbb6ec8b9ab11f (diff)
downloadrails-f209b176c3a39af11f25e7a77fd856b5ad55c996.tar.gz
rails-f209b176c3a39af11f25e7a77fd856b5ad55c996.tar.bz2
rails-f209b176c3a39af11f25e7a77fd856b5ad55c996.zip
Merge pull request #8977 from Soylent/master
Resovled issue rails#7774
Diffstat (limited to 'actionpack')
-rw-r--r--actionpack/lib/action_dispatch/http/mime_negotiation.rb2
-rw-r--r--actionpack/test/dispatch/request_test.rb4
2 files changed, 5 insertions, 1 deletions
diff --git a/actionpack/lib/action_dispatch/http/mime_negotiation.rb b/actionpack/lib/action_dispatch/http/mime_negotiation.rb
index 57660e93c4..89a7b12818 100644
--- a/actionpack/lib/action_dispatch/http/mime_negotiation.rb
+++ b/actionpack/lib/action_dispatch/http/mime_negotiation.rb
@@ -122,7 +122,7 @@ module ActionDispatch
def valid_accept_header
(xhr? && (accept || content_mime_type)) ||
- (accept && accept !~ BROWSER_LIKE_ACCEPTS)
+ (accept.present? && accept !~ BROWSER_LIKE_ACCEPTS)
end
def use_accept_header
diff --git a/actionpack/test/dispatch/request_test.rb b/actionpack/test/dispatch/request_test.rb
index 39923d0d2b..8a01b29340 100644
--- a/actionpack/test/dispatch/request_test.rb
+++ b/actionpack/test/dispatch/request_test.rb
@@ -587,6 +587,10 @@ class RequestTest < ActiveSupport::TestCase
request.expects(:parameters).at_least_once.returns({})
assert_equal [Mime::HTML], request.formats
+ request = stub_request 'HTTP_ACCEPT' => ''
+ request.expects(:parameters).at_least_once.returns({})
+ assert_equal [Mime::HTML], request.formats
+
request = stub_request 'CONTENT_TYPE' => 'application/xml; charset=UTF-8',
'HTTP_X_REQUESTED_WITH' => "XMLHttpRequest"
request.expects(:parameters).at_least_once.returns({})