Skip to content

Commit 1efc8e0

Browse files
sivaschenkosivaschenko
authored
ENGCOM-3986: [Backport] Added option to exclude discount amount from minimum order amount calculation #18524 
2 parents aea5b08 + 50c4a5a commit 1efc8e0

File tree

6 files changed

+56
-3
lines changed

6 files changed

+56
-3
lines changed

‎app/code/Magento/Quote/Model/Quote.php

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2218,6 +2218,11 @@ public function validateMinimumAmount($multishipping = false)
22182218
if (!$minOrderActive) {
22192219
return true;
22202220
}
2221+
$includeDiscount = $this->_scopeConfig->getValue(
2222+
'sales/minimum_order/include_discount_amount',
2223+
\Magento\Store\Model\ScopeInterface::SCOPE_STORE,
2224+
$storeId
2225+
);
22212226
$minOrderMulti = $this->_scopeConfig->isSetFlag(
22222227
'sales/minimum_order/multi_address',
22232228
\Magento\Store\Model\ScopeInterface::SCOPE_STORE,
@@ -2251,7 +2256,10 @@ public function validateMinimumAmount($multishipping = false)
22512256
$taxes = ($taxInclude) ? $address->getBaseTaxAmount() : 0;
22522257
foreach ($address->getQuote()->getItemsCollection() as $item) {
22532258
/** @var \Magento\Quote\Model\Quote\Item $item */
2254-
$amount = $item->getBaseRowTotal() - $item->getBaseDiscountAmount() + $taxes;
2259+
$amount = $includeDiscount ?
2260+
$item->getBaseRowTotal() - $item->getBaseDiscountAmount() + $taxes :
2261+
$item->getBaseRowTotal() + $taxes;
2262+
22552263
if ($amount < $minAmount) {
22562264
return false;
22572265
}
@@ -2261,7 +2269,9 @@ public function validateMinimumAmount($multishipping = false)
22612269
$baseTotal = 0;
22622270
foreach ($addresses as $address) {
22632271
$taxes = ($taxInclude) ? $address->getBaseTaxAmount() : 0;
2264-
$baseTotal += $address->getBaseSubtotalWithDiscount() + $taxes;
2272+
$baseTotal += $includeDiscount ?
2273+
$address->getBaseSubtotalWithDiscount() + $taxes :
2274+
$address->getBaseSubtotal() + $taxes;
22652275
}
22662276
if ($baseTotal < $minAmount) {
22672277
return false;

‎app/code/Magento/Quote/Model/Quote/Address.php

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1150,6 +1150,11 @@ public function validateMinimumAmount()
11501150
return true;
11511151
}
11521152

1153+
$includeDiscount = $this->_scopeConfig->getValue(
1154+
'sales/minimum_order/include_discount_amount',
1155+
\Magento\Store\Model\ScopeInterface::SCOPE_STORE,
1156+
$storeId
1157+
);
11531158
$amount = $this->_scopeConfig->getValue(
11541159
'sales/minimum_order/amount',
11551160
\Magento\Store\Model\ScopeInterface::SCOPE_STORE,
@@ -1160,9 +1165,12 @@ public function validateMinimumAmount()
11601165
\Magento\Store\Model\ScopeInterface::SCOPE_STORE,
11611166
$storeId
11621167
);
1168+
11631169
$taxes = $taxInclude ? $this->getBaseTaxAmount() : 0;
11641170

1165-
return ($this->getBaseSubtotalWithDiscount() + $taxes >= $amount);
1171+
return $includeDiscount ?
1172+
($this->getBaseSubtotalWithDiscount() + $taxes >= $amount) :
1173+
($this->getBaseSubtotal() + $taxes >= $amount);
11661174
}
11671175

11681176
/**

‎app/code/Magento/Quote/Test/Unit/Model/Quote/AddressTest.php

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -216,6 +216,7 @@ public function testValidateMiniumumAmountVirtual()
216216
$scopeConfigValues = [
217217
['sales/minimum_order/active', ScopeInterface::SCOPE_STORE, $storeId, true],
218218
['sales/minimum_order/amount', ScopeInterface::SCOPE_STORE, $storeId, 20],
219+
['sales/minimum_order/include_discount_amount', ScopeInterface::SCOPE_STORE, $storeId, true],
219220
['sales/minimum_order/tax_including', ScopeInterface::SCOPE_STORE, $storeId, true],
220221
];
221222

@@ -240,6 +241,31 @@ public function testValidateMiniumumAmount()
240241
$scopeConfigValues = [
241242
['sales/minimum_order/active', ScopeInterface::SCOPE_STORE, $storeId, true],
242243
['sales/minimum_order/amount', ScopeInterface::SCOPE_STORE, $storeId, 20],
244+
['sales/minimum_order/include_discount_amount', ScopeInterface::SCOPE_STORE, $storeId, true],
245+
['sales/minimum_order/tax_including', ScopeInterface::SCOPE_STORE, $storeId, true],
246+
];
247+
248+
$this->quote->expects($this->once())
249+
->method('getStoreId')
250+
->willReturn($storeId);
251+
$this->quote->expects($this->once())
252+
->method('getIsVirtual')
253+
->willReturn(false);
254+
255+
$this->scopeConfig->expects($this->once())
256+
->method('isSetFlag')
257+
->willReturnMap($scopeConfigValues);
258+
259+
$this->assertTrue($this->address->validateMinimumAmount());
260+
}
261+
262+
public function testValidateMiniumumAmountWithoutDiscount()
263+
{
264+
$storeId = 1;
265+
$scopeConfigValues = [
266+
['sales/minimum_order/active', ScopeInterface::SCOPE_STORE, $storeId, true],
267+
['sales/minimum_order/amount', ScopeInterface::SCOPE_STORE, $storeId, 20],
268+
['sales/minimum_order/include_discount_amount', ScopeInterface::SCOPE_STORE, $storeId, false],
243269
['sales/minimum_order/tax_including', ScopeInterface::SCOPE_STORE, $storeId, true],
244270
];
245271

@@ -263,6 +289,7 @@ public function testValidateMiniumumAmountNegative()
263289
$scopeConfigValues = [
264290
['sales/minimum_order/active', ScopeInterface::SCOPE_STORE, $storeId, true],
265291
['sales/minimum_order/amount', ScopeInterface::SCOPE_STORE, $storeId, 20],
292+
['sales/minimum_order/include_discount_amount', ScopeInterface::SCOPE_STORE, $storeId, true],
266293
['sales/minimum_order/tax_including', ScopeInterface::SCOPE_STORE, $storeId, true],
267294
];
268295

‎app/code/Magento/Quote/Test/Unit/Model/QuoteTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -951,6 +951,7 @@ public function testValidateMiniumumAmount()
951951
['sales/minimum_order/active', ScopeInterface::SCOPE_STORE, $storeId, true],
952952
['sales/minimum_order/multi_address', ScopeInterface::SCOPE_STORE, $storeId, true],
953953
['sales/minimum_order/amount', ScopeInterface::SCOPE_STORE, $storeId, 20],
954+
['sales/minimum_order/include_discount_amount', ScopeInterface::SCOPE_STORE, $storeId, true],
954955
['sales/minimum_order/tax_including', ScopeInterface::SCOPE_STORE, $storeId, true],
955956
];
956957
$this->scopeConfig->expects($this->any())
@@ -977,6 +978,7 @@ public function testValidateMiniumumAmountNegative()
977978
['sales/minimum_order/active', ScopeInterface::SCOPE_STORE, $storeId, true],
978979
['sales/minimum_order/multi_address', ScopeInterface::SCOPE_STORE, $storeId, true],
979980
['sales/minimum_order/amount', ScopeInterface::SCOPE_STORE, $storeId, 20],
981+
['sales/minimum_order/include_discount_amount', ScopeInterface::SCOPE_STORE, $storeId, true],
980982
['sales/minimum_order/tax_including', ScopeInterface::SCOPE_STORE, $storeId, true],
981983
];
982984
$this->scopeConfig->expects($this->any())

‎app/code/Magento/Sales/etc/adminhtml/system.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,11 @@
8989
<label>Minimum Amount</label>
9090
<comment>Subtotal after discount</comment>
9191
</field>
92+
<field id="include_discount_amount" translate="label" sortOrder="12" type="select" showInDefault="1" showInWebsite="1" showInStore="0" canRestore="1">
93+
<label>Include Discount Amount</label>
94+
<source_model>Magento\Config\Model\Config\Source\Yesno</source_model>
95+
<comment>Choosing yes will be used subtotal after discount, otherwise only subtotal will be used</comment>
96+
</field>
9297
<field id="tax_including" translate="label" sortOrder="15" type="select" showInDefault="1" showInWebsite="1" showInStore="0" canRestore="1">
9398
<label>Include Tax to Amount</label>
9499
<source_model>Magento\Config\Model\Config\Source\Yesno</source_model>

‎app/code/Magento/Sales/etc/config.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
<allow_zero_grandtotal>1</allow_zero_grandtotal>
2323
</zerograndtotal_creditmemo>
2424
<minimum_order>
25+
<include_discount_amount>1</include_discount_amount>
2526
<tax_including>1</tax_including>
2627
</minimum_order>
2728
<orders>

0 commit comments

Comments
 (0)