a
Size: a a a
2020 September 26
Д
Если ты про https://github.com/yiisoft/yii-demo/commit/1510ef2199b0c6c6a9bdf7f009a00ffec74ccc32, то оно не работает
a
по крайней мере связано с DI - там его рефакторили
a
Если ты про https://github.com/yiisoft/yii-demo/commit/1510ef2199b0c6c6a9bdf7f009a00ffec74ccc32, то оно не работает
Кстати, вроде работало - просто там еще что то заливали в DI
a
@hiqsol дебажит
АН
Если ты про https://github.com/yiisoft/yii-demo/commit/1510ef2199b0c6c6a9bdf7f009a00ffec74ccc32, то оно не работает
Там последние коммиты от @hiqsol, сломало, но вроде пофиксил, но не проверял, так как откатился до посл. рабочего коммита.
AV
Пытаюсь разобраться что не так с app. yii-demo заработал
Д
https://github.com/yiisoft/factory/blob/master/src/Definitions/CallableDefinition.php#L22
В общем, проблема здесь.
has возвращает true, но на самом деле в контейнере еще нет сконфигурированного инжектора.
Это бага контейнера, которую никто чинить не хочет
В общем, проблема здесь.
has возвращает true, но на самом деле в контейнере еще нет сконфигурированного инжектора.
Это бага контейнера, которую никто чинить не хочет
Д
Решение:
if (class_exists(Injector::class)) {
return (new Injector($container))->invoke($callback);
}Д
Либо инжектор переводить на интерфейс, чтобы наш контейнер не ломался...
AV
> has возвращает true, но на самом деле в контейнере еще нет сконфигурированного инжектора.
> Это бага контейнера, которую никто чинить не хочет
В чём бага? has возвращает то что и должен возвращать в соответствии с описаним psr интерфейса
> Это бага контейнера, которую никто чинить не хочет
В чём бага? has возвращает то что и должен возвращать в соответствии с описаним psr интерфейса
AV
Решение:
if (class_exists(Injector::class)) {
return (new Injector($container))->invoke($callback);
}Это полумера. Так как сделано сейчас позволяет использовать другой инжектор задав определение в контейнере
AV
2020 September 27
AM
Тоже считаю что надо добавить интерфейс. @samdark Injector interface нет умышленно ?
Да
AM
Потому что нет и не будет других имплементаций
AV
Потому что нет и не будет других имплементаций
я так и подумал что идея такая. Только почему такая уверенность? Типа заинжектить как-то по другому совершенно невозможно?
Д
В общем, вот
https://github.com/yiisoft/di/pull/161 - это должно падать
https://github.com/yiisoft/di/pull/162 - это должно работать
сейчас всё наоборот.
https://github.com/yiisoft/di/pull/161 - это должно падать
https://github.com/yiisoft/di/pull/162 - это должно работать
сейчас всё наоборот.
AV
В общем, вот
https://github.com/yiisoft/di/pull/161 - это должно падать
https://github.com/yiisoft/di/pull/162 - это должно работать
сейчас всё наоборот.
https://github.com/yiisoft/di/pull/161 - это должно падать
https://github.com/yiisoft/di/pull/162 - это должно работать
сейчас всё наоборот.
👍 с этим уже можно работать
AV
Потому что нет и не будет других имплементаций
если это так, то можно выпилить возможность переопределять инжектор и всё сильно упростится