T
But I think that packages should not have configurations, and there if we have a clean container in the application.
Size: a a a
2020 August 07
А
EventDispatcherProvider - полезная нагрузка 2 строки, файл 25 строк
+ 2 строки в providers.php
T
+ 2 строки в providers.php
Yes, but there are a maximum of 4 providers, all have parameters or logic, and if you ask me what I prefer, it is better providers, it is easier to debug than the configuration in the container.
T
The configuration in the container if the configurations of the nested packages come sometimes you do not know if the error is from the app or from another package.
T
If you want to debug and leave the container blank you will always have the configuration of the other packages.
А
TradersVE
Yes, but there are a maximum of 4 providers, all have parameters or logic, and if you ask me what I prefer, it is better providers, it is easier to debug than the configuration in the container.
не соглашусь. It depends. Если я захочу переопределить реализацию какого-то интерфейса и запишу её в
common.php, то я могу даже не подозревать, что интерфейс переопределяется каким-то провайдером. Как новоиспечённый пользователь я потеряю много времени, прежде чем пойму, что произошлоT
не соглашусь. It depends. Если я захочу переопределить реализацию какого-то интерфейса и запишу её в
common.php, то я могу даже не подозревать, что интерфейс переопределяется каким-то провайдером. Как новоиспечённый пользователь я потеряю много времени, прежде чем пойму, что произошлоFor example you are a novice, and you don't configure neither assets nor yii-web but it still works, ideally you should see the configuration and learn how it works.
T
With providers the settings are in the application by component, which makes you see how it works.
AB
не соглашусь. It depends. Если я захочу переопределить реализацию какого-то интерфейса и запишу её в
common.php, то я могу даже не подозревать, что интерфейс переопределяется каким-то провайдером. Как новоиспечённый пользователь я потеряю много времени, прежде чем пойму, что произошлопровайдеры только в app? сами yii-* пакеты их не предоставляют?
А
TradersVE
For example you are a novice, and you don't configure neither assets nor yii-web but it still works, ideally you should see the configuration and learn how it works.
это так не работает :) мы живём не в идеальном мире
А
провайдеры только в app? сами yii-* пакеты их не предоставляют?
до тех пор, пока нет PSR на сервис провайдеры (очень надеюсь и жду нормального решения от https://github.com/container-interop/service-provider/issues/51), их там и не будет
T
это так не работает :) мы живём не в идеальном мире
It is correct but we must eliminate the magic of the auto configurations of the packages, they can bring problems.
AM
@romkatsu пушнул именование другое, посмотри https://github.com/yiisoft/yii-web/pull/296/files
RT
@romkatsu пушнул именование другое, посмотри https://github.com/yiisoft/yii-web/pull/296/files
👍
RT
только
RT
вот это уже не нужно получается
if ($this->contentType !== null) {
return $this->contentType;
}
if ($this->contentType !== null) {
return $this->contentType;
}
AM
разве?
AM
а, да
AM
смёржил
RT