[PORTLETBRIDGE-18] ExternalContext.getViewIdFromPath missing an else statement

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0.0-alpha-2
• Fix Version/s: 1.0.0-alpha
• Component/s: Impl
• Assignee: Unassigned
• Reporter: Michael Freedman
• Created: 30/Nov/07 18:41
• Updated: 19/Feb/08 17:51
• Resolved: 30/Nov/07 18:51

Description

Code in EC.getViewIdFromPath is missing an else clause causing us to drop the suffix translation. I.e. code should have an else before the if (isPrefixedMapped) --inserted code (else) is in all caps:
// Okay now figure out whether this is prefix or suffixed mapped
if (isSuffixedMapped(url, mFacesMappings))

{ viewId = viewIdFromSuffixMapping( url, mFacesMappings, mPortletContext .getInitParameter(ViewHandler.DEFAULT_SUFFIX_PARAM_NAME)); }

ELSE if (isPrefixedMapped(url, mFacesMappings))

{ viewId = viewIdFromPrefixMapping(url, mFacesMappings); }

else

{ // Set to what follows the URL viewId = url; }

return viewId;