Permalink
Browse files

Merge pull request #8995 from schneems/schneems/formatting-the-format…

…ters

Remove "Application" section title from routes
  • Loading branch information...
2 parents 40e7978 + f654c3c commit 5da8491e9f5daa5915410dc891b06172f295929c @guilleiguaran guilleiguaran committed Jan 18, 2013
Showing with 13 additions and 6 deletions.
  1. +13 −6 actionpack/lib/action_dispatch/routing/inspector.rb
@@ -73,10 +73,11 @@ def format(formatter, filter = nil)
routes_to_display = filter_routes(filter)
routes = collect_routes(routes_to_display)
- formatter.section :application, 'Application routes', routes
+ formatter.section routes
@engines.each do |name, engine_routes|
- formatter.section :engine, "Routes for #{name}", engine_routes
+ formatter.section_title "Routes for #{name}"
+ formatter.section engine_routes
end
formatter.result
@@ -125,8 +126,11 @@ def result
@buffer.join("\n")
end
- def section(type, title, routes)
- @buffer << "\n#{title}:" unless type == :application
+ def section_title(title)
+ @buffer << "\n#{title}:"
+ end
+
+ def section(routes)
@buffer << draw_section(routes)
end
@@ -148,8 +152,11 @@ def initialize(view)
@buffer = []
end
- def section(type, title, routes)
- @buffer << %(<tr><th colspan="4">#{title}</th></tr>)
+ def section_title(title)
+ @buffer << "<tr><th colspan="4">#{title}</th></tr>"
+ end
+
+ def section(routes)
@buffer << @view.render(partial: "routes/route", collection: routes)
end

0 comments on commit 5da8491

Please sign in to comment.