Do not change inventoryInStock and inventoryAvailableToSell from null to 0

[aldeed]

Prerequisites

• Are you running the latest version?
• Are you able to consistently reproduce the issue?
• Did you search the issue queue for existing issue? Search issues

Issue Description

In https://github.com/reactioncommerce/reaction/blob/develop/imports/plugins/core/catalog/server/no-meteor/utils/createCatalogProduct.js#L30-L31 we have || 0 for inventoryInStock and inventoryAvailableToSell everywhere. This would convert null to 0, but these might be null if we are not tracking inventory, and converting to 0 would change the meaning. null means not tracked while 0 means tracked and unavailable.

Possible Solution

Remove || 0 for inventoryInStock and inventoryAvailableToSell everywhere in that file. This is a simple task, but we will need to test carefully that this does not break anything in the storefront that might be expecting always to get a number and not properly handling null value.

Versions

2.0.0 RC9