pivot-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Roger Whitcomb (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PIVOT-924) Standard handling of exceptions in BXMLSerializer
Date Tue, 22 Oct 2013 15:29:42 GMT

    [ https://issues.apache.org/jira/browse/PIVOT-924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13801914#comment-13801914

Roger Whitcomb commented on PIVOT-924:

Thanks, Karel.  I will pull your patch and give it a try.  I think we would use this ourselves,
so definitely a good change/addition.

> Standard handling of exceptions in BXMLSerializer
> -------------------------------------------------
>                 Key: PIVOT-924
>                 URL: https://issues.apache.org/jira/browse/PIVOT-924
>             Project: Pivot
>          Issue Type: Improvement
>          Components: core-serialization
>    Affects Versions: 2.0.3
>            Reporter: Karel Hübl
>         Attachments: patch_trunk_rev1534200
> We require handling of all uncaught exceptions using standard mechanism (implementing
UncaughtExceptionHandler interface on PivotApplication class).
> Similar to recently resolved https://issues.apache.org/jira/browse/PIVOT-916, it is currently
not possible also in BXMLSerializer class. To resolve this, I would propose to:
> 1) Handle all exception reporting in BXMLSerializer in single protected method to enable
overriding this method in BXMLSerializer descendant. I will upload patch for this.
> 2) Change scope of static ApplicationContext.handleUncaughtException method to public.
This will enable the delegation of exception handling from BXMLSerializer descendants to ApplicationContext.
So hacks like this http://svn.codespot.com/a/apache-extras.org/pivot-contrib/trunk/pivot_contrib.util/src/pivot_contrib/util/serializer/InjectingSerializer.java
wont be required anymore ;-).
> Please let me know, if someone can see more elegant solution.

This message was sent by Atlassian JIRA

View raw message