Skip to content

Commit d79eb08

Browse files
committed
Added refunded discount amount to the total revenue calculation and fixed Negative order amount in dashboard when viewing particular store data
1 parent fb9e645 commit d79eb08

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

app/code/Magento/Reports/Model/ResourceModel/Order/Collection.php

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -773,6 +773,7 @@ public function addRevenueToSelect($convertCurrency = false)
773773
!$convertCurrency,
774774
$this->getConnection()->getIfNullSql('main_table.base_subtotal_refunded', 0),
775775
$this->getConnection()->getIfNullSql('main_table.base_subtotal_canceled', 0),
776+
$this->getConnection()->getIfNullSql('main_table.base_discount_refunded', 0),
776777
$this->getConnection()->getIfNullSql('main_table.base_discount_canceled', 0)
777778
);
778779
$this->getSelect()->columns(['revenue' => $expr]);
@@ -795,6 +796,7 @@ public function addSumAvgTotals($storeId = 0)
795796
$storeId,
796797
$this->getConnection()->getIfNullSql('main_table.base_subtotal_refunded', 0),
797798
$this->getConnection()->getIfNullSql('main_table.base_subtotal_canceled', 0),
799+
$this->getConnection()->getIfNullSql('main_table.base_discount_refunded', 0),
798800
$this->getConnection()->getIfNullSql('main_table.base_discount_canceled', 0)
799801
);
800802

@@ -813,20 +815,22 @@ public function addSumAvgTotals($storeId = 0)
813815
* @param int $storeId
814816
* @param string $baseSubtotalRefunded
815817
* @param string $baseSubtotalCanceled
818+
* @param string $baseDiscountRefunded
816819
* @param string $baseDiscountCanceled
817820
* @return string
818821
*/
819822
protected function getTotalsExpression(
820823
$storeId,
821824
$baseSubtotalRefunded,
822825
$baseSubtotalCanceled,
826+
$baseDiscountRefunded,
823827
$baseDiscountCanceled
824828
) {
825829
$template = ($storeId != 0)
826-
? '(main_table.base_subtotal - %2$s - %1$s - ABS(main_table.base_discount_amount) - %3$s)'
827-
: '((main_table.base_subtotal - %1$s - %2$s - ABS(main_table.base_discount_amount) + %3$s) '
830+
? '(main_table.base_subtotal - %2$s - %1$s - ABS(main_table.base_discount_amount) + %3$s + %4$s)'
831+
: '((main_table.base_subtotal - %1$s - %2$s - ABS(main_table.base_discount_amount) + %3$s + %4$s) '
828832
. ' * main_table.base_to_global_rate)';
829-
return sprintf($template, $baseSubtotalRefunded, $baseSubtotalCanceled, $baseDiscountCanceled);
833+
return sprintf($template, $baseSubtotalRefunded, $baseSubtotalCanceled, $baseDiscountRefunded, $baseDiscountCanceled);
830834
}
831835

832836
/**

0 commit comments

Comments
 (0)