From 873dcc373f4fa9652c28d11e26c03e27a2611ff6 Mon Sep 17 00:00:00 2001
From: luisramos0 <luisramos0@gmail.com>
Date: Thu, 6 Feb 2020 15:06:50 +0000
Subject: [PATCH] Small refactoring to make next commit easier

---
 app/controllers/checkout_controller.rb | 41 ++++++++++++++++++--------
 1 file changed, 29 insertions(+), 12 deletions(-)

diff --git a/app/controllers/checkout_controller.rb b/app/controllers/checkout_controller.rb
index 3e00127d8a..309ff0701a 100644
--- a/app/controllers/checkout_controller.rb
+++ b/app/controllers/checkout_controller.rb
@@ -165,7 +165,7 @@ class CheckoutController < Spree::StoreController
       return update_failed
     end
 
-    update_result
+    update_response
   end
 
   def redirect_to_payment_gateway
@@ -193,27 +193,37 @@ class CheckoutController < Spree::StoreController
     end
   end
 
-  def update_result
-    if @order.state == "complete" || @order.completed?
-      save_order_addresses_as_user_default
-      ResetOrderService.new(self, current_order).call
-
-      update_succeeded
+  def update_response
+    if order_complete?
+      checkout_succeeded
+      update_succeeded_response
     else
-      update_failed
+      checkout_failed
+      update_failed_response
     end
   end
 
+  def order_complete?
+    @order.state == "complete" || @order.completed?
+  end
+
+  def checkout_succeeded
+    save_order_addresses_as_user_default
+    ResetOrderService.new(self, current_order).call
+
+    session[:access_token] = current_order.token
+    flash[:notice] = t(:order_processed_successfully)
+  end
+
   def save_order_addresses_as_user_default
+    return unless params[:order]
+
     user_default_address_setter = UserDefaultAddressSetter.new(@order, spree_current_user)
     user_default_address_setter.set_default_bill_address if params[:order][:default_bill_address]
     user_default_address_setter.set_default_ship_address if params[:order][:default_ship_address]
   end
 
-  def update_succeeded
-    session[:access_token] = current_order.token
-    flash[:notice] = t(:order_processed_successfully)
-
+  def update_succeeded_response
     respond_to do |format|
       format.html do
         respond_with(@order, location: order_path(@order))
@@ -225,9 +235,16 @@ class CheckoutController < Spree::StoreController
   end
 
   def update_failed
+    checkout_failed
+    update_failed_response
+  end
+
+  def checkout_failed
     current_order.updater.shipping_address_from_distributor
     RestartCheckout.new(@order).call
+  end
 
+  def update_failed_response
     respond_to do |format|
       format.html do
         render :edit