DS
в фабрике мб, но в контейнере не надо - цикличность хз как проверять
Да, с цикличностью там засада
Size: a a a
2020 April 04
А
поэтому, раз контейнер юзает автовайринг из фабрики, то лучше в фабрику инъектор не пихать )
А
плюс там правила свои на '__construct()' и прочие
DS
поэтому, раз контейнер юзает автовайринг из фабрики, то лучше в фабрику инъектор не пихать )
Да я и без него знаю как сделать) Это я так, вариант рассмотрел)
А
Да я и без него знаю как сделать) Это я так, вариант рассмотрел)
ну ты же поднял вопрос переиспользования кода, не?
DS
ну ты же поднял вопрос переиспользования кода, не?
Та не, там другой способ)
DS
Но именованные параметры там бы тоже не помешали
А
они там специально отсутствуют
DS
Это я о фабрике и методе create
А
блин, как всегда
А
смержил PR и заметил косяк
А
заголовок не того уровня )
DS
А нам в CallableDefinition точно параметры нужно передавать? Есть юзкейсы?
AM
фабрика?
DS
фабрика?
Ну да, фабрика, контейнер
DS
@samdark, не спи, замерзнешь) Я выпилил параметры из ризолва всех дефинишинов. Все твои тесты проходят, можно выпиливать второй параметр из контейнера.
https://github.com/yiisoft/factory/pull/17
https://github.com/yiisoft/factory/pull/17
DS
Там сейчас есть только два кейса, которые не работают в новом варианте фабрики, но работали в старом. Не передаются параметры в коллейбл дефинишн, хз зачем они там вообще нужны. И еще один интересный, тестом не покрыт. Найдешь сам?😉
AM
тесты фабрики проходят?
AM
очень подозреваю что нет
DS
тесты фабрики проходят?
Невнимательно читаешь, все проходят. Тесты не менял, все твои