Skip to content

Commit 1f96b87

Browse files
[26054-Do not duplicate SEO meta data when duplicating a product]
1 parent 90a479f commit 1f96b87

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

app/code/Magento/Catalog/Model/Product/Copier.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,9 @@ public function copy(Product $product)
7878
$productData = $this->removeStockItem($productData);
7979
$duplicate->setData($productData);
8080
$duplicate->setOptions([]);
81+
$duplicate->setMetaTitle(null);
82+
$duplicate->setMetaKeyword(null);
83+
$duplicate->setMetaDescription(null);
8184
$duplicate->setIsDuplicate(true);
8285
$duplicate->setOriginalLinkId($product->getData($metadata->getLinkField()));
8386
$duplicate->setStatus(\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_DISABLED);
@@ -89,10 +92,6 @@ public function copy(Product $product)
8992
$this->setDefaultUrl($product, $duplicate);
9093
$this->setStoresUrl($product, $duplicate);
9194
$this->getOptionRepository()->duplicate($product, $duplicate);
92-
$product->getResource()->duplicate(
93-
$product->getData($metadata->getLinkField()),
94-
$duplicate->getData($metadata->getLinkField())
95-
);
9695
return $duplicate;
9796
}
9897

app/code/Magento/Catalog/Test/Unit/Model/Product/CopierTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -197,6 +197,9 @@ public function testCopy()
197197
$duplicateMock->expects($this->once())->method('setCreatedAt')->with(null);
198198
$duplicateMock->expects($this->once())->method('setUpdatedAt')->with(null);
199199
$duplicateMock->expects($this->once())->method('setId')->with(null);
200+
$duplicateMock->expects($this->once())->method('setMetaTitle')->with(null);
201+
$duplicateMock->expects($this->once())->method('setMetaKeyword')->with(null);
202+
$duplicateMock->expects($this->once())->method('setMetaDescription')->with(null);
200203
$duplicateMock->expects($this->atLeastOnce())->method('getStoreIds')->willReturn([]);
201204
$duplicateMock->expects($this->atLeastOnce())->method('setData')->willReturn($duplicateMock);
202205
$this->copyConstructorMock->expects($this->once())->method('build')->with($this->productMock, $duplicateMock);

0 commit comments

Comments
 (0)