Merge branch '2.4-develop' of https://github.com/magento-commerce/magento2ce into PR-2021-12-21

Author: chittima

app/code/Magento/Analytics/Plugin/BearerTokenValidatorPlugin.php

@@ -0,0 +1,49 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+declare(strict_types=1);
+
+namespace Magento\Analytics\Plugin;
+
+use Magento\Framework\App\Config\ScopeConfigInterface;
+use Magento\Integration\Model\Integration;
+use Magento\Integration\Model\Validator\BearerTokenValidator;
+
+/**
+ * Overrides authorization config to always allow analytics token to be used as bearer
+ */
+class BearerTokenValidatorPlugin
+{
+    /**
+     * @var ScopeConfigInterface
+     */
+    private ScopeConfigInterface $config;
+
+    /**
+     * @param ScopeConfigInterface $config
+     */
+    public function __construct(ScopeConfigInterface $config)
+    {
+        $this->config = $config;
+    }
+
+    /***
+     * Always allow access token for analytics to be used as bearer
+     *
+     * @param BearerTokenValidator $subject
+     * @param bool $result
+     * @param Integration $integration
+     * @return bool
+     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
+     */
+    public function afterIsIntegrationAllowedAsBearerToken(
+        BearerTokenValidator $subject,
+        bool $result,
+        Integration $integration
+    ): bool {
+        return $result || $integration->getName() === $this->config->getValue('analytics/integration_name');
+    }
+}

app/code/Magento/Analytics/Test/Unit/Plugin/BearerTokenValidatorPluginTest.php

@@ -0,0 +1,72 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+declare(strict_types=1);
+
+namespace Magento\Analytics\Test\Unit\Plugin;
+
+use Magento\Analytics\Plugin\BearerTokenValidatorPlugin;
+use Magento\Framework\App\Config\ScopeConfigInterface;
+use Magento\Integration\Model\Integration;
+use Magento\Integration\Model\Validator\BearerTokenValidator;
+use PHPUnit\Framework\MockObject\MockObject;
+use PHPUnit\Framework\TestCase;
+
+class BearerTokenValidatorPluginTest extends TestCase
+{
+    /**
+     * @var BearerTokenValidatorPlugin
+     */
+    private BearerTokenValidatorPlugin $plugin;
+
+    /**
+     * @var BearerTokenValidator|MockObject
+     */
+    private $validator;
+
+    public function setUp(): void
+    {
+        $config = $this->createMock(ScopeConfigInterface::class);
+        $config->method('getValue')
+            ->with('analytics/integration_name')
+            ->willReturn('abc');
+        $this->plugin = new BearerTokenValidatorPlugin($config);
+        $this->validator = $this->createMock(BearerTokenValidator::class);
+    }
+
+    public function testTrueIsPassedThrough()
+    {
+        $integration = $this->createMock(Integration::class);
+        $integration->method('__call')
+            ->with('getName')
+            ->willReturn('invalid');
+
+        $result = $this->plugin->afterIsIntegrationAllowedAsBearerToken($this->validator, true, $integration);
+        self::assertTrue($result);
+    }
+
+    public function testFalseWhenIntegrationDoesntMatch()
+    {
+        $integration = $this->createMock(Integration::class);
+        $integration->method('__call')
+            ->with('getName')
+            ->willReturn('invalid');
+
+        $result = $this->plugin->afterIsIntegrationAllowedAsBearerToken($this->validator, false, $integration);
+        self::assertFalse($result);
+    }
+
+    public function testTrueWhenIntegrationMatches()
+    {
+        $integration = $this->createMock(Integration::class);
+        $integration->method('__call')
+            ->with('getName')
+            ->willReturn('abc');
+
+        $result = $this->plugin->afterIsIntegrationAllowedAsBearerToken($this->validator, true, $integration);
+        self::assertTrue($result);
+    }
+}

app/code/Magento/Analytics/etc/di.xml

