Extended ParamBulder to be included in an entire API or namespace.

README.md

@@ -501,9 +501,19 @@ Route string parameters will have precedence.
 
 By default parameters are available as `ActiveSupport::HashWithIndifferentAccess`. This can be changed to, for example, Ruby `Hash` or `Hashie::Mash` for the entire API.
 
-[TODO]
+```ruby
+class API < Grape::API
+  include Grape::Extensions::Hashie::Mash::ParamBuilder
+
+  params do
+    optional :color, type: String
+  end
+  get do
+    params.color # instead of params[:color]
+  end
+```
 
-The class can be overridden on individual parameter blocks using `build_with` as follows.
+The class can also be overridden on individual parameter blocks using `build_with` as follows.
 
 ```ruby
 params do

UPGRADING.md

@@ -5,9 +5,33 @@ Upgrading Grape
 
 #### Changes in Parameter Class
 
-The default class for `params` has changed from `Hashie::Mash` to `ActiveSupport::HashWithIndifferentAccess` and the `hashie` dependency has been removed. To restore the behavior of prior versions, add `hashie` to your `Gemfile` and use `TODO`.
+The default class for `params` has changed from `Hashie::Mash` to `ActiveSupport::HashWithIndifferentAccess` and the `hashie` dependency has been removed. This means that by default you can no longer access parameters by method name.
 
-[TODO]
+```
+class API < Grape::API
+  params do
+    optional :color, type: String
+  end
+  get do
+    params[:color] # use params[:color] instead of params.color
+  end
+end
+```
+
+To restore the behavior of prior versions, add `hashie` to your `Gemfile` and `include Grape::Extensions::Hashie::Mash::ParamBuilder` in your API.
+
+```
+class API < Grape::API
+  include Grape::Extensions::Hashie::Mash::ParamBuilder
+
+  params do
+    optional :color, type: String
+  end
+  get do
+    # params.color works
+  end
+end
+```
 
 This behavior can also be overridden on individual parameter blocks using `build_with`.
 

lib/grape.rb

@@ -26,7 +26,7 @@
 I18n.load_path << File.expand_path('../grape/locale/en.yml', __FILE__)
 
 module Grape
-  extend ActiveSupport::Autoload
+  extend ::ActiveSupport::Autoload
 
   eager_autoload do
     autoload :API
@@ -47,14 +47,14 @@ module Grape
   end
 
   module Http
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     eager_autoload do
       autoload :Headers
     end
   end
 
   module Exceptions
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     autoload :Base
     autoload :Validation
     autoload :ValidationArrayErrors
@@ -78,22 +78,27 @@ module Exceptions
   end
 
   module Extensions
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
 
     autoload :DeepMergeableHash
     autoload :DeepSymbolizeHash
     autoload :Hash
-    autoload :HashWithIndifferentAccess
+
+    module ActiveSupport
+      extend ::ActiveSupport::Autoload
+
+      autoload :HashWithIndifferentAccess
+    end
 
     module Hashie
-      extend ActiveSupport::Autoload
+      extend ::ActiveSupport::Autoload
 
       autoload :Mash
     end
   end
 
   module Middleware
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     autoload :Base
     autoload :Versioner
     autoload :Formatter
@@ -102,15 +107,15 @@ module Middleware
     autoload :Stack
 
     module Auth
-      extend ActiveSupport::Autoload
+      extend ::ActiveSupport::Autoload
       autoload :Base
       autoload :DSL
       autoload :StrategyInfo
       autoload :Strategies
     end
 
     module Versioner
-      extend ActiveSupport::Autoload
+      extend ::ActiveSupport::Autoload
       autoload :Path
       autoload :Header
       autoload :Param
@@ -119,7 +124,7 @@ module Versioner
   end
 
   module Util
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     autoload :InheritableValues
     autoload :StackableValues
     autoload :ReverseStackableValues
@@ -129,29 +134,29 @@ module Util
   end
 
   module ErrorFormatter
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     autoload :Base
     autoload :Json
     autoload :Txt
     autoload :Xml
   end
 
   module Formatter
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     autoload :Json
     autoload :SerializableHash
     autoload :Txt
     autoload :Xml
   end
 
   module Parser
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     autoload :Json
     autoload :Xml
   end
 
   module DSL
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     eager_autoload do
       autoload :API
       autoload :Callbacks
@@ -170,17 +175,17 @@ module DSL
   end
 
   class API
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     autoload :Helpers
   end
 
   module Presenters
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     autoload :Presenter
   end
 
   module ServeFile
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
     autoload :FileResponse
     autoload :FileBody
     autoload :SendfileResponse

lib/grape/dsl/parameters.rb

@@ -12,7 +12,7 @@ module Parameters
       #
       # @param build_with the ParamBuilder module to use when building request.params
       #   Available builders are;
-      #     * Grape::Extensions::HashWithIndifferentAccess::ParamBuilder (default)
+      #     * Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder (default)
       #     * Grape::Extensions::Hash::ParamBuilder
       #     * Grape::Extensions::Hashie::Mash::ParamBuilder
       #
@@ -30,7 +30,7 @@ module Parameters
       #       end
       #     end
       def build_with(build_with = nil)
-        @api.namespace_inheritable(:build_with, build_with)
+        @api.namespace_inheritable(:build_params_with, build_with)
       end
 
       # Include reusable params rules among current.

lib/grape/endpoint.rb

@@ -237,7 +237,7 @@ def equals?(e)
     def run
       ActiveSupport::Notifications.instrument('endpoint_run.grape', endpoint: self, env: env) do
         @header = {}
-        @request = Grape::Request.new(env, build_params_with: namespace_inheritable(:build_with))
+        @request = Grape::Request.new(env, build_params_with: namespace_inheritable(:build_params_with))
         @params = @request.params
         @headers = @request.headers
 

lib/grape/extensions/active_support/hash_with_indifferent_access.rb

@@ -0,0 +1,25 @@
+module Grape
+  module Extensions
+    module ActiveSupport
+      module HashWithIndifferentAccess
+        extend ::ActiveSupport::Concern
+
+        included do
+          namespace_inheritable(:build_params_with, Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder)
+        end
+
+        def params_builder
+          Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder
+        end
+
+        module ParamBuilder
+          def build_params
+            params = ::ActiveSupport::HashWithIndifferentAccess[rack_params]
+            params.deep_merge!(grape_routing_args) if env[Grape::Env::GRAPE_ROUTING_ARGS]
+            params
+          end
+        end
+      end
+    end
+  end
+end

lib/grape/extensions/hash.rb

@@ -2,6 +2,12 @@ module Grape
   module Extensions
     module Hash
       module ParamBuilder
+        extend ::ActiveSupport::Concern
+
+        included do
+          namespace_inheritable(:build_params_with, Grape::Extensions::Hash::ParamBuilder)
+        end
+
         def build_params
           params = Grape::Extensions::DeepMergeableHash[rack_params]
           params.deep_merge!(grape_routing_args) if env[Grape::Env::GRAPE_ROUTING_ARGS]

lib/grape/extensions/hashie/mash.rb

@@ -2,11 +2,17 @@ module Grape
   module Extensions
     module Hashie
       module Mash
-        def params_builder
-          Grape::Extensions::Hashie::Mash::ParamBuilder
-        end
-
         module ParamBuilder
+          extend ::ActiveSupport::Concern
+
+          included do
+            namespace_inheritable(:build_params_with, Grape::Extensions::Hashie::Mash::ParamBuilder)
+          end
+
+          def params_builder
+            Grape::Extensions::Hashie::Mash::ParamBuilder
+          end
+
           def build_params
             params = ::Hashie::Mash.new(rack_params)
             params.deep_merge!(grape_routing_args) if env[Grape::Env::GRAPE_ROUTING_ARGS]

lib/grape/request.rb

@@ -5,7 +5,7 @@ class Request < Rack::Request
     alias rack_params params
 
     def initialize(env, options = {})
-      extend options[:build_params_with] || Grape::Extensions::HashWithIndifferentAccess::ParamBuilder
+      extend options[:build_params_with] || Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder
       super(env)
     end
 

spec/grape/endpoint_spec.rb

@@ -269,7 +269,7 @@ def app
     context 'build params with HashWithIndifferentAccess' do
       it 'should be ActiveSupport::HashWithIndifferentAccess' do
         subject.params do
-          build_with Grape::Extensions::HashWithIndifferentAccess::ParamBuilder
+          build_with Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder
         end
 
         subject.get '/foo' do
@@ -283,7 +283,7 @@ def app
 
       it 'parse sub hash params' do
         subject.params do
-          build_with Grape::Extensions::HashWithIndifferentAccess::ParamBuilder
+          build_with Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder
 
           optional :a, type: Hash do
             optional :b, type: Hash do
@@ -1063,7 +1063,7 @@ def app
     context 'params presented as a HashWithIndifferentAccess' do
       it 'params are indifferent to symbol or string keys' do
         subject.params do
-          build_with Grape::Extensions::HashWithIndifferentAccess::ParamBuilder
+          build_with Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder
           optional :a, type: Hash do
             optional :b, type: Hash do
               optional :c, type: String
@@ -1083,7 +1083,7 @@ def app
 
       it 'responds to string keys' do
         subject.params do
-          build_with Grape::Extensions::HashWithIndifferentAccess::ParamBuilder
+          build_with Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder
           requires :a, type: String
         end
 

spec/grape/extensions/param_builders/has_with_indifferent_access_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder do
+  subject { Class.new(Grape::API) }
+
+  def app
+    subject
+  end
+
+  describe 'in an endpoint' do
+    context '#params' do
+      before do
+        subject.params do
+          build_with Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder
+        end
+
+        subject.get do
+          params.class
+        end
+      end
+
+      it 'should be of type Hash' do
+        get '/'
+        expect(last_response.status).to eq(200)
+        expect(last_response.body).to eq('ActiveSupport::HashWithIndifferentAccess')
+      end
+    end
+  end
+
+  describe 'in an api' do
+    before do
+      subject.send(:include, Grape::Extensions::ActiveSupport::HashWithIndifferentAccess::ParamBuilder)
+    end
+
+    context '#params' do
+      before do
+        subject.get do
+          params.class
+        end
+      end
+
+      it 'should be Hash' do
+        get '/'
+        expect(last_response.status).to eq(200)
+        expect(last_response.body).to eq('ActiveSupport::HashWithIndifferentAccess')
+      end
+    end
+  end
+end