DS
Но я против мёржа полуфабрикатов в yiisoft/di.
А где там полуфабрикаты? Там только с модификатором метода delegateLookup решить надо.
Size: a a a
2020 March 04
DS
И на пакеджист добавь
AM
1. Я не уверен что readme сейчас актуален с этими изменениями.
2. AbstractContainerConfigurator выглядит очень грязно когда set() не перекрывается. Но не знаю, можно ли тут нормально. @xepozz, @roxblnfk ваши идеи бы не помешали.
3. delegateLookup() я бы всё-таки снёс. Для него ни тестов ни реального клиента...
4. ContainerProxyInterface мы хотели выносить вроде, а он ещё там.
2. AbstractContainerConfigurator выглядит очень грязно когда set() не перекрывается. Но не знаю, можно ли тут нормально. @xepozz, @roxblnfk ваши идеи бы не помешали.
3. delegateLookup() я бы всё-таки снёс. Для него ни тестов ни реального клиента...
4. ContainerProxyInterface мы хотели выносить вроде, а он ещё там.
А
в том плане, что не везде перекрывается?
DS
1. Я не уверен что readme сейчас актуален с этими изменениями.
2. AbstractContainerConfigurator выглядит очень грязно когда set() не перекрывается. Но не знаю, можно ли тут нормально. @xepozz, @roxblnfk ваши идеи бы не помешали.
3. delegateLookup() я бы всё-таки снёс. Для него ни тестов ни реального клиента...
4. ContainerProxyInterface мы хотели выносить вроде, а он ещё там.
2. AbstractContainerConfigurator выглядит очень грязно когда set() не перекрывается. Но не знаю, можно ли тут нормально. @xepozz, @roxblnfk ваши идеи бы не помешали.
3. delegateLookup() я бы всё-таки снёс. Для него ни тестов ни реального клиента...
4. ContainerProxyInterface мы хотели выносить вроде, а он ещё там.
1. Это я поправлю сегодня.
2. Я не заню лучшего метода, если знаешь - скажи.
3. Клиенты есть - прокси контейнер, композитный контейнер
4. Вмержи мой пр в
2. Я не заню лучшего метода, если знаешь - скажи.
3. Клиенты есть - прокси контейнер, композитный контейнер
4. Вмержи мой пр в
container-proxy и я удалю этот интерфейс.AM
в том плане, что не везде перекрывается?
Да.
DS
в том плане, что не везде перекрывается?
Там его можно абстрактым сделать, но надо будет везде заглушку в потомках делать.
А
в контейнере перекрывается, остальные абстрактные
за абстрактными тоже не перекрывается?
за абстрактными тоже не перекрывается?
AM
А
Там его можно абстрактым сделать, но надо будет везде заглушку в потомках делать.
ну собсна да, если оно и будет перекрываться дальше, то такими же заглушками с Exception'ами :D
AM
@yiiliveext 4 готово.
AM
ну собсна да, если оно и будет перекрываться дальше, то такими же заглушками с Exception'ами :D
Это и не нравится...
А
а
set() вообще нужен где-нибудь кроме Container?А
что-нибудь делает проверку
instanceof AbstractContainerConfigurator?AM
да он и там не нужен
AM
что-нибудь делает проверку
instanceof AbstractContainerConfigurator?эта хреновина нужна чтобы дёрнуть не публичный метод класса из другого класса
AM
аля friendly class
AM
но не факт что так надо
DS
да он и там не нужен
Посмотри последний коммит в контейнере, сделал delegateLookup protected