Fixed default_error_formatter which takes a format symbol.

Author: vad4msiu

CHANGELOG.md

@@ -5,6 +5,10 @@ Next Release
 
 * [#475](https://github.com/intridea/grape/pull/475): Added support for the `:jsonapi`, `application/vnd.api+json` media type registered at http://jsonapi.org - [@bcm](https://github.com/bcm).
 * [#471](https://github.com/intridea/grape/issues/471): Added parameter validator for a list of allowed values - [@vickychijwani](https://github.com/vickychijwani).
+
+#### Fixes
+
+* [#477](https://github.com/intridea/grape/pull/477): Fixed `default_error_formatter` which takes a format symbol - [@vad4msiu](https://github.com/vad4msiu).
 * Your contribution here.
 
 0.6.0 (9/16/2013)

lib/grape/api.rb

@@ -147,8 +147,13 @@ def parser(content_type, new_parser)
       end
 
       # Specify a default error formatter.
-      def default_error_formatter(new_formatter = nil)
-        new_formatter ? set(:default_error_formatter, new_formatter) : settings[:default_error_formatter]
+      def default_error_formatter(new_formatter_name = nil)
+        if new_formatter_name
+          new_formatter = Grape::ErrorFormatter::Base.formatter_for(new_formatter_name, {})
+          set(:default_error_formatter, new_formatter)
+        else
+          settings[:default_error_formatter]
+        end
       end
 
       def error_formatter(format, options)

spec/grape/api_spec.rb

@@ -2191,4 +2191,17 @@ def before
       end
     end
   end
+
+  context 'with json default_error_formatter' do
+    it 'returns json error' do
+      subject.content_type :json, "application/json"
+      subject.default_error_formatter :json
+      subject.get '/something' do
+        'foo'
+      end
+      get '/something'
+      last_response.status.should == 406
+      last_response.body.should == "{\"error\":\"The requested format 'txt' is not supported.\"}"
+    end
+  end
 end