spree/spree-guides

Orders - OrderPopulator missing documentation

dustineichler opened this issue · 1 comments

dustineichler commented

Searched and didn't see a ticket for this.

Relevant link below. This ticket is a placeholder while this page hasn't been updated.

http://guides.spreecommerce.com/developer/orders.html

radar commented

Thanks @dustineichler. I investigated that today, and I think the OrderPopulator is severely more complex than it needs to be. It's only used in OrdersController#populate and that action gets its information from the _cart_form partial. That partial only passes through a variant_id and quantity.

I've got a commit on my master branch (radar/spree@9950ce8716318952817d9d34e6b0f114c64a4ff7) which does a huge simplification of that class. I will be updating the documentation on master appropriately, and removing the documentation from 2-1-x, as I think it's too confusing to be used publicly.