diff options
author | Andrew White <andyw@pixeltrix.co.uk> | 2013-03-03 19:18:01 +0000 |
---|---|---|
committer | Andrew White <andyw@pixeltrix.co.uk> | 2013-03-03 19:18:01 +0000 |
commit | 86cf7a2d166430fac1611aa7593b52b46eeb9f70 (patch) | |
tree | 998de220aab255a787bfbcf21341e71f5b6e6edb /actionpack/lib/action_dispatch/journey/route.rb | |
parent | 48c21e3255fa2f4b1875ca8616a5ad7706a7a4cb (diff) | |
download | rails-86cf7a2d166430fac1611aa7593b52b46eeb9f70.tar.gz rails-86cf7a2d166430fac1611aa7593b52b46eeb9f70.tar.bz2 rails-86cf7a2d166430fac1611aa7593b52b46eeb9f70.zip |
Use custom visitor class for optimized url helpers
Rather than trying to use gsub to remove the optional route segments,
which will fail with nested optional segments, use a custom visitor
class that returns a empty string for group nodes.
Closes #9524
Diffstat (limited to 'actionpack/lib/action_dispatch/journey/route.rb')
-rw-r--r-- | actionpack/lib/action_dispatch/journey/route.rb | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/actionpack/lib/action_dispatch/journey/route.rb b/actionpack/lib/action_dispatch/journey/route.rb index 063302e0f2..6fda085681 100644 --- a/actionpack/lib/action_dispatch/journey/route.rb +++ b/actionpack/lib/action_dispatch/journey/route.rb @@ -71,6 +71,10 @@ module ActionDispatch Visitors::Formatter.new(path_options).accept(path.spec) end + def optimized_path + Visitors::OptimizedPath.new.accept(path.spec) + end + def optional_parts path.optional_names.map { |n| n.to_sym } end |