Magento – checkQuoteItemQty using wrong qty for configurable product

event-observerinventory

I have a configurable product with associated simple products which I want to keep Inventory of. I set all the inventories to 10.

I've noticed that if I try to add 4 products to my cart, it won't let me. It says The requested quantity for [product] is not available.

I started digging for a solution.
I found that in app/core/Mage/CatalogInventory/Model/Stock/item.php in the function checkQuoteItemQty, I'm getting funny values.

This function gets called from the observer located one folder up, Observer.php.

I started logging the parameters fed to checkQuoteItemQty and have found that it is squaring my requested quantities. If I order 3 products, it checks inventory for 9. When I order 4, it looks for 16 (which is above the limit of 10).

The observer runs for each Item in the quote. The quote has the simple product AND the configurable product in it. And the observer has a line
$rowQty = $quoteItem->getParentItem()->getQty() * $qty; that is multiplying the two, and yielding the incorrect rowQty.

This seems too asinine to be the default behavior. Do I have something configured wrong? Why are the simple AND configurable products in the quote? Or why do they both have an qty value (of 4, in my case)

Best Answer

I modified core to fix this.

I know it's bad practice and maybe there's a downside. But I did this months ago and I don't think we've noticed any other problems...

Near line 478 in the Observer.php file, I commented out the original code and replaced it with my own that does not containt the * $qty. Here it is:

/**
 * This change made by __[myname]______ because the Qty was being squared
 */
 //$rowQty = $quoteItem->getParentItem()->getQty() * $qty;
 $rowQty = $quoteItem->getParentItem()->getQty();

Good luck.

Related Solutions

Event Observer – Observe Product Quantity Changes Only

OK, here's the function that triggers the event in 1.9.0.2

public function setQty($qty)
{
    $qty = $this->_prepareQty($qty);
    $oldQty = $this->_getData('qty');
    $this->setData('qty', $qty);

    Mage::dispatchEvent('sales_quote_item_qty_set_after', array('item' => $this));

    if ($this->getQuote() && $this->getQuote()->getIgnoreOldQty()) {
        return $this;
    }
    if ($this->getUseOldQty()) {
        $this->setData('qty', $oldQty);
    }

    return $this;
}

So, the observer is only being passed the current state of the item. If it's triggering everytime you load the cart then maybe this won't work, but you can use $item->dataHasChangedFor('qty') to see if it's changed since the object was loaded, and $item->getOrigQty() to get what it was when it was loaded.

Magento – Get Configurable Product Price from Simple Product in Order

In this case you need to use call parent item's row().As per magento when a configurable product is ordered,then there are two rows has been saved at sales_flat_order_item.One is configurable product data product_id and order item prices and another is simple products details.So need data fetch data from Configurable products.

So need to change

'UnitPrice'=> $orderItem->getParentItem()?$orderItem->getParentItem()-> getPrice():$orderItem-> getPrice(),

From

UnitPrice'           => $orderItem -> getPrice(),

Also To:

$linesTaxAmountSum += $orderItem->getParentItem()?$orderItem->getParentItem()->getTaxAmount():$orderItem->getTaxAmount();

From:

$linesTaxAmountSum += $orderItem->getTaxAmount();

Best Answer

Related Solutions

Event Observer – Observe Product Quantity Changes Only

Magento – Get Configurable Product Price from Simple Product in Order

Related Topic