AM
@predvoditelev посмотрел https://github.com/yiisoft/yii-demo/pull/122. Оставил комментов. Хорошие идеи в PR точно есть.
Size: a a a
2020 August 19
AM
Arman https://github.com/yiisoft/yii-web/pull/298 оставил вопросы.
AM
Похоже, мы немного надублировали...
AP
Arman https://github.com/yiisoft/yii-web/pull/298 оставил вопросы.
ок, я честно говоря AutoLoginMiddleware даже не смотрел пока (не успел), сейчас посмотрю что там.
СП
@predvoditelev посмотрел https://github.com/yiisoft/yii-demo/pull/122. Оставил комментов. Хорошие идеи в PR точно есть.
Отписал. Там часть не понял(
AM
Что не понял?
СП
Что не понял?
AM
ага, уточнил
СП
ага, уточнил
Да, когда это просто HTML - то удобно.
Но когда есть логика, например как в Csrf, то удобнее делать прямо в инъекции.
Но когда есть логика, например как в Csrf, то удобнее делать прямо в инъекции.
AM
Всё так. Просто там favicon зачем-то сделан по дефолту.
AM
А это HTML проще и лучше.
СП
Это я для примера сделал в
ApplicationViewInjection, чтобы было видно как это использовать :)AM
А, ок.
AM
С одной стороны мне нравится разделение, интерфейсы и вот это всё. Для более-менее сложных штук очень удобно будет.
AM
Но для простых нет.
СП
А для простых это вообще не использовать, просто в layout вставил и всё.
AM
Для простых тупо конфиг списком как сейчас в app в разы понятней и удобней.
СП
Это же не замена, а дополнение
СП
Про csrf - можно конечно в ViewRenderer сделать "withCsrf", но тогда потянутся зависимости, которые не нужны бы по идее там.
AM
Ну получается много способов сделать одно и то же. Слишком много:
1. HTML
2. Передать параметр, сунуть в HTML
3. Задать default в View
4. Через твои интерфейсы
1. HTML
2. Передать параметр, сунуть в HTML
3. Задать default в View
4. Через твои интерфейсы