Skip to content

Commit

Permalink
Move name_prefix merging into Scoping concern
Browse files Browse the repository at this point in the history
  • Loading branch information
josh committed Dec 8, 2009
1 parent e600b41 commit e86a82c
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions actionpack/lib/action_dispatch/routing/mapper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -258,7 +258,13 @@ def constraints(constraints = {})

def match(*args)
options = args.extract_options!

options = (@scope[:options] || {}).merge(options)

if @scope[:name_prefix]
options[:as] = "#{@scope[:name_prefix]}#{options[:as]}"
end

args.push(options)
super(*args)
end
Expand Down Expand Up @@ -391,9 +397,7 @@ def match(*args)
validate_defaults!(app, defaults, segment_keys)
app = Constraints.new(app, blocks)

name = @scope[:name_prefix] ? "#{@scope[:name_prefix]}#{options[:as]}" : options[:as]

@set.add_route(app, conditions, requirements, defaults, name)
@set.add_route(app, conditions, requirements, defaults, options[:as])

self
end
Expand Down

0 comments on commit e86a82c

Please sign in to comment.