Some time ago injection of Container/Config (and some other) were replaced by the direct call Container::getInstance() to be compatible with Octane (#38). The main problem with the current approach - implicit dependencies and (more important) implicit config (Container::getInstance()->make(Repository::class)->get(...)).
Recently I have reviewed all these places and looks like it is possible to convert all of them into explicit 🎉
[x] Switch to resolvers instead of Container::getInstance() (#160)
[ ] #152
[x] Model Casts (serializer) (77490f68ec78b1d481f2b6aa8cb456b35ab40480)
Some time ago injection of
Container
/Config
(and some other) were replaced by the direct callContainer::getInstance()
to be compatible with Octane (#38). The main problem with the current approach - implicit dependencies and (more important) implicit config (Container::getInstance()->make(Repository::class)->get(...)
).Recently I have reviewed all these places and looks like it is possible to convert all of them into explicit 🎉
Container::getInstance()
(#160)serializer
) (77490f68ec78b1d481f2b6aa8cb456b35ab40480)StreamOffset
) (#157, #160, #161)spa
) (#160)