diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG index de5543289b..33e5fa417b 100644 --- a/actionpack/CHANGELOG +++ b/actionpack/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Fixed :overwrite_param so it doesn't delete but reject elements from @request.parameters #982 [raphinou@yahoo.com] + * Added :method option to verify for ensuring that either GET, POST, etc is allowed #984 [Jamis Buck] * Added options to set cc, bcc, subject, and body for UrlHelper#mail_to #966 [DeLynn] diff --git a/actionpack/lib/action_controller/url_rewriter.rb b/actionpack/lib/action_controller/url_rewriter.rb index 6d7c99eff0..e79ac4daa1 100644 --- a/actionpack/lib/action_controller/url_rewriter.rb +++ b/actionpack/lib/action_controller/url_rewriter.rb @@ -38,7 +38,7 @@ module ActionController path, extras = Routing::Routes.generate(options, @request) if extras[:overwrite_params] - params_copy = @request.parameters.delete_if { |k,v| ["controller","action"].include? k } + params_copy = @request.parameters.reject { |k,v| ["controller","action"].include? k } params_copy.update extras[:overwrite_params] extras.delete(:overwrite_params) extras.update(params_copy)