Mutating shipments must not mutate line items

Open tvdeyen opened this issue 8 years ago • 3 comments

Steps to reproduce

Create an order. Split the shipment. Remove one of the shipments.

Expected behavior

The line item quantity must not change.

Actual behavior

The line item quantity changes.

System configuration

All Solidus versions affected

Sep 22 '17 09:09 tvdeyen

Was able to reproduce the Bug on the https://demo.solidus.io/.

You need to create an Order with 2+ line items so we can Split.

I made a recording to showcase it:

Untitled_ Nov 24, 2022 9_58 AM.webm

Nov 24 '22 12:11 cesartalves

I think a quick workaround would be an alert that tells people that they are going to delete the content of the shipment as well. That would be enough to allow people to move the item again before removing the shipment and its content.

Nov 24 '22 13:11 kennyadsl