Unable to add product to cart if stock is not configured.

[nickfrederiksen]

Which component is this issue related to?

Umbraco Commerce (Core)

Which Umbraco Commerce version are you using? (Please write the exact version, example: 10.1.0)

15.3.1

Bug summary

There's an issue where a product that has no stock information configured, can't be added to the cart.

I get this error:
Insufficient stock: Product '9bdde03e-7339-4e58-9321-f03c92d3c9f4' has 0,000000 item(s) left, but 1 were requested.

We have no custom validators and this issue seems to have been introduced with the 15.3.0 update. The product does not have stock and should therefore not be validated as such.

It's rather critical for this to be fixed.

Specifics

No response

Steps to reproduce

Install 15.3.0 (or 15.3.1).

Create product with no stock property.

Add that product to the basket.

Expected result / actual result

Product should be added. With no error.

Dependencies

No response

[mattbrailsford]

Does the product node have an actual stock property on it?

[nickfrederiksen]

It does not.

[niekvanderreest]

Having similar problems, there seems to be a major bug or breaking change introduced in 15.3.

I today updated a project to use Commerce 15.3.1 from 15.2.2 and suddenly it's impossible to add products, any product to the cart, also a workaround with adding a stock property doesn't work, it seems the Stock property isn't binding to IPublishedContent...

I do have a custom productadapter in place, removing it for the default makes adding products work again

[mattbrailsford]

Yea, in 15.3.0 some extra validation handlers were added to ensure an item being added to the cart doesn't exceed the stock of an item, however these shouldn't trigger if there is no stock property editor on your node.

@nickfrederiksen are you also using a custom product adapter?

I've got a workaround I can give you but I'd like to figure out what the actually bug is here as on my local environment, a product with no stock property editor on the product doc type doesn't trigger the validators

[nickfrederiksen]

@mattbrailsford
We are indeed using a custom product adapter. It only handles prices, why I thought it wasn't relevant to this.

[mattbrailsford]

Yea, product adapters aren't really responsible for managing stock levels so it's weird if they are connected 🤔

[mattbrailsford]

Ahh, ok, I think I might have found it

[niekvanderreest]

@mattbrailsford Calling GetProductSnapshotAsync on my product adapter calls the base class method:
await TryGetProductNodeAsync(productReference, productVariantReference);

The result has an iPublishedContent for item1 (the product). When i add a stock property to my document type i see the value correctly in the properties collection, but the actual property Stock on the model shows 0M as value, maybe a valuebinder problem?

[mattbrailsford]

So I think the problem is when we work out the stock, we have to try and work out where the stock is stored because we can have nodes with no variants, nodes with child variants, and nodes with multi variants. When we work that out, we request the a product snapshot and see if it is IUmbracoProductSnapshot and from there we can get the stock source. But if the product snapshot doesn't implement IUmbracoProductSnapshot then it will return a default stock of 0.

Ultimately I think this needs updating to return a default stock of null as it's more the case that no stock was found, than there actually being zero items.

A quick test your end might be to have your product adapter return a product snapshot that implements IUmbracoProductSnapshot either directly, or by extending the default UmbracoProductSnapshot.

I'll do a quick build though with the default stock defaulting to null and see if that works for you too.

[mattbrailsford]

There should be a 15.3.2--preview.4 build on the nightly feed that sets the default stock to null if either of you wants to try that.

[niekvanderreest]

@mattbrailsford Thanks for the quick action on this, just to make everything complete, I implemented the IUmbracoProductSnapshot interface on my custom ProductSnapshot that previously only implemented IProductSnapshot.

This also fixes the issue and makes adding products (without stock property) work again

[nickfrederiksen]

@mattbrailsford

Same issue.

All I do in my adapter, is to get the snapshot using base.GetProductSnapshotAsync, and then manipulate the Prices property.
If the base.GetProductSnapshotAsync does not return a UmbracoProductSnapshot, i will ignore it and just return it as is.

[mattbrailsford]

@nickfrederiksen by "same issue" you mean running 15.3.2--preview.4 still throws the same exception?