diff --git a/actionpack/lib/action_dispatch/routing/inspector.rb b/actionpack/lib/action_dispatch/routing/inspector.rb index 273f5b9306fb1706150e0612776d08140a9136e2..dc03a7370e8eea01272bbf458d61f66020b8b47c 100644 --- a/actionpack/lib/action_dispatch/routing/inspector.rb +++ b/actionpack/lib/action_dispatch/routing/inspector.rb @@ -16,7 +16,7 @@ def rack_app @rack_app ||= begin endpoint = app.app - if ActionDispatch::Routing::Redirect === endpoint || !app.dispatcher? + if app.redirect? || !app.dispatcher? endpoint end end diff --git a/actionpack/lib/action_dispatch/routing/mapper.rb b/actionpack/lib/action_dispatch/routing/mapper.rb index 08a5e7c637f064a51d0f15b65aa67c8dea6f2e3d..86357ded4fe03d1a1a9c6703ba26117e7d95527a 100644 --- a/actionpack/lib/action_dispatch/routing/mapper.rb +++ b/actionpack/lib/action_dispatch/routing/mapper.rb @@ -18,7 +18,7 @@ class Mapper class Constraints #:nodoc: attr_reader :app, :constraints - def initialize(app, constraints, request, dispatcher_p) + def initialize(app, constraints, request, dispatcher_p, redirect_p) # Unwrap Constraints objects. I don't actually think it's possible # to pass a Constraints object to this constructor, but there were # multiple places that kept testing children of this object. I @@ -28,12 +28,14 @@ def initialize(app, constraints, request, dispatcher_p) app = app.app end - @dispatcher = dispatcher_p + @dispatcher = dispatcher_p + @redirect = redirect_p @app, @constraints, @request = app, constraints, request end def dispatcher?; @dispatcher; end + def redirect?; @redirect; end def matches?(req) @constraints.all? do |constraint| @@ -218,21 +220,24 @@ def normalize_conditions! end def app + dispatcher_p = false + redirect = false + # Unwrap any constraints so we can see what's inside for route generation. # This allows the formatter to skip over any mounted applications or redirects # that shouldn't be matched when using a url_for without a route name. if to.respond_to?(:call) - dispatcher_p = false endpoint = to + redirect = Redirect === endpoint else dispatcher_p = true endpoint = dispatcher end if blocks.any? - Constraints.new(endpoint, blocks, @set.request_class, dispatcher_p) + Constraints.new(endpoint, blocks, @set.request_class, dispatcher_p, redirect) else - Constraints.new(endpoint, blocks, @set.request_class, dispatcher_p) + Constraints.new(endpoint, blocks, @set.request_class, dispatcher_p, redirect) end end