Skip to content

Commit c643d79

Browse files
author
Viktor Kopin
committed
MC-37665: Updating a category through the REST API will uncheck "Use Default Value" on a bunch of attributes
1 parent eaa08ee commit c643d79

File tree

1 file changed

+17
-29
lines changed

1 file changed

+17
-29
lines changed

dev/tests/api-functional/testsuite/Magento/Catalog/Api/CategoryRepositoryTest.php

Lines changed: 17 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
* Copyright © Magento, Inc. All rights reserved.
55
* See COPYING.txt for license details.
66
*/
7+
78
namespace Magento\Catalog\Api;
89

910
use Magento\Authorization\Model\Role;
@@ -157,7 +158,7 @@ public function testDelete()
157158
UrlRewrite::ENTITY_ID => $categoryId,
158159
UrlRewrite::ENTITY_TYPE => CategoryUrlRewriteGenerator::ENTITY_TYPE
159160
];
160-
/** @var \Magento\UrlRewrite\Service\V1\Data\UrlRewrite $urlRewrite*/
161+
/** @var \Magento\UrlRewrite\Service\V1\Data\UrlRewrite $urlRewrite */
161162
$urlRewrite = $storage->findOneByData($data);
162163

163164
// Assert that a url rewrite is auto-generated for the category created from the data fixture
@@ -432,14 +433,9 @@ protected function updateCategory($id, $data, ?string $token = null)
432433
if ($token) {
433434
$serviceInfo['rest']['token'] = $serviceInfo['soap']['token'] = $token;
434435
}
436+
$data['id'] = $id;
435437

436-
if (TESTS_WEB_API_ADAPTER == self::ADAPTER_SOAP) {
437-
$data['id'] = $id;
438-
return $this->_webApiCall($serviceInfo, ['id' => $id, 'category' => $data]);
439-
} else {
440-
$data['id'] = $id;
441-
return $this->_webApiCall($serviceInfo, ['id' => $id, 'category' => $data]);
442-
}
438+
return $this->_webApiCall($serviceInfo, ['id' => $id, 'category' => $data]);
443439
}
444440

445441
/**
@@ -619,31 +615,23 @@ protected function updateCategoryForSpecificStore(
619615
?string $token = null,
620616
string $storeCode = 'default'
621617
) {
622-
$serviceInfo =
623-
[
624-
'rest' => [
625-
'resourcePath' => self::RESOURCE_PATH . '/' . $id,
626-
'httpMethod' => \Magento\Framework\Webapi\Rest\Request::HTTP_METHOD_PUT,
627-
],
628-
'soap' => [
629-
'service' => self::SERVICE_NAME,
630-
'serviceVersion' => 'V1',
631-
'operation' => self::SERVICE_NAME . 'Save',
632-
],
633-
];
618+
$serviceInfo = [
619+
'rest' => [
620+
'resourcePath' => self::RESOURCE_PATH . '/' . $id,
621+
'httpMethod' => \Magento\Framework\Webapi\Rest\Request::HTTP_METHOD_PUT,
622+
],
623+
'soap' => [
624+
'service' => self::SERVICE_NAME,
625+
'serviceVersion' => 'V1',
626+
'operation' => self::SERVICE_NAME . 'Save',
627+
],
628+
];
634629
if ($token) {
635630
$serviceInfo['rest']['token'] = $serviceInfo['soap']['token'] = $token;
636631
}
632+
$data['id'] = $id;
637633

638-
if (TESTS_WEB_API_ADAPTER == self::ADAPTER_SOAP) {
639-
$data['id'] = $id;
640-
641-
return $this->_webApiCall($serviceInfo, ['id' => $id, 'category' => $data], null, $storeCode);
642-
} else {
643-
$data['id'] = $id;
644-
645-
return $this->_webApiCall($serviceInfo, ['id' => $id, 'category' => $data], null, $storeCode);
646-
}
634+
return $this->_webApiCall($serviceInfo, ['id' => $id, 'category' => $data], null, $storeCode);
647635
}
648636

649637
/**

0 commit comments

Comments
 (0)