FFWEB-3159: Support MSI - checking all inventory sources during export feed

Rayn93 · web-flow · commit e687137e31e1 · 2024-08-29T12:16:14.000+02:00
Support MSI - checking all inventory sources during export feed
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,4 +1,9 @@
 # Changelog
+## Unreleased
+### Fix
+- Support MSI - checking all inventory sources during export feed
+- Fix feed path for UI export type
+
 ## [v5.0.0] - 2024.07.08
 ### BREAKING
 - Upgrade FACT-Finder Web-Components to version 5.0.0-pre.1
@@ -45,7 +50,7 @@
 ### Fix
 - Fix problems in factfinder_feed_export cron job by casting `storeid` to int.
 
-## [v4.1.6] - 2023.07.19
+## [v4.1.6] - 2023.07.19 
 ### Fix
 - Set default value for $pushImportResult to prevent initialization exception
 
diff --git a/src/Controller/Adminhtml/Export/Feed.php b/src/Controller/Adminhtml/Export/Feed.php
@@ -90,7 +90,7 @@ public function execute()
                 $path     = $this->feedFileService->getExportPath($filename);
 
                 $this->feedGeneratorFactory->create($this->feedType)->generate($stream);
-                $messages[] = __('<li>Feed file for channel %1 has been generated under %2</li>', $channel, "$path/$filename");
+                $messages[] = __('<li>Feed file for channel %1 has been generated under %2</li>', $channel, $path);
 
                 try {
                     $this->ftpUploader->upload($filename, $stream);
diff --git a/src/Model/Export/Catalog/Entity/ProductVariation.php b/src/Model/Export/Catalog/Entity/ProductVariation.php
@@ -5,7 +5,7 @@
 namespace Omikron\Factfinder\Model\Export\Catalog\Entity;
 
 use Magento\Catalog\Model\Product;
-use Magento\CatalogInventory\Model\Stock\StockItemRepository as StockItem;
+use Magento\Inventory\Model\SourceItem\Command\GetSourceItemsBySku;
 use Omikron\Factfinder\Api\Export\ExportEntityInterface;
 use Omikron\Factfinder\Api\Export\FieldInterface;
 use Omikron\Factfinder\Model\Export\Catalog\FieldProvider;
@@ -20,22 +20,22 @@ class ProductVariation implements ExportEntityInterface
 
     /** @var string[] */
     private array $configurableData;
-    private StockItem $stockItem;
+    private GetSourceItemsBySku $getSourceItemsBySku;
 
     public function __construct(
         Product $product,
         Product $configurable,
         NumberFormatter $numberFormatter,
         FieldProvider $variantFieldProvider,
-        StockItem $stockItem,
+        GetSourceItemsBySku $getSourceItemsBySku,
         array $data = []
     ) {
         $this->product          = $product;
         $this->configurable     = $configurable;
         $this->numberFormatter  = $numberFormatter;
         $this->configurableData = $data;
         $this->fieldprovider    = $variantFieldProvider;
-        $this->stockItem = $stockItem;
+        $this->getSourceItemsBySku = $getSourceItemsBySku;
     }
 
     public function getId(): int
@@ -91,8 +91,14 @@ private function getAvailability(): bool
             return $this->product->isAvailable();
         }
 
-        $quantity = $this->stockItem->get($this->product->getId());
+        $sourceItems = $this->getSourceItemsBySku->execute($this->product->getSku());
 
-        return $quantity->getIsInStock();
+        foreach ($sourceItems as $sourceItem) {
+            if ($sourceItem->getStatus()) {
+                return true;
+            }
+        }
+
+        return false;
     }
 }
diff --git a/src/Model/Export/Catalog/ProductType/BundleDataProvider.php b/src/Model/Export/Catalog/ProductType/BundleDataProvider.php
@@ -6,21 +6,21 @@
 
 use Magento\Bundle\Model\Product\CatalogPrice;
 use Magento\Catalog\Model\Product;
