Skip to content

Commit d8ddd7e

Browse files
dkvashninbaydkvashninbay
committed
Merge remote-tracking branch 'main/develop' into MAGETWO-33534
2 parents 629bc9c + 16d73f7 commit d8ddd7e

File tree

2 files changed

+69
-23
lines changed

2 files changed

+69
-23
lines changed

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

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -586,10 +586,8 @@ public function getTypeId()
586586
*/
587587
public function getStatus()
588588
{
589-
if ($this->_getData(self::STATUS) === null) {
590-
$this->setData(self::STATUS, \Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED);
591-
}
592-
return $this->_getData(self::STATUS);
589+
$status = $this->_getData(self::STATUS);
590+
return $status !== null ? $status : \Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED;
593591
}
594592

595593
/**
@@ -974,6 +972,9 @@ public function afterDeleteCommit()
974972
*/
975973
protected function _afterLoad()
976974
{
975+
if (!$this->hasData(self::STATUS)) {
976+
$this->setData(self::STATUS, \Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED);
977+
}
977978
parent::_afterLoad();
978979
/**
979980
* Load product options
@@ -2277,8 +2278,8 @@ public function getIdentities()
22772278
$identities[] = self::CACHE_PRODUCT_CATEGORY_TAG . '_' . $categoryId;
22782279
}
22792280
}
2280-
if ($this->getOrigData('status') > $this->getData('status')) {
2281-
foreach ($this->getData('category_ids') as $categoryId) {
2281+
if ($this->getOrigData('status') != $this->getData('status')) {
2282+
foreach ($this->getCategoryIds() as $categoryId) {
22822283
$identities[] = self::CACHE_PRODUCT_CATEGORY_TAG . '_' . $categoryId;
22832284
}
22842285
}

‎app/code/Magento/Catalog/Test/Unit/Model/ProductTest.php

Lines changed: 62 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
use Magento\Catalog\Model\Product;
1212
use Magento\Framework\Api\Data\ImageContentInterface;
1313
use Magento\Framework\TestFramework\Unit\Helper\ObjectManager as ObjectManagerHelper;
14+
use Magento\Catalog\Model\Product\Attribute\Source\Status as Status;
1415

1516
/**
1617
* Product Test
@@ -156,6 +157,11 @@ class ProductTest extends \PHPUnit_Framework_TestCase
156157
*/
157158
protected $entityCollectionProviderMock;
158159

160+
/**
161+
* @var \Magento\Framework\Event\ManagerInterface|\PHPUnit_Framework_MockObject_MockObject
162+
*/
163+
protected $eventManagerMock;
164+
159165
/**
160166
* @SuppressWarnings(PHPMD.ExcessiveMethodLength)
161167
*/
@@ -203,7 +209,7 @@ public function setUp()
203209
->method('getAreaCode')
204210
->will($this->returnValue(\Magento\Backend\App\Area\FrontNameResolver::AREA_CODE));
205211

206-
$eventManagerMock = $this->getMock('Magento\Framework\Event\ManagerInterface');
212+
$this->eventManagerMock = $this->getMock('Magento\Framework\Event\ManagerInterface');
207213
$actionValidatorMock = $this->getMock(
208214
'\Magento\Framework\Model\ActionValidator\RemoveAction',
209215
[],
@@ -219,7 +225,9 @@ public function setUp()
219225
['getEventDispatcher', 'getCacheManager', 'getAppState', 'getActionValidator'], [], '', false
220226
);
221227
$contextMock->expects($this->any())->method('getAppState')->will($this->returnValue($stateMock));
222-
$contextMock->expects($this->any())->method('getEventDispatcher')->will($this->returnValue($eventManagerMock));
228+
$contextMock->expects($this->any())
229+
->method('getEventDispatcher')
230+
->will($this->returnValue($this->eventManagerMock));
223231
$contextMock->expects($this->any())
224232
->method('getCacheManager')
225233
->will($this->returnValue($cacheInterfaceMock));
@@ -415,16 +423,20 @@ public function testGetCategoryIds()
415423
$this->assertEquals([], $this->model->getCategoryIds());
416424
}
417425

