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