Skip to content

Commit 2c9140e

Browse files
irenelagnoirenelagno
committed
Merge branch 'main_develop' into switcher_stable
2 parents 73382f6 + f278923 commit 2c9140e

File tree

29 files changed

+622
-192
lines changed

29 files changed

+622
-192
lines changed

‎app/code/Magento/Bundle/Ui/DataProvider/Product/Form/Modifier/BundlePanel.php

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -569,6 +569,7 @@ protected function getBundleSelections()
569569
'arguments' => [
570570
'data' => [
571571
'config' => [
572+
'component' => 'Magento_Bundle/js/components/bundle-option-qty',
572573
'formElement' => Form\Element\Input::NAME,
573574
'componentType' => Form\Field::NAME,
574575
'dataType' => Form\Element\DataType\Number::NAME,
@@ -577,8 +578,12 @@ protected function getBundleSelections()
577578
'value' => '1',
578579
'sortOrder' => 100,
579580
'validation' => [
581+
'required-entry' => true,
580582
'validate-number' => true,
581583
],
584+
'imports' => [
585+
'isInteger' => '${ $.provider }:${ $.parentScope }.selection_qty_is_integer'
586+
],
582587
],
583588
],
584589
],

‎app/code/Magento/Bundle/Ui/DataProvider/Product/Form/Modifier/Composite.php

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,8 @@ public function modifyMeta(array $meta)
8888