-use Magento\CatalogInventory\Model\Stock\StockItemRepository as StockItem;
+use Magento\Inventory\Model\SourceItem\Command\GetSourceItemsBySku;
 use Magento\Directory\Model\PriceCurrency;
 use Omikron\Factfinder\Model\Formatter\NumberFormatter;
 
 class BundleDataProvider extends SimpleDataProvider
 {
     public function __construct(
-        protected Product         $product,
-        protected NumberFormatter $numberFormatter,
-        private readonly PriceCurrency     $priceCurrency,
-        private readonly CatalogPrice      $priceModel,
-        StockItem $stockItem,
-        protected array             $productFields = []
+        protected Product              $product,
+        protected NumberFormatter      $numberFormatter,
+        private readonly PriceCurrency $priceCurrency,
+        private readonly CatalogPrice  $priceModel,
+        protected GetSourceItemsBySku  $getSourceItemsBySku,
+        protected array                $productFields = []
     ) {
-        parent::__construct($product, $numberFormatter, $stockItem, $productFields);
+        parent::__construct($product, $numberFormatter, $getSourceItemsBySku, $productFields);
     }
 
     public function toArray(): array
diff --git a/src/Model/Export/Catalog/ProductType/ConfigurableDataProvider.php b/src/Model/Export/Catalog/ProductType/ConfigurableDataProvider.php
@@ -7,9 +7,9 @@
 use Magento\Catalog\Api\Data\ProductInterface;
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\Catalog\Model\Product;
-use Magento\CatalogInventory\Model\Stock\StockItemRepository as StockItem;
 use Magento\ConfigurableProduct\Model\Product\Type\Configurable as ConfigurableProductType;
 use Magento\Framework\Api\SearchCriteriaBuilder;
+use Magento\Inventory\Model\SourceItem\Command\GetSourceItemsBySku;
 use Omikron\Factfinder\Api\Export\ExportEntityInterface;
 use Omikron\Factfinder\Api\Filter\FilterInterface;
 use Omikron\Factfinder\Model\Export\Catalog\Entity\ProductVariationFactory;
@@ -18,17 +18,17 @@
 class ConfigurableDataProvider extends SimpleDataProvider
 {
     public function __construct(
-        protected Product                  $product,
-        protected NumberFormatter          $numberFormatter,
+        protected Product                           $product,
+        protected NumberFormatter                   $numberFormatter,
         private readonly ConfigurableProductType    $productType,
         private readonly FilterInterface            $filter,
         private readonly ProductVariationFactory    $variationFactory,
         private readonly ProductRepositoryInterface $productRepository,
         private readonly SearchCriteriaBuilder      $builder,
-        StockItem $stockItem,
-        protected array                    $productFields = []
+        protected GetSourceItemsBySku               $getSourceItemsBySku,
+        protected array                             $productFields = []
     ) {
-        parent::__construct($product, $numberFormatter, $stockItem, $productFields);
+        parent::__construct($product, $numberFormatter, $getSourceItemsBySku, $productFields);
     }
 
     public function getEntities(): iterable
diff --git a/src/Model/Export/Catalog/ProductType/SimpleDataProvider.php b/src/Model/Export/Catalog/ProductType/SimpleDataProvider.php
@@ -5,7 +5,7 @@
 namespace Omikron\Factfinder\Model\Export\Catalog\ProductType;
 
 use Magento\Catalog\Model\Product;
-use Magento\CatalogInventory\Model\Stock\StockItemRepository as StockItem;
+use Magento\Inventory\Model\SourceItem\Command\GetSourceItemsBySku;
 use Omikron\Factfinder\Api\Export\FieldInterface;
 use Omikron\Factfinder\Api\Export\DataProviderInterface;
 use Omikron\Factfinder\Api\Export\ExportEntityInterface;
@@ -16,7 +16,7 @@ class SimpleDataProvider implements DataProviderInterface, ExportEntityInterface
     public function __construct(
         protected Product $product,
         protected NumberFormatter $numberFormatter,
-        protected StockItem $stockItem,
+        protected GetSourceItemsBySku $getSourceItemsBySku,
         protected array $productFields = [],
     ) {
     }
@@ -70,8 +70,14 @@ private function getAvailability(): bool
             return $this->product->isAvailable();
         }
 
-        $quantity = $this->stockItem->get($this->product->getId());
+        $sourceItems = $this->getSourceItemsBySku->execute($this->product->getSku());
 
-        return $quantity->getIsInStock();
+        foreach ($sourceItems as $sourceItem) {
+            if ($sourceItem->getStatus()) {
+                return true;
+            }
+        }
+
+        return false;
     }
 }
