Size: a a a

2020 August 16

СП

Сергей Предводителев... in Yii Framework 3
Dmitriy S
Там должны были поменять codestyle, но не в курсе поменяли ли. PHPStorm последний?
Да, последний
источник

RM

Rustam Mamadaminov in Yii Framework 3
У меня было это, потом решил исправить в настройках
источник

СП

Сергей Предводителев... in Yii Framework 3
Rustam Mamadaminov
У меня было это, потом решил исправить в настройках
Спасибо, сделал.
источник

DS

Dmitriy S in Yii Framework 3
Я бы все-таки добавил private $layoutParameters, и вторым параметром его в withLayout()
источник

DS

Dmitriy S in Yii Framework 3
Создавать отделный класс инъекции для каждого простого параметра - так себе затея
источник

СП

Сергей Предводителев... in Yii Framework 3
Dmitriy S
в injections перенести массив с параметров
перетащил
источник

СП

Сергей Предводителев... in Yii Framework 3
Dmitriy S
Создавать отделный класс инъекции для каждого простого параметра - так себе затея
Зачем для каждого. Я думал то, что для всех будет в рамках одной инъекции ApplicationViewInjection.
источник

СП

Сергей Предводителев... in Yii Framework 3
Dmitriy S
Я бы все-таки добавил private $layoutParameters, и вторым параметром его в withLayout()
Если добавлять простые параметры, то это уже в рамках другого PR наверное.
источник

DS

Dmitriy S in Yii Framework 3
Сергей Предводителев
Зачем для каждого. Я думал то, что для всех будет в рамках одной инъекции ApplicationViewInjection.
А если мне надо для крнкретного экшина применить конкретный макет и передать туда дополнительный параметр, как это будет выглядеть?
источник

DS

Dmitriy S in Yii Framework 3
Я бы хотел что-то вроде этого
$this->viewRenderer->withLayout('@views/layout/custom', ['myParam' => 8657654])
источник

СП

Сергей Предводителев... in Yii Framework 3
Dmitriy S
Я бы хотел что-то вроде этого
$this->viewRenderer->withLayout('@views/layout/custom', ['myParam' => 8657654])
Да я понял. Но это вне инъекций. Это тогда сделать ещё дополнительно "withLayoutParameters" и в withLayout добавить тоже ещё один аргумент, да.
источник

DS

Dmitriy S in Yii Framework 3
Сергей Предводителев
Да я понял. Но это вне инъекций. Это тогда сделать ещё дополнительно "withLayoutParameters" и в withLayout добавить тоже ещё один аргумент, да.
Да, я об этом, просто не всегда нужны инъекции
источник

СП

Сергей Предводителев... in Yii Framework 3
Dmitriy S
Да, я об этом, просто не всегда нужны инъекции
Согласен.
Но для случаев, например, с CSRF - красиво выходит)
источник

DS

Dmitriy S in Yii Framework 3
Сергей Предводителев
Согласен.
Но для случаев, например, с CSRF - красиво выходит)
Ну как по мне, то применение которое я уже знаю - это плагины для разметки вроде twitter|facebook в метатегах.  Должна быть возможность установить плагин и сразу получить разметку
источник

DS

Dmitriy S in Yii Framework 3
В yii2 это делалось через события, в принципе и здесь так можно, ибо класс WebView не иммутабельный. Значит надо во ViewRenderer сделать clone $view
источник

DS

Dmitriy S in Yii Framework 3
То есть, я сейчас инъекции могу заменить обработчиком события
BeforeRender::class => fn (WebView $view) => $view->registerLinkTag(...)
источник

DS

Dmitriy S in Yii Framework 3
Но только если я не применил один из методов with...
источник

DS

Dmitriy S in Yii Framework 3
Dmitriy S
Но только если я не применил один из методов with...
Хотя не, всегда могу, объект представления не клонируется
источник

СП

Сергей Предводителев... in Yii Framework 3
Но через события не так прозрачно выходит, мне кажется.
источник

DS

Dmitriy S in Yii Framework 3
Сергей Предводителев
Но через события не так прозрачно выходит, мне кажется.
Дело даже не в прозрачности, нехорошо что класс представления сам по себе не иммутабельный
источник