mirumee/satchless — pull request #96 by brew (2012-01-30 17:54:17+00:00)
Title: Ensure delivery price by routing checkout through delivery-details

[Not necessarily a pull request, but want to illustrate what I'm talking about]

I'm using the multistep checkout from contrib, but it seems steps 2½ and 3½ aren't used (unless I'm missing something).

The delivery price isn't saved to the delivery group object unless the delivery-details step (2½) is used, as part of saving the delivery_queue.

In the attached code, the `delivery_method` redirects to `delivery_details`, rather than straight to `payment_method`.

Also, delivery groups are filtered for `require_shipping_address`, so delivery prices aren't added to things like e-books.