Skip to content

Commit 8023905

Browse files
committed
Merge remote-tracking branch 'l3/ACP2E-357' into PR_L3_05_04_2022
2 parents e24b886 + 1ddbded commit 8023905

File tree

2 files changed

+17
-5
lines changed

2 files changed

+17
-5
lines changed

app/code/Magento/Paypal/Model/Cart.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,11 @@ protected function _applyDiscountTaxCompensationWorkaround(
180180
) {
181181
$dataContainer = $salesEntity->getTaxContainer();
182182
$this->addTax((double)$dataContainer->getBaseDiscountTaxCompensationAmount());
183-
$this->addTax((double)$dataContainer->getBaseShippingDiscountTaxCompensationAmnt());
183+
if ($dataContainer->getBaseShippingDiscountTaxCompensationAmnt() !== null) {
184+
$this->addTax((double)$dataContainer->getBaseShippingDiscountTaxCompensationAmnt());
185+
} else {
186+
$this->addTax((double)$dataContainer->getBaseShippingDiscountTaxCompensationAmount());
187+
}
184188
}
185189

186190
/**

app/code/Magento/Paypal/Test/Unit/Model/CartTest.php

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,12 +152,14 @@ public function invalidGetAllItemsDataProvider()
152152
public function testInvalidTotalsGetAllItems($values, $transferDiscount)
153153
{
154154
$expectedSubtotal = $this->_prepareInvalidModelData($values, $transferDiscount);
155+
$baseShippingDiscountTaxCompensationAmount = $values['base_shipping_discount_tax_compensation_amount'] ??
156+
$values['base_shipping_discount_tax_compensation_amnt'];
155157
$this->assertEmpty($this->_model->getAllItems());
156158
$this->assertEquals($expectedSubtotal, $this->_model->getSubtotal());
157159
$this->assertEquals(
158160
$values['base_tax_amount'] +
159161
$values['base_discount_tax_compensation_amount'] +
160-
$values['base_shipping_discount_tax_compensation_amnt'],
162+
$baseShippingDiscountTaxCompensationAmount,
161163
$this->_model->getTax()
162164
);
163165
$this->assertEquals($values['base_shipping_amount'], $this->_model->getShipping());
@@ -177,6 +179,7 @@ public function invalidTotalsGetAllItemsDataProvider()
177179
[
178180
'base_discount_tax_compensation_amount' => 0,
179181
'base_shipping_discount_tax_compensation_amnt' => 0,
182+
'base_shipping_discount_tax_compensation_amount' => null,
180183
'base_subtotal' => 0,
181184
'base_tax_amount' => 0,
182185
'base_shipping_amount' => 0,
@@ -188,7 +191,8 @@ public function invalidTotalsGetAllItemsDataProvider()
188191
[
189192
[
190193
'base_discount_tax_compensation_amount' => 1,
191-
'base_shipping_discount_tax_compensation_amnt' => 2,
194+
'base_shipping_discount_tax_compensation_amount' => 2,
195+
'base_shipping_discount_tax_compensation_amnt' => null,
192196
'base_subtotal' => 3,
193197
'base_tax_amount' => 4,
194198
'base_shipping_amount' => 5,
@@ -267,8 +271,12 @@ protected function _prepareInvalidModelData($data, $transferDiscount)
267271
{
268272
$taxContainer = new DataObject(
269273
[
270-
'base_discount_tax_compensation_amount' => $data['base_discount_tax_compensation_amount'],
271-
'base_shipping_discount_tax_compensation_amnt' => $data['base_shipping_discount_tax_compensation_amnt'],
274+
'base_discount_tax_compensation_amount' =>
275+
$data['base_discount_tax_compensation_amount'],
276+
'base_shipping_discount_tax_compensation_amnt' =>
277+
$data['base_shipping_discount_tax_compensation_amnt'],
278+
'base_shipping_discount_tax_compensation_amount' =>
279+
$data['base_shipping_discount_tax_compensation_amount']
272280
]
273281
);
274282
$expectedSubtotal = $data['base_subtotal'];

0 commit comments

Comments
 (0)