Backport hotfix: Add support for products not having a SKU (#47)

* Add support for products not having a SKU

* Add test for product support with empty SKU

Author: mescalantea

src/BusinessLogic/Domain/PromotionalWidgets/Services/WidgetValidationService.php

@@ -102,17 +102,15 @@ public function isProductSupported(string $productId): bool
 
         $productSku = $this->productService->getProductsSkuByProductId($productId);
 
-        if (!$productSku) {
-            return false;
-        }
-
-        $excludedProducts = $generalSettings->getExcludedProducts() ?? [];
-
-        if (
-            $excludedProducts &&
-            in_array($productSku, $excludedProducts, true)
-        ) {
-            return false;
+        if ($productSku) {
+            $excludedProducts = $generalSettings->getExcludedProducts() ?? [];
+
+            if (
+                $excludedProducts &&
+                in_array($productSku, $excludedProducts, true)
+            ) {
+                return false;
+            }
         }
 
         $excludedCategories = $generalSettings->getExcludedCategories() ?? [];

tests/BusinessLogic/Domain/PromotionalWidgets/Services/WidgetValidationServiceTest.php

@@ -280,6 +280,36 @@ public function testIsProductSupportedSkuExcluded(): void
         self::assertFalse($result);
     }
 
+    /**
+     * @dataProvider dataProviderIsProductSupportedEmptySku
+     * @return void
+     */
+    public function testIsProductSupportedEmptySku($sku): void
+    {
+        //arrange
+        $this->mockGeneralSettingsService->saveGeneralSettings(
+            new GeneralSettings(
+                true,
+                null,
+                null,
+                null,
+                null
+            )
+        );
+        $this->mockProductService->setMockProductSku('');
+
+        // act
+        $result = $this->widgetValidationService->isProductSupported($sku);
+
+        // assert
+        self::assertTrue($result);
+    }
+
+    public function dataProviderIsProductSupportedEmptySku(): array
+    {
+        return [[''], ['0']];
+    }
+
     /**
      * @return void
      */