diff options
| author | Joshua Peek <josh@joshpeek.com> | 2009-01-31 23:47:03 -0600 |
|---|---|---|
| committer | Joshua Peek <josh@joshpeek.com> | 2009-01-31 23:51:41 -0600 |
| commit | 28b65c9120f347bd61569402a5e50ca6d4b9b6e7 (patch) | |
| tree | 87e9763901ed3d020173dd1c0ed5137925f7bf72 | |
| parent | fa7aa19a9940a6872efa01b70caf70c7e2915b3f (diff) | |
| download | rails-28b65c9120f347bd61569402a5e50ca6d4b9b6e7.tar.gz rails-28b65c9120f347bd61569402a5e50ca6d4b9b6e7.tar.bz2 rails-28b65c9120f347bd61569402a5e50ca6d4b9b6e7.zip | |
Removed Prototype specific Safari 2 AJAX hack.
The normal null character stripper is still there.
| -rw-r--r-- | actionpack/lib/action_controller/rack_ext/parse_query.rb | 1 | ||||
| -rw-r--r-- | actionpack/test/controller/request/url_encoded_params_parsing_test.rb | 6 |
2 files changed, 0 insertions, 7 deletions
diff --git a/actionpack/lib/action_controller/rack_ext/parse_query.rb b/actionpack/lib/action_controller/rack_ext/parse_query.rb index 2f21a57770..b1acef8e72 100644 --- a/actionpack/lib/action_controller/rack_ext/parse_query.rb +++ b/actionpack/lib/action_controller/rack_ext/parse_query.rb @@ -10,7 +10,6 @@ module Rack def parse_query(qs, d = '&;') qs = qs.dup qs.chop! if qs[-1] == 0 - qs.gsub!(/&_=$/, '') parse_query_without_ajax_body_cleanup(qs, d) end module_function :parse_query diff --git a/actionpack/test/controller/request/url_encoded_params_parsing_test.rb b/actionpack/test/controller/request/url_encoded_params_parsing_test.rb index 89239687de..90865afc4e 100644 --- a/actionpack/test/controller/request/url_encoded_params_parsing_test.rb +++ b/actionpack/test/controller/request/url_encoded_params_parsing_test.rb @@ -156,12 +156,6 @@ class UrlEncodedParamsParsingTest < ActionController::IntegrationTest assert_parses expected, query end - test "parses params with Prototype's hack around Safari 2 trailing null character" do - query = "selected[]=1&selected[]=2&selected[]=3&_=" - expected = { "selected" => [ "1", "2", "3" ] } - assert_parses expected, query - end - test "passes through rack middleware and parses params" do with_muck_middleware do assert_parses({ "a" => { "b" => "c" } }, "a[b]=c") |