Skip to content
This repository was archived by the owner on Dec 19, 2019. It is now read-only.

Commit 929378f

Browse files
author
Stanislav Idolov
authored
ENGCOM-1391: [2.3-develop] Forwardport of magento/magento2#11323 #184 
2 parents 330a33f + 3aa0cb2 commit 929378f

File tree

3 files changed

+195
-17
lines changed

3 files changed

+195
-17
lines changed

‎app/code/Magento/Catalog/Block/Product/View/Gallery.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ public function getGalleryImagesJson()
139139
'thumb' => $image->getData('small_image_url'),
140140
'img' => $image->getData('medium_image_url'),
141141
'full' => $image->getData('large_image_url'),
142-
'caption' => $image->getData('label'),
142+
'caption' => ($image->getLabel() ?: $this->getProduct()->getName()),
143143
'position' => $image->getData('position'),
144144
'isMain' => $this->isMainImage($image),
145145
'type' => str_replace('external-', '', $image->getMediaType()),

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

+85-16
Original file line numberDiff line numberDiff line change
@@ -167,23 +167,19 @@ public function execute($product, $arguments = [])
167167
if (empty($attrData) && empty($clearImages) && empty($newImages) && empty($existImages)) {
168168
continue;
169169
}
170-
if (in_array($attrData, $clearImages)) {
171-
$product->setData($mediaAttrCode, 'no_selection');
172-
}
173-
174-
if (in_array($attrData, array_keys($newImages))) {
175-
$product->setData($mediaAttrCode, $newImages[$attrData]['new_file']);
176-
$product->setData($mediaAttrCode . '_label', $newImages[$attrData]['label']);
177-
}
178-
179-
if (in_array($attrData, array_keys($existImages)) && isset($existImages[$attrData]['label'])) {
180-
$product->setData($mediaAttrCode . '_label', $existImages[$attrData]['label']);
181-
}
182-
if (!empty($product->getData($mediaAttrCode))) {
183-
$product->addAttributeUpdate(
170+
$this->processMediaAttribute(
171+
$product,
172+
$mediaAttrCode,
173+
$clearImages,
174+
$newImages
175+
);
176+
if (in_array($mediaAttrCode, ['image', 'small_image', 'thumbnail'])) {
177+
$this->processMediaAttributeLabel(
178+
$product,
184179
$mediaAttrCode,
185-
$product->getData($mediaAttrCode),
186-
$product->getStoreId()
180+
$clearImages,
181+
$newImages,
182+
$existImages
187183
);
188184
}
189185
}
@@ -449,4 +445,77 @@ private function getMediaAttributeCodes()
449445
}
450446
return $this->mediaAttributeCodes;
451447
}
448+
449+
/**
450+
* @param \Magento\Catalog\Model\Product $product
451+
* @param $mediaAttrCode
452+
* @param array $clearImages
453+
* @param array $newImages
454+
*/
455+
private function processMediaAttribute(
456+
\Magento\Catalog\Model\Product $product,
457+
$mediaAttrCode,
458+
array $clearImages,
459+
array $newImages
460+
) {
461+
$attrData = $product->getData($mediaAttrCode);
462+
if (in_array($attrData, $clearImages)) {
463+
$product->setData($mediaAttrCode, 'no_selection');
464+
}
465+
466+
if (in_array($attrData, array_keys($newImages))) {
467+
$product->setData($mediaAttrCode, $newImages[$attrData]['new_file']);
468+
}
469+
if (!empty($product->getData($mediaAttrCode))) {
470+
$product->addAttributeUpdate(
471+
$mediaAttrCode,
472+
$product->getData($mediaAttrCode),
473+
$product->getStoreId()
474+
);
475+
}
476+
}
477+
478+
/**
479+
* @param \Magento\Catalog\Model\Product $product
480+
* @param $mediaAttrCode
481+
* @param array $clearImages
482+
* @param array $newImages
483+
* @param array $existImages
484+
*/
485+
private function processMediaAttributeLabel(
486+
\Magento\Catalog\Model\Product $product,
487+
$mediaAttrCode,
488+
array $clearImages,
489+
array $newImages,
490+
array $existImages
491+
) {
492+
$resetLabel = false;
493+
$attrData = $product->getData($mediaAttrCode);
494+
if (in_array($attrData, $clearImages)) {
495+
$product->setData($mediaAttrCode . '_label', null);
496+
$resetLabel = true;
497+
}
498+
499+
if (in_array($attrData, array_keys($newImages))) {
500+
$product->setData($mediaAttrCode . '_label', $newImages[$attrData]['label']);
501+
}
502+
503+
if (in_array($attrData, array_keys($existImages)) && isset($existImages[$attrData]['label'])) {
504+
$product->setData($mediaAttrCode . '_label', $existImages[$attrData]['label']);
505+
}
506+
507+
if ($attrData === 'no_selection' && !empty($product->getData($mediaAttrCode . '_label'))) {
508+
$product->setData($mediaAttrCode . '_label', null);
509+
$resetLabel = true;
510+
}
511+
if (!empty($product->getData($mediaAttrCode . '_label'))
512+
|| $resetLabel === true
513+
) {
514+
$product->addAttributeUpdate(
515+
$mediaAttrCode . '_label',
516+
$product->getData($mediaAttrCode . '_label'),
517+
$product->getStoreId()
518+
);
519+
}
520+
}
452521
}

