diff options
author | Alberto Almagro <albertoalmagro@gmail.com> | 2018-12-11 23:45:57 +0100 |
---|---|---|
committer | Alberto Almagro <albertoalmagro@gmail.com> | 2019-05-22 23:03:56 +0200 |
commit | 29506d0f03a76b09497d1a392ef1adbddf776102 (patch) | |
tree | e210cd36f48e5f95f9417b4fba933752ddffd294 /actionpack/lib/action_dispatch/journey | |
parent | fb9117e190e39872fff7ae5d6b96bfb26ef8b32c (diff) | |
download | rails-29506d0f03a76b09497d1a392ef1adbddf776102.tar.gz rails-29506d0f03a76b09497d1a392ef1adbddf776102.tar.bz2 rails-29506d0f03a76b09497d1a392ef1adbddf776102.zip |
Use keyword arguments for ActionDispatch::Journey:Route constructor
This commit changes from constructor's argument list to keyword
arguments in order to remove the dependency of remember parameters'
positions.
The constructor already provided a default value for `internal`, this
commits takes the chance to also add default values for `precedence` and
`scope_options`.
Diffstat (limited to 'actionpack/lib/action_dispatch/journey')
-rw-r--r-- | actionpack/lib/action_dispatch/journey/route.rb | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/actionpack/lib/action_dispatch/journey/route.rb b/actionpack/lib/action_dispatch/journey/route.rb index 587ebcb873..d497b9d475 100644 --- a/actionpack/lib/action_dispatch/journey/route.rb +++ b/actionpack/lib/action_dispatch/journey/route.rb @@ -50,14 +50,15 @@ module ActionDispatch end def self.build(name, app, path, constraints, required_defaults, defaults) - request_method_match = verb_matcher(constraints.delete(:request_method)) - new name, app, path, constraints, required_defaults, defaults, request_method_match, 0, {} + new name: name, app: app, path: path, constraints: constraints, + required_defaults: required_defaults, defaults: defaults, + request_method_match: verb_matcher(constraints.delete(:request_method)) end ## # +path+ is a path constraint. # +constraints+ is a hash of constraints to be applied to this route. - def initialize(name, app, path, constraints, required_defaults, defaults, request_method_match, precedence, scope_options, internal = false) + def initialize(name:, app:, path:, constraints:, required_defaults:, defaults:, request_method_match:, precedence: 0, scope_options: {}, internal: false) @name = name @app = app @path = path |