Revert "Merge pull request rails#52605 from Shopify/scope_with_keywords"

This reverts commit 9f9deaf, reversing
changes made to 36f5a20.

Author: jhawthorn

actionpack/lib/action_dispatch/routing/mapper.rb

@@ -673,7 +673,7 @@ def default_url_options=(options)
         alias_method :default_url_options, :default_url_options=
 
         def with_default_scope(scope, &block)
-          scope(**scope) do
+          scope(scope) do
             instance_exec(&block)
           end
         end
@@ -883,7 +883,8 @@ module Scoping
         #     scope as: "sekret" do
         #       resources :posts
         #     end
-        def scope(*args, only: nil, except: nil, **options)
+        def scope(*args)
+          options = args.extract_options!.dup
           scope = {}
 
           options[:path] = args.flatten.join("/") if args.any?
@@ -904,8 +905,9 @@ def scope(*args, only: nil, except: nil, **options)
             block, options[:constraints] = options[:constraints], {}
           end
 
-          if only || except
-            scope[:action_options] = { only:, except: }
+          if options.key?(:only) || options.key?(:except)
+            scope[:action_options] = { only: options.delete(:only),
+                                       except: options.delete(:except) }
           end
 
           if options.key? :anchor
@@ -985,16 +987,18 @@ def controller(controller)
         #     namespace :admin, as: "sekret" do
         #       resources :posts
         #     end
-        def namespace(name, as: DEFAULT, path: DEFAULT, shallow_path: DEFAULT, shallow_prefix: DEFAULT, **options, &block)
-          name = name.to_s
-          options[:module] ||= name
-          as = name if as == DEFAULT
-          path = name if path == DEFAULT
-          shallow_path = path if shallow_path == DEFAULT
-          shallow_prefix = as if shallow_prefix == DEFAULT
+        def namespace(path, options = {}, &block)
+          path = path.to_s
+
+          defaults = {
+            module:         path,
+            as:             options.fetch(:as, path),
+            shallow_path:   options.fetch(:path, path),
+            shallow_prefix: options.fetch(:as, path)
+          }
 
-          path_scope(path) do
-            scope(**options, as:, shallow_path:, shallow_prefix:, &block)
+          path_scope(options.delete(:path) { path }) do
+            scope(defaults.merge!(options), &block)
           end
         end
 
@@ -1188,7 +1192,7 @@ module Resources
         class Resource # :nodoc:
           attr_reader :controller, :path, :param
 
-          def initialize(entities, api_only, shallow, only: nil, except: nil, **options)
+          def initialize(entities, api_only, shallow, options = {})
             if options[:param].to_s.include?(":")
               raise ArgumentError, ":param option can't contain colons"
             end
@@ -1201,8 +1205,8 @@ def initialize(entities, api_only, shallow, only: nil, except: nil, **options)
             @options    = options
             @shallow    = shallow
             @api_only   = api_only
-            @only       = only
-            @except     = except
+            @only       = options.delete :only
+            @except     = options.delete :except
           end
 
           def default_actions
@@ -1281,7 +1285,7 @@ def singleton?; false; end
         end
 
         class SingletonResource < Resource # :nodoc:
-          def initialize(entities, api_only, shallow, **options)
+          def initialize(entities, api_only, shallow, options)
             super
             @as         = nil
             @controller = (options[:controller] || plural).to_s
@@ -1346,17 +1350,19 @@ def resources_path_names(options)
         #
         # ### Options
         # Takes same options as [resources](rdoc-ref:#resources)
-        def resource(*resources, concerns: nil, **options, &block)
-          if apply_common_behavior_for(:resource, resources, concerns:, **options, &block)
+        def resource(*resources, &block)
+          options = resources.extract_options!.dup
+
+          if apply_common_behavior_for(:resource, resources, options, &block)
             return self
           end
 
           with_scope_level(:resource) do
             options = apply_action_options options
-            resource_scope(SingletonResource.new(resources.pop, api_only?, @scope[:shallow], **options)) do
+            resource_scope(SingletonResource.new(resources.pop, api_only?, @scope[:shallow], options)) do
               yield if block_given?
 
-              concerns(*concerns) if concerns
+              concerns(options[:concerns]) if options[:concerns]
 
               new do
                 get :new
@@ -1514,17 +1520,19 @@ def resource(*resources, concerns: nil, **options, &block)
         #
         #     # resource actions are at /admin/posts.
         #     resources :posts, path: "admin/posts"
-        def resources(*resources, concerns: nil, **options, &block)
-          if apply_common_behavior_for(:resources, resources, concerns:, **options, &block)
+        def resources(*resources, &block)
+          options = resources.extract_options!.dup
+
+          if apply_common_behavior_for(:resources, resources, options, &block)
             return self
           end
 
           with_scope_level(:resources) do
             options = apply_action_options options
-            resource_scope(Resource.new(resources.pop, api_only?, @scope[:shallow], **options)) do
+            resource_scope(Resource.new(resources.pop, api_only?, @scope[:shallow], options)) do
               yield if block_given?
 
-              concerns(*concerns) if concerns
+              concerns(options[:concerns]) if options[:concerns]
 
               collection do
                 get  :index if parent_resource.actions.include?(:index)
