Magento: неправильный расчет налога на суммы счетов

Я использую Magento 1.5.1.0, и раньше у меня были проблемы с расчетом налога на суммы счетов. В то время как расчет будет правильным для всех итогов в моем магазине, в представлении счета-фактуры и в счетах-фактурах в формате PDF будут отображаться неправильные итоги.

Разницу между неправильным отображаемым значением и правильным значением можно увидеть на этом рисунке: (Краткая версия: промежуточная сумма будет включать налог на доставку, хотя налог на доставку уже включен в стоимость доставки) http://i731.photobucket.com/albums/ww318/vitamin6/orderview_fixed.jpg

Поэтому я разместил эту проблему на freelancer.com, и кому-то удалось ее исправить. НО, как я узнал позже, исправление не распространяется на все сценарии — если у заказа есть бесплатная доставка, промежуточная сумма счета все равно будет неверной. Вот скриншот, показывающий разницу: http://i731.photobucket.com/albums/ww318/vitamin6/orderview_freeship.jpg

Фрилансер отредактировал следующий файл, чтобы исправить неправильный расчет налога: app\code\local\Mage\Sales\Model\Order\Invoice\Total\Subtotal.php

Там следующий код:

    if ($invoice->isLast()) {
        $subtotal = $allowedSubtotal;
        $baseSubtotal = $baseAllowedSubtotal;
        $subtotalInclTax = $allowedSubtotalInclTax;
        $baseSubtotalInclTax  = $baseAllowedSubtotalInclTax;

был заменен на этот:

    if ($invoice->isLast()) {
        $subtotal = $allowedSubtotal;
        $baseSubtotal = $baseAllowedSubtotal;
        //$subtotalInclTax = $allowedSubtotalInclTax;
        //$baseSubtotalInclTax  = $baseAllowedSubtotalInclTax;
        $subtotalInclTax = min($allowedSubtotalInclTax, $subtotalInclTax);
        $baseSubtotalInclTax = min($baseAllowedSubtotalInclTax, $baseSubtotalInclTax);

Может ли кто-нибудь указать мне правильное направление, как мне нужно еще изменить файл, чтобы исправление работало для заказов с бесплатной доставкой? При необходимости можно предоставить более подробную информацию о налоговых настройках и т. д. - заранее спасибо!


php magento magento-1.5
person loeffel    schedule 27.01.2012    source источник
comment
Исправил это сам, изменив два основных файла.   -  person loeffel    schedule 02.04.2012
comment
Также исправляет это для меня. Не очень понимаю, почему.   -  person Stephan Hoyer    schedule 15.05.2012
comment
Скажите, пожалуйста, что именно вы делали, чтобы исправить это самостоятельно?   -  person mniess    schedule 04.07.2012
comment
Я поддерживаю просьбу mniess   -  person simonthesorcerer    schedule 12.03.2015

Ответы (2)


arrow_upward
0
arrow_downward

Существует ошибка с сортировкой итогов, которая может вызвать довольно неприятные проблемы.

Есть ли у вас модули, которые добавляют итоги?

Взгляните на это: https://stackoverflow.com/a/11954867/288568

person Alex    schedule 03.10.2012

arrow_upward
0
arrow_downward

Это было давно, и для меня проблема была решена с одним из обновлений magento (сейчас у меня 1.8.1.0). Я просмотрел свои старые файлы, и все, что я мог найти, это это редактирование:

app\code\core\Mage\Sales\Model\Order\Invoice\Total\Subtotal.php (взято из 1.7.0.2)

<?php /**  * Magento  *  * NOTICE OF LICENSE  *  * This source file is subject to the Open Software License (OSL 3.0)  * that is bundled with this package in the file LICENSE.txt.  * It is also available through the world-wide-web at this URL:  * http://opensource.org/licenses/osl-3.0.php  * If you did not receive a copy of the license and are unable to  * obtain it through the world-wide-web, please send an email  * to [email protected] so we can send you a copy immediately.  *  * DISCLAIMER  *  * Do not edit or add to this file if you wish to upgrade Magento to newer  * versions in the future. If you wish to customize Magento for your  * needs please refer to http://www.magentocommerce.com for more information.  *  * @category    Mage  * @package     Mage_Sales  * @copyright   Copyright (c) 2012 Magento Inc. (http://www.magentocommerce.com)  * @license     http://opensource.org/licenses/osl-3.0.php  Open Software License (OSL
3.0)  */


class Mage_Sales_Model_Order_Invoice_Total_Subtotal extends Mage_Sales_Model_Order_Invoice_Total_Abstract {
    /**
     * Collect invoice subtotal
     *
     * @param   Mage_Sales_Model_Order_Invoice $invoice
     * @return  Mage_Sales_Model_Order_Invoice_Total_Subtotal
     */
    public function collect(Mage_Sales_Model_Order_Invoice $invoice)
    {
        $subtotal       = 0;
        $baseSubtotal   = 0;
        $subtotalInclTax= 0;
        $baseSubtotalInclTax = 0;

        $order = $invoice->getOrder();

        foreach ($invoice->getAllItems() as $item) {
            if ($item->getOrderItem()->isDummy()) {
                continue;
            }

            $item->calcRowTotal();

            $subtotal       += $item->getRowTotal();
            $baseSubtotal   += $item->getBaseRowTotal();
            $subtotalInclTax+= $item->getRowTotalInclTax();
            $baseSubtotalInclTax += $item->getBaseRowTotalInclTax();
        }

        $allowedSubtotal = $order->getSubtotal() - $order->getSubtotalInvoiced();
        $baseAllowedSubtotal = $order->getBaseSubtotal() - $order->getBaseSubtotalInvoiced();
        $allowedSubtotalInclTax = $allowedSubtotal + $order->getHiddenTaxAmount()
                + $order->getTaxAmount() - $order->getTaxInvoiced() - $order->getHiddenTaxInvoiced();
        $baseAllowedSubtotalInclTax = $baseAllowedSubtotal + $order->getBaseHiddenTaxAmount()
                + $order->getBaseTaxAmount() - $order->getBaseTaxInvoiced() - $order->getBaseHiddenTaxInvoiced();

        /**
         * Check if shipping tax calculation is included to current invoice.
         */
        $includeShippingTax = true;
        foreach ($invoice->getOrder()->getInvoiceCollection() as $previousInvoice) {
            if ($previousInvoice->getShippingAmount() && !$previousInvoice->isCanceled()) {
                $includeShippingTax = false;
                break;
            }
        }

        if ($includeShippingTax) {
            $allowedSubtotalInclTax     -= $order->getShippingTaxAmount();
            $baseAllowedSubtotalInclTax -= $order->getBaseShippingTaxAmount();
        } else {
            $allowedSubtotalInclTax     += $order->getShippingHiddenTaxAmount();
            $baseAllowedSubtotalInclTax += $order->getBaseShippingHiddenTaxAmount();
        }

        if ($invoice->isLast()) {
            $subtotal = $allowedSubtotal;
            $baseSubtotal = $baseAllowedSubtotal;
            $subtotalInclTax = $allowedSubtotalInclTax;
            $baseSubtotalInclTax  = $baseAllowedSubtotalInclTax;
        } else {
            $subtotal = min($allowedSubtotal, $subtotal);
            $baseSubtotal = min($baseAllowedSubtotal, $baseSubtotal);
            $subtotalInclTax = min($allowedSubtotalInclTax, $subtotalInclTax);
            $baseSubtotalInclTax = min($baseAllowedSubtotalInclTax, $baseSubtotalInclTax);
        }

        $invoice->setSubtotal($subtotal);
        $invoice->setBaseSubtotal($baseSubtotal);
        $invoice->setSubtotalInclTax($subtotalInclTax);
        $invoice->setBaseSubtotalInclTax($baseSubtotalInclTax);

        $invoice->setGrandTotal($invoice->getGrandTotal() + $subtotal);
        $invoice->setBaseGrandTotal($invoice->getBaseGrandTotal() + $baseSubtotal);
        return $this;
    } }
person loeffel    schedule 13.03.2015