Ilya
=)
Ilya
Меня уже за список такой чуть не забанили, ну правда он касался 2.0, хотя скорее всего уже 3.0 =)
Anonymous
Но там есть несколько моментов которые мне не нравятся
Ilya
Сцыкотно чет =)
Andrey
А я такой "Э-э-э?" =)
Pathologic
как использовать, понятно из статьи на modx.im, вот именно некоторые моменты сильно портят всю затею
Alexander
Ilya
Всеравно все что не пиши, бракуют, говорят это надо только нубасам, а мы прогеры все пишем руками и текстом
Ilya
=)
Ilya
Надо тебе список доков в тв пиши @eval return Че надо =)
Anonymous
Dmytro
Ну вот нет списка нет и решений все ж просто :)))
Dmytro
Anonymous
Ilya
Я сам себя убивал за это много раз =)
Dmytro
+ Больше спасибо !!!
P.s а то тут многие уже 10 лет с Evo а хоть бы как то помогли
Dmytro
да я не шарю, я не один сайт на Evo не сделал =)
Dmytro
Только 1 поддерживаю
Dmytro
так прикольная CMS, удобная, быстрая, простая
Dmytro
Понравилась CMS, поглядываю на git, на форум. Да и интересно следить за прогрессом, все прозрачно, не как с MODX LLC =)
Dmytro
В общем, еще раз спасибо за труд!
Dmytro
Dmytro
Вот реально благодаря таким людям и хочеться сделать EVO очень крутой!
Alexander
Ruslan
Ilya
Да мы вас тут и так хвалим =)
Пункт из списка, вшейте в ядро ) https://c2n.me/435O2kC Извиняюсь за рево =)
Dmytro
Dmytro
А то под пересланными сообщениями бот не понял )
Alexander
Anonymous
Ilya
Дыр бояться - в лес не ходить +)
Alexander
Потенциальная дыра при неправильном использовании
ага, во
eval ( string $code ) : mixed
Выполняет строку, переданную в параметре code, как код PHP.
Предостережение
Языковая конструкция eval() может быть очень опасной, поскольку позволяет выполнить произвольный код. Использование данной функции не рекомендуется. Если вы полностью убеждены, что нет другого способа, кроме использования этой конструкции, обратите особое внимание на то, чтобы не передавать какие-либо данные, предоставленые пользователем, без предварительной проверки.
Ilya
Получается все првильно, так как "полностью убежден, что нет другого способа" +))
Alexander
Pathologic
Aliaksandr
ага, во
eval ( string $code ) : mixed
Выполняет строку, переданную в параметре code, как код PHP.
Предостережение
Языковая конструкция eval() может быть очень опасной, поскольку позволяет выполнить произвольный код. Использование данной функции не рекомендуется. Если вы полностью убеждены, что нет другого способа, кроме использования этой конструкции, обратите особое внимание на то, чтобы не передавать какие-либо данные, предоставленые пользователем, без предварительной проверки.
это очень опасная функция, ее почти нигде не используют)
Ilya
Aliaksandr
ну разве что тут https://github.com/evolution-cms/evolution/blob/1.4.x/manager/includes/document.parser.class.inc.php#L1919 или тут https://github.com/evolution-cms/evolution/blob/1.4.x/manager/includes/document.parser.class.inc.php#L1965 ))
Dmytro
https://github.com/search?l=PHP&q=eval%28&type=Repositories
Ilya
Да, клянусь я про эту функцию даже не знал, до того как пересел на модикс =))
Alexander
Ilya
Всегда можно было написать <?php echo 1111; ?> Но в модиксе сами понимаете =))
Aliaksandr
значит не такая уж и опасная)
как только видит точку с запятой - сразу начинает эвалить - https://github.com/evolution-cms/evolution/blob/1.4.x/manager/includes/document.parser.class.inc.php#L1964-L1966 - а ты говоришь не опасно )
Anonymous
@webber_12 о я глянул бага почти исправлена. Но это просто эпик 🙂
Alexander
Ilya
Anonymous
@webber_12 просто перешлю
Anonymous
https://github.com/webber12/eFilter/blob/master/assets/snippets/eFilter/eFilter.class.php#L1096-L1123
Вот он через доклистер получает всех детей
Anonymous
https://github.com/webber12/eFilter/blob/master/assets/snippets/eFilter/eFilter.class.php#L1083
Потом вот он подтягивает из мультикатегории
Anonymous
А теперь внимание на строку выше и там ссылка на статью Макса где рассказано, что для того чтобы подтянуть через доклистер это дело. Нужно добавить в доклистер
&multiCategories=`1`
Anonymous
Ну и кто из нас тупой и не умеет читать документацию ссылки на которую вставляет прямо в свой код?
Anonymous
И городит функции которые уже есть в используемых им решениях?
Alexander
Aliaksandr
какое дело куда подтягивает, причем тут статья и твоя ссылка - ну вообще не в дугу - как обычно ))
Aliaksandr
а что плохого?
эта тупица предлагает вместо одного запроса в базу запустить еще раз весь доклистер в режиме апи, а потом разбирать json от него и пихать в массив )
Anonymous
а что плохого?
Зачем городить дополнительные функции, если они уже есть в решениях и достаточно просто добавить одну переменную
Anonymous
https://github.com/webber12/eFilter/blob/master/assets/snippets/eFilter/eFilter.class.php#L1096-L1123
Для умных, я предлагаю сюда параметр добавить 🙂
Anonymous
В тот вызов который уже есть
Aliaksandr
он же не понимает всю логику работы, так за одну штуку, как ему кажется, "зацепился" и не выдержал - две недели молчал, а тут поперло :)
Anonymous
А не ещё раз дёргать доклистер и подтягивать дополнительно
Aliaksandr
и как обычно с этих рассуждений можно разве что поржать в голос
Anonymous
Серьёзно? Т.е. эта функция делает что-то больше чем возврат списка детей? https://github.com/webber12/eFilter/blob/master/assets/snippets/eFilter/eFilter.class.php#L1096-L1123
Anonymous
Расскажите что же она такого делает ещё
Anonymous
И что мешает в список параметров подмешать
&multiCategories=`1`
кроме собственной гордости?
Anonymous
Dmytro
Ну да как то не логично:
Dmytro
https://take.ms/Gcucr
Ilya
Anonymous
Anonymous
При этом сам бревна в своём глазу не замечает
Aliaksandr
Anonymous
Ну ок, говнокодером не называл
Dmytro
https://take.ms/Gcucr
Так же не понимаю зачем вторая функция если в первую можно передать параметр мультикатегорийности и все
Anonymous