@@ -271,4 +271,7 @@
             <argument name="connectionFactory" xsi:type="object">Magento\Framework\Model\ResourceModel\Type\Db\ConnectionFactory</argument>
         </arguments>
     </type>
+    <type name="Magento\Integration\Model\Validator\BearerTokenValidator">
+        <plugin name="allow_bearer_token" type="Magento\Analytics\Plugin\BearerTokenValidatorPlugin"/>
+    </type>
 </config>

app/code/Magento/CatalogInventory/Model/StockManagement.php

@@ -14,9 +14,12 @@
 use Magento\CatalogInventory\Model\Spi\StockRegistryProviderInterface;
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\CatalogInventory\Model\ResourceModel\Stock as ResourceStock;
+use Magento\Framework\Exception\LocalizedException;
 
 /**
  * Implements a few interfaces for backward compatibility
+ *
+ * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
 class StockManagement implements StockManagementInterface, RegisterProductSaleInterface, RevertProductSaleInterface
 {
@@ -91,7 +94,8 @@ public function __construct(
      * @param string[] $items
      * @param int $websiteId
      * @return StockItemInterface[]
-     * @throws \Magento\Framework\Exception\LocalizedException
+     * @throws StockStateException
+     * @throws LocalizedException
      * @SuppressWarnings(PHPMD.CyclomaticComplexity)
      */
     public function registerProductsSale($items, $websiteId = null)
