СП
Добавь это в pull request, пожалуйста.
Добавил в коммент.
Size: a a a
2020 September 09
AB
Добавил в коммент.
по идеи такая инициализация должна быть где-нибудь в эвентах приложения (какое-нибудь
ApplicationStartup). Но вообще вопрос, зачем WidgetFactory - статичная? Выглядит как рудимент из 2-ой версииСП
по идеи такая инициализация должна быть где-нибудь в эвентах приложения (какое-нибудь
ApplicationStartup). Но вообще вопрос, зачем WidgetFactory - статичная? Выглядит как рудимент из 2-ой версииНаверное для того, чтобы виджеты так запускать: MyWidget::widget()
AB
Тогда не понятно почему вы паритесь по поводу прокидывания сервисов в ViewRenderer (вроде было тут похожее обсуждение), сделайте все статикой, как и виджеты. UrlMatcher, CSRF etc
В
Лично я ярый противник статики: это global state, который несет одни беды как в поде, так и при тестировании. Не говоря уже про всякие long-living приложения.
СП
Лично я ярый противник статики: это global state, который несет одни беды как в поде, так и при тестировании. Не говоря уже про всякие long-living приложения.
+1 со статикой одни неудобства
AB
Тогда может
$widgetFactory->create(MyWidget::class)->widget() сделать? Ну и подумать о каком-нибудь сахаре, типа $this->widget(MyWidget::class)AM
Тогда может
$widgetFactory->create(MyWidget::class)->widget() сделать? Ну и подумать о каком-нибудь сахаре, типа $this->widget(MyWidget::class)Думали. Лучше текущего не нашли пока.
AM
Если в контроллере или сервисе так норм, то в шаблоне view замучаешься так длинно писать. Там сотня виджетов на страницу бывает...
T
Тогда может
$widgetFactory->create(MyWidget::class)->widget() сделать? Ну и подумать о каком-нибудь сахаре, типа $this->widget(MyWidget::class)(new Widget())->methods() // work.
T
T
TradersVE
@predvoditelev good job 👍
AM
TradersVE
(new Widget())->methods() // work.
Nope. That won't allow having dependency autowiring in widget classes.
AM
TradersVE
@predvoditelev good job 👍
Tested on db?
T
Tested on db?
In Ar it works, it also has the previous tests, therefore it must pass.
T
Nope. That won't allow having dependency autowiring in widget classes.
It's corrects for simple widget.
T
Framework widgets have no dependencies so it will work for most of them.
T
Loading the factory in view is not a good idea.
DS
по идеи такая инициализация должна быть где-нибудь в эвентах приложения (какое-нибудь
ApplicationStartup). Но вообще вопрос, зачем WidgetFactory - статичная? Выглядит как рудимент из 2-ой версииДа, с ивентом нормальный вариант
СП
Да, с ивентом нормальный вариант
А как это сделать в пакете?
Есть пример?
Есть пример?