File tree Expand file tree Collapse file tree 2 files changed +9
-12
lines changed
app/code/Magento/CatalogUrlRewrite
Model/Category/Plugin/Category
Test/Unit/Model/Category/Plugin/Category Expand file tree Collapse file tree 2 files changed +9
-12
lines changed Original file line number Diff line number Diff 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
Original file line number Diff line number Diff 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 (
You can’t perform that action at this time.
0 commit comments