426+
public function testGetStatusInitial()
427+
{
428+
$this->assertEquals(Status::STATUS_ENABLED, $this->model->getStatus());
429+
}
430+
418431
public function testGetStatus()
419432
{
420433
$this->model->setStatus(null);
421-
$expected = \Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED;
422-
$this->assertEquals($expected, $this->model->getStatus());
434+
$this->assertEquals(Status::STATUS_ENABLED, $this->model->getStatus());
423435
}
424436

425437
public function testIsInStock()
426438
{
427-
$this->model->setStatus(\Magento\Catalog\Model\Product\Attribute\Source\Status::STATUS_ENABLED);
439+
$this->model->setStatus(Status::STATUS_ENABLED);
428440
$this->assertTrue($this->model->isInStock());
429441
}
430442

@@ -521,12 +533,12 @@ public function testGetIdentities($expected, $origData, $data, $isDeleted = fals
521533
public function getIdentitiesProvider()
522534
{
523535
return [
524-
[
536+
'no changes' => [
525537
['catalog_product_1'],
526538
['id' => 1, 'name' => 'value', 'category_ids' => [1]],
527539
['id' => 1, 'name' => 'value', 'category_ids' => [1]],
528540
],
529-
[
541+
'new product' => [
530542
['catalog_product_1', 'catalog_category_product_1'],
531543
null,
532544
[
@@ -537,24 +549,57 @@ public function getIdentitiesProvider()
537549
'is_changed_categories' => true
538550
]
539551
],
540-
[
541-
[0 => 'catalog_product_1', 1 => 'catalog_category_product_1'],
552+
'status and category change' => [
553+
[0 => 'catalog_product_1', 1 => 'catalog_category_product_1', 2 => 'catalog_category_product_2'],
542554
['id' => 1, 'name' => 'value', 'category_ids' => [1], 'status' => 2],
543-
['id' => 1, 'name' => 'value', 'category_ids' => [1], 'status' => 1],
555+
[
556+
'id' => 1,
557+
'name' => 'value',
558+
'category_ids' => [2],
559+
'status' => 1,
560+
'affected_category_ids' => [1, 2],
561+
'is_changed_categories' => true
562+
],
544563
],
545-
[
546-
[0 => 'catalog_product_1'],
547-
['id' => 1, 'name' => 'value', 'category_ids' => [1], 'status' => 1],
548-
['id' => 1, 'name' => 'value', 'category_ids' => [1], 'status' => 2],
564+
'status change only' => [
565+
[0 => 'catalog_product_1', 1 => 'catalog_category_product_7'],
566+
['id' => 1, 'name' => 'value', 'category_ids' => [7], 'status' => 1],
567+
['id' => 1, 'name' => 'value', 'category_ids' => [7], 'status' => 2],
549568
],
550-
[
569+
'status changed, category unassigned' => [
570+
[0 => 'catalog_product_1', 1 => 'catalog_category_product_5'],
571+
['id' => 1, 'name' => 'value', 'category_ids' => [5], 'status' => 2],
572+
[
573+
'id' => 1,
574+
'name' => 'value',
575+
'category_ids' => [],
576+
'status' => 1,
577+
'is_changed_categories' => true,
578+
'affected_category_ids' => [5]
579+
],
580+
],
581+
'no status changes' => [
551582
[0 => 'catalog_product_1'],
552-
['id' => 1, 'name' => 'value', 'category_ids' => [1], 'status' => 2],
553-
['id' => 1, 'name' => 'value', 'category_ids' => [], 'status' => 1],
583+
['id' => 1, 'name' => 'value', 'category_ids' => [1], 'status' => 1],
584+
['id' => 1, 'name' => 'value', 'category_ids' => [1], 'status' => 1],
554585
]
555586
];
556587
}
557588

589+
public function testStatusAfterLoad()
590+
{
591+
$this->resource->expects($this->once())->method('load')->with($this->model, 1, null);
592+
$this->eventManagerMock->expects($this->exactly(4))->method('dispatch');
593+
$this->model->load(1);
594+
$this->assertEquals(
595+
Status::STATUS_ENABLED,
596+
$this->model->getData(\Magento\Catalog\Model\Product::STATUS)
597+
);
598+
$this->assertFalse($this->model->hasDataChanges());
599+
$this->model->setStatus(Status::STATUS_DISABLED);
600+
$this->assertTrue($this->model->hasDataChanges());
601+
}
602+
558603
/**
559604
* Test retrieving price Info
560605
*/

0 commit comments

Comments
 (0)