MAGETWO-69667: Fix bug linked product position not updated if product link already exists #9810

Oleksii Korshenko · web-flow · commit bb98ac1b1d4c · 2017-06-12T13:34:38.000-05:00
diff --git a/app/code/Magento/CatalogImportExport/Model/Import/Product.php b/app/code/Magento/CatalogImportExport/Model/Import/Product.php
@@ -1228,15 +1228,15 @@ protected function _saveLinks()
                                         'linked_product_id' => $linkedId,
                                         'link_type_id' => $linkId,
                                     ];
-                                    if (!empty($linkPositions[$linkedKey])) {
-                                        $positionRows[] = [
-                                            'link_id' => $productLinkKeys[$linkKey],
-                                            'product_link_attribute_id' => $positionAttrId[$linkId],
-                                            'value' => $linkPositions[$linkedKey],
-                                        ];
-                                    }
-                                    $nextLinkId++;
                                 }
+                                if (!empty($linkPositions[$linkedKey])) {
+                                    $positionRows[] = [
+                                        'link_id' => $productLinkKeys[$linkKey],
+                                        'product_link_attribute_id' => $positionAttrId[$linkId],
+                                        'value' => $linkPositions[$linkedKey],
+                                    ];
+                                }
+                                $nextLinkId++;
                             }
                         }
                     }

Original file line number	Diff line number	Diff line change
`@@ -1228,15 +1228,15 @@ protected function _saveLinks()`
`1228`	`1228`	`'linked_product_id' => $linkedId,`
`1229`	`1229`	`'link_type_id' => $linkId,`
`1230`	`1230`	`];`
`1231`		`- if (!empty($linkPositions[$linkedKey])) {`
`1232`		`- $positionRows[] = [`
`1233`		`- 'link_id' => $productLinkKeys[$linkKey],`
`1234`		`- 'product_link_attribute_id' => $positionAttrId[$linkId],`
`1235`		`- 'value' => $linkPositions[$linkedKey],`
`1236`		`- ];`
`1237`		`- }`
`1238`		`- $nextLinkId++;`
`1239`	`1231`	`}`
	`1232`	`+ if (!empty($linkPositions[$linkedKey])) {`
	`1233`	`+ $positionRows[] = [`
	`1234`	`+ 'link_id' => $productLinkKeys[$linkKey],`
	`1235`	`+ 'product_link_attribute_id' => $positionAttrId[$linkId],`
	`1236`	`+ 'value' => $linkPositions[$linkedKey],`
	`1237`	`+ ];`
	`1238`	`+ }`
	`1239`	`+ $nextLinkId++;`
`1240`	`1240`	`}`
`1241`	`1241`	`}`
`1242`	`1242`	`}`