[MYFACES-3198] Unify exception handling from render_response component.encodeAll

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Component/s: General
• Environment: myfaces core trunk
• Assignee: Unassigned
• Reporter: Martin Kočí
• Created: 29/Jun/11 18:07
• Updated: 30/May/20 11:11
• Resolved: 30/May/20 11:11

Description

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.