ACP2E-3255: [GRAPHQL] model value should be specified when getting customerCart

 - Fixed the issue.

Author: devarul

app/code/Magento/Quote/Model/QuoteManagement.php

@@ -43,6 +43,7 @@
 use Magento\Sales\Api\Data\OrderInterfaceFactory as OrderFactory;
 use Magento\Sales\Api\OrderManagementInterface as OrderManagement;
 use Magento\Store\Model\StoreManagerInterface;
+use Magento\Customer\Model\AddressRegistry;
 
 /**
  * Class for managing quote
@@ -183,6 +184,11 @@ class QuoteManagement implements CartManagementInterface, ResetAfterRequestInter
      */
     private $cartMutex;
 
+    /**
+     * @var AddressRegistry
+     */
+    private $addressRegistry;
+
     /**
      * @param EventManager $eventManager
      * @param SubmitQuoteValidator $submitQuoteValidator
@@ -210,6 +216,7 @@ class QuoteManagement implements CartManagementInterface, ResetAfterRequestInter
      * @param RemoteAddress|null $remoteAddress
      * @param LockManagerInterface $lockManager
      * @param CartMutexInterface|null $cartMutex
+     * @param AddressRegistry|null $addressRegistry
      * @SuppressWarnings(PHPMD.ExcessiveParameterList)
      * @SuppressWarnings(PHPMD.UnusedFormalParameter)
      */
@@ -239,7 +246,8 @@ public function __construct(
         RequestInterface $request = null,
         RemoteAddress $remoteAddress = null,
         LockManagerInterface $lockManager = null,
-        ?CartMutexInterface $cartMutex = null
+        ?CartMutexInterface $cartMutex = null,
+        ?AddressRegistry $addressRegistry = null
     ) {
         $this->eventManager = $eventManager;
         $this->submitQuoteValidator = $submitQuoteValidator;
@@ -271,6 +279,8 @@ public function __construct(
             ->get(RemoteAddress::class);
         $this->cartMutex = $cartMutex
             ?? ObjectManager::getInstance()->get(CartMutexInterface::class);
+        $this->addressRegistry = $addressRegistry
+            ?? ObjectManager::getInstance()->get(AddressRegistry::class);
     }
 
     /**
@@ -696,7 +706,9 @@ protected function _prepareCustomerQuote($quote)
                 }
                 //save here new customer address
                 $shippingAddress->setCustomerId($quote->getCustomerId());
-                $this->addressRepository->save($shippingAddress);
+                if ($this->validateAddress((int) $shippingAddress->getId()) === true) {
+                    $this->addressRepository->save($shippingAddress);
+                }
                 $quote->addCustomerAddress($shippingAddress);
                 $shipping->setCustomerAddressData($shippingAddress);
                 $this->addressesToSync[] = $shippingAddress->getId();
@@ -728,7 +740,9 @@ protected function _prepareCustomerQuote($quote)
                     $billingAddress->setIsDefaultBilling(true);
                 }
                 $billingAddress->setCustomerId($quote->getCustomerId());
-                $this->addressRepository->save($billingAddress);
+                if ($this->validateAddress((int) $billingAddress->getId()) === true) {
+                    $this->addressRepository->save($billingAddress);
+                }
                 $quote->addCustomerAddress($billingAddress);
                 $billing->setCustomerAddressData($billingAddress);
                 $this->addressesToSync[] = $billingAddress->getId();
@@ -791,4 +805,17 @@ public function _resetState(): void
     {
         $this->addressesToSync = [];
     }
+
+    /**
+     * Validate address
+     *
+     * @param int|null $addressId
+     * @return array|bool
+     * @throws NoSuchEntityException
+     */
+    private function validateAddress(?int $addressId): bool|array
+    {
+        $addressModel = $this->addressRegistry->retrieve($addressId);
+        return $addressModel->validate();
+    }
 }