Skip to content

Commit 8491be6

Browse files
committed
Refactor helper to have less changes
1 parent 41fbd60 commit 8491be6

File tree

1 file changed

+10
-7
lines changed
  • app/code/Magento/Catalog/Controller/Adminhtml/Product/Initialization

1 file changed

+10
-7
lines changed

app/code/Magento/Catalog/Controller/Adminhtml/Product/Initialization/Helper.php

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ class Helper
7474
* @var \Magento\Framework\Stdlib\DateTime\Filter\DateTime
7575
*/
7676
private $dateTimeFilter;
77-
77+
7878
/**
7979
* @var \Magento\Catalog\Model\Product\LinkTypeProvider
8080
*/
@@ -253,13 +253,16 @@ protected function setProductLinks(\Magento\Catalog\Model\Product $product)
253253

254254
$product = $this->productLinks->initializeLinks($product, $links);
255255
$productLinks = $product->getProductLinks();
256+
$linkTypes = [];
256257

257-
/** @var \Magento\Catalog\Api\Data\ProductLinkTypeInterface $linkType */
258-
foreach ($this->linkTypeProvider->getItems() as $linkType) {
259-
$readonly = $product->getData($linkType->getName() . '_readonly');
258+
/** @var \Magento\Catalog\Api\Data\ProductLinkTypeInterface $linkTypeObject */
259+
foreach ($this->linkTypeProvider->getItems() as $linkTypeObject) {
260+
$linkTypes[$linkTypeObject->getName()] = $product->getData($linkTypeObject->getName() . '_readonly');
261+
}
260262

261-
if (isset($links[$linkType->getName()]) && !$readonly) {
262-
foreach ((array) $links[$linkType->getName()] as $linkData) {
263+
foreach ($linkTypes as $linkType => $readonly) {
264+
if (isset($links[$linkType]) && !$readonly) {
265+
foreach ((array)$links[$linkType] as $linkData) {
263266
if (empty($linkData['id'])) {
264267
continue;
265268
}
@@ -268,7 +271,7 @@ protected function setProductLinks(\Magento\Catalog\Model\Product $product)
268271
$link = $this->getProductLinkFactory()->create();
269272
$link->setSku($product->getSku())
270273
->setLinkedProductSku($linkProduct->getSku())
271-
->setLinkType($linkType->getName())
274+
->setLinkType($linkType)
272275
->setPosition(isset($linkData['position']) ? (int)$linkData['position'] : 0);
273276
$productLinks[] = $link;
274277
}

0 commit comments

Comments
 (0)