myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Freedman (JIRA)" <...@myfaces.apache.org>
Subject [jira] Created: (PORTLETBRIDGE-18) ExternalContext.getViewIdFromPath missing an else statement
Date Fri, 30 Nov 2007 18:41:42 GMT
ExternalContext.getViewIdFromPath missing an else statement
-----------------------------------------------------------

                 Key: PORTLETBRIDGE-18
                 URL: https://issues.apache.org/jira/browse/PORTLETBRIDGE-18
             Project: MyFaces Portlet Bridge
          Issue Type: Bug
          Components: Impl
    Affects Versions: 1.0.0-SNAPSHOT
            Reporter: Michael Freedman
             Fix For: 1.0.0-SNAPSHOT


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;

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message