伊锐数码为 Magento 最新版本修正 Bug

作为专业的 Magento 开发团队，EveBit 近日帮助修正了 Magento 最新版本 v1.4.0.1 的一个销售数据计算错误，并在本站公布了修正办法。

关于 Magento v1.4.01 的这个bug， 同时可以在 Magento 的官方社区里面读到: http://www.magentocommerce.com/boards/viewthread/198624/, 以下，是我们对这个问题的中文描述及解决方案:

----------------------------------------------------------------------------------------------

MAgento 的这个bug, 是关于 Lifetime Sales calculation 的.

程序在统计 Lifetime Sale的时候出现了问题，这个问题是由于错误的计算了 DISCOUNT_COUNT所致。

而Average Sales = Lifetime Sales / Order Count 来算出来的，所以，当Lifetime Sales不正确的时候，Average Sales也不正确。

/app/code/core/Mage/Adminhtml/Block/Dashboard/Sales.php

the Lifetime Sales SQL 语句如下:

SELECT `e`.*, SUM((e.base_subtotal-IFNULL(e.base_subtotal_refunded,0)-IFNULL(e.base_subtotal_canceled,0)-IFNULL(e.base_discount_amount,0)+IFNULL(e.base_discount_refunded,0))*e.base_to_global_rate) AS `lifetime`, AVG((e.base_subtotal-IFNULL(e.base_subtotal_refunded,0)-IFNULL(e.base_subtotal_canceled,0)-IFNULL(e.base_discount_amount,0)+IFNULL(e.base_discount_refunded,0))*e.base_to_global_rate) AS `average` FROM `sales_order` AS `e` WHERE (e.entity_type_id = 11) AND (e.state != canceled) GROUP BY `e`.`entity_type_id`

这个问题我想是在SQL语句中的SUM函数。在SQL语句 e.base_discount_amount是负数（-59.4000）。所以负负为正。所以这里，其实是 e.base_subtotal（60.0000） - discount_amount（-59.4000）。这里有7个order，和5个order是免费的，不用付钱。两个支付命令是相同的。

最后:

Lifetime Sales = (e.base_subtotal(60.0000) - discount_amount(-59.4000)) * 2;
Average Orders = Lifetime Sales / 7

我还检查了发货规则，action tab，the Discount amount是一个正值，并套用设定的产品价格折扣率。

---------------------------------------------------------------------------------------------

----------------------------------------------------------------------------------------------

解决办法:

/app/code/local/Mage/Reports/Model/Mysql4/Order/Collection.php 这个文件。

line 204:
Original:
({{base_subtotal}}-IFNULL({{base_subtotal_refunded}},0)-IFNULL({{base_subtotal_canceled}},0)I-FNULL({{base_discount_amount}},0)+IFNULL({{base_discount_refunded}},0))*{{base_to_global_rate}}
Modified:
({{base_subtotal}}-IFNULL({{base_subtotal_refunded}},0)-IFNULL({{base_subtotal_canceled}},0)+IFNULL({{base_tax_amount}},0)+IFNULL({{base_discount_amount}},0)+IFNULL({{base_discount_refunded}},0))*{{base_to_global_rate}}

line 205:
Original:
array('base_subtotal', 'base_to_global_rate', 'base_subtotal_refunded', 'base_subtotal_canceled','base_discount_amount','base_discount_refunded')
Modified:
array('base_subtotal', 'base_to_global_rate', 'base_subtotal_refunded', 'base_tax_amount', 'base_subtotal_canceled','base_discount_amount','base_discount_refunded')

line 209:
Original:
({{base_subtotal}}-IFNULL({{base_subtotal_refunded}},0)-IFNULL({{base_subtotal_canceled}},0)-IFNULL({{base_discount_amount}},0)+IFNULL({{base_discount_amount}},0))
Modified:
({{base_subtotal}}-IFNULL({{base_subtotal_refunded}},0)-IFNULL({{base_subtotal_canceled}},0)+IFNULL({{base_tax_amount}},0)+IFNULL({{base_discount_amount}},0)+IFNULL({{base_discount_amount}},0))

base_discount_amount','base_discount_refunded')
Modified:
array('base_subtotal', 'base_subtotal_refunded', 'base_tax_amount', 'base_subtotal_canceled','base_discount_amount','base_discount_refunded')