кастомный getProducts для категории

а как к этому отнесутся фильтры? 

мне кажется подобные модели трогать категорически запрещено не модами 

я бы сделал чтоб getProduct и getProducts принимали параметры типа не считать скидку, не считать рейтинг и тп

28 минут назад, spectre сказал:

а как к этому отнесутся фильтры? 

 

мне кажется подобные модели трогать категорически запрещено не модами 

 

я бы сделал чтоб getProduct и getProducts принимали параметры типа не считать скидку, не считать рейтинг и тп

дримфильтр на такой кастомизации работает исправно. окфильтр надо будет потестить

про то, что надо делать модом согласен

Вообще такой запрос полезен там, где нужно сделать подборку товаров аля модуль какой-нибудь вывода товаров

Подправьте код со временем.

Никогда такого не было и вот опять.
Все результаты запросов актуальны в формате вывода SELEC SQL_NO_CACHE.

Где EXPLAIN запросов  с планом запроса ?

Что за цифры с потолка, как же group by и sort по сторонним таблицам.
Кто то тут крепко свистит, да еще и предлагает какую то дичайшую несовместимую дичь, которая приведет к конфликтам со всем!

1 час назад, Yoda сказал:

Кто то тут крепко свистит, да еще и предлагает какую то дичайшую несовместимую дичь, которая приведет к конфликтам со всем!

во тут реально конструктивно было. конфликты будут с микроволной печью в первую очередь. затестил цифры с sql_no_cache и разрыв в первом байте стал еще вкуснее на каких-то 5-10мс

Отображение строк 0 - 49 (50 всего, Запрос занял 0.0276 сек.)
SELECT SQL_NO_CACHE p.image, p.price, p.quantity, pd.name , (SELECT price FROM oc_product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '1' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special, p.product_id FROM oc_product_to_category p2c LEFT JOIN oc_product p ON (p2c.product_id = p.product_id) LEFT JOIN oc_product_description pd ON (p.product_id = pd.product_id) LEFT JOIN oc_product_to_store p2s ON (p.product_id = p2s.product_id) WHERE pd.language_id = '1' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '0' AND p2c.category_id = '118' GROUP BY p.product_id ORDER BY (p.quantity>0) DESC,p.sort_order ASC, LCASE(pd.name) ASC LIMIT 0,50

Отображение строк 0 - 49 (50 всего, Запрос занял 0.0200 сек.)
SELECT SQL_NO_CACHE p.product_id, (SELECT price FROM oc_product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '1' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special, p.product_id FROM oc_product_to_category p2c LEFT JOIN oc_product p ON (p2c.product_id = p.product_id) LEFT JOIN oc_product_description pd ON (p.product_id = pd.product_id) LEFT JOIN oc_product_to_store p2s ON (p.product_id = p2s.product_id) WHERE pd.language_id = '1' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '0' AND p2c.category_id = '118' GROUP BY p.product_id ORDER BY (p.quantity>0) DESC,p.sort_order ASC, LCASE(pd.name) ASC LIMIT 0,50

как полагаешь разница в 0.0076 секунды сможет компенсировать 50 таких запросов(ниже)?

Отображение строк 0 - 0 (1 всего, Запрос занял 0.0016 сек.)
SELECT SQL_NO_CACHE DISTINCT *, pd.name AS name, p.image, (SELECT md.name FROM oc_manufacturer_description md WHERE md.manufacturer_id = p.manufacturer_id AND md.language_id = '1') AS manufacturer, (SELECT price FROM oc_product_discount pd2 WHERE pd2.product_id = p.product_id AND pd2.customer_group_id = '1' AND pd2.quantity = '1' AND ((pd2.date_start = '0000-00-00' OR pd2.date_start < NOW()) AND (pd2.date_end = '0000-00-00' OR pd2.date_end > NOW())) ORDER BY pd2.priority ASC, pd2.price ASC LIMIT 1) AS discount, (SELECT price FROM oc_product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '1' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special, (SELECT points FROM oc_product_reward pr WHERE pr.product_id = p.product_id AND customer_group_id = '1') AS reward, (SELECT ss.name FROM oc_stock_status ss WHERE ss.stock_status_id = p.stock_status_id AND ss.language_id = '1') AS stock_status, (SELECT wcd.unit FROM oc_weight_class_description wcd WHERE p.weight_class_id = wcd.weight_class_id AND wcd.language_id = '1') AS weight_class, (SELECT lcd.unit FROM oc_length_class_description lcd WHERE p.length_class_id = lcd.length_class_id AND lcd.language_id = '1') AS length_class, (SELECT AVG(rating) AS total FROM oc_review r1 WHERE r1.product_id = p.product_id AND r1.status = '1' GROUP BY r1.product_id) AS rating, (SELECT COUNT(*) AS total FROM oc_review r2 WHERE r2.product_id = p.product_id AND r2.status = '1' GROUP BY r2.product_id) AS reviews, p.sort_order FROM oc_product p LEFT JOIN oc_product_description pd ON (p.product_id = pd.product_id) LEFT JOIN oc_product_to_store p2s ON (p.product_id = p2s.product_id) LEFT JOIN oc_manufacturer m ON (p.manufacturer_id = m.manufacturer_id) WHERE p.product_id = '2510842' AND pd.language_id = '1' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '0'

зачем тебе explain? смешной. условия запроса никак не менялись. экплейн идентичен для двух этих запросов, я думал это само собой понятно... но не тебе 

не убирать дискаунт, чтобы добиться хорошей совместимости будет максимально логично, но мне интересно сделать легче и быстрее

Вопросов больше нет!

А смысл рассматривать частный случай?

В 05.08.2021 в 17:35, kJlukOo сказал:

окфильтр надо будет потестить

отпишитесь пожалуйста после тестов

В 09.08.2021 в 00:05, max1985 сказал:

отпишитесь пожалуйста после тестов

Должен работать как и прежде.

По поводу этой разработки.

getProduct не просто так сделан в цикле. Это единственный и правильный путь для стандартизации структуры данных. Один метод на все случаи.

Логика такая: наборы product_id ищем любым удобным способом, а сами конечные элементы выбираем одним (!) на всю систему методом с нужными нам полями. То есть не плодим себе кучу мест, где нужно добавлять новые поля или форматировать их, следить за всем этим.

Чего не хватает в этом методе, так это кэширования с ограниченным временем жизни. И будь-то вызов товаров из блока "Похожие", или "Просмотренные",  любой другой модуль, всегда нужно искать product_id максимально простым запросом без кэширования, а сам product брать из кэша.

Также в бэкенде изменяя товар не нужно удалять весь кэш товаров, достаточно delete('product.' . $product_id) и все остальные останутся целыми.

Ну и ко всему прочему, сам по себе запрос выборки одного товара по primary key со всем навесным не такой уж тяжелый. 

Ваш метод тоже имеет свои преимущества, но кроме списка товаров в категории есть же еще масса других страниц, где можно всегда обращаться только к одному методу получения товара, а внутри него уже делаем что хотим, кэшируем или разбиваем большой запрос на разные (может со своими условиями), форматируем и т.д.