@@ -50,11 +50,12 @@ module Rack return if k.empty? if after == "" - cur = params[k] - if cur.is_a?(Array) - params[k] << v - elsif cur && name == $1 - params[k] = [cur, v] + if cur = params[k] + if cur.is_a?(Array) + params[k] << v + else + params[k] = [cur, v] + end else params[k] = v end lib/rack/utils.rb @@ -62,7 +62,7 @@ context "Rack::Utils" do Rack::Utils.parse_query("x[y][z][]=1"). should.equal "x" => {"y" => {"z" => ["1"]}} Rack::Utils.parse_query("x[y][z]=1&x[y][z]=2"). - should.equal "x" => {"y" => {"z" => "2"}} + should.equal "x" => {"y" => {"z" => ["1", "2"]}} Rack::Utils.parse_query("x[y][z][]=1&x[y][z][]=2"). should.equal "x" => {"y" => {"z" => ["1", "2"]}} test/spec_rack_utils.rb 94a08fe93e785f34520d31fc2413d31bc211416d Joshua Peek josh@joshpeek.com http://github.com/scytrin/rack/commit/c9b4159e66860188c1abeebb90b1efc9d8fe53de c9b4159e66860188c1abeebb90b1efc9d8fe53de 2009-02-10T16:30:49-08:00 2009-02-10T16:30:49-08:00 Revert "Don't clobber nested parameter values. Fixes checkboxes in nested forms." This reverts commit 94a08fe93e785f34520d31fc2413d31bc211416d. a8c812ad8f9836aaf7acb0d28be9b37429a8f0da Joshua Peek josh@joshpeek.com