Stock check does not take variants into account about oc-mall-plugin HOT 4 CLOSED

According to the scripts.htm partial this method is only called if no properties are available - so it's technically a base product without variants.

If properties are available the onChangeProperty event handler is called via JS event which takes different variants into account.

I guess we should refactor this and display the available stock directly if the management method is set to single. Otherwise reuse the stock calculation from the onChangeProperty linked above.

from oc-mall-plugin.

I'm not sure if I get it correctly, however if I set up variants and group by a single property (which is package size in my case), then the default component markup renders simple links with images to change the variant: https://github.com/OFFLINE-GmbH/oc-mall-plugin/blob/develop/components/product/cart.htm#L6-L11

Then there are no other properties left which would render the select inputs which actually trigger the onChangeProperty event. I guess this only works for the case that "Attribute for variant grouping" is "None". In this case the select input correctly renders and triggers the event. But in the former case firstProperty.length will yield 0 and incorreclty onCheckProductStock will be run. This now fails to calculate the stock with {"X_OCTOBER_ERROR_FIELDS":[["Missing input data"]],"X_OCTOBER_ERROR_MESSAGE":"Missing input data"}.

from oc-mall-plugin.

This should be fixed in 128daca

from oc-mall-plugin.

This is looking good in the latest version. I'll close this for now.

from oc-mall-plugin.