Pathologic
я скопировал
Сергій
Pathologic
а баг как увидеть теперь
Pathologic
если я копию выберу, то хули я увижу
Pathologic
с конфигом
Volodymyr
$this->app->getConfig('manager_theme', 'default'); должно вернуть название твой темы
Volodymyr
а не default
Pathologic
я решал проблему иначе совсем
Aliaksandr
Мне кажется эту тему все тот же никола года полтора назад поднимал уже))
Anonymous
Да кто её только не подымал, а решил условно только @dzhuryn
Volodymyr
А решил, потому что работать влом
Volodymyr
еще есть баги которые пофиксить нужно?
Anonymous
А решил, потому что работать влом
Никому о этом не рассказывай, просто решил взять и показать всем кто тут папка 😊
Volodymyr
Ну до папки мне далеко, но кое что умею)) Ладно попытаюсь часик таски поделать
Aliaksandr
Я не совсем понял, чего решали в итоге?)
Volodymyr
Я не совсем понял, чего решали в итоге?)
трабла, была из кастомными темами
Aliaksandr
Меняли местами загрузку конфигов манагера или просто название темы правильно возвращали?
Volodymyr
и получили значение по умолчанию default
Aliaksandr
Так а сейчас конфиг есть? Или только тема появилась правильная?
Aliaksandr
Или просто стали позднее его брать, это название?))
Volodymyr
Или просто стали позднее его брать, это название?))
стали брать в момент, вызова сервиса, а не при регистрации
Volodymyr
терялась суть di мы что-то делали при регистрации зависимости, а не при первом использовании
Aliaksandr
Немного скромности на ночь не помешает https://vm.tiktok.com/w6dKpy/ :)
Aliaksandr
а как там обстановка у Батьки?
Лучше всех в мире:) и это никто уже не оспаривает ))
Pathologic
+
Pathologic
но тема эта скопированная все равно как-то подглючивает
Pathologic
styles.min.css не загружается
Aliaksandr
что-то мутное вы по ночам мутите))
Pathologic
не, это я так понимаю баг, на который Камиль отправил pr
Volodymyr
что-то мутное вы по ночам мутите))
угу, multitv из режимом dbtable изучаю
Volodymyr
хочу отзывы сделать на нем
Pathologic
https://github.com/evolution-cms/evolution/pull/1459
Pathologic
Volodymyr
не страдай херней
ну изначально было сделано, обычно хранение в тв
Pathologic
я наверное даже знаю, кто делал ((:
Volodymyr
я решил поигратся, прочитал доку по multitv, конечно, больше похоже на некрофилию, но все же
Pathologic
а я думал Либер
Volodymyr
Ну меня попросили так сделать
Volodymyr
теперь нужно, еще фотку добавить, вот я решил, сделать по уму
Pathologic
у меня есть готовые отзывы с фотками ((:
Volodymyr
у меня есть готовые отзывы с фотками ((:
блин, нужно было сразу у тебя купить
Volodymyr
но вообще я хотел на comments делать, но там нужно было из твиг разобратся
Volodymyr
а тут я за пол часика наклепал, про траблу из большым количеством я предупредил
Volodymyr
все были довольны
Pathologic
на comments слишком жирно отзывы делать
Aliaksandr
а почему всю эту муть вообще надо делать в сервис-провайдере, а не в конструкторе манагертемы? )
Volodymyr
а почему всю эту муть вообще надо делать в сервис-провайдере, а не в конструкторе манагертемы? )
Ну, можеш напрямую инициализировать managerTheme в нужном месте, и забить на di
Aliaksandr
причем тут всякие loadChunks к di ?
Volodymyr
причем тут всякие loadChunks к di ?
ну ты ж объект конфигурируешь
Volodymyr
ну как вариант, можна фасад сделать
Aliaksandr
я просто не могу понять логику - почему мы тему менеджера получаем где-то в сервис-провайдере, а не просто сразу в конструкторе менеджера)
Volodymyr
ну это да
Volodymyr
ну я хз, как дальше мы это юзаем
Aliaksandr
так получается просто конфига нет, туда сразу и передавалась дефолтная. и дальше уже не проверялась. А если бы мы ее получали сразу в construct, то там уже конфиг будет - там и тема нужная будет
Pathologic
я так вообще не понимаю, зачем для этой херни сервис-провайдер
Pathologic
ну, я и переделал, чтобы было без него
Aliaksandr
я так вообще не понимаю, зачем для этой херни сервис-провайдер
на случай, если ты напишешь свой \\EvolutionCMS\\ManagerThemeCustom, видимо ))
Aliaksandr
но тему и всю эту сопутствующую фигню мне кажется прямо в манагере уже и получать надо)
Pathologic
оно по факту так и происходит, это же отложенный провайдер
Aliaksandr
не зря ж пишут, что в регистере надо только определить класс, но You should never attempt to register any event listeners, routes, or any other piece of functionality within the register method.
Aliaksandr
а вот эти все лоадчанки и лоадсниппеты это ж чисто "other piece of functionality" ))
Pathologic
верно
Volodymyr
А я впервые сам phx заюзал в 2020 году
Volodymyr
надеюсь не уволят за такое
Aliaksandr
ну кстати да. в мультитв оно до сих пор работает наверно))
Aliaksandr
смотри не нарвись на теги в поле message ))
Volodymyr
смотри не нарвись на теги в поле message ))
а что будет, кстате, limit не могу задать, только дефолтное значение
Aliaksandr
а что будет, кстате, limit не могу задать, только дефолтное значение
ну как обычно - отрежет закрывающий тег и будешь отвечать на письмо "поломался сайт" )))
Aliaksandr
там же вроде :striptags есть
Volodymyr
ну на фронте, html не пройдет, а админы врятли в textarea будут html тулить
Aliaksandr
не стоит недооценивать фантазию админов ))))
Volodymyr
А вообще учитывая когда сделали multitv то функционал, мощный заложили
Aliaksandr
о - notags )