В
Коммит там же в пр
Мерси
Size: a a a
2020 June 30
В
Не работает этот вариант. Если дефинишина нет ни в одном контейнере, то надо с первого инстанцировать.
Чтобы заработало, надо в тесте
strict подвинуть в тот контейнер, где указывается реализация :) Ну и да, надо учитывать порядок добавления контейнеров в мета-контейнер.В
И, полагаю, надо такой же
strict добавить и в фабрику.DS
Чтобы заработало, надо в тесте
strict подвинуть в тот контейнер, где указывается реализация :) Ну и да, надо учитывать порядок добавления контейнеров в мета-контейнер.Так нельзя делать, не будет основной контейнер работать
А
strict === !autowiring ?
А
если помечать контейнеры этим флагом при добавлении в композитный, то почему это не решение проблемы?
В
Так нельзя делать, не будет основной контейнер работать
не понял
А
@yiiliveext , какие проблемы будут у такой схемы:
Все контейнеры без автовайринга. Автовайринг в самом композитном контейнере, срабатывает в последнюю очередь и использует при автоподстановке все strict контейнеры.
Все контейнеры без автовайринга. Автовайринг в самом композитном контейнере, срабатывает в последнюю очередь и использует при автоподстановке все strict контейнеры.
DS
не понял
Основной контейнер не будет работать если его strict сделать. В общем сделал с методом hasDefinition, только спрятал его под protected
В
Основной контейнер не будет работать если его strict сделать. В общем сделал с методом hasDefinition, только спрятал его под protected
С чего бы вдруг он не стал работать? Просто он будет грузить только явно заданные зависимости, что от него и требуется. Если нужен контейнер, который будет грузить классы автолоудом, в мета-контейнер надо добавить такой. Это же то самое поведение, которое ты и хочешь воспроизвести.
В
Мне неясно само понятие "основного контейнера"
DS
С чего бы вдруг он не стал работать? Просто он будет грузить только явно заданные зависимости, что от него и требуется. Если нужен контейнер, который будет грузить классы автолоудом, в мета-контейнер надо добавить такой. Это же то самое поведение, которое ты и хочешь воспроизвести.
DS
Можно расшифровать?
Ты предлагаешь контейнер который прилетает в register сделать strict. А это основной контейнер приложения. То есть, в нем не будет работать инстанцирование сервисов по class_exists, если поставить ему strict.
В
Ты предлагаешь контейнер который прилетает в register сделать strict. А это основной контейнер приложения. То есть, в нем не будет работать инстанцирование сервисов по class_exists, если поставить ему strict.
Не, ты не так меня понял
В
strict - он для того и есть, чтобы автолоадинг не работал в контейнере. Т.е. за исключением каких-то редких случаев он будет работать в составе композитного контейнера. Если в основном приложении и будет использоваться такой, то в составе композитного, и в
register должен прилететь именно композитный контейнер.DS
strict - он для того и есть, чтобы автолоадинг не работал в контейнере. Т.е. за исключением каких-то редких случаев он будет работать в составе композитного контейнера. Если в основном приложении и будет использоваться такой, то в составе композитного, и в
register должен прилететь именно композитный контейнер.Это уже будет костыль с добавлением фиктивного пустого контейнера для автолоадинга.
В
Это уже будет костыль с добавлением фиктивного пустого контейнера для автолоадинга.
Не костыль, а адекватное поведение. Есть и другой вариант: выкинуть вообще композитный контейнер.
В
Я потыкал твой тест. Там надо перенести определение
EngineMarkOne во второй контейнер, который оставить strict. А в самом контейнере в buildInternal надо пересмотреть работу со strict: оно падает, когда пытается инстанцировать EngineMarkOne через EngineMarkTwo, т.к. это все-таки "примитив".DS
Я потыкал твой тест. Там надо перенести определение
EngineMarkOne во второй контейнер, который оставить strict. А в самом контейнере в buildInternal надо пересмотреть работу со strict: оно падает, когда пытается инстанцировать EngineMarkOne через EngineMarkTwo, т.к. это все-таки "примитив".Там ничего не падает. Все тесты проходят, логика работы не та.