Fix to ignored the product don't have combinable

Shape-and-Shift · Jul 1, 2022 · a2799a6 · a2799a6
1 parent ad4a1c1
commit a2799a6
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/CHANGELOG_de-DE.md b/CHANGELOG_de-DE.md
@@ -1,6 +1,7 @@
 # 1.1.0 - Erweiterung
 - [Issue #1](https://github.com/Shape-and-Shift/shopware-variant-switch/issues/1) Zeige auch Varianten, die nicht auf Lager sind
 - [Issue #2](https://github.com/Shape-and-Shift/shopware-variant-switch/issues/2) Ausblenden von geteilten Varianten
+- [Issue #6](https://github.com/Shape-and-Shift/shopware-variant-switch/issues/6) Ignoriert, dass das Produkt nicht kombinierbar ist
 
 # 1.0.0 - Erste Veröffentlichung
 - Variantenschalter auf der Produktliste anzeigen

diff --git a/CHANGELOG_en-GB.md b/CHANGELOG_en-GB.md
@@ -1,6 +1,7 @@
 # 1.1.0 - Enhancement
 - [Issue #1](https://github.com/Shape-and-Shift/shopware-variant-switch/issues/1) Show also variants which are out of stock
 - [Issue #2](https://github.com/Shape-and-Shift/shopware-variant-switch/issues/2) Hide split variants
+- [Issue #6](https://github.com/Shape-and-Shift/shopware-variant-switch/issues/6) Ignored the product don't have combinable
 
 # 1.0.0 - First release
 - Show variant switch on product listing card

diff --git a/src/Storefront/Page/ProductListingConfigurationLoader.php b/src/Storefront/Page/ProductListingConfigurationLoader.php
@@ -43,6 +43,9 @@ public function loadListing(ProductCollection $products, SalesChannelContext $co
         }));
 
         $allCombinations = $this->loadCombinations($productIds, $context->getContext());
+        if (\count($allCombinations) === 0) {
+            return;
+        }
 
         /** @var SalesChannelProductEntity $product */
         foreach ($products as $product) {
@@ -58,6 +61,10 @@ public function loadListing(ProductCollection $products, SalesChannelContext $co
 
             $groups = $this->sortSettings($productSetting, $product);
 
+            if (!array_key_exists($product->getParentId(), $allCombinations)) {
+                continue;
+            }
+
             $combinations = $allCombinations[$product->getParentId()];
 
             $current = $this->buildCurrentOptions($product, $groups);