Skip to content

Commit 2243b48

Browse files
author
Serhii Balko
committed
Merge remote-tracking branch 'origin/MC-40371' into 2.4-develop-pr52
2 parents 2b943ed + dfe5af4 commit 2243b48

File tree

2 files changed

+9
-12
lines changed
  • app/code/Magento/CatalogUrlRewrite

2 files changed

+9
-12
lines changed

app/code/Magento/CatalogUrlRewrite/Model/Category/Plugin/Category/Move.php

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -67,13 +67,11 @@ public function afterChangeParent(
6767
$categoryStoreId = $category->getStoreId();
6868
foreach ($category->getStoreIds() as $storeId) {
6969
$category->setStoreId($storeId);
70-
if (!$this->isGlobalScope($storeId)) {
71-
$this->updateCategoryUrlKeyForStore($category);
72-
$category->unsUrlPath();
73-
$category->setUrlPath($this->categoryUrlPathGenerator->getUrlPath($category));
74-
$category->getResource()->saveAttribute($category, 'url_path');
75-
$this->updateUrlPathForChildren($category);
76-
}
70+
$this->updateCategoryUrlKeyForStore($category);
71+
$category->unsUrlPath();
72+
$category->setUrlPath($this->categoryUrlPathGenerator->getUrlPath($category));
73+
$category->getResource()->saveAttribute($category, 'url_path');
74+
$this->updateUrlPathForChildren($category);
7775
}
7876
$category->setStoreId($categoryStoreId);
7977

app/code/Magento/CatalogUrlRewrite/Test/Unit/Model/Category/Plugin/Category/MoveTest.php

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -91,27 +91,26 @@ protected function setUp(): void
9191
public function testAfterChangeParent()
9292
{
9393
$urlPath = 'test/path';
94-
$storeIds = [1];
94+
$storeIds = [0, 1];
9595
$originalCategory = $this->getMockBuilder(Category::class)
9696
->disableOriginalConstructor()
9797
->getMock();
9898
$this->categoryFactory->method('create')
9999
->willReturn($originalCategory);
100-
101100
$this->categoryMock->method('getResource')
102101
->willReturn($this->subjectMock);
103102
$this->categoryMock->expects($this->once())
104103
->method('getStoreIds')
105104
->willReturn($storeIds);
106-
$this->childrenCategoriesProviderMock->expects($this->once())
105+
$this->childrenCategoriesProviderMock->expects($this->exactly(2))
107106
->method('getChildren')
108107
->with($this->categoryMock, true)
109108
->willReturn([]);
110-
$this->categoryUrlPathGeneratorMock->expects($this->once())
109+
$this->categoryUrlPathGeneratorMock->expects($this->exactly(2))
111110
->method('getUrlPath')
112111
->with($this->categoryMock)
113112
->willReturn($urlPath);
114-
$this->categoryMock->expects($this->once())
113+
$this->categoryMock->expects($this->exactly(2))
115114
->method('setUrlPath')
116115
->with($urlPath);
117116
$this->assertSame(

0 commit comments

Comments
 (0)