From f727bd24c211316a80b5d3c81e867f64d6ad7a09 Mon Sep 17 00:00:00 2001 From: Aaron Patterson Date: Mon, 10 Aug 2015 16:21:06 -0700 Subject: pull path conditional up we only need to check for `path` once. --- actionpack/lib/action_dispatch/routing/mapper.rb | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) (limited to 'actionpack/lib/action_dispatch/routing') diff --git a/actionpack/lib/action_dispatch/routing/mapper.rb b/actionpack/lib/action_dispatch/routing/mapper.rb index a0d12144c4..1c945dd496 100644 --- a/actionpack/lib/action_dispatch/routing/mapper.rb +++ b/actionpack/lib/action_dispatch/routing/mapper.rb @@ -1733,16 +1733,18 @@ module ActionDispatch end def path_for_action(action, path) #:nodoc: - if path.blank? && canonical_action?(action) + return "#{@scope[:path]}/#{path}" if path + + if canonical_action?(action) @scope[:path].to_s else - "#{@scope[:path]}/#{action_path(action, path)}" + "#{@scope[:path]}/#{action_path(action)}" end end - def action_path(name, path = nil) #:nodoc: + def action_path(name) #:nodoc: name = name.to_sym if name.is_a?(String) - path || @scope[:path_names][name] || name.to_s + @scope[:path_names][name] || name.to_s end def prefix_name_for_action(as, action) #:nodoc: -- cgit v1.2.3