TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Make the helpers a required argument

Browse Source
This commit is contained in:
Evan Phoenix 2015-02-19 15:11:08 -08:00
parent 43dae99645
commit e002a68a4e
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
actionpack/lib/action_dispatch/routing/routes_proxy.rb
View File

@ -8,9 +8,9 @@ class RoutesProxy #:nodoc:
attr_accessor :scope, :routes attr_accessor :scope, :routes
alias :_routes :routes alias :_routes :routes
def initialize(routes, scope, helpers=nil) def initialize(routes, scope, helpers)
@routes, @scope = routes, scope @routes, @scope = routes, scope
@helpers = helpers || routes.url_helpers @helpers = helpers
end end
def url_options def url_options

2
actionview/test/activerecord/polymorphic_routes_test.rb
View File

@ -113,7 +113,7 @@ def test_symbol_with_options
def test_passing_routes_proxy def test_passing_routes_proxy
with_namespaced_routes(:blog) do with_namespaced_routes(:blog) do
proxy = ActionDispatch::Routing::RoutesProxy.new(_routes, self) proxy = ActionDispatch::Routing::RoutesProxy.new(_routes, self, _routes.url_helpers)
@blog_post.save @blog_post.save
assert_url "http://example.com/posts/#{@blog_post.id}", [proxy, @blog_post] assert_url "http://example.com/posts/#{@blog_post.id}", [proxy, @blog_post]
end end