[MYFACES-968] LifecycleImpl's restoreView throws NPE when viewId is null in a portlet context

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.0.9m9, 1.1.0, 1.1.1
• Fix Version/s: 1.1.5
• Component/s: General
• Assignee: Martin Marinschek
• Reporter: David Philip Brondsema
• Created: 21/Dec/05 05:14
• Updated: 14/Feb/07 00:30
• Resolved: 25/Sep/06 04:12

Description

LifecycleImpl's restoreView throws exception when viewId is null and in a portlet context

In restoreView:
if(viewId == null)
{
ExternalContext externalContext = facesContext.getExternalContext();
if(!externalContext.getRequestServletPath().endsWith("/"))

In a portlet context getRequestServletPath() returns null by definition. Not only should we avoid the NPE, but we should do something appropriate for portlets (return to the 'default-view')