Merge pull request #9424 from magento-gl/spartans_pr_12122024

[Spartans] Bugfix Delivery

Author: glo71317

app/code/Magento/Catalog/Model/Product/Price/Validation/TierPriceValidator.php

@@ -1,7 +1,7 @@
 <?php
 /**
- * Copyright © Magento, Inc. All rights reserved.
- * See COPYING.txt for license details.
+ * Copyright 2024 Adobe
+ * All Rights Reserved.
  */
 
 namespace Magento\Catalog\Model\Product\Price\Validation;
@@ -10,10 +10,8 @@
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\Catalog\Model\Product\Type;
 use Magento\Catalog\Model\ProductIdLocatorInterface;
-use Magento\Customer\Api\GroupRepositoryInterface;
 use Magento\Directory\Model\Currency;
-use Magento\Framework\Api\FilterBuilder;
-use Magento\Framework\Api\SearchCriteriaBuilder;
+use Magento\Framework\App\ResourceConnection;
 use Magento\Framework\Exception\LocalizedException;
 use Magento\Framework\ObjectManager\ResetAfterRequestInterface;
 use Magento\Store\Api\WebsiteRepositoryInterface;
@@ -35,21 +33,6 @@ class TierPriceValidator implements ResetAfterRequestInterface
      */
     private $productIdLocator;
 
-    /**
-     * @var SearchCriteriaBuilder
-     */
-    private $searchCriteriaBuilder;
-
-    /**
-     * @var FilterBuilder
-     */
-    private $filterBuilder;
-
-    /**
-     * @var GroupRepositoryInterface
-     */
-    private $customerGroupRepository;
-
     /**
      * @var WebsiteRepositoryInterface
      */
@@ -60,13 +43,6 @@ class TierPriceValidator implements ResetAfterRequestInterface
      */
     private $validationResult;
 
-    /**
-     * Groups by code cache.
-     *
-     * @var array
-     */
-    private $customerGroupsByCode = [];
-
     /**
      * @var InvalidSkuProcessor
      */
@@ -97,6 +73,16 @@ class TierPriceValidator implements ResetAfterRequestInterface
      */
     private $productsCacheBySku = [];
 
+    /**
+     * @var ResourceConnection
+     */
+    private $resourceConnection;
+
+    /**
+     * @var array Customer group check cache
+     */
+    private $customerGroupCheck = [];
+
     /**
      * @var ScopeConfigInterface
      */
@@ -106,38 +92,32 @@ class TierPriceValidator implements ResetAfterRequestInterface
      * TierPriceValidator constructor.
      *
      * @param ProductIdLocatorInterface $productIdLocator
-     * @param SearchCriteriaBuilder $searchCriteriaBuilder
-     * @param FilterBuilder $filterBuilder
-     * @param GroupRepositoryInterface $customerGroupRepository
      * @param WebsiteRepositoryInterface $websiteRepository
      * @param Result $validationResult
      * @param InvalidSkuProcessor $invalidSkuProcessor
      * @param ProductRepositoryInterface $productRepository
      * @param array $allowedProductTypes [optional]
+     * @param ResourceConnection|null $resourceConnection
      * @param ScopeConfigInterface|null $scopeConfig
      * @SuppressWarnings(PHPMD.ExcessiveParameterList)
      */
     public function __construct(
         ProductIdLocatorInterface $productIdLocator,
-        SearchCriteriaBuilder $searchCriteriaBuilder,
-        FilterBuilder $filterBuilder,
-        GroupRepositoryInterface $customerGroupRepository,
         WebsiteRepositoryInterface $websiteRepository,
         Result $validationResult,
         InvalidSkuProcessor $invalidSkuProcessor,
         ProductRepositoryInterface $productRepository,
         array $allowedProductTypes = [],
+        ?ResourceConnection $resourceConnection = null,
         ?ScopeConfigInterface $scopeConfig = null
     ) {
         $this->productIdLocator = $productIdLocator;
-        $this->searchCriteriaBuilder = $searchCriteriaBuilder;
-        $this->filterBuilder = $filterBuilder;
-        $this->customerGroupRepository = $customerGroupRepository;
         $this->websiteRepository = $websiteRepository;
         $this->validationResult = $validationResult;
         $this->invalidSkuProcessor = $invalidSkuProcessor;
         $this->productRepository = $productRepository;
         $this->allowedProductTypes = $allowedProductTypes;
+        $this->resourceConnection = $resourceConnection ?: ObjectManager::getInstance()->get(ResourceConnection::class);
         $this->scopeConfig = $scopeConfig ?: ObjectManager::getInstance()->get(ScopeConfigInterface::class);
     }
 
@@ -504,32 +484,19 @@ private function checkGroup(TierPriceInterface $price, $key, Result $validationR
      */
     private function retrieveGroupValue(string $code)
     {
-        if (!isset($this->customerGroupsByCode[$code])) {
-            $searchCriteria = $this->searchCriteriaBuilder->addFilters(
-                [
-                    $this->filterBuilder->setField('customer_group_code')->setValue($code)->create()
-                ]
+        if (!isset($this->customerGroupCheck[$code])) {
+            $connection = $this->resourceConnection->getConnection();
+            $select = $connection->select()->from(
+                $this->resourceConnection->getTableName('customer_group'),
+                'customer_group_id'
+            )->where(
+                'customer_group_code = ?',
+                $code
             );
-            $items = $this->customerGroupRepository->getList($searchCriteria->create())->getItems();
-            $item = array_shift($items);
-
-            if (!$item) {
-                $this->customerGroupsByCode[$code] = false;
-                return false;
-            }
-
-            $itemCode = $item->getCode();
-            $itemId = $item->getId();
-
-            if (strtolower($itemCode) !== $code) {
-                $this->customerGroupsByCode[$code] = false;
-                return false;
-            }
-
-            $this->customerGroupsByCode[strtolower($itemCode)] = $itemId;
+            $this->customerGroupCheck[$code] = $connection->fetchOne($select);
         }
 
-        return $this->customerGroupsByCode[$code];
+        return $this->customerGroupCheck[$code];
     }
 
     /**
@@ -576,6 +543,6 @@ private function compareWebsiteValueNewPrice(TierPriceInterface $price, TierPric
      */
     public function _resetState(): void
     {
-        $this->customerGroupsByCode = [];
+        $this->customerGroupCheck = [];
     }
 }