ACP2E-790: [Magento Cloud] - Long value for custom options breaks order

arnobsh · arnobsh · commit 0cd49e8c4aa3 · 2022-07-07T12:52:17.000-05:00
- with test
diff --git a/app/code/Magento/Catalog/Test/Fixture/Product.php b/app/code/Magento/Catalog/Test/Fixture/Product.php
@@ -7,12 +7,14 @@
 
 namespace Magento\Catalog\Test\Fixture;
 
+use Magento\Catalog\Api\Data\ProductCustomOptionInterface;
 use Magento\Catalog\Api\Data\ProductInterface;
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\Catalog\Model\Product\Attribute\Source\Status;
 use Magento\Catalog\Model\Product\Type;
 use Magento\Catalog\Model\Product\Visibility;
 use Magento\Framework\DataObject;
+use Magento\Framework\Exception\NoSuchEntityException;
 use Magento\TestFramework\Fixture\Api\DataMerger;
 use Magento\TestFramework\Fixture\Api\ServiceFactory;
 use Magento\TestFramework\Fixture\RevertibleDataFixtureInterface;
@@ -137,6 +139,7 @@ private function prepareData(array $data): array
         }
 
         $data['product_links'] = $this->prepareLinksData($data);
+        $data['options'] = $this->prepareOptions($data);
 
         return $this->dataProcessor->process($this, $data);
     }
@@ -146,6 +149,7 @@ private function prepareData(array $data): array
      *
      * @param array $data
      * @return array
+     * @throws NoSuchEntityException
      */
     private function prepareLinksData(array $data): array
     {
@@ -181,4 +185,33 @@ private function prepareLinksData(array $data): array
 
         return $links;
     }
+
+    /**
+     *
+     * Prepare custom option fixtures
+     *
+     * @param array $data
+     * @return array
+     */
+    private function prepareOptions(array $data): array
+    {
+        $options = [];
+        $default = [
+            'product_sku' => $data['sku'],
+            'title' => 'customoption%uniqid%',
+            'type' => ProductCustomOptionInterface::OPTION_TYPE_FIELD,
+            'is_require' => true,
+            'price' => 10.0,
+            'price_type' => 'fixed',
+            'sku' => 'customoption%uniqid%',
+            'max_characters' => null,
+            'values' => null,
+        ];
+        $sortOrder = 1;
+        foreach ($data['options'] as $item) {
+            $options[] = $item + ['sort_order' => $sortOrder++] + $default;
+        }
+
+        return $options;
+    }
 }
diff --git a/app/code/Magento/Sales/etc/db_schema.xml b/app/code/Magento/Sales/etc/db_schema.xml
@@ -475,7 +475,7 @@
         <column xsi:type="int" name="product_id" unsigned="true" nullable="true" identity="false"
                 comment="Product ID"/>
         <column xsi:type="varchar" name="product_type" nullable="true" length="255" comment="Product Type"/>
-        <column xsi:type="text" name="product_options" nullable="true" comment="Product Options"/>
+        <column xsi:type="longtext" name="product_options" nullable="true" comment="Product Options"/>
         <column xsi:type="decimal" name="weight" scale="4" precision="12" unsigned="false" nullable="true" default="0"
                 comment="Weight"/>
         <column xsi:type="smallint" name="is_virtual" unsigned="true" nullable="true" identity="false"
diff --git a/dev/tests/integration/testsuite/Magento/Sales/Model/ResourceModel/OrderTest.php b/dev/tests/integration/testsuite/Magento/Sales/Model/ResourceModel/OrderTest.php
@@ -8,7 +8,12 @@
 
 namespace Magento\Sales\Model\ResourceModel;
 
+use Magento\Catalog\Api\ProductRepositoryInterface;
+use Magento\Catalog\Model\Product\Type;
+use Magento\Catalog\Test\Fixture\Product as ProductFixture;
 use Magento\Framework\Event\ManagerInterface;
+use Magento\Framework\Exception\AlreadyExistsException;
+use Magento\Framework\Exception\NoSuchEntityException;
 use Magento\Framework\ObjectManagerInterface;
 use Magento\Framework\Registry;
 use Magento\Sales\Api\OrderRepositoryInterface;
@@ -19,6 +24,7 @@
 use Magento\Sales\Model\ResourceModel\Order\CollectionFactory as OrderCollectionFactory;
 use Magento\Store\Api\StoreRepositoryInterface;
 use Magento\Store\Model\StoreManagerInterface;
