MC-41093: [CE/EE] Fatal error during saving Simple Product by admin

Author: OlgaVasyltsun

app/code/Magento/CatalogInventory/Model/Stock/StockItemRepository.php

@@ -25,6 +25,7 @@
 use Magento\Framework\Stdlib\DateTime\DateTime;
 use Magento\Framework\Stdlib\DateTime\TimezoneInterface;
 use Magento\Catalog\Model\ResourceModel\Product\CollectionFactory;
+use Psr\Log\LoggerInterface as PsrLogger;
 
 /**
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
@@ -98,8 +99,11 @@ class StockItemRepository implements StockItemRepositoryInterface
     protected $productCollectionFactory;
 
     /**
-     * Constructor
-     *
+     * @var PsrLogger
+     */
+    private $psrLogger;
+
+    /**
      * @param StockConfigurationInterface $stockConfiguration
      * @param StockStateProviderInterface $stockStateProvider
      * @param StockItemResource $resource
@@ -111,7 +115,8 @@ class StockItemRepository implements StockItemRepositoryInterface
      * @param TimezoneInterface $localeDate
      * @param Processor $indexProcessor
      * @param DateTime $dateTime
-     * @param \Magento\Catalog\Model\ResourceModel\Product\CollectionFactory|null $collectionFactory
+     * @param CollectionFactory|null $productCollectionFactory
+     * @param PsrLogger|null $psrLogger
      * @SuppressWarnings(PHPMD.ExcessiveParameterList)
      */
     public function __construct(
@@ -126,7 +131,8 @@ public function __construct(
         TimezoneInterface $localeDate,
         Processor $indexProcessor,
         DateTime $dateTime,
-        \Magento\Catalog\Model\ResourceModel\Product\CollectionFactory $productCollectionFactory = null
+        \Magento\Catalog\Model\ResourceModel\Product\CollectionFactory $productCollectionFactory = null,
+        PsrLogger $psrLogger = null
     ) {
         $this->stockConfiguration = $stockConfiguration;
         $this->stockStateProvider = $stockStateProvider;
@@ -141,6 +147,8 @@ public function __construct(
         $this->dateTime = $dateTime;
         $this->productCollectionFactory = $productCollectionFactory ?: ObjectManager::getInstance()
             ->get(CollectionFactory::class);
+        $this->psrLogger = $psrLogger ?: ObjectManager::getInstance()
+            ->get(PsrLogger::class);
     }
 
     /**
@@ -184,6 +192,7 @@ public function save(\Magento\CatalogInventory\Api\Data\StockItemInterface $stoc
 
             $this->resource->save($stockItem);
         } catch (\Exception $exception) {
+            $this->psrLogger->error($exception->getMessage());
             throw new CouldNotSaveException(__('The stock item was unable to be saved. Please try again.'), $exception);
         }
         return $stockItem;

app/code/Magento/Elasticsearch/Model/ResourceModel/Index.php

@@ -6,6 +6,7 @@
 namespace Magento\Elasticsearch\Model\ResourceModel;
 
 use Magento\Catalog\Model\Indexer\Product\Price\DimensionCollectionFactory;
+use Magento\Framework\Exception\NoSuchEntityException;
 use Magento\Framework\Model\ResourceModel\Db\Context;
 use Magento\Store\Model\StoreManagerInterface;
 use Magento\Catalog\Api\ProductRepositoryInterface;
@@ -137,7 +138,11 @@ public function getFullCategoryProductIndexData($storeId = null, $productIds = n
 
         foreach ($categoryPositions as $productId => $positions) {
             foreach ($positions as $categoryId => $position) {
-                $category = $this->categoryRepository->get($categoryId, $storeId);
+                try {
+                    $category = $this->categoryRepository->get($categoryId, $storeId);
+                } catch (NoSuchEntityException $e) {
+                    continue;
+                }
                 $categoryName = $category->getName();
                 $categoryData[$productId][] = [
                     'id' => $categoryId,

dev/tests/integration/testsuite/Magento/Catalog/Controller/Adminhtml/Product/Save/DeleteCategoryTest.php

@@ -0,0 +1,54 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento\Catalog\Controller\Adminhtml\Product\Save;
+
+use Magento\Catalog\Api\ProductRepositoryInterface;
+use Magento\Framework\App\Request\Http as HttpRequest;
+use Magento\Framework\Message\MessageInterface;
+use Magento\TestFramework\TestCase\AbstractBackendController;
+
+/**
+ * @magentoDbIsolation disabled
+ * @magentoDataFixture Magento/Catalog/_files/category_product.php
+ * @magentoAppArea adminhtml
+ */
+class DeleteCategoryTest extends AbstractBackendController
+{
+    /**
+     * @var ProductRepositoryInterface
+     */
+    private $productRepository;
+
+    /**
+     * @inheritdoc
+     */
+    protected function setUp(): void
+    {
+        parent::setUp();
+        $this->productRepository = $this->_objectManager->get(ProductRepositoryInterface::class);
+
+    }
+
+    /**
+     * Checks product saving with deleted category before reindex is done
+     */
+    public function testDeleteCustomOptionWithTypeField(): void
+    {
+        $category = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->create(\Magento\Catalog\Model\Category::class);
+        $category->load(333);
+        $category->delete();
+        $product = $this->productRepository->get('simple333');
+        $this->productRepository->save($product);
+        $this->getRequest()->setMethod(HttpRequest::METHOD_POST);
+        $this->dispatch('backend/catalog/product/save/id/' . $product->getEntityId());
+        $this->assertSessionMessages(
+            $this->equalTo([(string)__('You saved the product.')]),
+            MessageInterface::TYPE_SUCCESS
+        );
+    }
+}