[WICKET-5794] Make DefaultExceptionMapper extensible

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Implemented
• Affects Version/s: 6.18.0, 7.0.0-M4
• Fix Version/s: 7.0.0-M5, 6.19.0
• Component/s: wicket
• Assignee: Andrea Del Bene
• Reporter: Hannes Dohrn
• Created: 25/Dec/14 16:22
• Updated: 31/Dec/14 09:49
• Resolved: 31/Dec/14 09:49

Description

I'd like to send an email when wicket catches an unhandled exception. Also I want to customize the ExceptionErrorPage. Wicket offers getApplicationSettings().setInternalErrorPage(...) interface to set a custom internal error page. However, unlike the ExceptionErrorPage, InternalErrorPage does not receive the caught exception object or the current page object.

In https://github.com/hannesd/wicket/commit/a33c6dff I extracted some code from DefaultExceptionMapper#internalMap into an extra method. Also I changed private to protected methods to make DefaultExceptionMapper reusable and extensible.

I'd be nice if this patch or something similar enters wicket 6.x so that I don't have to duplicate 99% of DefaultExceptionMapper just to get my own version of ExceptionErrorPage.

Issue Links

• is related to: WICKET-5628 Introduce a marker interface for exception which are recommended to be handler by the framework