myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rob Decker (JIRA)" <myfaces-...@incubator.apache.org>
Subject [jira] Commented: (MYFACES-159) Deleted tree2 nodes cause fatal exception
Date Wed, 27 Apr 2005 22:22:41 GMT
     [ http://issues.apache.org/jira/browse/MYFACES-159?page=comments#action_63930 ]
     
Rob Decker commented on MYFACES-159:
------------------------------------

I think throwing a fatal exception in an app under any circumstances that isn't fatal just
isnt' right. A runtime exception is fatal in servlet container. It just gets thrown up and
handled by the servlet container or caught by the app developers catchall error page(s). I
don't see anything fatal about a node no longer in a tree in a stateless environment like
http.

That said, a hook is a good idea for letting the developer set a method where they can add
their own message or not. But the message bundles let you do that as is. Just set the message
to empty.

What could a component user do in a method hook? Stop the tree layout from continuing or not?
I don't know enough about jsf to add anything really useful to that idea.

> Deleted tree2 nodes cause fatal exception
> -----------------------------------------
>
>          Key: MYFACES-159
>          URL: http://issues.apache.org/jira/browse/MYFACES-159
>      Project: MyFaces
>         Type: Bug
>     Versions: 1.0.9 beta
>  Environment: All
>     Reporter: Rob Decker
>     Assignee: sean schofield

>
> If a node gets deleted between the time it is clicked on and when the server processes
> the request I get this error:
>  
>  java.lang.IllegalArgumentException: Node with id 0:1:1. Failed to parse 0:1:1
>  
> Basically the node's child was deleted and the index of the child list is no longer
> valid. It was valid when the tree was displayed. This gets thrown from
> TreeModel.getNodeById() when an IndexOutOfBoundsException is thrown by the child list.
> This should be handled more gracefully, like by adding a faces context message, "The
selected item no longer exists.", and the processing of the rest of the tree nodes should
continue. The end user can choose whether or not to show the messages.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


Mime
View raw message