+use Magento\TestFramework\Fixture\DataFixture;
 use Magento\TestFramework\Helper\Bootstrap;
 use PHPUnit\Framework\TestCase;
 
@@ -54,6 +60,9 @@ class OrderTest extends TestCase
      */
     private $storeRepository;
 
+    /** @var ProductRepositoryInterface */
+    private $productRepository;
+
     /**
      * @inheritdoc
      */
@@ -65,6 +74,8 @@ protected function setUp(): void
         $this->orderIncrementId = '100000001';
         $this->storeManager = $this->objectManager->get(StoreManagerInterface::class);
         $this->storeRepository = $this->objectManager->get(StoreRepositoryInterface::class);
+        $this->productRepository = $this->objectManager->get(ProductRepositoryInterface::class);
+        $this->productRepository->cleanCache();
     }
 
     /**
@@ -93,6 +104,7 @@ protected function tearDown(): void
      *
      * @magentoDataFixture Magento/Catalog/_files/product_simple.php
      * @return void
+     * @throws AlreadyExistsException
      */
     public function testSaveOrder(): void
     {
@@ -151,12 +163,117 @@ public function testSaveOrder(): void
         $this->assertNotNull($order->getUpdatedAt());
     }
 
+    /**
+     * Test save order with custom option text field
+     *
+     * @return void
+     * @throws AlreadyExistsException|NoSuchEntityException
+     */
+    #[
+        DataFixture(
+            ProductFixture::class,
+            [
+                'sku' => 'simple1',
+                'options' => [
+                    [
+                        'type' => 'field'
+                    ]
+                ],
+                'price' => 10
+            ],
+            'p1'
+        ),
+    ]
+    public function testSaveOrderWithCustomOptionTextField(): void
+    {
+        $addressData = [
+            'region' => 'CA',
+            'postcode' => '11111',
+            'lastname' => 'lastname',
+            'firstname' => 'firstname',
+            'street' => 'street',
+            'city' => 'Los Angeles',
+            'email' => 'admin@example.com',
+            'telephone' => '11111111',
+            'country_id' => 'US'
+        ];
+
+        $billingAddress = $this->objectManager->create(Address::class, ['data' => $addressData]);
+        $billingAddress->setAddressType('billing');
+
+        $shippingAddress = clone $billingAddress;
+        $shippingAddress->setId(null)->setAddressType('shipping');
+
+        $payment = $this->objectManager->create(Payment::class);
+        $payment->setMethod('checkmo');
+        $buyRequest = [
+            'qty' => 2,
+            'options' => [
+                $this->getLongTextOption()
+            ]
+        ];
+
+        $product = $this->productRepository->get('simple1');
+
+        $orderItemRepository = $this->objectManager->create(\Magento\Sales\Api\OrderItemRepositoryInterface::class);
+
+        /** @var Item $orderItem */
+        $orderItem = $this->objectManager->create(Item::class);
+        $orderItem->setProductId($product->getId())
+            ->setQtyOrdered(2)
+            ->setBasePrice($product->getPrice())
+            ->setPrice($product->getPrice())
+            ->setRowTotal(20)
+            ->setProductType(Type::TYPE_SIMPLE)
+            ->setProductOptions(['info_buyRequest' => $buyRequest]);
+
+        /** @var OrderModel $order */
+        $order = $this->objectManager->create(OrderModel::class);
+        $order->setIncrementId($this->orderIncrementId)
+            ->setState(OrderModel::STATE_PROCESSING)
+            ->setStatus($order->getConfig()->getStateDefaultStatus(OrderModel::STATE_PROCESSING))
+            ->setSubtotal(100)
+            ->setBaseSubtotal(100)
+            ->setBaseGrandTotal(100)
+            ->setCustomerIsGuest(true)
+            ->setCustomerEmail('customer@null.com')
+            ->setBillingAddress($billingAddress)
+            ->setShippingAddress($shippingAddress)
+            ->setStoreId(
+                $this->objectManager
+                    ->get(StoreManagerInterface::class)
+                    ->getStore()
+                    ->getId()
+            )
+            ->addItem($orderItem)
+            ->setPayment($payment);
+
+        $this->resourceModel->save($order);
+        $this->assertNotNull($order->getCreatedAt());
+        $this->assertNotNull($order->getUpdatedAt());
+        $orderRepository = $this->objectManager->create(OrderRepositoryInterface::class);
+        $order = $orderRepository->get($order->getId());
+        foreach ($order->getItems() as $item) {
+            if ($item->getProductType() === Type::TYPE_SIMPLE) {
+                $orderItem = $orderItemRepository->get($item->getItemId());
+                $productOptions = $orderItem->getProductOptions();
+                $infoBuyRequest = $productOptions['info_buyRequest'];
+                $this->assertNotEmpty($infoBuyRequest);
+                $options = current($infoBuyRequest['options']);
+                $this->assertNotEmpty($options);
+                $this->assertEquals(strlen($options), strlen($this->getLongTextOption()));
+            }
+        }
+    }
+
     /**
      * Check that store name and x_forwarded_for with length within 255 chars can be saved in table sales_order
      *
      * @magentoDataFixture Magento/Store/_files/store_with_long_name.php
      * @magentoDbIsolation disabled
      * @return void
+     * @throws AlreadyExistsException
+     * @throws NoSuchEntityException
      */
     public function testSaveLongNames(): void
     {
@@ -185,4 +302,16 @@ public function testSaveLongNames(): void
         $this->assertStringContainsString($store->getWebsite()->getName(), $order->getStoreName());
         $this->assertStringContainsString($store->getGroup()->getName(), $order->getStoreName());
     }
