AM
Ну можно и так, конечно.
Size: a a a
2020 June 01
AM
Wilmer can you adjust it in app template? (will move it to yii-web later)
AM
name() should be implemented in the base controller class getting the name from controller class name
DS
Кто хочет сделать?
Только везде поменять не забудьте
T
it is better to create the controller on yii-web and then change it everywhere.
T
@xepozz if you have a clear idea, you can create the pr on yii-web.
AM
TradersVE
it is better to create the controller on yii-web and then change it everywhere.
Alright, agree.
Д
я бы вынес "ручную" установку в статическую переменную базового класса, чтобы не было никаких getLayoutPath и прочих методов, т.к. а) пугает юзеров б) не дает писать свои собственные методы с таким именем.
а статическая переменная - самое то. например, так сделано в Symfony\Command, переменная $defaultName или как-то так
а статическая переменная - самое то. например, так сделано в Symfony\Command, переменная $defaultName или как-то так
T
and if we do it automatically subtracting the word Controller.
AM
TradersVE
and if we do it automatically subtracting the word Controller.
That's good default.
AM
я бы вынес "ручную" установку в статическую переменную базового класса, чтобы не было никаких getLayoutPath и прочих методов, т.к. а) пугает юзеров б) не дает писать свои собственные методы с таким именем.
а статическая переменная - самое то. например, так сделано в Symfony\Command, переменная $defaultName или как-то так
а статическая переменная - самое то. например, так сделано в Symfony\Command, переменная $defaultName или как-то так
Метод так и так понадобится как раз потому что дефолт тоже надо вычислять как Wilmer заметил.
Д
зачем?
BaseController:
$layout = null;
render() {
$layout = $this->layout
if $layout !== null {
$layout = $this->detectLayout()
}
....
}
Controller extends BaseController
AM
Oh! I remember now why I wasn't too excited about having base controller in yii-web.
AM
There's no yiisoft/view dependency now.
Д
There's no yiisoft/view dependency now.
да, было такое)
T
We must make a convention with the aliases being overwritten.
AM
@yiiliveext мы же фиксили https://github.com/yiisoft/yii-demo/issues/102, не?
Д
так давай занесём?
T
aliases directory no params.
AM
так давай занесём?
А если у меня JSON API и View не нужен?