Aliaksandr
с другой стороны никто не мешает запустить на шопкипере (раз уж все готово), а если тема взлетит, то на вырученные деньги или поддерживать шопкипер или переделать на коммерс )
Aliaksandr
если нет четкой уверенности что это принесет деньги - то может и не стоит торопиться с переписыванием
Senya
Да, разумно, торопиться не буду, а для того чтобы комерс пощупать , новый проект заведу )
Senya
Спасибо ,что помогли собрать мысли в кучу)
Aliaksandr
зачем тут про $tvsArray забыли - поломали мне кусок ефильтра? )) https://github.com/evolution-cms/evolution/blob/2.0.x/manager/actions/mutate_content.dynamic.php#L8
EVO bot Лёшка
Aliaksandr
не понял про апи. раньше у меня внутри плагина на OnDocFormRender были видны и $content и $tvsArray, а теперь второй пропал
Volodymyr
Aliaksandr
ну я вот как бы совсем и не спрашиваю как мне получить тв))
Aliaksandr
я говорю что раньше работало, а теперь перестало )
Pathologic
Ну работу глобалов поддерживать это не очень решение
Pathologic
Лучше прямо в событие закинуть эти тв
Pathologic
А еще лучше было бы модель туда передавать
Anonymous
Ну работу глобалов поддерживать это не очень решение
+ в целом все глобалы надо убивать 🙂
Anonymous
Оно кстати не факт что они работают. Там очень много глобалов ломалось и часть из них переписывались на что-то из чего можно найти концы, а часть оживлялось как глобалы
Pathologic
еще и события через жопу сделаны с давних времен
Aliaksandr
Я просто долго искал концы, почему $content остался, а $tvsArray пропал
Aliaksandr
В целом разговор не про то, что хорошо, а что плохо, а про то, что после рендера тв мы нигде не можем получить их значения, хотя раньше как раз в этом событии она была
Pathologic
так ты же знал, что глобалы это плохо ((:
Aliaksandr
Ну глобал контента хорошо, а глобал его тв плохо - сейчас так :)
Aliaksandr
Если б мои правки принимали, я б передавал туда тв параметром в плагин :)
Aliaksandr
Да и контент)
Pathologic
так ты не захотел же безкостыльно сделать 🤠
Aliaksandr
Но туда передают только шаблон вроде почему-то)
Aliaksandr
так ты не захотел же безкостыльно сделать 🤠
Я просто нашел другой, более простой костыль))
Pathologic
который сломался ((:
Aliaksandr
Не, тот еще не сломали, которым я себе выпиливал тв
Aliaksandr
Хотя не факт, может и там уже кранты))
Pathologic
всюду еще сломался глобал lang_attribute
Aliaksandr
В общем фигня какая то, когда нигде нет доступа к тв, это факт, остальное нюансы:)
Aliaksandr
Интересно, global $_lang еще работает, которым можно было поля веб юзера переназвать:)
Aliaksandr
Вместо штата влупить рабочий телефон
Aliaksandr
?
Pathologic
мне бы такое даже в голову не пришло ((:
Pathologic
но если и работает то сломается когда-нибудь точно
Aliaksandr
Ну а где взять еще пару полей вебюзера для рабочего телефона и скайпа, когда простаивает какой-нибудь zip )
Pathologic
добавить свои
Pathologic
это лучше чем потом гадать почему zip это скайп (:
Aliaksandr
Наверно extended-атрибуты и до сих пор нифига в админке не редактируются
Aliaksandr
это лучше чем потом гадать почему zip это скайп (:
Дык там написано будет, что это скайп, пока не поломается:)
Pathologic
поломается
Pathologic
на фронте с языками стало получше благодаря тому что @dzhuryn не стал козлить с блангом ((:
Pathologic
остается внедрить в админку языки из ларавеля
Aliaksandr
Да тут ломается периодически то, что пять лет работало, так что это как карта ляжет, не зависит костылем сделано или нет
Pathologic
проблема костылей в том, что никто не знает кто какие костыли себе сделал
Pathologic
я хотел привести пользователей к одному виду, но обнаружилось непреодолимое препятствие - многие же напрямую в таблицы пишут, как тут совместимость обеспечить
Pathologic
я, кстати, сделал поле verified для пользователей, чтобы убрать свой костыль для проверки почты через поле logincount ((: скоро обновлю все, чтобы это работало нормально
Aliaksandr
Ну костылем я бы это назвал, если был другой доступ к полученным тв. А так это скорее единственное возможное решение, которое в итоге поломали и взамен вообще ничего не осталось :)
Pathologic
можно перекинуть в событие эти переменные
Aliaksandr
Ну так кто перекинет?:)
Pathologic
тут хотя бы можно совместимость обеспечить
Pathologic
может @Serious_t91 перекинет (:
Aliaksandr
Упрётся, ему не надо, скажет ))
Pathologic
тогда сделай сам, Дима все равно принимает не глядя (((:
Anonymous
Создайте issue и прокину, а так забуду
Aliaksandr
Зайду через яму)
Pathologic
яма прокинет так, что потом пару релизов будут страдания
Aliaksandr
Через форич каждое тв
Pathologic
и побольше sprintf
Pathologic
но с foreach мог облажаться любой, на самом деле
Aliaksandr
Ну вообще конечно странно, если все id полей тв у нас начинаются с tv, то почему они до сих пор не заменены на %s
Pathologic
это как с $modx->event я потратил день, пока понял, в чем проблема
Aliaksandr
И потом в конце спринтф)
Pathologic
или как @dzhuryn долбался с параметрами
Aliaksandr
Ну вот так и я вчера долбался с плагинами коммерса, которые работали из файлов и не работали из админки, а сегодня с этим внезапным tvsArray )
Pathologic
ну, зато тебя никто не обвинял что это ты виноват
Pathologic
я вот почему-то был виноват, что из базы не возвращаются только строки, что $lang_attribute не работает и еще что-то
Pathologic
даже в том, что скрипты на страницу некоторые не знают как вставить правильно - тоже я был виноват
Aliaksandr
Да, меня даже перестали обвинять, когда не работает эвобабель )) вышел на новый уровень абстракции ))
Aliaksandr
Раньше то проклятья сыпались))
Pathologic
как раз прямо сегодня один человек что-то сам себе надумал мимо инструкции и получил проблемы на ровном месте
Aliaksandr
Просмотрел я вчера сегодня с десяток модулей скидок для опенкарта и все однотипно: дата от, до, выбор категории, товаров или производителя, группы пользователей и собственно все
Aliaksandr
Ничего прям оригинального не нашел
Aliaksandr
Ну еще в одном вроде было минимальное количество, после которого действует скидка
Pathologic
Aliaksandr
Сама скидка везде в %
Pathologic
на рево посмотри, может там что-нибудь хорошее придумали (:
Aliaksandr
Мне почему-то кажется, что там 70-80% усилий тратится не на сам функционал, а на то, как сделать на рево, но максимально его обойти:)