Dmytro
просто рано или поздно доберемся делать нормально и будет печалька
Aliaksandr
Ничего не понял, что вы там хотите, ну да как хотите :)
Pathologic
смотри, если плагин ничего не возвращает, то выполняется обычный запрос как сейчас
Pathologic
а если плагин вернет массив с тв, то этот запрос не будет выполняться
Pathologic
вот и все
Aliaksandr
Какой массив с тв? Дублировать в каждом плагине тот запрос с пятью джоинами для получения тв? Зачем?
Aliaksandr
В системе двести событий из которых хорошо если десять используются, а для нормальной фильтрации сделать два события на тв все против. Не хотите как хотите, я себе сделаю, а люди пусть и дальше скрывают тв с jquery
Aliaksandr
Рассуждая "модх для многого непригоден":)
Pathologic
так если делать, то делать нормально как-то, а не хаки и яма-теги
Pathologic
что тебя смущает в запросе с 5 джойнами?
Aliaksandr
То, что кроме where там модифицировать нечего, так что и переписывать его нет смысла в плагин. Особенно если завтра сам запрос там поменяется
Pathologic
зато появится возможность использовать данные из других таблиц
Pathologic
выводить их как тв
Aliaksandr
Боюсь что клиенты до этой возможности не доживут:)
Aliaksandr
В чем конкретно ненормальность, встречный вопрос?
Aliaksandr
В том что решаем конкретную проблему модх, а не подключаем твиг, другую базу и другой парсер?:)
Pathologic
по идее так для каждого запроса нужно делать плагины, чтобы где-то where поменять, где-то order
Pathologic
где-то список полей
Pathologic
ненормально привязываться к запросу, я так думаю
Pathologic
в общем мое дело обосрать, а если оставшиеся три человека проголосуют за, то и хрен с ним ((:
Pathologic
по мне так это из той же серии, что использовать поле donthit для того, чтобы скрывать дочерние
Aliaksandr
Ну по большому счету мне и вообще никакого дела до этого нет, т.к. себе я и так сделаю :)
Aliaksandr
Я им и скрываю :)
Aliaksandr
Понятно что без того же crecourse/customtables и тому подобного функционала это нафиг не нужно, но не всем нравится в дереве видеть 20000 товаров
Pathologic
я не про то, что скрывать дочерние это плохо, а про то что пропихнули хак, вместо того, чтобы сделать еще одну колонку
Aliaksandr
Вообще конечно интересно было бы услышать как в принципе в эво каталоги люди делают, все молчат что-то
Aliaksandr
Неужели прям все пишут свои таблицы и логику и только я интересуюсь штатными возможностями)
Pathologic
🙃
Pathologic
сомневаюсь я что-то, что все пишут
Pathologic
cresource видел один раз только, кроме тебя
Aliaksandr
Ну так а какой выход? Делать тысячу шаблонов товаров с разными тв?
Aliaksandr
Как потом эти товары в админке смотреть, фильтровать, сортировать вообще хз:)
Aliaksandr
Какую дичь заказчику втирать. Если ему нужнн новый тип товара
Pathologic
я на рево делал (:
Pathologic
там из коробки это все
Pathologic
это для простеньких каталогов
Aliaksandr
Да ну, у меня из одинаковых тольео картинка, цена, бренд, категории
Aliaksandr
Тем более, что делать даже если один два параметра отличаются? Говорить вот это тут не заполняйте, это не для этого?:)
Сергей
Да ну, у меня из одинаковых тольео картинка, цена, бренд, категории
цвет, вес, старая цена, лейбл, состояние, доступность, артикул, комплектующие, доп. изображения.. и там ещё много подобных
Сергей
Сергей
все свойства товарв складывать в одну категорию, через templateEdit2 выводить твшки из нужной категории на определённой вкладке (это если они доступны)
И через кастом ТВ, сделать выбор свойств товара
Сергей
в templateEdit2 поставить условие.. если твшка пустая, то не показывать
Pathologic
Aliaksandr
Anonymous
Я бы вешал событие на "до получения ТВ"
Anonymous
Что бы вносить ограничения в запрос и меньше данных выбирать
Сергей
»вопрос в том, что нафига выбирать из базы 2000 тв, чтобы вывести 5
хм... логично)))
но они привязаны к доку)
но событие по сути для твшек не нужно, там есть событие же.
https://github.com/evolution-cms/evolution/blob/develop/manager/actions/mutate_content.dynamic.php#L638
Anonymous
На весь запрос есть же событие
Pathologic
Pathologic
Сергей
@webber_12
Почему всю форму не заменить по "OnDocFormTemplateRender" ?
Aliaksandr
Нет там никакого подходящего события
Pathologic
Aliaksandr
Никто не будет всю логику переписывать. Которя к тому же может поменяться
Сергей
Aliaksandr
Это путь к геморрою:)
Pathologic
если бы страница была шаблонизирована, то еще можно ctrl+c
Aliaksandr
Причем прямой))
Pathologic
а так это пиздец просто
Pathologic
я после внедрения вкладок для тв не понимаю даже что там как
Pathologic
переменные типа $vs тоже способствуют
Aliaksandr
Вот был у меня в плагине гет pid, а его взяли и выпилили
Сергей
Это путь к геморрою:)
посмотри templateEdit2, всё там работает, и не менял уже ничего там давно, и годами всё прекрасно пашет.
Aliaksandr
Сергей
Aliaksandr
В общем ясно, в рево надо иметь свой магазин и сообщество, в эво достаточно фильтра/формы и системы подмены тв :)
Anonymous
Чёт у меня вопросик к этому событию. А каким именно масивом нужно возвращать данные?
Pathologic
таким, чтобы дальше вывелось
Aliaksandr
К какому
Anonymous
Хмммм, дайте подумать, а какой пул тут сейчас обсуждали?
Aliaksandr
Обсуждали как минимум три события
Anonymous
Вот я сижу и думаю, как мне построить массив, что бы получился нормальный where
Aliaksandr
Если пул, то просто строку дополнительную для where
Сергей