Форум ReadyScript — Дополнительный фильтр по наличию фотографий товара

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Максим) — Tue, 11 Oct 2016 04:46:08 +0000

Финальный вариант, если кого вдруг заинтересует:

$adv_config = \RS\Config\Loader::byModule("custom"); /*Проверяем настройку показа товаров без фотографий*/
        if ($adv_config['hide_nophoto_product'] == 1) {

            $q = $this->queryObj();

            if ($q->select === '*') {
                $q->select = null;
                $q->select('A.*, X.*');
            }

            $q->join(new \Photo\Model\Orm\Image, 'I.linkid = A.id', 'I');
            $q->groupby('A.id');

        }

Выяснилось, что при использовании подзапроса в секции WHERE - ломается напрочь поиск и крайне медленно выполняется запрос в случае использования фильтров на странице каталога. Сделал в итоге через JOIN и добавил условие, так как при поиске должны выбираться только поля из таблицы `A`, а таблица `X` - вообще не джойнится при поиске. Косяков не обнаружено и скорость отличная.

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Закусило Александр) — Fri, 30 Sep 2016 16:18:01 +0000

ВЫ можете через запятую указать те таблицы которые нужны в SELECT.

A.*, B.* и т.д.

Т.е. исключите картинки.

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Максим) — Fri, 30 Sep 2016 15:30:31 +0000

Все, теперь я доволен.

$adv_config = \RS\Config\Loader::byModule("custom");
        if ($adv_config['hide_nophoto_product'] == 1) {
            $qImage = new \RS\Orm\Request();
            $qImage
                ->select('linkid')
                ->from(new \Photo\Model\Orm\Image(), 'Img');
            $this->queryObj()
                ->where("A.id IN (".$qImage->toSql().")");

        }

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Максим) — Fri, 30 Sep 2016 15:17:35 +0000

только вот мне не нравится, что замес получился с sql синтаксисом. Хочется как-то в отдельный запрос вывести то, что в скобках, но не пойму, как.

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Максим) — Fri, 30 Sep 2016 15:15:17 +0000

Не, так не пойдет. Тогда приджойненные цены не выведутся. Но я нашел способ. Вместо join использовал where с подзапросом и условием, что id товара находится в результатах этого запроса.

$this->queryObj()
                ->where("A.id IN (SELECT `linkid` FROM `readyscriptyoung`.`rs_images`)");

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Закусило Александр) — Fri, 30 Sep 2016 15:07:32 +0000

Как и во всех SQL запросах, нужно указать то что собираетесь выбирать конкретно.

SELECT A.*

->select('A.*')

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Максим) — Fri, 30 Sep 2016 13:13:33 +0000

Александр, спасибо. Заработало. Вот текст запроса. Новая проблема - после JOIN в поле id товара передается теперь id изображения. Соответственно, в каталоге начинается вакханалия, ибо везде в переменной $product.id теперь id картинки. Как-то можно исключить из секции SELECT поле Img.id?

SELECT * FROM `readyscriptyoung`.`rs_product` as A 
INNER JOIN `readyscriptyoung`.`rs_product_x_dir` as X ON A.id = X.product_id 
INNER JOIN `readyscriptyoung`.`rs_images` as Img ON A.id = Img.linkid 
WHERE (`A`.`site_id` = '1') AND `X`.`dir_id` IN ('675','676','677','678','679','680','681','682','683','684','685','686','687','688','689','690','691','692','834','835','837','845','846','847','848','849','850','851','852','853','854','855','907','674','694','695','696','698','699','700','701','702','703','838','856','857','858','875','876','900','901','902','904','905','906','693','673','672') AND `A`.`public` = '1' AND `A`.`num` > '0' 
ORDER BY dateof DESC

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Максим) — Fri, 30 Sep 2016 10:16:48 +0000

Ух ты, похоже, что да, оно. Да и из своего модуля все-таки кошернее. Отпишусь по-результату. Спасибо.

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Закусило Александр) — Fri, 30 Sep 2016 10:04:31 +0000

filterSection по идее срабатывает тогда когда вы пытаетесь фильтровать что-то. Вам нужен вывод сразу во всём каталоге только тех что с фото? Я правильно понимаю?

Если да, то тогда лучше всего подвеситься из своего модуля на хук страницы со списком товаров. И уже у объекта API добавить нужный фильтр.
controller.beforeexec.catalog-front-listproducts

Там первым аргументом будет.

/**
* @param \Catalog\Controller\Front\ListProducts $controller - объект контроллера
* @param string $action - наименование метода, который отработал (использовать для нужного)
*/
public static function controllerBeforeexecCatalogFrontListproducts($controller, $action)
{
     $q = $controller->api->queryObj(); //Потому что у фронт контроллера публичное свойство $api, в котором объект API товаров
     //И нужные мне действия с \RS\Orm\Request в $q, т.е. обычным запросом
}

Попробуйте, если это оно.

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Максим) — Fri, 30 Sep 2016 09:37:53 +0000

Ничего не выводит, похоже, что я совсем не туда вставляю код. Помогите найти, где происходит первичное формирование запроса для вывода каталога.

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Закусило Александр) — Thu, 29 Sep 2016 16:38:39 +0000

Ах да там же у Вас innerjoin. В проверьте у Вас в эту ветку заходит.
Можно вывести итоговый SQL так:

var_dump($q->toSql());

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Максим) — Thu, 29 Sep 2016 14:22:50 +0000

А разве сам по себе join не подразумевает, что в выборку попадает только пересечение? Таким образом моя логика говорит мне, что останутся в выборке только товары, у которых есть соответствие в таблице с изображениями. Разве не так?

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Закусило Александр) — Wed, 28 Sep 2016 16:22:34 +0000

Ну прибавили Вы таблицу с фото. А где WHERE с условием?

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Максим) — Wed, 28 Sep 2016 15:14:16 +0000

Не работает ))

Re: Дополнительный фильтр по наличию фотографий товара

null@example.com (Закусило Александр) — Wed, 28 Sep 2016 15:13:07 +0000

И в чём вопрос?)