‎app/code/Magento/Catalog/Test/Unit/Block/Product/View/GalleryTest.php

+109
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,89 @@ protected function setUp()
9191
]);
9292
}
9393

94+
public function testGetGalleryImagesJsonWithLabel()
95+
{
96+
$this->prepareGetGalleryImagesJsonMocks();
97+
$json = $this->model->getGalleryImagesJson();
98+
$decodedJson = json_decode($json, true);
99+
$this->assertEquals('product_page_image_small_url', $decodedJson[0]['thumb']);
100+
$this->assertEquals('product_page_image_medium_url', $decodedJson[0]['img']);
101+
$this->assertEquals('product_page_image_large_url', $decodedJson[0]['full']);
102+
$this->assertEquals('test_label', $decodedJson[0]['caption']);
103+
$this->assertEquals('2', $decodedJson[0]['position']);
104+
$this->assertEquals(false, $decodedJson[0]['isMain']);
105+
$this->assertEquals('test_media_type', $decodedJson[0]['type']);
106+
$this->assertEquals('test_video_url', $decodedJson[0]['videoUrl']);
107+
}
108+
109+
public function testGetGalleryImagesJsonWithoutLabel()
110+
{
111+
$this->prepareGetGalleryImagesJsonMocks(false);
112+
$json = $this->model->getGalleryImagesJson();
113+
$decodedJson = json_decode($json, true);
114+
$this->assertEquals('test_product_name', $decodedJson[0]['caption']);
115+
}
116+
117+
private function prepareGetGalleryImagesJsonMocks($hasLabel = true)
118+
{
119+
$storeMock = $this->getMockBuilder(\Magento\Store\Model\Store::class)
120+
->disableOriginalConstructor()
121+
->getMock();
122+
123+
$productMock = $this->getMockBuilder(\Magento\Catalog\Model\Product::class)
124+
->disableOriginalConstructor()
125+
->getMock();
126+
127+
$productTypeMock = $this->getMockBuilder(\Magento\Catalog\Model\Product\Type\AbstractType::class)
128+
->disableOriginalConstructor()
129+
->getMock();
130+
$productTypeMock->expects($this->any())
131+
->method('getStoreFilter')
132+
->with($productMock)
133+
->willReturn($storeMock);
134+
135+
$productMock->expects($this->any())
136+
->method('getTypeInstance')
137+
->willReturn($productTypeMock);
138+
$productMock->expects($this->any())
139+
->method('getMediaGalleryImages')
140+
->willReturn($this->getImagesCollectionWithPopulatedDataObject($hasLabel));
141+
$productMock->expects($this->any())
142+
->method('getName')
143+
->willReturn('test_product_name');
144+
145+
$this->registry->expects($this->any())
146+
->method('registry')
147+
->with('product')
148+
->willReturn($productMock);
149+
150+
$this->imageHelper->expects($this->any())
151+
->method('init')
152+
->willReturnMap([
153+
[$productMock, 'product_page_image_small', [], $this->imageHelper],
154+
[$productMock, 'product_page_image_medium_no_frame', [], $this->imageHelper],
155+
[$productMock, 'product_page_image_large_no_frame', [], $this->imageHelper],
156+
])
157+
->willReturnSelf();
158+
$this->imageHelper->expects($this->any())
159+
->method('setImageFile')
160+
->with('test_file')
161+
->willReturnSelf();
162+
$this->imageHelper->expects($this->at(2))
163+
->method('getUrl')
164+
->willReturn('product_page_image_small_url');
165+
$this->imageHelper->expects($this->at(5))
166+
->method('getUrl')
167+
->willReturn('product_page_image_medium_url');
168+
$this->imageHelper->expects($this->at(8))
169+
->method('getUrl')
170+
->willReturn('product_page_image_large_url');
171+
172+
$this->galleryImagesConfigMock->expects($this->exactly(2))
173+
->method('getItems')
174+
->willReturn($this->getGalleryImagesConfigItems());
175+
}
176+
94177
public function testGetGalleryImages()
95178
{
96179
$productMock = $this->createMock(Product::class);
@@ -163,4 +246,30 @@ private function getGalleryImagesConfigItems()
163246
])
164247
];
165248
}
249+
250+
/**
251+
* @return \Magento\Framework\Data\Collection
252+
*/
253+
private function getImagesCollectionWithPopulatedDataObject($hasLabel)
254+
{
255+
$collectionMock = $this->getMockBuilder(\Magento\Framework\Data\Collection::class)
256+
->disableOriginalConstructor()
257+
->getMock();
258+
259+
$items = [
260+
new \Magento\Framework\DataObject([
261+
'file' => 'test_file',
262+
'label' => ($hasLabel ? 'test_label' : ''),
263+
'position' => '2',
264+
'media_type' => 'external-test_media_type',
265+
"video_url" => 'test_video_url'
266+
]),
267+
];
268+
269+
$collectionMock->expects($this->any())
270+
->method('getIterator')
271+
->willReturn(new \ArrayIterator($items));
272+
273+
return $collectionMock;
274+
}
166275
}

0 commit comments

Comments
 (0)