LineItemId no longer unique per cart?

Stian

Vote:

We recently updated Commerce from 13.8.0 to 13.25.0

Has there been a change that LineItemId is no longer unique per cart? I see that if I add line items they get get id, -1, -2, -3 etc but if I remove the first line item with id -1 and add another item that item gets -2 as the id. This results in 2 line items having the same id.

I also tried this in the latest Quicksilver master, 13.26.0 and I see the same thing here. We have some logic on the site that expects the LineItemId to be unique per cart. Do we have to change this or is this a bug?

#231113

Edited, Nov 19, 2020 9:49

Quan Mai

Vote:

I have asked COmmerce team to look into this. Probably a bug. thanks for bringing this into our attention.

#231124

Nov 19, 2020 11:52

Stian - Nov 20, 2020 8:31

Thank you. Do you have any updates on this?

Quan Mai - Nov 20, 2020 8:59

Commerce team will look into this. I have no further update currently unfortunately

Simon

Vote:

Running 13.25.0

I'm experiencing the same issue.

I found out while debugging a custom promotion that was acting strange: the wrong item gets removed from the shipment in PromotionExcludedCatalogItemHandler.cs, when a promotion with a product exclusion is evaluated.

My analysis: The code uses the LineItemId from the exclusion map to find and remove LineItems in the shipment, but since the IDs are duplicate like Stian explained, the wrong LineItem is removed. The item in the exclusion map (which was supposed to be removed from the shipment) is then re-added to the shipment after the evaluation in RestoreItems() in PromotionExcludedCatalogItemHandler.cs resulting in a cart that misses an item and instead has another item duplicated.

#231529

Nov 30, 2020 10:40

Quan Mai

Vote:

This will be fixed in Commerce 13.27, releasing in near future

#231551

Dec 01, 2020 13:36