Skip to content

Commit 115f70f

Browse files
committed
Merge pull request #215 from magento-sparta/MDVA-33
[SUPPORT] MDVA-33: Format of SKUs for auto generated simple products associated to a configurable product follow the name not the sku of the parent configurable
2 parents 48de512 + 3d4ab48 commit 115f70f

File tree

2 files changed

+10
-4
lines changed
  • app/code/Magento/ConfigurableProduct/view/adminhtml

2 files changed

+10
-4
lines changed

app/code/Magento/ConfigurableProduct/view/adminhtml/templates/catalog/product/edit/super/matrix.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ if ($variations) {
176176
</a>
177177
<!-- /ko -->
178178
<!-- ko ifnot: variation.readonly -->
179-
<input type="text" data-bind="attr: {id: $parent.getRowId(variation, 'name'), name: $parent.getVariationRowName(variation, 'name'), value: variation.sku}"/>
179+
<input type="text" data-bind="attr: {id: $parent.getRowId(variation, 'name'), name: $parent.getVariationRowName(variation, 'name'), value: variation.name}"/>
180180
<input type="hidden" data-bind="attr: {id: $parent.getRowId(variation, 'configurable_attribute'), name: $parent.getVariationRowName(variation, 'configurable_attribute'), value: variation.attribute}"/>
181181
<input type="hidden" data-role="status" data-bind="attr: {name: $parent.getVariationRowName(variation, 'status'), value: variation.status}"/>
182182
<!-- /ko -->

app/code/Magento/ConfigurableProduct/view/adminhtml/web/js/variations/steps/summary.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,17 +46,22 @@ define([
4646
},
4747
generateGrid: function (variations, getSectionValue) {
4848
//['a1','b1','c1','d1'] option = {label:'a1', value:'', section:{img:'',inv:'',pri:''}}
49+
var productSku = this.variationsComponent().getProductValue('sku');
4950
var productName = this.variationsComponent().getProductValue('name');
5051
var productPrice = this.variationsComponent().getProductValue('price');
5152
this.variations = [];
5253
return _.map(variations, function (options) {
53-
var variation = [], images, sku, quantity, price;
54+
var variation = [], images, sku, name, quantity, price;
5455
images = getSectionValue('images', options);
5556
variation.push(images);
56-
sku = productName + _.reduce(options, function (memo, option) {
57+
58+
var suffix = _.reduce(options, function (memo, option) {
5759
return memo + '-' + option.label;
5860
}, '');
61+
sku = productSku + suffix;
5962
variation.push(sku);
63+
name = productName + suffix;
64+
6065
quantity = getSectionValue('quantity', options);
6166
variation.push(quantity);
6267
//attributes
@@ -66,7 +71,8 @@ define([
6671
price = getSectionValue('price', options);
6772
price = price || productPrice;
6873
variation.push('$ ' + price);
69-
this.variations.push({options: options, images: images, sku: sku, quantity: quantity, price: price});
74+
this.variations.push({options: options, images: images, sku: sku, name: name,
75+
quantity: quantity, price: price});
7076
return variation;
7177
}, this);
7278
},

0 commit comments

Comments
 (0)