diff --git a/src/Test/Unit/Model/Export/Catalog/Entity/ProductVariationTest.php b/src/Test/Unit/Model/Export/Catalog/Entity/ProductVariationTest.php
@@ -5,9 +5,9 @@
 namespace Omikron\Factfinder\Model\Export\Catalog\Entity;
 
 use Magento\Catalog\Model\Product;
-use Magento\CatalogInventory\Api\Data\StockItemInterface;
-use Magento\CatalogInventory\Model\Stock\StockItemRepository;
 use Magento\Framework\Model\AbstractModel;
+use Magento\Inventory\Model\SourceItem;
+use Magento\Inventory\Model\SourceItem\Command\GetSourceItemsBySku;
 use Omikron\Factfinder\Model\Export\Catalog\FieldProvider;
 use Omikron\Factfinder\Model\Export\Catalog\ProductField\FilterAttributes;
 use Omikron\Factfinder\Model\Export\Catalog\ProductField\ProductImage;
@@ -53,13 +53,8 @@ public function test_variant_data_will_override_the_parent()
             $this->createMock(Product::class),
             new NumberFormatter(),
             $fieldProviderMock,
-            $this->createConfiguredMock(StockItemRepository::class, [
-                'get' => $this->createConfiguredMock(
-                    StockItemInterface::class,
-                    [
-                    'getIsInStock' => true
-                    ]
-                )
+            $this->createConfiguredMock(GetSourceItemsBySku::class, [
+                'execute' => [$this->createConfiguredMock(SourceItem::class, ['getStatus' => 1])]
             ]),
             $this->configurableProductData
         );
@@ -99,13 +94,8 @@ public function test_configurable_attributes_should_be_merged_with_filter_attrib
             $this->createMock(Product::class),
             new NumberFormatter(),
             $fieldProviderMock,
-            $this->createConfiguredMock(StockItemRepository::class, [
-                'get' => $this->createConfiguredMock(
-                    StockItemInterface::class,
-                    [
-                    'getIsInStock' => true
-                    ]
-                )
+            $this->createConfiguredMock(GetSourceItemsBySku::class, [
+                'execute' => [$this->createConfiguredMock(SourceItem::class, ['getStatus' => 1])]
             ]),
             ['FilterAttributes' => '|Color=Red|Size=XS|'] + $this->configurableProductData
         );
diff --git a/src/Test/Unit/Model/Export/Catalog/ProductType/ConfigurableDataProviderTest.php b/src/Test/Unit/Model/Export/Catalog/ProductType/ConfigurableDataProviderTest.php
@@ -6,10 +6,10 @@
 
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\Catalog\Model\Product;
-use Magento\CatalogInventory\Api\Data\StockItemInterface;
-use Magento\CatalogInventory\Model\Stock\StockItemRepository;
 use Magento\ConfigurableProduct\Model\Product\Type\Configurable as ConfigurableProductType;
 use Magento\Framework\Api\SearchCriteriaBuilder;
+use Magento\Inventory\Model\SourceItem;
+use Magento\Inventory\Model\SourceItem\Command\GetSourceItemsBySku;
 use Omikron\Factfinder\Api\Filter\FilterInterface;
 use Omikron\Factfinder\Model\Export\Catalog\Entity\ProductVariationFactory;
 use Omikron\Factfinder\Model\Formatter\NumberFormatter;
@@ -114,11 +114,8 @@ protected function setUp(): void
             $this->variantFactoryMock,
             $this->repositoryMock,
             $this->builderMock,
-            $this->createConfiguredMock(StockItemRepository::class, [
-                'get' => $this->createConfiguredMock(
-                    StockItemInterface::class,
-                    ['getIsInStock' => true]
-                )
+            $this->createConfiguredMock(GetSourceItemsBySku::class, [
+                'execute' => [$this->createConfiguredMock(SourceItem::class, ['getStatus' => 1])]
             ]),
         );
     }
