Modify getProducts

[AndyLaci]

Hi everyone,

 

I would need help with modifying this function. I would like to display products in product-list which are not out of stock (all of their attributes have quantity 0), but would be seen if user somehow get to old product by old link.

 

Easier: old products stay, but are not wisible in product list.

 

 

 

public static function getProducts($id_lang, $start, $limit, $order_by, $order_way, $id_category = false,

$only_active = false, Context $context = null)

{

if (!$context)

$context = Context::getContext();

 

$front = true;

if (!in_array($context->controller->controller_type, array('front', 'modulefront')))

$front = false;

 

if (!Validate::isOrderBy($order_by) || !Validate::isOrderWay($order_way))

die (Tools::displayError());

if ($order_by == 'id_product' || $order_by == 'price' || $order_by == 'date_add' || $order_by == 'date_upd')

$order_by_prefix = 'p';

else if ($order_by == 'name')

$order_by_prefix = 'pl';

else if ($order_by == 'position')

$order_by_prefix = 'c';

 

if (strpos($order_by, '.') > 0)

{

$order_by = explode('.', $order_by);

$order_by_prefix = $order_by[0];

$order_by = $order_by[1];

}

$sql = 'SELECT p.*, product_shop.*, pl.* , m.`name` AS manufacturer_name, s.`name` AS supplier_name

FROM `'._DB_PREFIX_.'product` p

'.Shop::addSqlAssociation('product', 'p').'

LEFT JOIN `'._DB_PREFIX_.'product_lang` pl ON (p.`id_product` = pl.`id_product` '.Shop::addSqlRestrictionOnLang('pl').')

LEFT JOIN `'._DB_PREFIX_.'manufacturer` m ON (m.`id_manufacturer` = p.`id_manufacturer`)

LEFT JOIN `'._DB_PREFIX_.'supplier` s ON (s.`id_supplier` = p.`id_supplier`)'.

($id_category ? 'LEFT JOIN `'._DB_PREFIX_.'category_product` c ON (c.`id_product` = p.`id_product`)' : '').'

WHERE pl.`id_lang` = '.(int)$id_lang.

($id_category ? ' AND c.`id_category` = '.(int)$id_category : '').

($front ? ' AND product_shop.`visibility` IN ("both", "catalog")' : '').

($only_active ? ' AND product_shop.`active` = 1' : '').'

ORDER BY '.(isset($order_by_prefix) ? pSQL($order_by_prefix).'.' : '').'`'.pSQL($order_by).'` '.pSQL($order_way).

($limit > 0 ? ' LIMIT '.(int)$start.','.(int)$limit : '');

$rq = Db::getInstance(_PS_USE_SQL_SLAVE_)->executeS($sql);

if ($order_by == 'price')

Tools::orderbyPrice($rq, $order_way);

 

foreach ($rq as &$row)

$row = Product::getTaxesInformations($row);

 

return ($rq);

}

 

 

This would save me tons of time, because the database is optimised, but it is bit hard to get to understand it.

[vekia]

hello

 

i've got nice and easy in use solution for this, check please this tutorial: disable out of stock products

in my opinion it's better than modyfing the source codes of prestashop

[AndyLaci]

hello i've got nice and easy in use solution for this, check please this tutorial: disable out of stock products in my opinion it's better than modyfing the source codes of prestashop

 

This solution is good, but I want other thing. This disales the view of the product, I would like just to hide the product from product list with ability to find it on google or relink to product, not to deactivate it from all views, maybe it could be done in productController with this solutiun plus enabling to view deactivated product, but it is not the best solution. The best would be changing the code, where the products load to product-list and filter them by quantity greater than 0.

 

Changing the core can be done in override section am I right?

[vekia]

yup, that's right. Now i know what you exactly expect, thanks for explanation.

what ps version you use? everything depends on it

[AndyLaci]

I use 1.5.3

[vekia]

to the query that you mentioned above add ps_stock_available AS stock via LEFT JOIN

add also WHERE stock.quantity > 0

 

if you've got any questions related to this case - feel free to write

[AndyLaci]

wouldnt there be problem with attributes? We are talking about shop with clothes, so allmost every product have some attributes. I found somewhere in one tutorial that you can remove the products by if statement which was something like: if quantity 0 and all attributes quantity 0 then dont show. Do I have to care about attribute quantyties in this case? Or prestashop set this stock.quantity automatically when there is 0 of every attribute?

[AndyLaci]

please, do you have some solution for this, or should I start digging up in the prestashop database to reach this result?

[vekia]

each attribute is a "different" product in the stock table, it should work as you expect.