+
+    /**
+     * Get long text for option
+     */
+    private function getLongTextOption(): string
+    {
+        // phpcs:disable
+        return <<< HEREDOC
+            Lorem ipsum dolor sit amet, consectetuer adipiscing elit. Aenean commodo ligula eget dolor. Aenean massa. Cum sociis natoque penatibus et magnis dis parturient montes, nascetur ridiculus mus. Donec quam felis, ultricies nec, pellentesque eu, pretium quis, sem. Nulla consequat massa quis enim. Donec pede justo, fringilla vel, aliquet nec, vulputate eget, arcu. In enim justo, rhoncus ut, imperdiet a, venenatis vitae, justo. Nullam dictum felis eu pede mollis pretium. Integer tincidunt. Cras dapibus. Vivamus elementum semper nisi. Aenean vulputate eleifend tellus. Aenean leo ligula, porttitor eu, consequat vitae, eleifend ac, enim. Aliquam lorem ante, dapibus in, viverra quis, feugiat a, tellus. Phasellus viverra nulla ut metus varius laoreet. Quisque rutrum. Aenean imperdiet. Etiam ultricies nisi vel augue. Curabitur ullamcorper ultricies nisi. Nam eget dui. Etiam rhoncus. Maecenas tempus, tellus eget condimentum rhoncus, sem quam semper libero, sit amet adipiscing sem neque sed ipsum. Nam quam nunc, blandit vel, luctus pulvinar, hendrerit id, lorem. Maecenas nec odio et ante tincidunt tempus. Donec vitae sapien ut libero venenatis faucibus. Nullam quis ante. Etiam sit amet orci eget eros faucibus tincidunt. Duis leo. Sed fringilla mauris sit amet nibh. Donec sodales sagittis magna. Sed consequat, leo eget bibendum sodales, augue velit cursus nunc, quis gravida magna mi a libero. Fusce vulputate eleifend sapien. Vestibulum purus quam, scelerisque ut, mollis sed, nonummy id, metus. Nullam accumsan lorem in dui. Cras ultricies mi eu turpis hendrerit fringilla. Vestibulum ante ipsum primis in faucibus orci luctus et ultrices posuere cubilia Curae; In ac dui quis mi consectetuer lacinia. Nam pretium turpis et arcu. Duis arcu tortor, suscipit eget, imperdiet nec, imperdiet iaculis, ipsum. Sed aliquam ultrices mauris. Integer ante arcu, accumsan a, consectetuer eget, posuere ut, mauris. Praesent adipiscing. Phasellus ullamcorper ipsum rutrum nunc. Nunc nonummy metus. Vestibulum volutpat pretium libero. Cras id dui. Aenean ut eros et nisl sagittis vestibulum. Nullam nulla eros, ultricies sit amet, nonummy id, imperdiet feugiat, pede. Sed lectus. Donec mollis hendrerit risus. Phasellus nec sem in justo pellentesque facilisis. Etiam imperdiet imperdiet orci. Nunc nec neque. Phasellus leo dolor, tempus non, auctor et, hendrerit quis, nisi. Curabitur ligula sapien, tincidunt non, euismod vitae, posuere imperdiet, leo. Maecenas malesuada. Praesent congue erat at massa. Sed cursus turpis vitae tortor. Donec posuere vulputate arcu. Phasellus accumsan cursus velit. Vestibulum ante ipsum primis in faucibus orci luctus et ultrices posuere cubilia Curae; Sed aliquam, nisi quis porttitor congue, elit erat euismod orci, ac placerat dolor lectus quis orci. Phasellus consectetuer vestibulum elit. Aenean tellus metus, bibendum sed, posuere ac, mattis non, nunc. Vestibulum fringilla pede sit amet augue. In turpis. Pellentesque posuere. Praesent turpis. Aenean posuere, tortor sed cursus feugiat, nunc augue blandit nunc, eu sollicitudin urna dolor sagittis lacus. Donec elit libero, sodales nec, volutpat a, suscipit non, turpis. Nullam sagittis. Suspendisse pulvinar, augue ac venenatis condimentum, sem libero volutpat nibh, nec pellentesque velit pede quis nunc. Vestibulum ante ipsum primis in faucibus orci luctus et ultrices posuere cubilia Curae; Fusce id purus. Ut varius tincidunt libero. Phasellus dolor. Maecenas vestibulum mollis diam. Pellentesque ut neque. Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. In dui magna, posuere eget, vestibulum et, tempor auctor, justo. In ac felis quis tortor malesuada pretium. Pellentesque auctor neque nec urna. Proin sapien ipsum, porta a, auctor quis, euismod ut, mi. Aenean viverra rhoncus pede. Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Ut non enim eleifend felis pretium feugiat. Vivamus quis mi. Phasellus a est. Phasellus magna. In hac habitasse platea dictumst. Curabitur at lacus ac velit ornare lobortis. Curabitur a felis in nunc fringilla tristique. Morbi mattis ullamcorper velit. Phasellus gravida semper nisi. Nullam vel sem. Pellentesque libero tortor, tincidunt et, tincidunt eget, semper nec, quam. Sed hendrerit. Morbi ac felis. Nunc egestas, augue at pellentesque laoreet, felis eros vehicula leo, at malesuada velit leo quis pede. Donec interdum, metus et hendrerit aliquet, dolor diam sagittis ligula, eget egestas libero turpis vel mi. Nunc nulla. Fusce risus nisl, viverra et, tempor et, pretium in, sapien. Donec venenatis vulputate lorem. Morbi nec metus. Phasellus blandit leo ut odio. Maecenas ullamcorper, dui et placerat feugiat, eros pede varius nisi, condimentum viverra felis nunc et lorem. Sed magna purus, fermentum eu, tincidunt eu, varius ut, felis. In auctor lobortis lacus. Quisque libero metus, condimentum nec, tempor a, commodo mollis, magna. Vestibulum ullamcorper mauris at ligula. Fusce fermentum. Nullam cursus lacinia erat. Praesent blandit laoreet nibh. Fusce convallis metus id felis luctus adipiscing. Pellentesque egestas, neque sit amet convallis pulvinar, justo nulla eleifend augue, ac auctor orci leo non est. Quisque id mi. Ut tincidunt tincidunt erat. Etiam feugiat lorem non metus. Vestibulum dapibus nunc ac augue. Curabitur vestibulum aliquam leo. Praesent egestas neque eu enim. In hac habitasse platea dictumst. Fusce a quam. Etiam ut purus mattis mauris sodales aliquam. Curabitur nisi. Quisque malesuada placerat nisl. Nam ipsum risus, rutrum vitae, vestibulum eu, molestie vel, lacus. Sed augue ipsum, egestas nec, vestibulum et, malesuada adipiscing, dui. Vestibulum facilisis, purus nec pulvinar iaculis, ligula mi congue nunc, vitae euismod ligula urna in dolor. Mauris sollicitudin fermentum libero. Praesent nonummy mi in odio. Nunc interdum lacus sit amet orci. Vestibulum rutrum, mi nec elementum vehicula, eros quam gravida nisl, id fringilla neque ante vel mi. Morbi mollis tellus ac sapien. Phasellus volutpat, metus eget egestas mollis, lacus lacus blandit dui, id egestas quam mauris ut lacus. Fusce vel dui. Sed in libero ut nibh placerat accumsan. Proin faucibus arcu quis ante. In consectetuer turpis ut velit. Nulla sit amet est. Praesent metus tellus, elementum eu, semper a, adipiscing nec, purus. Cras risus ipsum, faucibus ut, ullamcorper id, varius ac, leo. Suspendisse feugiat. Suspendisse enim turpis, dictum sed, iaculis a, condimentum nec, nisi. Praesent nec nisl a purus blandit viverra. Praesent ac massa at ligula laoreet iaculis. Nulla neque dolor, sagittis eget, iaculis quis, molestie non, velit. Mauris turpis nunc, blandit et, volutpat molestie, porta ut, ligula. Fusce pharetra convallis urna. Quisque ut nisi. Donec mi odio, faucibus at, scelerisque quis, convallis in, nisi. Suspendisse non nisl sit amet velit hendrerit rutrum. Ut leo. Ut a nisl id ante tempus hendrerit. Proin pretium, leo ac pellentesque mollis, felis nunc ultrices eros, sed gravida augue augue mollis justo. Suspendisse eu ligula. Nulla facilisi. Donec id justo. Praesent porttitor, nulla vitae posuere iaculis, arcu nisl dignissim dolor, a pretium mi sem ut ipsum. Curabitur suscipit suscipit tellus. Praesent vestibulum dapibus nibh. Etiam iaculis nunc ac metus. Ut id nisl quis enim dignissim sagittis. Etiam sollicitudin, ipsum eu pulvinar rutrum, tellus ipsum laoreet sapien, quis venenatis ante odio sit amet eros. Proin magna. Duis vel nibh at velit scelerisque suscipit. Curabitur turpis. Vestibulum suscipit nulla quis orci. Fusce ac felis sit amet ligula pharetra condimentum. Maecenas egestas arcu quis ligula mattis placerat. Duis lobortis massa imperdiet quam. Suspendisse potenti. Pellentesque commodo eros a enim. Vestibulum turpis sem, aliquet eget, lobortis pellentesque, rutrum eu, nisl. Sed libero. Aliquam erat volutpat. Etiam vitae tortor. Morbi vestibulum volutpat enim. Aliquam eu nunc. Nunc sed turpis. Sed mollis, eros et ultrices tempus, mauris ipsum aliquam libero, non adipiscing dolor urna a orci. Nulla porta dolor. Class aptent taciti sociosqu ad litora torquent per conubia nostra, per inceptos hymenaeos. Pellentesque dapibus hendrerit tortor. Praesent egestas tristique nibh. Sed a libero. Cras varius. Donec vitae orci sed dolor rutrum auctor. Fusce egestas elit eget lorem. Suspendisse nisl elit, rhoncus eget, elementum ac, condimentum eget, diam. Nam at tortor in tellus interdum sagittis. Aliquam lobortis. Donec orci lectus, aliquam ut, faucibus non, euismod id, nulla. Curabitur blandit mollis lacus. Nam adipiscing. Vestibulum eu odio. Vivamus laoreet. Nullam tincidunt adipiscing enim. Phasellus tempus. Proin viverra, ligula sit amet ultrices semper, ligula arcu tristique sapien, a accumsan nisi mauris ac eros. Fusce neque. Suspendisse faucibus, nunc et pellentesque egestas, lacus ante convallis tellus, vitae iaculis lacus elit id tortor. Vivamus aliquet elit ac nisl. Fusce fermentum odio nec arcu. Vivamus euismod mauris. In ut quam vitae odio lacinia tincidunt. Praesent ut ligula non mi varius sagittis. Cras sagittis. Praesent ac sem eget est egestas volutpat. Vivamus consectetuer hendrerit lacus. Cras non dolor. Vivamus in erat ut urna cursus vestibulum. Fusce commodo aliquam arcu. Nam commodo suscipit quam. Quisque id odio. Praesent venenatis metus at tortor pulvinar varius. Lorem ipsum dolor sit amet, consectetuer adipiscing elit. Aenean commodo ligula eget dolor. Aenean massa. Cum sociis natoque penatibus et magnis dis parturient montes, nascetur ridiculus mus. Donec quam felis, ultricies nec, pellentesque eu, pretium quis, sem. Nulla consequat massa quis enim. Donec pede justo, fringilla vel, aliquet nec, vulputate eget, arcu. In enim justo, rhoncus ut, imperdiet a, venenatis vitae, justo. Nullam dictum felis eu pede mollis pretium. Integer tincidunt. Cras dapibus. Vivamus elementum semper nisi. Aenean vulputate eleifend tellus. Aenean leo ligula, porttitor eu, consequat vita
+        HEREDOC;
+        // phpcs:enable
+    }
 }
