[TASK] Use more DI in FE RequestHandler
Runtime static `FilePathSanitizer` and stateless `TypoScriptService` can be injected in ext:frontend `RequestHandler`. Also, the `ServiceProvider` entry is useless since the class implements `RequestHandlerInterface`, which sets those classes public. Change-Id: Ia7469d8e4d4f940871c4856d4782d53b70e4cee4 Resolves: #101375 Releases: main Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80076 Reviewed-by:Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Stefan B�rk <stefan@buerk.tech> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Benjamin Franzke <ben@bnf.dev> Reviewed-by:
Stefan B�rk <stefan@buerk.tech>
Showing
- typo3/sysext/core/Classes/ServiceProvider.php 0 additions, 6 deletionstypo3/sysext/core/Classes/ServiceProvider.php
- typo3/sysext/frontend/Classes/Http/RequestHandler.php 20 additions, 20 deletionstypo3/sysext/frontend/Classes/Http/RequestHandler.php
- typo3/sysext/frontend/Classes/ServiceProvider.php 22 additions, 11 deletionstypo3/sysext/frontend/Classes/ServiceProvider.php
- typo3/sysext/frontend/Tests/Unit/Http/RequestHandlerTest.php 10 additions, 0 deletionstypo3/sysext/frontend/Tests/Unit/Http/RequestHandlerTest.php
Please register or sign in to comment