2.4 : When saving a product using the product repository, the price is not saved

[kanduvisla]

I'm trying to programmatically create a product, but upon the first save, the price is not saved, thus returning 'NULL'. On a second save (with the identical object), the price gets saved.

Preconditions

1. Magento 2.4-develop
2. php v 7.2

Steps to reproduce

/** @var \Magento\Catalog\Model\Product $product */
$product = $this->productFactory->create();
$product->setSku('abc');
$product->setName('Name');
$product->setPrice(9.95);
$product->setAttributeSetId(4);

// Save once, the price does not get saved:
$this->productRepository->save($product);

// Check if the product is saved:
$product = $this->productRepository->get('abc');
$this->assertEquals('abc', $product->getSku());          // passes
$this->assertEquals('Name', $product->getName());  // passes
$this->assertEquals(9.95, $product->getPrice());  // fails (null does not equal 9.95

When I add an extra save()-instruction it works:

// Save twice, the price gets saved:
$this->productRepository->save($product);
$this->productRepository->save($product);

Expected result

The price should be saved in the first run

Actual result

The price only gets saved the second time the save()-method is executed

[magento-engcom-team]

@kanduvisla, thank you for your report.
We've acknowledged the issue and added to our backlog.

[kanduvisla]

I've found another strange behavior with this:

Upon the second save, the catalog_product_entity_xxx-tables are populated with duplicate values for every store view. So to recap:

• Single save does save for store ID 0, but does not save the price.
• Second save saves duplicate values for every store view.

Even if I explicitly use setStoreId(0), the (duplicate) values get saved for all store views.

[willtran]

We are working on it #MLAU18

[zolat]

We are working on it #MLAU18

[willtran]

I have investigated this issue but unfortunately this can not be fixed easily.

Although the symptoms is the same (ProductRepository doesn't save price attribute on creating new product), there are 2 different causes depending on the area which the code was called from.

First, if you are calling the above code snippet from an area different than adminhtml (frontend, global, crontab ...), product type does not default to simple, hence when the Resource class tries to save the product it doesn't load the product type attribute set and strips out a whole heaps of attributes, price is one of them. This can be fixed but I will create a separate issue.
In short, this code will work as a work around for now:

$this->state->setAreaCode('global'); // or frontend / crontab...
$product = $this->productFactory->create();
$product->setSku('abcd');
$product->setName('abcd');
$product->setPrice(9.95);
$product->setAttributeSetId(10);
$product->setTypeId('simple');  

$this->productRepository->save($product);
$product = $this->productRepository->get('abcd');

Now, if you are calling the above code in a setup script and set the area as adminhtml. There is an observer that check if the current admin user has permission to create new product with price:

// Magento\PricePermissions\Observer\CatalogProductSaveBeforeObserver

if ($product->isObjectNew() && !$this->observerData->isCanReadProductPrice()) {
    $product->setPrice($this->observerData->getDefaultProductPriceString());
}

Since we are calling this from a setup script, there is not admin user, hence the ACL check will fail and the product's price attribute is stripped out.

I think the correct fix for this would be to create another area for setup script and refactor the code involved in ProductRepository to consider the new area, but this seems to be a very big changes.
Moreover it would be best to prevent product creation from certain area such as frontend.
@okorshenko @magento-engcom-team can you please provide some suggestions.

[kanduvisla]

@willtran : for full disclosure: I'm trying to create a product while in emulated global area code:

$this->state->emulateAreaCode(Area::AREA_GLOBAL, [$this, 'createProduct'], [$product]);

[keevitaja]

There is an easy fix :)

When creating product, assign the type id and price magically appears!

$product->setTypeId('simple')
    //->setStoreId(0)
    ->setName($name)
    ->setSku($name)
    ->setUrlKey($name)
    // ->setVisibility(Visibility::VISIBILITY_BOTH)
    // ->setStatus(Status::STATUS_ENABLED)
    ->setPrice(4)
    ->setAttributeSetId(4)
    ->save();