8989
/**
9090
* {@inheritdoc}
91+
* @SuppressWarnings(PHPMD.CyclomaticComplexity)
92+
* @SuppressWarnings(PHPMD.NPathComplexity)
9193
*/
9294
public function modifyData(array $data)
9395
{
@@ -103,6 +105,12 @@ public function modifyData(array $data)
103105
/** @var \Magento\Bundle\Api\Data\LinkInterface $productLink */
104106
foreach ($option->getProductLinks() as $productLink) {
105107
$linkedProduct = $this->productRepository->get($productLink->getSku());
108+
$integerQty = 1;
109+
if ($linkedProduct->getExtensionAttributes()->getStockItem()) {
110+
if ($linkedProduct->getExtensionAttributes()->getStockItem()->getIsQtyDecimal()) {
111+
$integerQty = 0;
112+
}
113+
}
106114
$selections[] = [
107115
'selection_id' => $productLink->getId(),
108116
'option_id' => $productLink->getOptionId(),
@@ -112,8 +120,9 @@ public function modifyData(array $data)
112120
'is_default' => ($productLink->getIsDefault()) ? '1' : '0',
113121
'selection_price_value' => $productLink->getPrice(),
114122
'selection_price_type' => $productLink->getPriceType(),
115-
'selection_qty' => $productLink->getQty(),
123+
'selection_qty' => (bool)$integerQty ? (int)$productLink->getQty() : $productLink->getQty(),
116124
'selection_can_change_qty' => $productLink->getCanChangeQuantity(),
125+
'selection_qty_is_integer' => (bool)$integerQty,
117126
'position' => $productLink->getPosition(),
118127
];
119128
}

‎app/code/Magento/Bundle/view/adminhtml/templates/product/edit/bundle/option/selection.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ Bundle.Selection.prototype = {
164164
if (data.can_read_price != undefined && !data.can_read_price) {
165165
data.selection_price_value = '';
166166
} else {
167-
data.selection_price_value = Number(data.selection_price_value).toFixed(2);
167+
data.selection_price_value = Number(Math.round(data.selection_price_value + "e+2") + "e-2").toFixed(2);
168168
}
169169

170170
data.index = this.itemsCount++;

‎app/code/Magento/Bundle/view/adminhtml/web/js/components/bundle-option-qty.js

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
/**
2+
* Copyright © 2016 Magento. All rights reserved.
3+
* See COPYING.txt for license details.
4+
*/
5+
6+
define([
7+
'Magento_Ui/js/form/element/abstract'
8+
], function (Abstract) {
9+
'use strict';
10+
11+
return Abstract.extend({
12+
defaults: {
13+
valueUpdate: 'input',
14+
isInteger: true
15+
},
16+
17+
/**
18+
* update event
19+
*/
20+
onUpdate: function () {
21+
this.validation['validate-number'] = true;
22+
this.validation['validate-digits'] = this.isInteger;
23+
this.validate();
24+
}
25+
});
26+
});

‎app/code/Magento/Catalog/view/base/web/js/price-utils.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ define([
4343
pattern = format.pattern || '%s',
4444
s = '',
4545
i, pad,
46-
j, re, r;
46+
j, re, r, am;
4747

4848
if (isShowSign === undefined || isShowSign === true) {
4949
s = amount < 0 ? '-' : (isShowSign ? '+' : '');
@@ -52,7 +52,9 @@ define([
5252
}
5353
pattern = pattern.indexOf('{sign}') < 0 ? s + pattern : pattern.replace('{sign}', s);
5454

55-
i = parseInt(amount = Math.abs(+amount || 0).toFixed(precision), 10) + '';
55+
// we're avoiding the usage of to fixed, and using round instead with the e representation to address
56+
// numbers like 1.005 = 1.01. Using ToFixed to only provide trailig zeroes in case we have a whole number
57+
i = parseInt(amount = Number(Math.round(Math.abs(+amount || 0) + 'e+' + precision) + ('e-' + precision)) , 10) + '';
5658
pad = (i.length < integerRequired) ? (integerRequired - i.length) : 0;
5759

5860
i = stringPad('0', pad) + i;
@@ -63,9 +65,12 @@ define([
6365
// replace(/-/, 0) is only for fixing Safari bug which appears
6466
// when Math.abs(0).toFixed() executed on '0' number.
6567
// Result is '0.-0' :(
68+
69+
70+
am = Number(Math.round(Math.abs(amount - i) + 'e+' + precision) + ('e-' + precision));
6671
r = (j ? i.substr(0, j) + groupSymbol : '') +
6772
i.substr(j).replace(re, '$1' + groupSymbol) +
68-
(precision ? decimalSymbol + Math.abs(amount - i).toFixed(precision).replace(/-/, 0).slice(2) : '');
73+
(precision ? decimalSymbol + am.toFixed(2).replace(/-/, 0).slice(2) : '');
6974

7075
return pattern.replace('%s', r).replace(/^\s\s*/, '').replace(/\s\s*$/, '');
7176
}

‎app/code/Magento/ConfigurableProduct/view/adminhtml/web/js/configurable.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ Product.Config.prototype = {
214214
}
215215
}
216216

217-
var roundedPrice = (Math.round(price*100)/100).toString();
217+
var roundedPrice = Number(Math.round(price + "e+2") + "e-2").toString();
218218

219219
if (this.prices && this.prices[roundedPrice]) {
220220
str+= this.prices[roundedPrice];

‎app/code/Magento/ConfigurableProduct/view/adminhtml/web/js/variations/variations.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ define([
124124
},
125125
name: product.name || product.sku,
126126
options: options,
127-
price: parseFloat(product.price.replace(/[^\d.]+/g, '')).toFixed(4),
127+
price: parseFloat(Math.round(product.price.replace(/[^\d.]+/g, '') + "e+4") + "e-4").toFixed(4),
128128
productId: productId,
129129
productUrl: this.buildProductUrl(productId),
130130
quantity: product.quantity || null,

‎app/code/Magento/Eav/Model/Entity/AbstractEntity.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -931,13 +931,13 @@ public function checkAttributeUniqueValue(AbstractAttribute $attribute, $object)
931931
$select = $connection->select();
932932
if ($attribute->getBackend()->getType() === 'static') {
933933
$value = $object->getData($attribute->getAttributeCode());
934-
$bind = ['attribute_code' => trim($value)];
934+
$bind = ['value' => trim($value)];
935935

936936
$select->from(
937937
$this->getEntityTable(),
938938
$this->getEntityIdField()
939939
)->where(
940-
$attribute->getAttributeCode() . ' = :attribute_code'
940+
$attribute->getAttributeCode() . ' = :value'
941941
);
942942
} else {
943943
$value = $object->getData($attribute->getAttributeCode());
@@ -950,7 +950,7 @@ public function checkAttributeUniqueValue(AbstractAttribute $attribute, $object)
950950
];
951951
$select->from(
952952
$attribute->getBackend()->getTable(),
953-
$attribute->getBackend()->getEntityIdField()
953+
$object->getResource()->getLinkField()
954954
)->where(
955955
'attribute_id = :attribute_id'
956956
)->where(

‎app/code/Magento/Eav/Model/ResourceModel/UpdateHandler.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,7 @@ public function execute($entityType, $data)
134134
if ((!array_key_exists($attribute->getAttributeCode(), $snapshot)
135135
|| $snapshot[$attribute->getAttributeCode()] === false)
136136
&& array_key_exists($attribute->getAttributeCode(), $data)
137+
&& $data[$attribute->getAttributeCode()] !== false
137138
&& !$attribute->isValueEmpty($data[$attribute->getAttributeCode()])
138139
) {
139140
$this->attributePersistor->registerInsert(
@@ -147,6 +148,7 @@ public function execute($entityType, $data)
147148
if (array_key_exists($attribute->getAttributeCode(), $snapshot)
148149
&& $snapshot[$attribute->getAttributeCode()] !== false
149150
&& array_key_exists($attribute->getAttributeCode(), $data)
151+
&& $data[$attribute->getAttributeCode()] !== false
150152
&& $snapshot[$attribute->getAttributeCode()] != $data[$attribute->getAttributeCode()]
151153
&& !$attribute->isValueEmpty($data[$attribute->getAttributeCode()])
152154
) {

‎app/code/Magento/Quote/etc/fieldset.xml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -247,6 +247,9 @@
247247
<field name="row_total">
248248
<aspect name="to_order_item" />
249249
</field>
250+
<field name="base_price">
251+
<aspect name="to_order_item" />
252+
</field>
250253
<field name="base_original_price">
251254
<aspect name="to_order_item" />
252255
</field>

0 commit comments

Comments
 (0)