Remove duplicated responder configs and include from api

ActionController::Rendering already includes AbstractController::Rendering and we had duplicated respond_to calls in basecontroller and controller_setup
bsylvain · Oct 19, 2013 · ae23991 · ae23991
1 parent 546dd18
commit ae23991
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 5 deletions.
diff --git a/api/app/controllers/spree/api/base_controller.rb b/api/app/controllers/spree/api/base_controller.rb
@@ -10,10 +10,6 @@ class BaseController < ActionController::Metal
       include ::ActionController::Head
       include ::ActionController::ConditionalGet
 
-      self.responder = Spree::Api::Responders::AppResponder
-
-      respond_to :json
-
       attr_accessor :current_api_user
 
       before_filter :set_content_type

diff --git a/api/lib/spree/api/controller_setup.rb b/api/lib/spree/api/controller_setup.rb
@@ -5,7 +5,6 @@ module Api
     module ControllerSetup
       def self.included(klass)
         klass.class_eval do
-          include AbstractController::Rendering
           include AbstractController::ViewPaths
           include AbstractController::Callbacks
           include AbstractController::Helpers