Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
openSUSE:12.2
rubygem-actionpack-2_3
2-3-null_array_param.patch
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File 2-3-null_array_param.patch of Package rubygem-actionpack-2_3
Index: lib/action_controller/request.rb =================================================================== --- lib/action_controller/request.rb.orig 2012-07-18 16:28:20.433076369 +0200 +++ lib/action_controller/request.rb 2012-07-18 16:29:56.813242333 +0200 @@ -495,17 +495,19 @@ EOM # Remove nils from the params hash def deep_munge(hash) + keys = hash.keys.find_all { |k| hash[k] == [nil] } + keys.each { |k| hash[k] = nil } + hash.each_value do |v| case v when Array v.grep(Hash) { |x| deep_munge(x) } + v.compact! when Hash deep_munge(v) end end - keys = hash.keys.find_all { |k| hash[k] == [nil] } - keys.each { |k| hash[k] = nil } hash end Index: test/controller/request/query_string_parsing_test.rb =================================================================== --- test/controller/request/query_string_parsing_test.rb.orig 2012-07-18 16:28:20.436077868 +0200 +++ test/controller/request/query_string_parsing_test.rb 2012-07-18 16:32:20.072439141 +0200 @@ -89,6 +89,10 @@ class QueryStringParsingTest < ActionCon assert_parses({"action"=>{"foo"=>[{"bar"=>nil}]}}, "action[foo][][bar]") end + def test_array_parses_without_nil + assert_parses({"action" => ['1']}, "action[]=1&action[]") + end + test "query string with empty key" do assert_parses( { "action" => "create_customer", "full_name" => "David Heinemeier Hansson" },
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor