Skip to content

Commit f0e5c5f

Browse files
committed
ACP2E-3498: Incorrect discount value when multiple cart price rules are applied simultaneously with discounted/special priced products
1 parent de77474 commit f0e5c5f

File tree

2 files changed

+6
-7
lines changed
  • app/code/Magento/SalesRule/Model/Rule/Action/Discount
  • dev/tests/integration/testsuite/Magento/SalesRule/Model/Rule/Action/Discount

2 files changed

+6
-7
lines changed

app/code/Magento/SalesRule/Model/Rule/Action/Discount/CartFixed.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,6 @@ public function calculate($rule, $item, $qty)
9393
$baseItemPrice = $this->validator->getItemBasePrice($item);
9494
$itemOriginalPrice = $this->validator->getItemOriginalPrice($item);
9595
$baseItemOriginalPrice = $this->validator->getItemBaseOriginalPrice($item);
96-
$baseItemDiscountAmount = (float) $item->getBaseDiscountAmount();
9796

9897
$cartRules = $quote->getCartFixedRules();
9998
if (!isset($cartRules[$rule->getId()])) {
@@ -132,8 +131,8 @@ public function calculate($rule, $item, $qty)
132131
$ruleDiscount,
133132
$qty,
134133
$baseItemPrice,
135-
$baseItemDiscountAmount,
136-
$baseRuleTotals - $baseItemDiscountAmount,
134+
0,
135+
$baseRuleTotals,
137136
$discountType
138137
);
139138
}

dev/tests/integration/testsuite/Magento/SalesRule/Model/Rule/Action/Discount/CartFixedTest.php

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

@@ -526,11 +526,11 @@ public static function discountByPercentDataProvider()
526526
],
527527
[
528528
'percentDiscount' => 15.5,
529-
'expectedDiscounts' => ['simple1' => 3.65, 'simple2' => 9.76, 'totalDiscount' => -13.41]
529+
'expectedDiscounts' => ['simple1' => 3.66, 'simple2' => 9.75, 'totalDiscount' => -13.41]
530530
],
531531
[
532532
'percentDiscount' => 50,
533-
'expectedDiscounts' => ['simple1' => 5.72, 'simple2' => 15.27, 'totalDiscount' => -20.99]
533+
'expectedDiscounts' => ['simple1' => 5.73, 'simple2' => 15.26, 'totalDiscount' => -20.99]
534534
],
535535
[
536536
'percentDiscount' => 100,

0 commit comments

Comments
 (0)