Skip to content

Commit 588a8b0

Browse files
author
Stanislav Idolov
authored
ENGCOM-1655: [Fix] forgot to add lowercase conversion on grouped product assignation #15312
2 parents c4d9a99 + 7320a74 commit 588a8b0

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

app/code/Magento/GroupedImportExport/Model/Import/Product/Type/Grouped.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ public function saveData()
9999
}
100100
$scope = $this->_entityModel->getRowScope($rowData);
101101
if (Product::SCOPE_DEFAULT == $scope) {
102-
$productData = $newSku[$rowData[Product::COL_SKU]];
102+
$productData = $newSku[strtolower($rowData[Product::COL_SKU])];
103103
} else {
104104
$colAttrSet = Product::COL_ATTR_SET;
105105
$rowData[$colAttrSet] = $productData['attr_set_code'];

app/code/Magento/GroupedImportExport/Test/Unit/Model/Import/Product/Type/GroupedTest.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -192,26 +192,26 @@ public function saveDataProvider()
192192
'skus' => [
193193
'newSku' => [
194194
'sku_assoc1' => ['entity_id' => 1],
195-
'productSku' => ['entity_id' => 3, 'attr_set_code' => 'Default', 'type_id' => 'grouped']
195+
'productsku' => ['entity_id' => 3, 'attr_set_code' => 'Default', 'type_id' => 'grouped']
196196
],
197197
'oldSku' => ['sku_assoc2' => ['entity_id' => 2]]
198198
],
199199
'bunch' => [
200200
'associated_skus' => 'sku_assoc1=1, sku_assoc2=2',
201-
'sku' => 'productSku',
201+
'sku' => 'productsku',
202202
'product_type' => 'grouped'
203203
]
204204
],
205205
[
206206
'skus' => [
207207
'newSku' => [
208-
'productSku' => ['entity_id' => 1, 'attr_set_code' => 'Default', 'type_id' => 'grouped']
208+
'productsku' => ['entity_id' => 1, 'attr_set_code' => 'Default', 'type_id' => 'grouped']
209209
],
210210
'oldSku' => []
211211
],
212212
'bunch' => [
213213
'associated_skus' => '',
214-
'sku' => 'productSku',
214+
'sku' => 'productsku',
215215
'product_type' => 'grouped'
216216
]
217217
],
@@ -227,13 +227,13 @@ public function saveDataProvider()
227227
'skus' => [
228228
'newSku' => [
229229
'sku_assoc1' => ['entity_id' => 1],
230-
'productSku' => ['entity_id' => 3, 'attr_set_code' => 'Default', 'type_id' => 'grouped']
230+
'productsku' => ['entity_id' => 3, 'attr_set_code' => 'Default', 'type_id' => 'grouped']
231231
],
232232
'oldSku' => []
233233
],
234234
'bunch' => [
235235
'associated_skus' => 'sku_assoc1=1',
236-
'sku' => 'productSku',
236+
'sku' => 'productsku',
237237
'product_type' => 'simple'
238238
]
239239
]
@@ -247,7 +247,7 @@ public function testSaveDataScopeStore()
247247
{
248248
$this->entityModel->expects($this->once())->method('getNewSku')->will($this->returnValue([
249249
'sku_assoc1' => ['entity_id' => 1],
250-
'productSku' => ['entity_id' => 2, 'attr_set_code' => 'Default', 'type_id' => 'grouped']
250+
'productsku' => ['entity_id' => 2, 'attr_set_code' => 'Default', 'type_id' => 'grouped']
251251
]));
252252
$this->entityModel->expects($this->once())->method('getOldSku')->will($this->returnValue([
253253
'sku_assoc2' => ['entity_id' => 3]
@@ -257,7 +257,7 @@ public function testSaveDataScopeStore()
257257

258258
$bunch = [[
259259
'associated_skus' => 'sku_assoc1=1, sku_assoc2=2',
260-
'sku' => 'productSku',
260+
'sku' => 'productsku',
261261
'product_type' => 'grouped'
262262
]];
263263
$this->entityModel->expects($this->at(2))->method('getNextBunch')->will($this->returnValue($bunch));

0 commit comments

Comments
 (0)