#622: No possibility to update customizable_options in updateCartItem…

…s mutation - code style + minor fixes
magento · Jun 24, 2019 · d3734f8 · d3734f8
1 parent 1cf357b
commit d3734f8
Show file tree

Hide file tree

Showing 5 changed files with 22 additions and 48 deletions.
diff --git a/app/code/Magento/QuoteGraphQl/Model/Cart/AddSimpleProductToCart.php b/app/code/Magento/QuoteGraphQl/Model/Cart/AddSimpleProductToCart.php
@@ -114,27 +114,4 @@ private function extractQuantity(array $cartItemData): float
         }
         return $quantity;
     }
-
-    /**
-     * Extract Customizable Options from cart item data
-     *
-     * @param array $cartItemData
-     * @return array
-     */
-    private function extractCustomizableOptions(array $cartItemData): array
-    {
-        if (!isset($cartItemData['customizable_options']) || empty($cartItemData['customizable_options'])) {
-            return [];
-        }
-
-        $customizableOptionsData = [];
-        foreach ($cartItemData['customizable_options'] as $customizableOption) {
-            if (isset($customizableOption['value_string'])) {
-                $customizableOptionsData[$customizableOption['id']] = $this->convertCustomOptionValue(
-                    $customizableOption['value_string']
-                );
-            }
-        }
-        return $customizableOptionsData;
-    }
 }
diff --git a/app/code/Magento/QuoteGraphQl/Model/Cart/CreateBuyRequest.php b/app/code/Magento/QuoteGraphQl/Model/Cart/CreateBuyRequest.php
@@ -47,12 +47,14 @@ public function execute(float $qty, array $customizableOptionsData): DataObject
             }
         }
 
-        return $this->dataObjectFactory->create([
-            'data' => [
-                'qty' => $qty,
-                'options' => $customizableOptions,
-            ],
-        ]);
+        return $this->dataObjectFactory->create(
+            [
+                'data' => [
+                    'qty' => $qty,
+                    'options' => $customizableOptions,
+                ],
+            ]
+        );
     }
 
     /**

diff --git a/app/code/Magento/QuoteGraphQl/Model/Cart/UpdateCartItem.php b/app/code/Magento/QuoteGraphQl/Model/Cart/UpdateCartItem.php
@@ -18,7 +18,6 @@
 
 /**
  * Update cart item
- *
  */
 class UpdateCartItem
 {
@@ -86,18 +85,13 @@ public function execute(Quote $cart, int $cartItemId, float $quantity, array $cu
             );
         }
 
-        if (is_string($result)) {
-            throw new GraphQlInputException(__(
-                'Could not update cart item: %message',
-                ['message' => $result]
-            ));
-        }
-
         if ($result->getHasError()) {
-            throw new GraphQlInputException(__(
-                'Could not update cart item: %message',
-                ['message' => $result->getMessage(true)]
-            ));
+            throw new GraphQlInputException(
+                __(
+                    'Could not update cart item: %message',
+                    ['message' => $result->getMessage()]
+                )
+            );
         }
 
         $this->quoteRepository->save($cart);

diff --git a/...sts/api-functional/testsuite/Magento/GraphQl/Quote/EditQuoteItemWithCustomOptionsTest.php b/...sts/api-functional/testsuite/Magento/GraphQl/Quote/EditQuoteItemWithCustomOptionsTest.php
@@ -240,7 +240,6 @@ private function getQuoteItemIdBySku(string $sku): int
         return (int)$quoteItem->getId();
     }
 
-
     /**
      * Generate an array with test values for customizable options
      * based on the option type

diff --git a/...ts/integration/testsuite/Magento/GraphQl/Quote/_files/add_simple_product_with_options.php b/...ts/integration/testsuite/Magento/GraphQl/Quote/_files/add_simple_product_with_options.php
@@ -38,12 +38,14 @@
     $cartItemCustomOptions[$productOption->getId()] = 'initial value';
 }
 
-$request = $dataObjectFactory->create([
-    'data' => [
-        'qty' => 1.0,
-        'options' => $cartItemCustomOptions,
-    ],
-]);
+$request = $dataObjectFactory->create(
+    [
+        'data' => [
+            'qty' => 1.0,
+            'options' => $cartItemCustomOptions,
+        ],
+    ]
+);
 
 $quote = $quoteFactory->create();
 $quoteResource->load($quote, 'test_quote', 'reserved_order_id');