Skip to content

Commit 29060f4

Browse files
author
Sergey Shvets
committed
MAGETWO-97625: [Magento Cloud] Can't delete video images on Duplicate products
1 parent 54007dc commit 29060f4

File tree

1 file changed

+33
-0
lines changed

1 file changed

+33
-0
lines changed

app/code/Magento/ProductVideo/Model/Plugin/Catalog/Product/Gallery/CreateHandler.php

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,8 @@ class CreateHandler extends AbstractHandler
1919
const ADDITIONAL_STORE_DATA_KEY = 'additional_store_data';
2020

2121
/**
22+
* Execute before Plugin
23+
*
2224
* @param \Magento\Catalog\Model\Product\Gallery\CreateHandler $mediaGalleryCreateHandler
2325
* @param \Magento\Catalog\Model\Product $product
2426
* @param array $arguments
@@ -44,6 +46,8 @@ public function beforeExecute(
4446
}
4547

4648
/**
49+
* Execute plugin
50+
*
4751
* @param \Magento\Catalog\Model\Product\Gallery\CreateHandler $mediaGalleryCreateHandler
4852
* @param \Magento\Catalog\Model\Product $product
4953
* @return \Magento\Catalog\Model\Product
@@ -73,6 +77,8 @@ public function afterExecute(
7377
}
7478

7579
/**
80+
* Saves video data
81+
*
7682
* @param array $videoDataCollection
7783
* @param int $storeId
7884
* @return void
@@ -86,6 +92,8 @@ protected function saveVideoData(array $videoDataCollection, $storeId)
8692
}
8793

8894
/**
95+
* Saves additioanal video data
96+
*
8997
* @param array $videoDataCollection
9098
* @return void
9199
*/
@@ -102,6 +110,8 @@ protected function saveAdditionalStoreData(array $videoDataCollection)
102110
}
103111

104112
/**
113+
* Saves video data
114+
*
105115
* @param array $item
106116
* @return void
107117
*/
@@ -114,6 +124,8 @@ protected function saveVideoValuesItem(array $item)
114124
}
115125

116126
/**
127+
* Excludes current store data
128+
*
117129
* @param array $mediaCollection
118130
* @param int $currentStoreId
119131
* @return array
@@ -129,6 +141,8 @@ function ($item) use ($currentStoreId) {
129141
}
130142

131143
/**
144+
* Prepare video data for saving
145+
*
132146
* @param array $rowData
133147
* @return array
134148
*/
@@ -146,6 +160,8 @@ protected function prepareVideoRowDataForSave(array $rowData)
146160
}
147161

148162
/**
163+
* Loads video data
164+
*
149165
* @param array $mediaCollection
150166
* @param int $excludedStore
151167
* @return array
@@ -168,6 +184,8 @@ protected function loadStoreViewVideoData(array $mediaCollection, $excludedStore
168184
}
169185

170186
/**
187+
* Collect video data
188+
*
171189
* @param array $mediaCollection
172190
* @return array
173191
*/
@@ -185,6 +203,8 @@ protected function collectVideoData(array $mediaCollection)
185203
}
186204

187205
/**
206+
* Extract video data
207+
*
188208
* @param array $rowData
189209
* @return array
190210
*/
@@ -197,6 +217,8 @@ protected function extractVideoDataFromRowData(array $rowData)
197217
}
198218

199219
/**
220+
* Collect items for additional data adding
221+
*
200222
* @param array $mediaCollection
201223
* @return array
202224
*/
@@ -212,6 +234,8 @@ protected function collectVideoEntriesIdsToAdditionalLoad(array $mediaCollection
212234
}
213235

214236
/**
237+
* Add additional data
238+
*
215239
* @param array $mediaCollection
216240
* @param array $data
217241
* @return array
@@ -232,6 +256,8 @@ protected function addAdditionalStoreData(array $mediaCollection, array $data):
232256
}
233257

234258
/**
259+
* Creates additional video data
260+
*
235261
* @param array $storeData
236262
* @param int $valueId
237263
* @return array
@@ -250,6 +276,8 @@ protected function createAdditionalStoreDataCollection(array $storeData, $valueI
250276
}
251277

252278
/**
279+
* Collect new videos
280+
*
253281
* @param array $mediaCollection
254282
* @return array
255283
*/
@@ -265,6 +293,8 @@ private function collectNewVideos(array $mediaCollection): array
265293
}
266294

267295
/**
296+
* Checks if gallery item is video
297+
*
268298
* @param array $item
269299
* @return bool
270300
*/
@@ -276,6 +306,8 @@ private function isVideoItem(array $item): bool
276306
}
277307

278308
/**
309+
* Checks if video is new
310+
*
279311
* @param array $item
280312
* @return bool
281313
*/
@@ -287,6 +319,7 @@ private function isNewVideo(array $item): bool
287319
}
288320

289321
/**
322+
* Mark all videos as new
290323
*
291324
* @param int $entityId
292325
* @param array $mediaCollection

0 commit comments

Comments
 (0)