Permalink
Browse files

Make rake routes gracefully handle routes with no name

Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
  • Loading branch information...
1 parent a2c547f commit 829df8007d5898ee5f68c68412034d39adbad5c5 @rubys rubys committed with spastorino Aug 29, 2010
Showing with 5 additions and 5 deletions.
  1. +5 −5 railties/lib/rails/tasks/routes.rake
@@ -13,16 +13,16 @@ task :routes => :environment do
reqs[:to] = route.app unless route.app.class.name.to_s =~ /^ActionDispatch::Routing/
reqs = reqs.empty? ? "" : reqs.inspect
- {:name => route.name, :verb => route.verb.to_s, :path => route.path, :reqs => reqs}
+ {:name => route.name.to_s, :verb => route.verb.to_s, :path => route.path, :reqs => reqs}
end
routes.reject! { |r| r[:path] =~ %r{/rails/info/properties} } # Skip the route if it's internal info route
- name_width = routes.map{ |r| r[:name].length if r[:name] }.max
- verb_width = routes.map{ |r| r[:verb].length if r[:verb] }.max
- path_width = routes.map{ |r| r[:path].length if r[:path] }.max
+ 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.each do |r|
puts "#{r[:name].rjust(name_width)} #{r[:verb].ljust(verb_width)} #{r[:path].ljust(path_width)} #{r[:reqs]}"
end
-end
+end

0 comments on commit 829df80

Please sign in to comment.