Merge branch 'nil_script_name'
This commit is contained in:
commit
d3dafaa0b3
@ -749,7 +749,7 @@ def optimize_routes_generation?
|
||||
end
|
||||
|
||||
def find_script_name(options)
|
||||
options.delete(:script_name) { '' }
|
||||
options.delete(:script_name) || ''
|
||||
end
|
||||
|
||||
def path_for(options, route_name = nil) # :nodoc:
|
||||
|
@ -290,6 +290,13 @@ def test_relative_url_root_is_respected_for_named_routes
|
||||
end
|
||||
end
|
||||
|
||||
def test_using_nil_script_name_properly_concats_with_original_script_name
|
||||
add_host!
|
||||
assert_equal('https://www.basecamphq.com/subdir/c/a/i',
|
||||
W.new.url_for(:controller => 'c', :action => 'a', :id => 'i', :protocol => 'https', :script_name => nil, :original_script_name => '/subdir')
|
||||
)
|
||||
end
|
||||
|
||||
def test_only_path
|
||||
with_routing do |set|
|
||||
set.draw do
|
||||
|
Loading…
Reference in New Issue
Block a user