SQL query improvement to check active products

[Prestaplugins]

Questions	Answers
Description?	SQL query optimization, so "active" flag is only checked on viewed products instead of the whole catalog.
Type?	improvement
BC breaks?	no
Deprecations?	no
Fixed ticket?
How to test?	See beelow
Related Pr	#37

How to test :
Activate the "Viewed product" module
Turn debug profiling ON
Enter a product page
Then go to another one
Check the query in the debug profiling, it must contains "AND p.id_product IN ( .... )" with a small number of rows returned.

[mohaidoss]

Perhaps it's better to keep the same method name, and the same default behavior which is a select without condition.

And only fair to co-author @joshjackson 😀

[Touxten]

The addSqlRestriction method: accepts the first parameter as false only since 8.0.0. You must modify these files for the tests to work.

https://github.com/PrestaShop/ps_viewedproduct/tree/dev/tests/phpstan

- '#Parameter \#1 \$share of static method ShopCore::addSqlRestriction\(\) expects int, false given.#'

@mohaidoss

Perhaps it's better to keep the same method name

The method is private no problem but i aggree you. If someone has overridden the private method, their override will no longer work.

the same default behavior which is a select without condition.

The shop condition optilmize the request

Change on tests

[jf-viguier]

@PrestaShop/qa-functional can we have your eyes here ? it's an easy one.

[mohaidoss]

Up voted

[SiraDIOP]

Hello @Prestaplugins,

Thanks for your PR,

I tested it and it's good for me

Enregistrement.de.l.ecran.2026-02-04.a.17.28.51.mov

Thank you