Exclude out of stock products from cross selling module

[AcidLava]

Hello,

I'm looking for a way to exclude out of stock produts from the native cross selling module. I want to keep these products displayed in category product lists only.

Thanks,

Regards.

[Daresh]

Hello,

elsewhere you said that you are setting visibility for out of stock products to `none` so you can try to modify the PHP code of the module changing:

AND product_shop.active = 1

to:

AND product_shop.active = 1 AND product_shop.visibility != \'none\'

Of course to make it correctly, it should be done with a module override, but for first testing you can change it directly in the module and see if it works.

[AcidLava]

Hello,

I tried but out of stock products still display.

            $order_products = Db::getInstance((bool) _PS_USE_SQL_SLAVE_)->executeS('
                SELECT DISTINCT od.product_id
                FROM ' . _DB_PREFIX_ . 'order_detail od
                LEFT JOIN ' . _DB_PREFIX_ . 'product p ON (p.id_product = od.product_id)
                ' . Shop::addSqlAssociation('product', 'p') .
                $sql_groups_join . '
                WHERE od.id_order IN (' . $list . ')
                AND od.product_id NOT IN (' . $list_product_ids . ')
                AND product_shop.visibility IN (\'both\',\'catalog\')
                AND product_shop.active = 1 AND product_shop.visibility != \'none\'
                ' . $sql_groups_where . '
                ORDER BY RAND()
                LIMIT ' . (int) Configuration::get('CROSSSELLING_NBR')
            );
        }

 

[AcidLava]

Up.

[Daresh]

 I just noticed that the is already that line there:

AND product_shop.visibility IN (\'both\',\'catalog\')

AND product_shop.visibility IN (\'both\',\'catalog\')

So no changes should be required as long as your out of stock products have visibility set to 'none', so you need to make sure they do.

[AcidLava]

Hello,

I want to keep the out of stock product displayed in categories list. I use your module for this. But still, I want to remove them from cross selling module.

[Daresh]

So try to change:

AND product_shop.visibility IN (\'both\',\'catalog\')

to

AND product_shop.visibility IN (\'both\')

 

[AcidLava]

It still don't work.

[Daresh]

After this change, a product with its visibility set to 'catalog' should be excluded from the cross-selling module. Please double-check by taking a product that is incorrectly displayed there and verifying its visibility setting.

[AcidLava]

Hello @Daresh,

Yes, the out of stock product is set to 'Catalog only'. Stills appear in the cross selling list...

With this code :

AND product_shop.visibility IN (\'both\')

[AcidLava]

It's on Prestashop 8.1.7.