search

solidusio / solidus

🛒 Solidus, the open-source eCommerce framework for industry trailblazers.
https://solidus.io
Other
5.02k stars 1.29k forks source link

Specifying stock locations on cart page doesn't work for existing line items #1695

Closed jhawthorn closed 5 years ago

jhawthorn commented 7 years ago

Steps to reproduce

Start with a store with two stock locations

On a new order's cart page, add an item, taking quantity 1 from each stock location

Do the same again with the same item

View the shipments

Expected behavior

Two shipments, one from each location, each with quantity two, as the user specified.

Actual behavior

Three shipments are created. The quantities from each location aren't what the user specified.

https://github.com/solidusio/solidus/blob/33797b434986cf8fdecf6369467d29d6bd31979b/core/app/models/spree/order_contents.rb#L101-L103

System configuration

Solidus Version: Any Extensions in use: None

aayush2610 commented 6 years ago

Hey @jhawthorn I would like to contribute to this issue. I was not able to create a store with two stock locations, can you help with that?

jacobherrington commented 6 years ago

I don't think I can recreate this on Solidus 2.7.0

kennyadsl commented 5 years ago

This has been closed with #1709