Comments on: Yet Another “Hiding Out-Of-Stock Products” Update

By: magento themes

magento themes — Thu, 14 Apr 2011 16:47:13 +0000

Bug fixing is a very important aspect in programming.It also caused several sleepless nights for me.Coding may be different i can get the idea.Thanks for sharing.

By: Colin Mollenhour

Colin Mollenhour — Tue, 07 Dec 2010 20:45:52 +0000

Yes, you would just need to add a product attribute (can be done via

Catalog > Attributes) and then modify the filter to account for that

attribute's value.

By: Jon

Jon — Tue, 07 Dec 2010 14:17:52 +0000

Hi Colin, Thanks for writing this module, it is very useful. One question, for certain products I want them to be able to be backordered. Is there anyway I can disable the hiding for certain products?

By: colin

colin — Thu, 14 Jan 2010 15:19:07 +0000

I would instead observe an event. Off the top of my head I think cataloginventory_stock_status_save_after would be a good starting point. If the status is changed to out of stock then disable the related product. I haven’t fully thought this through or tested but I think it could work. Note, then, that you would no longer need the Lucky_InStockOnly module.

By: point4design

point4design — Thu, 14 Jan 2010 12:59:05 +0000

For example, is there a script I could run (by cron) to set products to disabled if the stock equals zero?

By: point4design

point4design — Wed, 13 Jan 2010 21:23:41 +0000

Thanks for the quick response. Do you know a way then to easily ‘disable’ products that are out of stock?

By: colin

colin — Wed, 13 Jan 2010 18:46:48 +0000

This is the “Known Bug” which is really a limitation of Magento’s query design. I fixed this for a client (triercopenhagen.com) a few months ago but it took *forever* and the solution is not something that is readily distributable and it isn’t tested with grouped products and simple products and various other Magento features. The way the layered nav query is constructed is completely backwards from the original and it is quite complex so I don’t know if I can fully recommend going down that path.. I think Varien made a huge design mistake when they decided to not have the ability to hide out of stock products built into the core.

By: point4design

point4design — Wed, 13 Jan 2010 17:13:44 +0000

I’m having a problem with the extension. I have a ‘size’ attribute that I use on many of my configurable products. I have some of these configurable items in a clearance section and this is the only category I use layered navigation on (just activated it). I’ve noticed that in the layered navigation for the size attribute it is showing products that don’t have any stock. Category is here: http://www.wildernessrunning.com/shop/index.php/screaming-deals. If you click the ‘extra large’ in the layered nav on the right, the first product listed shouldn’t be there as it doesn’t have any extra large in stock. Can you help me figure this out?

By: colin

colin — Sat, 07 Nov 2009 17:32:29 +0000

saho, the filter can be applied like so:

Mage::getSingleton('cataloginventory/instockonly')->addInStockFilterToCollection($products);

Although the above is not necessary if you are already applying the typical "visibility" filter since the extension overrides those methods. Example:

Mage::getSingleton('catalog/product_visibility')->addVisibleInCatalogFilterToCollection($products);

That is the proper replacement for the addAttributeToFilter('visibility', 4) in your code.

By: saho

saho — Fri, 06 Nov 2009 21:54:01 +0000

Hi Colin,

I could not live without this extension. Great work!!
I was wondering if this class could be used in other places for instance I am using a gbase type feed but it pulls all configurable products regardless of whether or not they have out of stock associated products.

You mention above “First, the function that can be used on any product collection to filter out of stock products:”

Is it possible it could be used in a feed script in this way?

//GET THE PRODUCTS
$products = Mage::getModel(‘catalog/product’)->getCollection();
$products->addAttributeToFilter(‘status’, 1); // enabled
$products->addAttributeToFilter(‘visibility’, 4); // catalog, search
$products->addAttributeToSelect(‘*’);
$prodIds = $products->getAllIds();

Thanks in advance for any insight.