app/controllers/comable/orders_controller.rb in comable_frontend-0.2.3 vs app/controllers/comable/orders_controller.rb in comable_frontend-0.3.0

@@ -1,126 +1,86 @@
 module Comable
   class OrdersController < Comable::ApplicationController
+    # TODO: Change the method name to load_order_with_params
+    before_filter :load_order
+    before_filter :ensure_cart_not_empty
+    before_filter :ensure_saleable_stocks
+
+    prepend Comable::SigninAction
     prepend Comable::ShipmentAction
     prepend Comable::PaymentAction
     include Comable::PermittedAttributes
 
-    helper_method :next_order_path
-
-    # TODO: Change the method name to load_order_with_params
-    before_filter :load_order
-    before_filter :verify
-    # TODO: Remove
-    after_filter :save_order, except: :create
-
-    rescue_from Comable::InvalidOrder, with: :order_invalid
-
-    def new
-      redirect_to next_order_path unless agreement_required?
-    end
-
-    def orderer
-      case request.method_symbol
-      when :post
-        redirect_to next_order_path if @order.save
+    def edit
+      if @order.state?(params[:state]) || @order.stated?(params[:state])
+        render params[:state]
+      else
+        redirect_to next_order_path
       end
     end
 
-    def delivery
-      case request.method_symbol
-      when :post
-        redirect_to next_order_path if @order.save
+    def update
+      if @order.stated?(params[:state]) ? @order.save : @order.next_state
+        redirect_to next_order_path
+      else
+        render @order.state
       end
     end
 
     def create
-      order = current_customer.order
-      if order.complete?
-        flash[:notice] = I18n.t('comable.orders.success')
+      if @order.state?(:confirm) && @order.next_state
+        flash.now[:notice] = Comable.t('orders.success')
         send_order_complete_mail
       else
-        flash[:alert] = I18n.t('comable.orders.failure')
-        redirect_to comable.confirm_order_path
+        flash[:alert] = Comable.t('orders.failure')
+        redirect_to next_order_path
       end
     end
 
     private
 
     def send_order_complete_mail
       Comable::OrderMailer.complete(@order).deliver if current_store.email_activate?
     end
 
-    # TODO: Switch to state_machine
-    # rubocop:disable all
-    def next_order_path(target_action_name = nil)
-      case (target_action_name || action_name).to_sym
-      when :new
-        orderer_required? ? comable.orderer_order_path : next_order_path(:orderer)
-      when :orderer
-        delivery_required? ? comable.delivery_order_path : next_order_path(:delivery)
-      when :delivery
-        shipment_required? ? comable.shipment_order_path : next_order_path(:shipment)
-      when :shipment
-        payment_required? ? comable.payment_order_path : next_order_path(:payment)
-      else
-        comable.confirm_order_path
-      end
+    def ensure_cart_not_empty
+      return if current_customer.cart.any?
+      flash[:alert] = Comable.t('carts.empty')
+      redirect_to comable.cart_path
     end
-    # rubocop:enable all
 
-    def agreement_required?
-      @order.customer.nil?
-    end
-
-    def orderer_required?
-      @order.bill_address.nil?
-    end
-
-    def delivery_required?
-      @order.ship_address.nil?
-    end
-
-    def verify
-      return if current_customer.cart.any?
-      flash[:alert] = I18n.t('comable.carts.empty')
+    def ensure_saleable_stocks
+      return if current_order.soldout_stocks.empty?
+      flash[:alert] = Comable.t('errors.messages.products_soldout')
       redirect_to comable.cart_path
     end
 
     def load_order
-      @order = current_customer.preorder(order_params || {})
+      @order = current_order
+      @order.attributes = order_params if order_params
     end
 
-    def save_order
-      @order.save
-    end
-
     def order_params
       return unless params[:order]
-      case action_name.to_sym
+      return unless params[:state]
+      case params[:state].to_sym
       when :orderer
         order_params_for_orderer
       when :delivery
         order_params_for_delivery
       end
     end
 
     def order_params_for_orderer
       params.require(:order).permit(
-        :family_name, # TODO: Remove
-        :first_name,  # TODO: Remove
         :email,
         bill_address_attributes: permitted_address_attributes
       )
     end
 
     def order_params_for_delivery
       params.require(:order).permit(
         ship_address_attributes: permitted_address_attributes
       )
-    end
-
-    def order_invalid
-      flash[:alert] = I18n.t('comable.orders.failure')
-      redirect_to comable.cart_path
     end
   end
 end