diff options
author | Rafael França <rafaelmfranca@gmail.com> | 2016-10-10 02:10:20 -0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-10-10 02:10:20 -0300 |
commit | 0cbb0be329e1375e415ad37c9c20ea9f74ff1ac3 (patch) | |
tree | 0d0363ee64c840cadf9e0e0d7e32de95633287ae /actionpack/lib/action_dispatch/routing/mapper.rb | |
parent | 034e4d38b508cbf2a8eaa9742df3e1948e482827 (diff) | |
parent | de9542acd56f60d281465a59eac11e15ca8b3323 (diff) | |
download | rails-0cbb0be329e1375e415ad37c9c20ea9f74ff1ac3.tar.gz rails-0cbb0be329e1375e415ad37c9c20ea9f74ff1ac3.tar.bz2 rails-0cbb0be329e1375e415ad37c9c20ea9f74ff1ac3.zip |
Merge pull request #26746 from rails/deprecations
Remove all deprecations from Action Pack
Diffstat (limited to 'actionpack/lib/action_dispatch/routing/mapper.rb')
-rw-r--r-- | actionpack/lib/action_dispatch/routing/mapper.rb | 19 |
1 files changed, 2 insertions, 17 deletions
diff --git a/actionpack/lib/action_dispatch/routing/mapper.rb b/actionpack/lib/action_dispatch/routing/mapper.rb index 4ec1b8ee1f..c481c190bf 100644 --- a/actionpack/lib/action_dispatch/routing/mapper.rb +++ b/actionpack/lib/action_dispatch/routing/mapper.rb @@ -1558,11 +1558,7 @@ module ActionDispatch options = path path, to = options.find { |name, _value| name.is_a?(String) } - if path.nil? - ActiveSupport::Deprecation.warn "Omitting the route path is deprecated. "\ - "Specify the path with a String or a Symbol instead." - path = "" - end + raise ArgumentError, "Route path not specified" if path.nil? case to when Symbol @@ -1844,18 +1840,7 @@ module ActionDispatch path_types.fetch(String, []).each do |_path| route_options = options.dup if _path && option_path - ActiveSupport::Deprecation.warn <<-eowarn -Specifying strings for both :path and the route path is deprecated. Change things like this: - - match #{_path.inspect}, :path => #{option_path.inspect} - -to this: - - match #{option_path.inspect}, :as => #{_path.inspect}, :action => #{_path.inspect} - eowarn - route_options[:action] = _path - route_options[:as] = _path - _path = option_path + raise ArgumentError, "Ambigous route definition. Both :path and the route path where specified as strings." end to = get_to_from_path(_path, to, route_options[:action]) decomposed_match(_path, controller, route_options, _path, to, via, formatted, anchor, options_constraints) |