Impossible to update product sold by weight with product import

[RachL]

Description

We can't update product sold by weight anymore. when using volume (L) or items, no issues.

This means users are creating lots of duplicate products...

Steps to Reproduce

1. Log in as enterprise manager of "Release farm" on staging FR
2. Use this file
   product_list_template.csv
3. Try to change price or stock
4. Observe a new product will always be created instead of updating the current one

Animated Gif/Screenshot

Changing price and stock

Changing only price / same result

Workaround

update manually

Severity

bug-s2: a non-critical feature is broken, no workaround

Your Environment

• Version used: v5
• Browser name and version: firefox and chrome

[chahmedejaz]

Hi @RachL - I did a spike on this one, and it seems like it's the existing functionality that units column should have the unscaled unit value of the product variant being updated. For example:

• You are trying to update the Pizza by kg product with units equals 1 and unit type equals kg
• We are expecting the unscaled unit in place of units i.e. 1000

Please try using units equals 1000 and it should work. Thanks.

P.S. I think it's not because of product-refactor but it was the case even before as well.

[NiftyGaloot]

I've had a report here in the US of the same issue. Testing has shown that it happens to products that have the unit type set to weight (lb, oz) and not those that are set to item.

[nicolasVr]

Hi @chahmedejaz ! You're right, by doing as you say, I can update a product, but :

• I'm pretty sure it was not like that before
• In the @RachL example the updated product results with a weight of 1000 kg/ 1T and can't be updated itself

[RachL]

@chahmedejaz I'm not sure I follow the logic of the needed change. My pizza weight 1 kg, why do we want to update it to 1T ? 😂

Note I've created the product with import and when creating a product, 1 KG is a value that is allowed. all I've changed is the stock and price info.

so we should be able to reuse the same file when updating (and I agree with @nicolasVr - that was definitely working before).

[chahmedejaz]

Yes, @RachL @nicolasVr, this is weird 😅.
I'll work on its fix now, thanks 👍

[nicolasVr]

Hi @chahmedejaz ! Did you have success in working with this issue ?

[chahmedejaz]

Yes, @nicolasVr - It's been fixed. Just working on its specs. Will be raising the PR soon. Thanks

[nicolasVr]

Tks ! Hope that it will be deployed next wek.

[chahmedejaz]

Sorry @nicolasVr - I think it will not be part of this release. I hope it gets deployed by the next release. Thanks.

[chahmedejaz]

@nicolasVr - Update:
This fix will be deployed in this release. Thanks.