@@ -1609,19 +1617,19 @@ def nested(&block)
             if shallow? && shallow_nesting_depth >= 1
               shallow_scope do
                 path_scope(parent_resource.nested_scope) do
-                  scope(**nested_options, &block)
+                  scope(nested_options, &block)
                 end
               end
             else
               path_scope(parent_resource.nested_scope) do
-                scope(**nested_options, &block)
+                scope(nested_options, &block)
               end
             end
           end
         end
 
         # See ActionDispatch::Routing::Mapper::Scoping#namespace.
-        def namespace(name, as: DEFAULT, path: DEFAULT, shallow_path: DEFAULT, shallow_prefix: DEFAULT, **options, &block)
+        def namespace(path, options = {})
           if resource_scope?
             nested { super }
           else
@@ -1776,21 +1784,22 @@ def parent_resource
             @scope[:scope_level_resource]
           end
 
-          def apply_common_behavior_for(method, resources, shallow: nil, **options, &block)
+          def apply_common_behavior_for(method, resources, options, &block)
             if resources.length > 1
-              resources.each { |r| public_send(method, r, shallow:, **options, &block) }
+              resources.each { |r| public_send(method, r, options, &block) }
               return true
             end
 
-            if shallow
-              self.shallow do
-                public_send(method, resources.pop, **options, &block)
+            if options[:shallow]
+              options.delete(:shallow)
+              shallow do
+                public_send(method, resources.pop, options, &block)
               end
               return true
             end
 
             if resource_scope?
-              nested { public_send(method, resources.pop, shallow:, **options, &block) }
+              nested { public_send(method, resources.pop, options, &block) }
               return true
             end
 
@@ -1799,9 +1808,9 @@ def apply_common_behavior_for(method, resources, shallow: nil, **options, &block
             end
 
             scope_options = options.slice!(*RESOURCE_OPTIONS)
-            if !scope_options.empty? || !shallow.nil?
-              scope(**scope_options, shallow:) do
-                public_send(method, resources.pop, **options, &block)
+            unless scope_options.empty?
+              scope(scope_options) do
+                public_send(method, resources.pop, options, &block)
               end
               return true
             end
@@ -1877,10 +1886,9 @@ def canonical_action?(action)
           end
 
           def shallow_scope
-            @scope = @scope.new(
-              as: @scope[:shallow_prefix],
-              path: @scope[:shallow_path],
-            )
+            scope = { as: @scope[:shallow_prefix],
+                      path: @scope[:shallow_path] }
+            @scope = @scope.new scope
 
             yield
           ensure
@@ -2144,7 +2152,8 @@ def concern(name, callable = nil, &block)
         #     namespace :posts do
         #       concerns :commentable
         #     end
-        def concerns(*args, **options)
+        def concerns(*args)
+          options = args.extract_options!
           args.flatten.each do |name|
             if concern = @concerns[name]
               concern.call(self, options)

actionpack/test/controller/resources_test.rb

@@ -76,7 +76,7 @@ def test_multiple_default_restful_routes
   def test_multiple_resources_with_options
     expected_options = { controller: "threads", action: "index" }
 
-    with_restful_routing :messages, :comments, controller: "threads" do
+    with_restful_routing :messages, :comments, expected_options.slice(:controller) do
       assert_recognizes(expected_options, path: "comments")
       assert_recognizes(expected_options, path: "messages")
     end
@@ -1109,15 +1109,17 @@ def test_singleton_resource_name_is_not_singularized
   end
 
   private
-    def with_restful_routing(*args, **options)
+    def with_restful_routing(*args)
+      options = args.extract_options!
       collection_methods = options.delete(:collection)
       member_methods = options.delete(:member)
       path_prefix = options.delete(:path_prefix)
+      args.push(options)
 
       with_routing do |set|
         set.draw do
           scope(path_prefix || "") do
-            resources(*args, **options) do
+            resources(*args) do
               if collection_methods
                 collection do
                   collection_methods.each do |name, method|

actionpack/test/dispatch/routing/concerns_test.rb

@@ -7,14 +7,14 @@ class ReviewsController < ResourcesController; end
 class RoutingConcernsTest < ActionDispatch::IntegrationTest
   class Reviewable
     def self.call(mapper, options = {})
-      mapper.resources :reviews, **options
+      mapper.resources :reviews, options
     end
   end
 
   Routes = ActionDispatch::Routing::RouteSet.new.tap do |app|
     app.draw do
       concern :commentable do |options|
-        resources :comments, **options
+        resources :comments, options
       end
 
       concern :image_attachable do

actionpack/test/dispatch/routing_test.rb

@@ -990,13 +990,13 @@ def test_resources_for_uncountable_names
 
   def test_resource_does_not_modify_passed_options
     options = { id: /.+?/, format: /json|xml/ }
-    draw { resource :user, **options }
+    draw { resource :user, options }
     assert_equal({ id: /.+?/, format: /json|xml/ }, options)
   end
 
   def test_resources_does_not_modify_passed_options
     options = { id: /.+?/, format: /json|xml/ }
-    draw { resources :users, **options }
+    draw { resources :users, options }
     assert_equal({ id: /.+?/, format: /json|xml/ }, options)
   end