diff options
author | Aaron Patterson <aaron.patterson@gmail.com> | 2014-05-29 16:13:29 -0700 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2014-05-29 16:13:29 -0700 |
commit | 74b73f132aab596ce509082fdf2b0e9dd4226e9c (patch) | |
tree | 821733b6379d00555a7c3e21e9c546bf4f9a958d | |
parent | be137b0ac389c5bed93df4b5a76e19549b8e1cfa (diff) | |
download | rails-74b73f132aab596ce509082fdf2b0e9dd4226e9c.tar.gz rails-74b73f132aab596ce509082fdf2b0e9dd4226e9c.tar.bz2 rails-74b73f132aab596ce509082fdf2b0e9dd4226e9c.zip |
Mapping never actually uses @set, so rm
-rw-r--r-- | actionpack/lib/action_dispatch/routing/mapper.rb | 6 | ||||
-rw-r--r-- | actionpack/test/dispatch/mapper_test.rb | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/actionpack/lib/action_dispatch/routing/mapper.rb b/actionpack/lib/action_dispatch/routing/mapper.rb index e4fd63cf0e..3bc578b379 100644 --- a/actionpack/lib/action_dispatch/routing/mapper.rb +++ b/actionpack/lib/action_dispatch/routing/mapper.rb @@ -66,8 +66,8 @@ module ActionDispatch attr_reader :scope, :options, :requirements, :conditions, :defaults attr_reader :to, :default_controller, :default_action - def initialize(set, scope, path, options) - @set, @scope = set, scope + def initialize(scope, path, options) + @scope = scope @requirements, @conditions, @defaults = {}, {}, {} options = scope[:options].merge(options) if scope[:options] @@ -1515,7 +1515,7 @@ module ActionDispatch options[:as] = name_for_action(options[:as], action) end - mapping = Mapping.new(@set, @scope, URI.parser.escape(path), options) + mapping = Mapping.new(@scope, URI.parser.escape(path), options) app, conditions, requirements, defaults, as, anchor = mapping.to_route @set.add_route(app, conditions, requirements, defaults, as, anchor) end diff --git a/actionpack/test/dispatch/mapper_test.rb b/actionpack/test/dispatch/mapper_test.rb index e3dcf9b88a..bf82e09f39 100644 --- a/actionpack/test/dispatch/mapper_test.rb +++ b/actionpack/test/dispatch/mapper_test.rb @@ -38,7 +38,7 @@ module ActionDispatch def test_mapping_requirements options = { :controller => 'foo', :action => 'bar', :via => :get } - m = Mapper::Mapping.new FakeSet.new, {}, '/store/:name(*rest)', options + m = Mapper::Mapping.new({}, '/store/:name(*rest)', options) _, _, requirements, _ = m.to_route assert_equal(/.+?/, requirements[:rest]) end |