From a53dc3a8c19213c9f8b0ae6a195e8014b7176ee5 Mon Sep 17 00:00:00 2001 From: Luis Ramos Date: Tue, 10 Mar 2020 14:46:16 +0000 Subject: [PATCH 1/2] Remove usage of the responder as this is a json only controller --- app/controllers/admin/proxy_orders_controller.rb | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/app/controllers/admin/proxy_orders_controller.rb b/app/controllers/admin/proxy_orders_controller.rb index b6d0140845..296bf83e67 100644 --- a/app/controllers/admin/proxy_orders_controller.rb +++ b/app/controllers/admin/proxy_orders_controller.rb @@ -9,9 +9,7 @@ module Admin def cancel if @proxy_order.cancel - respond_with(@proxy_order) do |format| - format.json { render_as_json @proxy_order } - end + render_as_json @proxy_order else respond_with(@proxy_order) do |format| format.json { render json: { errors: [t('admin.proxy_orders.cancel.could_not_cancel_the_order')] }, status: :unprocessable_entity } @@ -21,13 +19,9 @@ module Admin def resume if @proxy_order.resume - respond_with(@proxy_order) do |format| - format.json { render_as_json @proxy_order } - end + render_as_json @proxy_order else - respond_with(@proxy_order) do |format| - format.json { render json: { errors: [t('admin.proxy_orders.resume.could_not_resume_the_order')] }, status: :unprocessable_entity } - end + render json: { errors: [t('admin.proxy_orders.resume.could_not_resume_the_order')] }, status: :unprocessable_entity end end end From bc0a1d9baee1dcec02da915d0a97eaa4dea2e6d4 Mon Sep 17 00:00:00 2001 From: Luis Ramos Date: Tue, 10 Mar 2020 15:56:08 +0000 Subject: [PATCH 2/2] Remove one more responder and fix rubocop issues --- .rubocop_manual_todo.yml | 1 - app/controllers/admin/proxy_orders_controller.rb | 8 ++++---- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/.rubocop_manual_todo.yml b/.rubocop_manual_todo.yml index 77b598ad3c..08ed408d85 100644 --- a/.rubocop_manual_todo.yml +++ b/.rubocop_manual_todo.yml @@ -33,7 +33,6 @@ Layout/LineLength: - app/controllers/admin/inventory_items_controller.rb - app/controllers/admin/manager_invitations_controller.rb - app/controllers/admin/product_import_controller.rb - - app/controllers/admin/proxy_orders_controller.rb - app/controllers/admin/schedules_controller.rb - app/controllers/admin/subscriptions_controller.rb - app/controllers/admin/variant_overrides_controller.rb diff --git a/app/controllers/admin/proxy_orders_controller.rb b/app/controllers/admin/proxy_orders_controller.rb index 296bf83e67..8e92f307f4 100644 --- a/app/controllers/admin/proxy_orders_controller.rb +++ b/app/controllers/admin/proxy_orders_controller.rb @@ -11,9 +11,8 @@ module Admin if @proxy_order.cancel render_as_json @proxy_order else - respond_with(@proxy_order) do |format| - format.json { render json: { errors: [t('admin.proxy_orders.cancel.could_not_cancel_the_order')] }, status: :unprocessable_entity } - end + render json: { errors: [t('admin.proxy_orders.cancel.could_not_cancel_the_order')] }, + status: :unprocessable_entity end end @@ -21,7 +20,8 @@ module Admin if @proxy_order.resume render_as_json @proxy_order else - render json: { errors: [t('admin.proxy_orders.resume.could_not_resume_the_order')] }, status: :unprocessable_entity + render json: { errors: [t('admin.proxy_orders.resume.could_not_resume_the_order')] }, + status: :unprocessable_entity end end end