From 942d3702569534b0cdda55fe4ef2ed2085a275d4 Mon Sep 17 00:00:00 2001
From: Mark Scherer <dereuromark@users.noreply.github.com>
Date: Tue, 28 Dec 2021 11:51:59 +0100
Subject: [PATCH] TE-10370 Fix up cloaking checks that are not needed. (#8983)

* TE-10370 Fix up cloaking checks that are not needed.

* TE-10370 Manually fix up remaining cloaking checks that are not needed.

* TE-10370 Manually fix up remaining cloaking checks that are not needed.

* Remove code smell and phpstan issue.
---
 .../ProductListRestrictionFilter.php                        | 6 +++---
 .../ProductListRestrictionValidator.php                     | 2 +-
 .../RestrictedItemsFilter/RestrictedItemsFilter.php         | 4 ++--
 .../Zed/ProductList/Persistence/ProductListRepository.php   | 4 ++--
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/Spryker/Zed/ProductList/Business/ProductListRestrictionFilter/ProductListRestrictionFilter.php b/src/Spryker/Zed/ProductList/Business/ProductListRestrictionFilter/ProductListRestrictionFilter.php
index fe14c10..d868609 100644
--- a/src/Spryker/Zed/ProductList/Business/ProductListRestrictionFilter/ProductListRestrictionFilter.php
+++ b/src/Spryker/Zed/ProductList/Business/ProductListRestrictionFilter/ProductListRestrictionFilter.php
@@ -34,20 +34,20 @@ public function __construct(
      */
     public function filterRestrictedProductConcreteSkus(array $productConcreteSkus, array $customerBlacklistIds, array $customerWhitelistIds): array
     {
-        if (empty($productConcreteSkus)) {
+        if (!$productConcreteSkus) {
             return [];
         }
 
         $restrictedProductConcreteSkus = [];
 
-        if (!empty($customerWhitelistIds)) {
+        if ($customerWhitelistIds) {
             $productConcreteSkusInWhitelist = $this->productListReader
                 ->getProductConcreteSkusInWhitelists($productConcreteSkus, $customerWhitelistIds);
 
             $restrictedProductConcreteSkus = array_diff($productConcreteSkus, $productConcreteSkusInWhitelist);
         }
 
-        if (!empty($customerBlacklistIds)) {
+        if ($customerBlacklistIds) {
             $productConcreteSkusInBlacklist = $this->productListReader
                 ->getProductConcreteSkusInBlacklists($productConcreteSkus, $customerBlacklistIds);
 
diff --git a/src/Spryker/Zed/ProductList/Business/ProductListRestrictionValidator/ProductListRestrictionValidator.php b/src/Spryker/Zed/ProductList/Business/ProductListRestrictionValidator/ProductListRestrictionValidator.php
index 41debd5..36146b7 100644
--- a/src/Spryker/Zed/ProductList/Business/ProductListRestrictionValidator/ProductListRestrictionValidator.php
+++ b/src/Spryker/Zed/ProductList/Business/ProductListRestrictionValidator/ProductListRestrictionValidator.php
@@ -70,7 +70,7 @@ public function validateItemAddition(CartChangeTransfer $cartChangeTransfer): Ca
                 $customerWhitelistIds,
             );
 
-        if (empty($restrictedProductConcreteSkus)) {
+        if (!$restrictedProductConcreteSkus) {
             return $cartPreCheckResponseTransfer;
         }
 
diff --git a/src/Spryker/Zed/ProductList/Business/RestrictedItemsFilter/RestrictedItemsFilter.php b/src/Spryker/Zed/ProductList/Business/RestrictedItemsFilter/RestrictedItemsFilter.php
index de97466..db1979b 100644
--- a/src/Spryker/Zed/ProductList/Business/RestrictedItemsFilter/RestrictedItemsFilter.php
+++ b/src/Spryker/Zed/ProductList/Business/RestrictedItemsFilter/RestrictedItemsFilter.php
@@ -79,7 +79,7 @@ protected function removeRestrictedItemsFromQuote(
         array $blacklistIds,
         array $whitelistIds
     ): void {
-        if (empty($blacklistIds) && empty($whitelistIds)) {
+        if (!$blacklistIds && !$whitelistIds) {
             return;
         }
 
@@ -89,7 +89,7 @@ protected function removeRestrictedItemsFromQuote(
 
         $restrictedProductConcreteSkus = $this->productListRestrictionFilter->filterRestrictedProductConcreteSkus($quoteSkus, $blacklistIds, $whitelistIds);
 
-        if (empty($restrictedProductConcreteSkus)) {
+        if (!$restrictedProductConcreteSkus) {
             return;
         }
 
diff --git a/src/Spryker/Zed/ProductList/Persistence/ProductListRepository.php b/src/Spryker/Zed/ProductList/Persistence/ProductListRepository.php
index 72bee84..06aeabe 100644
--- a/src/Spryker/Zed/ProductList/Persistence/ProductListRepository.php
+++ b/src/Spryker/Zed/ProductList/Persistence/ProductListRepository.php
@@ -209,11 +209,11 @@ public function getProductConcreteProductListIdsForType(int $idProduct, string $
      */
     protected function getConcreteProductSkusInList(array $productConcreteSkus, string $listType, array $productListIds): array
     {
-        if (empty($productConcreteSkus)) {
+        if (!$productConcreteSkus) {
             return [];
         }
 
-        if (empty($productListIds)) {
+        if (!$productListIds) {
             return [];
         }