@@ -118,7 +122,7 @@ public function registerProductsSale($items, $websiteId = null)
                 && !$this->stockState->checkQty($productId, $orderedQty, $stockItem->getWebsiteId())
             ) {
                 $this->getResource()->commit();
-                throw new \Magento\Framework\Exception\LocalizedException(
+                throw new StockStateException(
                     __('Some of the products are out of stock.')
                 );
             }

app/code/Magento/CatalogInventory/Model/StockStateException.php

@@ -0,0 +1,17 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+namespace Magento\CatalogInventory\Model;
+
+use Magento\Framework\Exception\LocalizedException;
+
+/**
+ * Exception class reflecting when an operation cannot be completed due to the current stock status of an inventory item
+ *
+ * @api
+ */
+class StockStateException extends LocalizedException
+{
+}

app/code/Magento/CatalogInventory/Test/Unit/Model/StockManagementTest.php

@@ -203,7 +203,7 @@ public function testRegisterProductsSale(
      */
     public function testRegisterProductsSaleException(array $items, array $lockedItems)
     {
-        $this->expectException('Magento\Framework\Exception\LocalizedException');
+        $this->expectException('Magento\CatalogInventory\Model\StockStateException');
         $this->expectExceptionMessage('Some of the products are out of stock.');
         $this->stockResourceMock
             ->expects($this->once())

app/code/Magento/CatalogInventoryGraphQl/Helper/Error/MessageFormatters/StockStateExceptionMessageFormatter.php

@@ -0,0 +1,46 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento\CatalogInventoryGraphQl\Helper\Error\MessageFormatters;
+
+use Magento\CatalogInventory\Model\StockStateException;
+use Magento\Framework\Exception\LocalizedException;
+use Magento\Framework\GraphQl\Config\Element\Field;
+use Magento\Framework\GraphQl\Exception\GraphQlInputException;
+use Magento\Framework\GraphQl\Query\Resolver\ContextInterface;
+use Magento\Framework\GraphQl\Schema\Type\ResolveInfo;
+use Magento\GraphQl\Helper\Error\ExceptionMessageFormatterInterface;
+
+/**
+ * Check if an internally-thrown exception is from an item stock status issue and re-throw with the message intact if so
+ */
+class StockStateExceptionMessageFormatter implements ExceptionMessageFormatterInterface
+{
+    /**
+     * If the thrown exception was from an item stock status issue, allow the message to go through
+     *
+     * @param LocalizedException $e
+     * @param string $messagePrefix
+     * @param Field $field
+     * @param ContextInterface $context
+     * @param ResolveInfo $info
+     *
+     * @return GraphQlInputException|null
+     */
+    public function getFormatted(
+        LocalizedException $e,
+        string $messagePrefix,
+        Field $field,
+        ContextInterface $context,
+        ResolveInfo $info
+    ): ?GraphQlInputException {
+        if ($e instanceof StockStateException) {
+            return new GraphQlInputException(__("$messagePrefix: %message", ['message' => $e->getMessage()]), $e);
+        }
+        return null;
+    }
+}

app/code/Magento/CatalogInventoryGraphQl/composer.json

@@ -7,7 +7,8 @@
         "magento/framework": "*",
         "magento/module-store": "*",
         "magento/module-catalog": "*",
-        "magento/module-catalog-inventory": "*"
+        "magento/module-catalog-inventory": "*",
+        "magento/module-graph-ql": "*"
     },
     "license": [
         "OSL-3.0",

app/code/Magento/ConfigurableProductGraphQl/Model/Cart/BuyRequest/SuperAttributeDataProvider.php

@@ -10,6 +10,7 @@
 use Magento\Catalog\Api\Data\ProductInterface;
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\CatalogInventory\Api\StockStateInterface;
+use Magento\CatalogInventory\Model\StockStateException;
 use Magento\ConfigurableProductGraphQl\Model\Options\Collection as OptionCollection;
 use Magento\Framework\EntityManager\MetadataPool;
 use Magento\Framework\Exception\LocalizedException;
@@ -146,12 +147,13 @@ private function checkProductStock(string $sku, float $qty, int $scopeId): void
      *
      * @param string $parentSku
      * @param array $superAttributesData
+     * @throws StockStateException
      * @throws LocalizedException
      */
     private function checkSuperAttributeData(string $parentSku, array $superAttributesData): void
     {
         if (empty($superAttributesData)) {
-            throw new LocalizedException(
+            throw new StockStateException(
                 __('The product with SKU %sku is out of stock.', ['sku' => $parentSku])
             );
         }

app/code/Magento/GraphQl/Helper/Error/AggregateExceptionMessageFormatter.php

@@ -0,0 +1,64 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento\GraphQl\Helper\Error;
+
+use GraphQL\Error\ClientAware;
+use Magento\Framework\Exception\LocalizedException;
+use Magento\Framework\GraphQl\Config\Element\Field;
+use Magento\Framework\GraphQl\Exception\GraphQlInputException;
+use Magento\Framework\GraphQl\Query\Resolver\ContextInterface;
+use Magento\Framework\GraphQl\Schema\Type\ResolveInfo;
+use Magento\Framework\Phrase;
+
+/**
+ * Class for formatting internally-thrown errors if they match allowed exception types or using a default message if not
+ */
+class AggregateExceptionMessageFormatter
+{
+    /**
+     * @var ExceptionMessageFormatterInterface[]
+     */
+    private $messageFormatters;
+
+    /**
+     * @param ExceptionMessageFormatterInterface[] $messageFormatters
+     */
+    public function __construct(array $messageFormatters)
+    {
+        $this->messageFormatters = $messageFormatters;
+    }
+
+    /**
+     * Format a thrown exception message if it matches one of the supplied formatters, otherwise use a default message
+     *
+     * @param LocalizedException $e
+     * @param Phrase $defaultMessage
+     * @param string $messagePrefix
+     * @param Field $field
+     * @param ContextInterface $context
+     * @param ResolveInfo $info
+     *
+     * @return ClientAware
+     */
+    public function getFormatted(
+        LocalizedException $e,
+        Phrase $defaultMessage,
+        string $messagePrefix,
+        Field $field,
+        ContextInterface $context,
+        ResolveInfo $info
+    ): ClientAware {
+        foreach ($this->messageFormatters as $formatter) {
+            $formatted = $formatter->getFormatted($e, $messagePrefix, $field, $context, $info);
+            if ($formatted) {
+                return $formatted;
+            }
+        }
+        return new GraphQlInputException($defaultMessage, $e);
+    }
+}