DS
Size: a a a
2020 July 22
А
а, ок
А
а, вон оно как интересно
AM
А
а почему там инжектор каждый раз создаётся? он же стейтлесс, его можно вне цикла создать 1 раз
А
(new Injector($this->container)) вынести перед всеми foreachAM
а почему там инжектор каждый раз создаётся? он же стейтлесс, его можно вне цикла создать 1 раз
pull request!
DS
Была. Но мне норм и выпилить, раз мешает.
Нет, там была проверка is_callable(), можешь поднять код)
AM
Нет, там была проверка is_callable(), можешь поднять код)
Умгу, которая перестала срабатывать.
А
pull request!
сейчас там точно кто-то ковыряется, мб сделает. Если нет, то я потом сделаю Сейчас прод мучаю
DS
а почему там инжектор каждый раз создаётся? он же стейтлесс, его можно вне цикла создать 1 раз
Да там и создания класса можно в кложу внести
AM
DS
Умгу, которая перестала срабатывать.
Ну вот и я о том же, так что проверку фактически можно ограничить максимум $container->has()
А
Ну вот и я о том же, так что проверку фактически можно ограничить максимум $container->has()
просто has тоже спекулятивная штука
А
недопроверка
DS
просто has тоже спекулятивная штука
Та можно и вообще без него, так оба варианта листенеров будут фактически в равных условиях
А
Проверка хороша когда в dev. Конфиг events поменял, но чтобы проверить валидность конфига - надо дойти в проекте до того места, где вызывается событие, либо вызывать его самому - не удобно. А в проде это всё лишнее
DS
А нафига вообще в ивентах DeferredProvider? Что-то так и не понял из тестов его назначения
В
Там и сейчас так можно, только зачем контейнер передавать, если инжектор разрешит зависимости
Я ж специально посмотрел код перед тем, как написать: в инжектор можно передавать только готовые callable, а зависимости он уже разрешит.