[ http://issues.apache.org/jira/browse/MYFACES-159?page=comments#action_63972 ] sean schofield commented on MYFACES-159: ----------------------------------------
A runtime exception in a webapp is not necessarily fatal. You can catch the exception and redirect to a special error page. Struts does this all the time. Still, I see your point on how it might be a little overkill in this case. I guess we could also add a component message as well just to help out the user should they chose to take advantage of it. One thing the component user could do in the hook is abort JSF processing and redirect to an error page. So that would be an option there. Also, the user could log the message as WARNING or ERROR using log4j, etc. so the admin of the application would be aware of the issue. > 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
