- Issue created by @scuba_fly
- 🇳🇱Netherlands scuba_fly Netherlands, Vinkeveen
Added WIP on the cocoon_media-3394045 branch.
- @scuba_fly opened merge request.
- Status changed to Needs work
about 1 year ago 7:26pm 13 October 2023
Currently the cocoon controller takes arguments in it's constructor.
The idea here is to inject the config factory, and let the cocoon controller get the arguments from config.
This has some advantages:
- No longer needs the code get the config in every place that the controller is called.
- There is only one place where the config is called. ( bit of the same as above ) which makes it more maintainable
- Unit tests gets easier as we can prophesize the injected classes
As the signature is changed of the cocoon controller class, we need to change this in the whole project and announce that is it changed for projects extending this class.
Needs work
3.1
Code
Added WIP on the cocoon_media-3394045 branch.