Magento – How to Merge Quote Items

cartee-1.12magento-enterprisequote

I'm trying to merge like items in the cart by taking the total quantity of an item by SKU (the ID field is unreliable as multiple items may return as empty/null), setting the first instance to the total quantity, then either deleting or changing the rest to 0. This doesn't seem to work, however. This runs in an observer for the events checkout_cart_product_add_after and checkout_cart_product_update_after.

Edit: Brought in the code from removeItem() directly since it only works by ID, which may sometimes be blank. Now I'm unable to add items to the cart at all.

$version = floatval(Mage::getVersion());
$quantities = array();

foreach($quote->getAllVisibleItems() as $item) {
    if(!array_key_exists($item->getSku(),$quantities))
        { $quantities[$item->getSku()] = $item->getQty(); }
    else {
        $quantities[$item->getSku()] += $item->getQty();

        if($version >= 1.13)
            { $quote->deleteItem($item); }
        else {
            $item->isDeleted(true);
            if($item->getHasChildren()) {
                foreach($item->getChildren() as $child)
                    { $child->isDeleted(true); }
            }
            $parent = $item->getParentItem();
            if($parent)
                { $parent->isDeleted(true); }
            Mage::dispatchEvent('sales_quote_remove_item',array('quote_item'=>$item));
        }
    }
}

$quote->setTotalsCollectedFlag(false)->collectTotals()->save();

What am I doing wrong?

Best Answer

The function Mage_Sales_Model_Quote::removeItem() expects $itemId as a parameter, not an $item object.

So, your first foreach statement should be:

foreach($quote->getAllVisibleItems() as $item) {
    if(!array_key_exists($item->getSku(),$quantities))
        { $quantities[$item->getSku()] = $item->getQty(); }
    else {
        $quantities[$item->getSku()] += $item->getQty();
        $quote->removeItem($item->getId());
    }
}

This function is available in Enterprise 1.13. It gives the option to delete a quote item that doesn't have an id:

public function deleteItem(Mage_Sales_Model_Quote_Item $item)
{
    if ($item->getId()) {
        $this->removeItem($item->getId());
    } else {
        $quoteItems = $this->getItemsCollection();
        $items = array($item);
        if ($item->getHasChildren()) {
            foreach ($item->getChildren() as $child) {
                $items[] = $child;
            }
        }
        foreach ($quoteItems as $key => $quoteItem) {
            foreach ($items as $item) {
                if ($quoteItem->compare($item)) {
                    $quoteItems->removeItemByKey($key);
                }
            }
        }
    }

    return $this;
}

Related Solutions

Cart Event Observer Tax – Modify Tax Rate on Cart Quote Items and Recalculate

My suggestion would be to put an observer on the sales_quote_collect_totals_before event which is fired in the Mage_Sales_Model_Quote::collectTotals method before it starts the total collection process. Then from inside this observer method, iterate the quote items and change the tax class on the (already loaded) product object you can retrieve from the quote item.

After you set the information on the product object, whatever you do, DO NOT try and save it to the database. Having the tax class set as needed on the product object in memory will be good enough to have the collect totals logic found in Mage_Tax_Model_Sales_Total_Quote_Tax pickup which tax class it should base it's calculations on. Saving the product (as you seem to be trying to do in your code sample above) will cause major performance issues, will create race conditions in the calculation process, and is simply not good practice.

The reason that the events you are trying to work with are not enabling you to accomplish what you are trying to accomplish is because they all come after the total calculation, a process which is only be run once prior to saving the quote.

Worth pointing out about the collect totals process is that once run, without doing extra work, you cannot call it again to have it re-calculate based on changes you've made to the quote items. See this tie-bit I've taken from the blog series a colleage of mine recently put together on the collect totals process:

Now that you understand what occurs during the totals collection process, you may find it convenient or necessary to call it directly yourself. Before you start feeling too confident with using collectTotals for your own purposes, though, keep the following rule in mind:

Products cannot be added to the quote after collectTotals is run!

. . . unless the quote addresses' item caches are cleared.

Nearly every total model's "collect" method relies on fetching the quote items from the address and looping through them. The first time getAllItems is run on a quote address, the item collection is actually cached with a unique key, and it's this cached collection that is returned on subsequent calls.

If you do happen to have an inkling for really diving into the depths of how the collect totals process works, you can check out the first of the four part series on total collection here for more in-depth reading: Unravelling Magento's collectTotals: Introduction

To summarize, you need to be catching an event which runs before the collect totals process (and before getAllItems is called on the quote addresses) so that changes you make to the items will be used by the total collectors. I've not verified that the suggested sales_quote_collect_totals_before event runs before any calls to the getAllItems on the quote address, but I'm almost certain that it will work for what you need. But if not, hopefully I've provided enough context for you to figure out which event you need to catch to make it work.

Magento 1.7 – How to Copy Cart Items from One Quote to Another

Have a look at the merge() method in Mage_Sales_Model_Quote. I've never used it, but it should be something such as:

$quoteA = Mage::getModel('sales/quote')->load('5');
$quoteB = Mage::getModel('sales/quote')->load('77');

$quoteB->merge($quoteA);

// Also?
$quoteB->collectTotals()->save();

Zend_Debug::dump($quoteB->getAllItems());

// edit - I looked at what Magento does in the checkout session model, and it also calls collectTotals() and save(), so that's probably necessary as well.

Have a look also at Mage_Checkout_Model_Session

Best Answer

Related Solutions

Cart Event Observer Tax – Modify Tax Rate on Cart Quote Items and Recalculate

Magento 1.7 – How to Copy Cart Items from One Quote to Another

Related Topic