Merge pull request solidusio#590 from bonobos/determine_order_user_hook

refactor #determine_order_user helper method

api/app/controllers/spree/api/orders_controller.rb

@@ -27,14 +27,7 @@ def cancel
 
  def create
  authorize! :create, Order
-
- order_user = if order_params[:user_id]
- Spree.user_class.find(order_params[:user_id])
- else
- current_api_user
- end
-
- @order = Spree::Core::Importer::Order.import(order_user, order_params)
+ @order = Spree::Core::Importer::Order.import(determine_order_user, order_params)
  respond_with(@order, default_template: :show, status: 201)
  end
 
@@ -116,6 +109,15 @@ def normalize_params
  params[:order][:bill_address_attributes] = params[:order].delete(:bill_address) if params[:order][:bill_address].present?
  end
 
+ # @api public
+ def determine_order_user
+ if order_params[:user_id].present?
+ Spree.user_class.find(order_params[:user_id])
+ else
+ current_api_user
+ end
+ end
+
  def permitted_order_attributes
  can?(:admin, Spree::Order) ? (super + admin_order_attributes) : super
  end