mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
reduce conditionals in url_for
This commit is contained in:
parent
2d73a73d2f
commit
9df60693d7
1 changed files with 6 additions and 3 deletions
|
@ -655,7 +655,7 @@ module ActionDispatch
|
|||
options = default_url_options.merge(options || {})
|
||||
|
||||
user, password = extract_authentication(options)
|
||||
recall = options.delete(:_recall)
|
||||
recall = options.delete(:_recall) { {} }
|
||||
|
||||
original_script_name = options.delete(:original_script_name).presence
|
||||
script_name = options.delete(:script_name).presence || _generate_prefix(options)
|
||||
|
@ -668,8 +668,11 @@ module ActionDispatch
|
|||
RESERVED_OPTIONS.each { |ro| path_options.delete ro }
|
||||
path_options = yield(path_options) if block_given?
|
||||
|
||||
path, params = generate(path_options, recall || {})
|
||||
params.merge!(options[:params] || {})
|
||||
path, params = generate(path_options, recall)
|
||||
|
||||
if options.key? :params
|
||||
params.merge! options[:params]
|
||||
end
|
||||
|
||||
ActionDispatch::Http::URL.url_for(options.merge!({
|
||||
:path => path,
|
||||
|
|
Loading…
Reference in a new issue