bonobos/spree — pull request #102 by athal7 (2014-09-12 17:32:06+00:00)
Title: Implement cart and shipment steps in order admin

Fix line item proper currency check

This resolves #5132. The admin manual order cart is much more like the frontend now in that you start off by adding items to the cart(line items). You then add an address, the shipments are calculated based on the frontend rules you have set up but you as an admin can ignore those rules and muck with the shipments on the Shipments tab. You can also skip adding items to the cart altogether by just entering an address and then adding shipments manually, the way the old backend admin did before this patch.

Removing unnecessary code for getting the total_on_hand for a variant through the API

Extracting the advanceOrder function into a function in the Spree namespace so that both the shipmentsJS and line_itemsJS can share it

Advancing the order after we add variants on the cart page. If we have already created shipments, then when we edit a line item the order gets reset to the address state. This call pushes it back to the payment state and destroys the old shipments, recreating them to match the new line item state.

In checkout.rb we first create_proposed_shipments but here we check if addresses are valid. If not we need to destroy_all instead of delete_all so that the inventory_units that were created by create_propsed_shipments will also be destroyed and not clutter up the DB

Fixing the Split tool. IF success we reload the page, else display a generic error. Would like to in the future provide more specific errors. Not allowing splits to the same shipment nor to a shipment or location that does not have the stock and is NOT backorderable. Next commit will have a boatload of automated tests.

Updating my admin changes in master to be compat with spree-2-3

Conflicts:
api/app/controllers/spree/api/line_items_controller.rb
api/app/controllers/spree/api/shipments_controller.rb
api/app/controllers/spree/api/variants_controller.rb
api/config/routes.rb
backend/app/assets/javascripts/spree/backend/variant_autocomplete.js.coffee.erb
backend/app/controllers/spree/admin/orders_controller.rb
backend/app/views/spree/admin/orders/edit.html.erb
backend/spec/controllers/spree/admin/orders_controller_spec.rb
backend/spec/features/admin/orders/new_order_spec.rb
backend/spec/features/admin/orders/order_details_spec.rb
core/app/models/spree/line_item.rb
core/app/models/spree/order.rb
core/app/models/spree/shipment.rb

Change-Id: I2f15c34e7b5b79f1a2dc015c05bccf3fd1b46df6

Cleanup of #5166

Don't rescue exceptions

order shipments in admin by created_at

Raise an exception if an item can't be added

Previously this could fail silently without adding an item to the order.

Remove unnecessary stock check

If stock is unavailable an error will be triggered anyways rolling back
the transaction.

api/shipments: simplify load_transfer_params

Use transfer_to_shipment from transfer_to_location

Conflicts:
core/app/models/spree/shipment.rb

Change-Id: I8591ac0415bbf2dc4464a2ca377dccc96b470da4

remove duplicate, non-js variant autocomplete

the coffee one has all the templates we want