[BUGFIX] Clean up error handler for site handling
This patch handles several issues related to the new site error handling introduced in TYPO3 v9.2.0: 1. It adds unit tests 2. It adds a missing exception when an error handler is configured which does not implement the PageErrorHandlerInterface 3. It fixes one minor issue in PageContentErrorHandler where a wrong "InvalidArgumentException" was thrown 4. All PageErrorHandler logic was moved from EXT:frontend to EXT:core, as this would be a penalty across packages - the "Site" entity depends on them, so they must go to the same package (core). Releases: master Resolves: #85101 Change-Id: Ibdc05024abd7c719dd8d5dcb3388bf1679b69990 Reviewed-on: https://review.typo3.org/57057 Tested-by:TYPO3com <no-reply@typo3.com> Reviewed-by:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
Wouter Wolters <typo3@wouterwolters.nl> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
Georg Ringer <georg.ringer@gmail.com>
Showing
- typo3/sysext/core/Classes/Error/PageErrorHandler/FluidPageErrorHandler.php 1 addition, 1 deletion.../Classes/Error/PageErrorHandler/FluidPageErrorHandler.php
- typo3/sysext/core/Classes/Error/PageErrorHandler/InvalidPageErrorHandlerException.php 27 additions, 0 deletions...ror/PageErrorHandler/InvalidPageErrorHandlerException.php
- typo3/sysext/core/Classes/Error/PageErrorHandler/PageContentErrorHandler.php 3 additions, 4 deletions...lasses/Error/PageErrorHandler/PageContentErrorHandler.php
- typo3/sysext/core/Classes/Error/PageErrorHandler/PageErrorHandlerInterface.php 5 additions, 3 deletions...sses/Error/PageErrorHandler/PageErrorHandlerInterface.php
- typo3/sysext/core/Classes/Site/Entity/Site.php 14 additions, 12 deletionstypo3/sysext/core/Classes/Site/Entity/Site.php
- typo3/sysext/core/Tests/Unit/Site/Entity/SiteTest.php 116 additions, 0 deletionstypo3/sysext/core/Tests/Unit/Site/Entity/SiteTest.php
Please register or sign in to comment