You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Martin Grigorov (JIRA)" <ji...@apache.org> on 2014/12/26 15:31:14 UTC
[jira] [Commented] (WICKET-5794) Make DefaultExceptionMapper
extensible
[ https://issues.apache.org/jira/browse/WICKET-5794?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14259085#comment-14259085 ]
Martin Grigorov commented on WICKET-5794:
-----------------------------------------
Why not use IRequestCycleListener#onException() to send the mail ?
> Make DefaultExceptionMapper extensible
> --------------------------------------
>
> Key: WICKET-5794
> URL: https://issues.apache.org/jira/browse/WICKET-5794
> Project: Wicket
> Issue Type: Improvement
> Components: wicket
> Affects Versions: 6.19.0
> Reporter: Hannes Dohrn
> Priority: Minor
> Attachments: 0001-Extracted-code-in-DefautlExceptionMapper-internalMap.patch
>
> Original Estimate: 0h
> Remaining Estimate: 0h
>
> 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.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)