From 2e175d35cd4fc4438f6d263c8fe205939cd4b95f Mon Sep 17 00:00:00 2001 From: Nicholas Seckar Date: Mon, 18 Jul 2005 03:25:53 +0000 Subject: Fix last changeset to pass unittests git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@1858 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- actionpack/lib/action_controller/assertions.rb | 4 +++- actionpack/lib/action_controller/url_rewriter.rb | 10 +++++----- 2 files changed, 8 insertions(+), 6 deletions(-) (limited to 'actionpack/lib/action_controller') diff --git a/actionpack/lib/action_controller/assertions.rb b/actionpack/lib/action_controller/assertions.rb index 20650c8654..bb81ffac75 100644 --- a/actionpack/lib/action_controller/assertions.rb +++ b/actionpack/lib/action_controller/assertions.rb @@ -138,7 +138,9 @@ module Test #:nodoc: # Load routes.rb if it hasn't been loaded. ActionController::Routing::Routes.reload if ActionController::Routing::Routes.empty? - generated_path, found_extras = ActionController::Routing::Routes.generate(options, extras) + generated_path, extra_keys = ActionController::Routing::Routes.generate(options, extras) + found_extras = options.reject {|k, v| ! extra_keys.include? k} + msg = build_message(message, "found extras , not ", found_extras, extras) assert_block(msg) { found_extras == extras } diff --git a/actionpack/lib/action_controller/url_rewriter.rb b/actionpack/lib/action_controller/url_rewriter.rb index 546bd13eb7..ecbc6851f4 100644 --- a/actionpack/lib/action_controller/url_rewriter.rb +++ b/actionpack/lib/action_controller/url_rewriter.rb @@ -31,13 +31,13 @@ module ActionController rewritten_url end - def rewrite_path(original_options) - options = original_options.symbolize_keys - options.update(params.symbolize_keys) if (params = options[:params]) + def rewrite_path(options) + options = options.symbolize_keys + options.update(options[:params].symbolize_keys) if options[:params] RESERVED_OPTIONS.each {|k| options.delete k} - path, extra_keys = Routing::Routes.generate(options, @request) # Warning: Routes will mutate and violate the options hash + path, extra_keys = Routing::Routes.generate(options.dup, @request) # Warning: Routes will mutate and violate the options hash - path << build_query_string(original_options.symbolize_keys, extra_keys) unless extra_keys.empty? + path << build_query_string(options, extra_keys) unless extra_keys.empty? path end -- cgit v1.2.3