Unify exception handling from render_response component.encodeAll
-----------------------------------------------------------------

                 Key: MYFACES-3198
                 URL: https://issues.apache.org/jira/browse/MYFACES-3198
             Project: MyFaces Core
          Issue Type: Sub-task
          Components: General
         Environment: myfaces core trunk
            Reporter: Martin Kočí


Myfaces 
PartialViewContextImpl.PhaseAwareVisitCallback.processRenderComponent(UIComponent)
  catches only IOExpcetion and nothing else.

I've checked Mojarra and RichFaces (they have own PartialViewContextImpl) and 
both those implementations catch all exception; that isolates rendering of a 
component (and it children) - in case of exception, rest of component tree can 
still be rendered.

Similar concept of isolating execution for one component can be seen in 
UIInput.updateModel - problem in one component does not have influence on other 
components in tree. Maybe we should check all code for this symmetry.

previous is valid for partial lifecycle. For full lifecycle, rendering begins 
with UIViewRoot.encodeAll .


Also catched exception should not be logged, but queued with 
ExceptionQueuedEvent.


--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to