diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG index de5543289be73b66a32aa9d7e90ca967294fa1bc..33e5fa417bd587dcf1bfd255bba9b8aa83f72f28 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 6d7c99eff01d98d789a27d835080c0aeef941ca0..e79ac4daa1a350b2145884c0cd18821469d83e56 100644 --- a/actionpack/lib/action_controller/url_rewriter.rb +++ b/actionpack/lib/action_controller/url_rewriter.rb @@ -38,7 +38,7 @@ def rewrite_path(options) 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)