From 8554537e48d6ed18ef08cb3ac0a9da96ac3c5cd6 Mon Sep 17 00:00:00 2001 From: Steve Klabnik Date: Fri, 14 Dec 2012 15:44:57 -0500 Subject: Revert "Merge pull request #8499 from schneems/schneems/html-route-inspector" This reverts commit ae68fc3864e99ab43c18fd12577744e1583f6b64, reversing changes made to 0262a18c7b0ab6f60fee842b3007388f9ffeb0fa. See here: https://github.com/rails/rails/pull/8499#issuecomment-11356417 --- .../lib/action_dispatch/routing/inspector.rb | 31 +++++----------------- 1 file changed, 7 insertions(+), 24 deletions(-) (limited to 'actionpack') diff --git a/actionpack/lib/action_dispatch/routing/inspector.rb b/actionpack/lib/action_dispatch/routing/inspector.rb index 63d394be75..8d7461ecc3 100644 --- a/actionpack/lib/action_dispatch/routing/inspector.rb +++ b/actionpack/lib/action_dispatch/routing/inspector.rb @@ -67,19 +67,15 @@ module ActionDispatch @engines = Hash.new end - def format(all_routes, filter = nil, format = :txt) + def format(all_routes, filter = nil) if filter all_routes = all_routes.select{ |route| route.defaults[:controller] == filter } end routes = collect_routes(all_routes) - routes = formatted_routes(routes, format) + formatted_routes_for_engines(format) - if format == :html - routes.join('') - else - routes - end + formatted_routes(routes) + + formatted_routes_for_engines end def collect_routes(routes) @@ -105,32 +101,19 @@ module ActionDispatch end end - def formatted_routes_for_engines(format) + def formatted_routes_for_engines @engines.map do |name, routes| - ["\nRoutes for #{name}:"] + formatted_routes(routes, format) + ["\nRoutes for #{name}:"] + formatted_routes(routes) end.flatten end - def formatted_routes(routes, format) + def formatted_routes(routes) name_width = routes.map{ |r| r[:name].length }.max verb_width = routes.map{ |r| r[:verb].length }.max path_width = routes.map{ |r| r[:path].length }.max routes.map do |r| - if format == :txt - "#{r[:name].rjust(name_width)} " + - "#{r[:verb].ljust(verb_width)} " + - "#{r[:path].ljust(path_width)} " + - "#{r[:reqs]}" - elsif format == :html - route = r - "" + - "#{route[:name] + "_path" if route[:name].present?}" + - "#{route[:verb]}" + - "#{route[:path]}" + - "#{route[:reqs]}" + - "" - end + "#{r[:name].rjust(name_width)} #{r[:verb].ljust(verb_width)} #{r[:path].ljust(path_width)} #{r[:reqs]}" end end end -- cgit v1.2.3