Browse files

don't need the begin / end

  • Loading branch information...
1 parent 72c290c commit fb3e09a877f6b4649dff8c9fc4a991a3ac7fa3cf @tenderlove tenderlove committed Dec 29, 2011
Showing with 17 additions and 21 deletions.
  1. +17 −21 actionpack/lib/action_dispatch/routing/route_set.rb
View
38 actionpack/lib/action_dispatch/routing/route_set.rb
@@ -307,33 +307,29 @@ def #{name}
end
def url_helpers
- @url_helpers ||= begin
- routes = self
+ routes = self
- helpers = Module.new do
- extend ActiveSupport::Concern
- include UrlFor
+ @url_helpers ||= Module.new {
+ extend ActiveSupport::Concern
+ include UrlFor
- @_routes = routes
- def self.url_for(options)
- @_routes.url_for options
- end
+ @_routes = routes
+ def self.url_for(options)
+ @_routes.url_for options
+ end
- extend routes.named_routes.module
+ extend routes.named_routes.module
- # ROUTES TODO: install_helpers isn't great... can we make a module with the stuff that
- # we can include?
- # Yes plz - JP
- included do
- routes.install_helpers(self)
- singleton_class.send(:redefine_method, :_routes) { routes }
- end
-
- define_method(:_routes) { @_routes || routes }
+ # ROUTES TODO: install_helpers isn't great... can we make a module with the stuff that
+ # we can include?
+ # Yes plz - JP
+ included do
+ routes.install_helpers(self)
+ singleton_class.send(:redefine_method, :_routes) { routes }
end
- helpers
- end
+ define_method(:_routes) { @_routes || routes }
+ }
end
def empty?

0 comments on commit fb3e09a

Please sign in to comment.