From a5b55c6b9f8e09bb0936b30f5c096b28bafe00a4 Mon Sep 17 00:00:00 2001 From: David Heinemeier Hansson Date: Sat, 26 Mar 2005 13:31:01 +0000 Subject: [PATCH] Removed old UrlWriter methods that are no longer in use after Routes #942 git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@999 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- .../lib/action_controller/url_rewriter.rb | 49 +- actionpack/test/controller/helper_test.rb | 2 +- actionpack/test/controller/url_obsolete.rb | 487 ------------------ 3 files changed, 5 insertions(+), 533 deletions(-) delete mode 100644 actionpack/test/controller/url_obsolete.rb diff --git a/actionpack/lib/action_controller/url_rewriter.rb b/actionpack/lib/action_controller/url_rewriter.rb index 3e6ccf7a06..89245ceb37 100644 --- a/actionpack/lib/action_controller/url_rewriter.rb +++ b/actionpack/lib/action_controller/url_rewriter.rb @@ -2,10 +2,9 @@ module ActionController # Rewrites URLs for Base.redirect_to and Base.url_for in the controller. class UrlRewriter #:nodoc: - RESERVED_OPTIONS = [:anchor, :params, :only_path, :host, :protocol, :application_prefix] + RESERVED_OPTIONS = [:anchor, :params, :only_path, :host, :protocol] def initialize(request, parameters) @request, @parameters = request, parameters - @rewritten_path = @request.path ? @request.path.dup : "" end def rewrite(options = {}) @@ -19,13 +18,13 @@ def to_str alias_method :to_s, :to_str private - def rewrite_url(path, options) + def rewrite_url(path, options) rewritten_url = "" rewritten_url << (options[:protocol] || @request.protocol) unless options[:only_path] rewritten_url << (options[:host] || @request.host_with_port) unless options[:only_path] - rewritten_url << (options[:application_prefix] || @request.relative_url_root) + rewritten_url << @request.relative_url_root.to_s rewritten_url << path rewritten_url << "##{options[:anchor]}" if options[:anchor] @@ -34,7 +33,7 @@ def rewrite_url(path, options) def rewrite_path(options) options = options.symbolize_keys - options.update((options[:params]).symbolize_keys) if options[:params] + options.update((options[:params] || {}).symbolize_keys) RESERVED_OPTIONS.each {|k| options.delete k} path, extras = Routing::Routes.generate(options, @request) @@ -45,46 +44,6 @@ def rewrite_path(options) return path end - def action_name(options, action_prefix = nil, action_suffix = nil) - ensure_slash_suffix(options, :action_prefix) - ensure_slash_prefix(options, :action_suffix) - - prefix = options[:action_prefix] || action_prefix || "" - suffix = options[:action] == "index" ? "" : (options[:action_suffix] || action_suffix || "") - name = (options[:action] == "index" ? "" : options[:action]) || "" - - return prefix + name + suffix - end - - def controller_name(options, controller_prefix) - ensure_slash_suffix(options, :controller_prefix) - - controller_name = case options[:controller_prefix] - when String: options[:controller_prefix] - when false : "" - when nil : controller_prefix || "" - end - - controller_name << (options[:controller] + "/") if options[:controller] - return controller_name - end - - def path_params_in_list(options) - options[:path_params].inject("") { |path, pair| path += "/#{pair.last}" } - end - - def ensure_slash_suffix(options, key) - options[key] = options[key] + "/" if options[key] && !options[key].empty? && options[key][-1..-1] != "/" - end - - def ensure_slash_prefix(options, key) - options[key] = "/" + options[key] if options[key] && !options[key].empty? && options[key][0..1] != "/" - end - - def include_id_in_path_params(options) - options[:path_params] = (options[:path_params] || {}).merge({"id" => options[:id]}) if options[:id] - end - # Returns a query string with escaped keys and values from the passed hash. If the passed hash contains an "id" it'll # be added as a path element instead of a regular parameter pair. def build_query_string(hash) diff --git a/actionpack/test/controller/helper_test.rb b/actionpack/test/controller/helper_test.rb index 0a9840f705..2c2c48f15b 100644 --- a/actionpack/test/controller/helper_test.rb +++ b/actionpack/test/controller/helper_test.rb @@ -77,7 +77,7 @@ def test_declare_helper def test_declare_missing_helper assert_equal helper_methods, missing_methods - assert_raise(LoadError) { @controller_class.helper :missing } + assert_raise(MissingSourceFile) { @controller_class.helper :missing } end def test_declare_missing_file_from_helper diff --git a/actionpack/test/controller/url_obsolete.rb b/actionpack/test/controller/url_obsolete.rb deleted file mode 100644 index 4b6544dbf7..0000000000 --- a/actionpack/test/controller/url_obsolete.rb +++ /dev/null @@ -1,487 +0,0 @@ -require File.dirname(__FILE__) + '/../abstract_unit' -require 'action_controller/url_rewriter' - -MockRequest = Struct.new("MockRequest", :protocol, :host, :port, :path, :parameters, :path_parameters) -class MockRequest - def host_with_port - if (protocol == "http://" && port == 80) || (protocol == "https://" && port == 443) - host - else - host + ":#{port}" - end - end -end - -class UrlMockFactory - def self.create(path, parameters) - ActionController::UrlRewriter.new( - MockRequest.new("http://", "example.com", 80, path, parameters), - parameters - ) - end -end - -# old-style support for .new -module ActionController - class UrlRewriter - def self.old_new(request, controller, action) - request.parameters[:controller] = controller - request.parameters[:action] = action - return new(request, request.parameters) - end - end -end -class UrlTest < Test::Unit::TestCase - def setup - @library_url = ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", - "www.singlefile.com", - 80, - "/library/books/ISBN/0743536703/show", - { "type" => "ISBN", "code" => "0743536703" } - ), "books", "show") - - @library_url_using_module = ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", - "www.singlefile.com", - 80, - "/library/books/ISBN/0743536703/show", - { "type" => "ISBN", "code" => "0743536703", "module" => "library" } - ), "books", "show") - - @library_url_on_index = ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", - "www.singlefile.com", - 80, - "/library/books/ISBN/0743536703/", - { "type" => "ISBN", "code" => "0743536703" } - ), "books", "index") - - @clean_urls = [ - ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", "www.singlefile.com", 80, "/identity/", {} - ), "identity", "index"), - ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", "www.singlefile.com", 80, "/identity", {} - ), "identity", "index") - ] - - @clean_url_with_id = ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", "www.singlefile.com", 80, "/identity/show/5", { "id" => "5" } - ), "identity", "show") - - @clean_url_with_same_action_and_controller_name = ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", "www.singlefile.com", 80, "/login/login", { } - ), "login", "login") - - @clean_url_with_same_action_and_controller_and_module_name = ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", "www.singlefile.com", 80, "/login/login/login", { "module" => "login" } - ), "login", "login") - - @clean_url_with_id_as_char = ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", "www.singlefile.com", 80, "/teachers/show/t", { "id" => "t" } - ), "teachers", "show") - end - - def test_clean_action - assert_equal "http://www.singlefile.com/library/books/ISBN/0743536703/edit", @library_url.rewrite(:action => "edit") - end - - def test_clean_action_to_another_host - assert_equal( - "http://www.booksphere.com/library/books/ISBN/0743536703/edit", - @library_url.rewrite(:action => "edit", :host => "www.booksphere.com") - ) - end - - def test_clean_action_to_another_host_and_protocol - assert_equal( - "https://www.booksphere.com/library/books/ISBN/0743536703/edit", - @library_url.rewrite(:action => "edit", :host => "www.booksphere.com", :protocol => "https://") - ) - end - - def test_clean_action_with_only_path - assert_equal "/library/books/ISBN/0743536703/edit", @library_url.rewrite(:action => "edit", :only_path => true) - end - - def test_action_from_index - assert_equal "http://www.singlefile.com/library/books/ISBN/0743536703/edit", @library_url_on_index.rewrite(:action => "edit") - end - - def test_action_from_index_on_clean - @clean_urls.each do |url| - assert_equal "http://www.singlefile.com/identity/edit", url.rewrite(:action => "edit") - end - end - - def test_action_without_prefix - assert_equal "http://www.singlefile.com/library/books/", @library_url.rewrite(:action => "index", :action_prefix => "") - end - - def test_action_with_prefix - assert_equal( - "http://www.singlefile.com/library/books/XTC/123/show", - @library_url.rewrite(:action => "show", :action_prefix => "XTC/123") - ) - end - - def test_action_prefix_alone - assert_equal( - "http://www.singlefile.com/library/books/XTC/123/", - @library_url.rewrite(:action_prefix => "XTC/123") - ) - end - - def test_action_with_suffix - assert_equal( - "http://www.singlefile.com/library/books/show/XTC/123", - @library_url.rewrite(:action => "show", :action_prefix => "", :action_suffix => "XTC/123") - ) - end - - def test_clean_controller - assert_equal "http://www.singlefile.com/library/settings/", @library_url.rewrite(:controller => "settings") - end - - def test_clean_controller_prefix - assert_equal "http://www.singlefile.com/shop/", @library_url.rewrite(:controller_prefix => "shop") - end - - def test_clean_controller_with_module - assert_equal "http://www.singlefile.com/shop/purchases/", @library_url.rewrite(:module => "shop", :controller => "purchases") - end - - def test_getting_out_of_a_module - assert_equal "http://www.singlefile.com/purchases/", @library_url_using_module.rewrite(:module => false, :controller => "purchases") - end - - def test_controller_and_action - assert_equal "http://www.singlefile.com/library/settings/show", @library_url.rewrite(:controller => "settings", :action => "show") - end - - def test_controller_and_action_and_anchor - assert_equal( - "http://www.singlefile.com/library/settings/show#5", - @library_url.rewrite(:controller => "settings", :action => "show", :anchor => "5") - ) - end - - def test_controller_and_action_and_empty_overwrite_params_and_anchor - assert_equal( - "http://www.singlefile.com/library/settings/show?code=0743536703&type=ISBN#5", - @library_url.rewrite(:controller => "settings", :action => "show", :overwrite_params => {}, :anchor => "5") - ) - end - - def test_controller_and_action_and_overwrite_params_and_anchor - assert_equal( - "http://www.singlefile.com/library/settings/show?code=0000001&type=ISBN#5", - @library_url.rewrite(:controller => "settings", :action => "show", :overwrite_params => {"code"=>"0000001"}, :anchor => "5") - ) - end - - def test_controller_and_action_and_overwrite_params_with_nil_value_and_anchor - assert_equal( - "http://www.singlefile.com/library/settings/show?type=ISBN#5", - @library_url.rewrite(:controller => "settings", :action => "show", :overwrite_params => {"code" => nil}, :anchor => "5") - ) - end - - def test_controller_and_action_params_and_overwrite_params_and_anchor - assert_equal( - "http://www.singlefile.com/library/settings/show?code=0000001&version=5.0#5", - @library_url.rewrite(:controller => "settings", :action => "show", :params=>{"version" => "5.0"}, :overwrite_params => {"code"=>"0000001"}, :anchor => "5") - ) - end - - def test_controller_and_action_and_params_anchor - assert_equal( - "http://www.singlefile.com/library/settings/show?update=1#5", - @library_url.rewrite(:controller => "settings", :action => "show", :params => { "update" => "1"}, :anchor => "5") - ) - end - - def test_controller_and_index_action - assert_equal "http://www.singlefile.com/library/settings/", @library_url.rewrite(:controller => "settings", :action => "index") - end - - def test_same_controller_and_action_names - assert_equal "http://www.singlefile.com/login/logout", @clean_url_with_same_action_and_controller_name.rewrite(:action => "logout") - end - - def xtest_same_module_and_controller_and_action_names - assert_equal "http://www.singlefile.com/login/login/logout", @clean_url_with_same_action_and_controller_and_module_name.rewrite(:action => "logout") - end - - def test_controller_and_action_with_same_name_as_controller - @clean_urls.each do |url| - assert_equal "http://www.singlefile.com/anything/identity", url.rewrite(:controller => "anything", :action => "identity") - end - end - - def test_controller_and_index_action_without_controller_prefix - assert_equal( - "http://www.singlefile.com/settings/", - @library_url.rewrite(:controller => "settings", :action => "index", :controller_prefix => "") - ) - end - - def test_controller_and_index_action_with_controller_prefix - assert_equal( - "http://www.singlefile.com/fantastic/settings/show", - @library_url.rewrite(:controller => "settings", :action => "show", :controller_prefix => "fantastic") - ) - end - - def test_path_parameters - assert_equal "http://www.singlefile.com/library/books/EXBC/0743536703/show", @library_url.rewrite(:path_params => {"type" => "EXBC"}) - end - - def test_parameters - assert_equal( - "http://www.singlefile.com/library/books/ISBN/0743536703/show?delete=1&name=David", - @library_url.rewrite(:params => {"delete" => "1", "name" => "David"}) - ) - end - - def test_parameters_with_id - @clean_urls.each do |url| - assert_equal( - "http://www.singlefile.com/identity/show?name=David&id=5", - url.rewrite( - :action => "show", - :params => { "id" => "5", "name" => "David" } - ) - ) - end - end - - def test_parameters_with_array - @clean_urls.each do |url| - assert_equal( - "http://www.singlefile.com/identity/show?id[]=3&id[]=5&id[]=10", - url.rewrite( - :action => "show", - :params => { 'id' => [ 3, 5, 10 ] } ) - ) - end - end - - def test_action_with_id - assert_equal( - "http://www.singlefile.com/identity/show/7", - @clean_url_with_id.rewrite( - :action => "show", - :id => 7 - ) - ) - @clean_urls.each do |url| - assert_equal( - "http://www.singlefile.com/identity/index/7", - url.rewrite(:id => 7) - ) - end - end - - def test_parameters_with_id_and_away - assert_equal( - "http://www.singlefile.com/identity/show/25?name=David", - @clean_url_with_id.rewrite( - :path_params => { "id" => "25" }, - :params => { "name" => "David" } - ) - ) - end - - def test_parameters_with_index_and_id - @clean_urls.each do |url| - assert_equal( - "http://www.singlefile.com/identity/index/25?name=David", - url.rewrite( - :path_params => { "id" => "25" }, - :params => { "name" => "David" } - ) - ) - end - end - - def test_action_going_away_from_id - assert_equal( - "http://www.singlefile.com/identity/list", - @clean_url_with_id.rewrite( - :action => "list" - ) - ) - end - - def test_parameters_with_direct_id_and_away - assert_equal( - "http://www.singlefile.com/identity/show/25?name=David", - @clean_url_with_id.rewrite( - :id => "25", - :params => { "name" => "David" } - ) - ) - end - - def test_parameters_with_direct_id_and_away - assert_equal( - "http://www.singlefile.com/store/open/25?name=David", - @clean_url_with_id.rewrite( - :controller => "store", - :action => "open", - :id => "25", - :params => { "name" => "David" } - ) - ) - end - - def test_parameters_to_id - @clean_urls.each do |url| - %w(show index).each do |action| - assert_equal( - "http://www.singlefile.com/identity/#{action}/25?name=David", - url.rewrite( - :action => action, - :path_params => { "id" => "25" }, - :params => { "name" => "David" } - ) - ) - end - end - end - - def test_parameters_from_id - assert_equal( - "http://www.singlefile.com/identity/", - @clean_url_with_id.rewrite( - :action => "index" - ) - ) - end - - def test_id_as_char_and_part_of_controller - assert_equal( - "http://www.singlefile.com/teachers/skill/5", - @clean_url_with_id_as_char.rewrite( - :action => "skill", - :id => 5 - ) - ) - end - - def test_from_clean_to_library - @clean_urls.each do |url| - assert_equal( - "http://www.singlefile.com/library/books/ISBN/0743536703/show?delete=1&name=David", - url.rewrite( - :controller_prefix => "library", - :controller => "books", - :action_prefix => "ISBN/0743536703", - :action => "show", - :params => { "delete" => "1", "name" => "David" } - ) - ) - end - end - - def test_from_library_to_clean - assert_equal( - "http://www.singlefile.com/identity/", - @library_url.rewrite( - :controller => "identity", :controller_prefix => "" - ) - ) - end - - def test_from_another_port - @library_url = ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", - "www.singlefile.com", - 8080, - "/library/books/ISBN/0743536703/show", - { "type" => "ISBN", "code" => "0743536703" } - ), "books", "show") - - assert_equal( - "http://www.singlefile.com:8080/identity/", - @library_url.rewrite( - :controller => "identity", :controller_prefix => "" - ) - ) - end - - def test_basecamp - basecamp_url = ActionController::UrlRewriter.old_new(MockRequest.new( - "http://", - "projects.basecamp", - 80, - "/clients/disarray/1/msg/transcripts/", - {"category_name"=>"transcripts", "client_name"=>"disarray", "action"=>"index", "controller"=>"msg", "project_name"=>"1"} - ), "msg", "index") - - assert_equal( - "http://projects.basecamp/clients/disarray/1/msg/transcripts/1/comments", - basecamp_url.rewrite(:action_prefix => "transcripts/1", :action => "comments") - ) - end - - def test_on_explicit_index_page # My index page is very modest, thank you... - url = ActionController::UrlRewriter.old_new( - MockRequest.new( - "http://", "example.com", 80, "/controller/index", - {"controller"=>"controller", "action"=>"index"} - ), "controller", "index" - ) - assert_equal("http://example.com/controller/foo", url.rewrite(:action => 'foo')) - end - - def test_rewriting_on_similar_fragments - url = UrlMockFactory.create("/advertisements/advert/", {"controller"=>"advert", "action"=>"index"}) - assert_equal("http://example.com/advertisements/advert/news", url.rewrite(:action => 'news')) - end - - def test_rewriting_on_similar_fragments_with_action_prefixes - url = UrlMockFactory.create( - "/clients/prall/1/msg/all/", - { "category_name"=>"all", "client_name"=>"prall", "action"=>"index", "controller"=>"msg", "project_name"=>"1"} - ) - - assert_equal( - "http://example.com/clients/prall/1/msg/all/new", - url.rewrite({ :controller => "msg", :action_prefix => "all", :action => "new" }) - ) - - url = UrlMockFactory.create( - "/clients/prall/1/msg/all/", - { "category_name"=>"all", "client_name"=>"prall", "action"=>"index", "controller"=>"msg", "project_name"=>"1"} - ) - - assert_equal( - "http://example.com/clients/prall/1/msg/allous/new", - url.rewrite({ :controller => "msg", :action_prefix => "allous", :action => "new" }) - ) - end - - def test_clean_application_prefix - assert_equal "http://www.singlefile.com/namespace/library/books/ISBN/0743536703/show", - @library_url.rewrite(:application_prefix => "/namespace") - end - - def test_clean_application_prefix_with_controller_prefix - assert_equal "http://www.singlefile.com/namespace/shop/", - @library_url.rewrite(:application_prefix => "/namespace", - :controller_prefix => "shop" ) - end - - def test_blank_application_prefix - assert_equal "http://www.singlefile.com/library/books/ISBN/0743536703/show", - @library_url.rewrite(:application_prefix => "") - end - - def test_nil_application_prefix - assert_equal "http://www.singlefile.com/library/books/ISBN/0743536703/show", - @library_url.rewrite(:application_prefix => nil) - end -end -- GitLab