Skip to content

Commit 9eafdbb

Browse files
committed
AC-13075: FPT value in cart page and product page are different for same configurations for a dynamic product
1 parent 13b0ffc commit 9eafdbb

File tree

3 files changed

+28
-16
lines changed

3 files changed

+28
-16
lines changed

‎app/code/Magento/Tax/Model/Calculation/AbstractAggregateCalculator.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ protected function calculateWithTaxNotInPrice(QuoteDetailsItemInterface $item, $
121121
$deltaRoundingType = self::KEY_TAX_BEFORE_DISCOUNT_DELTA_ROUNDING;
122122
}
123123

124-
if ($round && strpos($item->getCode(), 'weee') !== false) {
124+
if ($round && strpos((string) $item->getCode(), 'weee') !== false) {
125125
$rowTaxPerRate = $this->calculationTool->round($rowTaxPerRate);
126126
} else {
127127
$rowTaxPerRate = $this->roundAmount($rowTaxPerRate, $taxId, false, $deltaRoundingType, $round, $item);

‎app/code/Magento/Tax/Test/Unit/Model/Calculation/TotalBaseCalculatorTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<?php
22
/**
3-
* Copyright 2014 Adobe
4-
* All Rights Reserved.
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
55
*/
66
declare(strict_types=1);
77

@@ -60,7 +60,7 @@ public function testCalculateWithTaxInPriceNoRounding()
6060
public function testCalculateWithTaxNotInPrice()
6161
{
6262
$this->initTotalBaseCalculator();
63-
$this->totalBaseCalculator->expects($this->exactly(1))
63+
$this->totalBaseCalculator->expects($this->exactly(2))
6464
->method('deltaRound')->willReturn(0);
6565
$this->initMocks(false);
6666

‎dev/tests/integration/testsuite/Magento/Tax/Model/TaxCalculationTest.php

Lines changed: 24 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1088,30 +1088,42 @@ public static function calculateTaxTaxInclDataProvider()
10881088
$productInclTaxWithoutStoreIdWithoutTaxClassId = $productTaxInclBase;
10891089

10901090
$productInclTaxWithStoreIdWithTaxClassId['storeId'] = 1;
1091-
$productInclTaxWithStoreIdWithTaxClassId['quoteDetailsData']['items'][] = $quoteDetailTaxInclItemWithDefaultProductTaxClass;
1091+
$productInclTaxWithStoreIdWithTaxClassId['quoteDetailsData']['items'][] =
1092+
$quoteDetailTaxInclItemWithDefaultProductTaxClass;
10921093
$productInclTaxWithStoreIdWithTaxClassId['expectedTaxDetails']['tax_amount'] = 0.70;
10931094
$productInclTaxWithStoreIdWithTaxClassId['expectedTaxDetails']['subtotal'] = 9.30;
10941095
$productInclTaxWithStoreIdWithTaxClassId['expectedTaxDetails']['applied_taxes'] = $quoteDetailAppliedTaxesBase;
1095-
$productInclTaxWithStoreIdWithTaxClassId['expectedTaxDetails']['items'] = $productTaxInclExpectedItemWithDefaultProductTaxClass;
1096+
$productInclTaxWithStoreIdWithTaxClassId['expectedTaxDetails']['items'] =
1097+
$productTaxInclExpectedItemWithDefaultProductTaxClass;
10961098

10971099
$productInclTaxWithStoreIdWithoutTaxClassId['storeId'] = 1;
10981100
$productInclTaxWithStoreIdWithoutTaxClassId['quoteDetailsData']['items'][] = $productTaxInclQuoteDetailItemBase;
1099-
$productInclTaxWithStoreIdWithoutTaxClassId['expectedTaxDetails']['items'] = $productTaxInclExpectedItemWithNoProductTaxClass;
1101+
$productInclTaxWithStoreIdWithoutTaxClassId['expectedTaxDetails']['items'] =
1102+
$productTaxInclExpectedItemWithNoProductTaxClass;
11001103

1101-
$productInclTaxWithoutStoreIdWithTaxClassId['quoteDetailsData']['items'][] = $quoteDetailTaxInclItemWithDefaultProductTaxClass;
1104+
$productInclTaxWithoutStoreIdWithTaxClassId['quoteDetailsData']['items'][] =
1105+
$quoteDetailTaxInclItemWithDefaultProductTaxClass;
11021106
$productInclTaxWithoutStoreIdWithTaxClassId['expectedTaxDetails']['tax_amount'] = 0.70;
11031107
$productInclTaxWithoutStoreIdWithTaxClassId['expectedTaxDetails']['subtotal'] = 9.30;
1104-
$productInclTaxWithoutStoreIdWithTaxClassId['expectedTaxDetails']['applied_taxes'] = $quoteDetailAppliedTaxesBase;
1105-
$productInclTaxWithoutStoreIdWithTaxClassId['expectedTaxDetails']['items'] = $productTaxInclExpectedItemWithDefaultProductTaxClass;
1108+
$productInclTaxWithoutStoreIdWithTaxClassId['expectedTaxDetails']['applied_taxes'] =
1109+
$quoteDetailAppliedTaxesBase;
1110+
$productInclTaxWithoutStoreIdWithTaxClassId['expectedTaxDetails']['items'] =
1111+
$productTaxInclExpectedItemWithDefaultProductTaxClass;
11061112

1107-
$productInclTaxWithoutStoreIdWithoutTaxClassId['quoteDetailsData']['items'][] = $productTaxInclQuoteDetailItemBase;
1108-
$productInclTaxWithoutStoreIdWithoutTaxClassId['expectedTaxDetails']['items'] = $productTaxInclExpectedItemWithNoProductTaxClass;
1113+
$productInclTaxWithoutStoreIdWithoutTaxClassId['quoteDetailsData']['items'][] =
1114+
$productTaxInclQuoteDetailItemBase;
1115+
$productInclTaxWithoutStoreIdWithoutTaxClassId['expectedTaxDetails']['items'] =
1116+
$productTaxInclExpectedItemWithNoProductTaxClass;
11091117

11101118
return [
1111-
'product incl tax with store id, with tax class id' => $productInclTaxWithStoreIdWithTaxClassId,
1112-
'product incl tax with store id, without tax class id' => $productInclTaxWithStoreIdWithoutTaxClassId,
1113-
'product incl tax without store id, with tax class id' => $productInclTaxWithoutStoreIdWithTaxClassId,
1114-
'product incl tax without store id, without tax class id' => $productInclTaxWithoutStoreIdWithoutTaxClassId,
1119+
'product incl tax with store id, with tax class id' =>
1120+
$productInclTaxWithStoreIdWithTaxClassId,
1121+
'product incl tax with store id, without tax class id' =>
1122+
$productInclTaxWithStoreIdWithoutTaxClassId,
1123+
'product incl tax without store id, with tax class id' =>
1124+
$productInclTaxWithoutStoreIdWithTaxClassId,
1125+
'product incl tax without store id, without tax class id' =>
1126+
$productInclTaxWithoutStoreIdWithoutTaxClassId,
11151127
];
11161128
}
11171129

0 commit comments

Comments
 (0)