myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Simon Kitching (JIRA)" <...@myfaces.apache.org>
Subject [jira] Commented: (MYFACES-1798) DefaultViewHandlerSupports fails detecting extension if there are dots in the path
Date Thu, 03 Jan 2008 12:49:35 GMT

    [ https://issues.apache.org/jira/browse/MYFACES-1798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12555557#action_12555557
] 

Simon Kitching commented on MYFACES-1798:
-----------------------------------------

What about paths like:
    /my.strange.path/faces/welcome

Using just "lastIndexOf" will not return the expected value. Agreed, it is no worse than the
current solution, but if this is being fixed now, it would be nice to fix it properly. How
about looking for lastIndexOf("/") and only falling into the "extension mapping" case when
the last dot is after the last slash?

And is it valid to have an extension with two dots, eg ".faces.jsf"? Ideally, it would be
best to first look up what suffixes we expect, then use "endsWith" to determine which one
is present on the url..don't know if that is possible or not though..

> DefaultViewHandlerSupports fails detecting extension if there are dots in the path
> ----------------------------------------------------------------------------------
>
>                 Key: MYFACES-1798
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1798
>             Project: MyFaces Core
>          Issue Type: Bug
>    Affects Versions:  1.2.0
>            Reporter: Mario Ivankovits
>            Assignee: Mario Ivankovits
>             Fix For: 1.2.2-SNAPSHOT
>
>
> For example: With "/myStrange.path/view.jsp" the extension ".path/view.jsp" will be derived.
> In DefaultViewHandlerSupport a wrong regular expression is used to extract the extension.
> I'll try to change this to a simple string.lastIndexOf('.') as this should do exactly
what we want.

-- 
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