Magento: Пользовательские итоги учитываются дважды? - PullRequest
Новая
       17

Magento: Пользовательские итоги учитываются дважды?

5 голосов
/ 24 ноября 2011

Хорошо, я создал пользовательский класс Total для добавления специальной скидки, и все, кажется, работает нормально, за исключением того, что по какой-то причине я не могу найти, мой итог рассчитывается дважды!Это приводит к двойной сумме скидки и неверной итоговой сумме.Теперь это происходит на странице корзины и на страницах оформления заказа ... НО ... когда я выполняю заказ, общая сумма рассчитывается только один раз, а общая сумма - в порядке.

Странно, что метод сбора вызывается дважды для страниц корзины, но только один раз при завершении заказа, но я могу «отследить, где все это происходит и почему.

пропустите ненужный код, я вставлю только важные 

     <sales>
        <quote>
            <totals>
                <mydiscount>
                    <class>ucon_mydiscount/total_mydiscount</class>
                    <before>subtotal</before>
                </mydiscount>
            </totals>
        </quote>
    </sales>

и методы сборщика 

    public function collect(Mage_Sales_Model_Quote_Address $address)
{
    parent::collect($address);

    $quote = $address->getQuote();
    $quoteId = $quote->getEntityId();

    $items = $quote->getAllItems();
    if (!count($items)) {
        return $this;
    }       


    $discount = 0;
    $productId = 2556;  

    foreach($items as $item)
    {       
        if($item->getProduct()->getId() == $productId)
        {
            $qty = $item->getQty();
            $totalPrice = round(($item->getRowTotal()+$item->getTaxAmount()),2);

            //discount 10%              
            $discount = round($totalPrice * 0.1,2);     

            $discount = 0 - $discount;
        }
    }

    if($discount == 0)
        return $this;

    $this->_setAmount($discount);
    $this->_setBaseAmount($discount);


    return $this;
}

и сборщик 

    public function fetch(Mage_Sales_Model_Quote_Address $address)
{
    $amount = $address->getMydiscountAmount();
    if ($amount != 0) {
        $title = Mage::helper('ucon_mydiscount')->__('My discount');
        $address->addTotal(array(
            'code' => $this->getCode(),
            'title' => $title,
            'value' => $amount
        ));
    }
    return $this;
}

редактировать: еще одна вещь, которую янайти очень странно - я делаю setValue в моем методе collect, а не addValue, поэтому даже если метод вызывается дважды, он не должен быть двойным значением, он просто должен дважды установить правильное значение.

Ответы [ 3 ]

16 голосов
/ 04 декабря 2011

Может ли быть проблема в том, что общий объект принадлежит объекту адреса, а заказы Magento обычно имеют ДВА адреса - один для доставки и один для выставления счетов?

Таким образом, ваш итог будет вызван для выполнения дважды - один раз с адресом выставления счета и один раз с адресом доставки, и сумма будет подсчитана по каждому адресу. Вы можете попробовать проверить, какой адрес был вам передан, и применить значение только к одному из них, например так: 

public function collect(Mage_Sales_Model_Quote_Address $address) {

  $this->_setAddress($address);
  $this->_setAmount(0);
  $this->_setBaseAmount(0);

  if ($address->getAddressType() == 'shipping') { 
    //only apply an actual value to the shipping address

    //... Do your calculation here as above ...

  } 

  return $this;
}

Вам также нужно будет сделать нечто подобное в методе выборки ... 

public function fetch(Mage_Sales_Model_Quote_Address $address) {

  $amount = $address->getMydiscountAmount();

  if ($amount != 0 && $address->getAddressType() == 'shipping') {

    $title = Mage::helper('ucon_mydiscount')->__('My discount');

    $address->addTotal(array(
        'code' => $this->getCode(),
        'title' => $title,
        'value' => $amount
    ));

  }

  return $this;
}

Я признаю, что функция collect может быть красивее, но, надеюсь, вы все равно поймете эту идею.

Попробуйте и посмотрите, правильно ли складываются ваши итоги на интерфейсе и в административной области.

2 голосов
/ 06 октября 2014

После поиска, есть другое решение 

public function collect(Mage_Sales_Model_Quote_Address $address) {
    parent::collect($address);

    //Pay attention to this code
    $items = $this->_getAddressItems($address);
    if (!count($items)) {
        return $this; //this makes only address type shipping to come through
    }

    //Do whatever you want here to add discount or fee...

    return $this;
}

При этом скидка или плата будет добавляться только к адресу доставки, и она будет засчитана один раз.Так что нам даже не нужно добавлять if ($address->getAddressType() == 'shipping') { в функцию fetch.

0 голосов
/ 25 ноября 2011

Возможно ли, что вы добавляете в свой макет XML-код для блока корзины?Если это так, есть большая вероятность, что блок вызывается дважды (один из базового кода, и снова для вашего кода - даже если вы просто его расширяете), таким образом дублируя итоговую цену.Если это так, вам нужно будет удалить (деструктивно с тегом <remove>) базовый макет XML для этого блока, и тогда все должно встать на свои места и работать.

...