You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by "Denis Delangle (JIRA)" <ji...@apache.org> on 2012/05/23 08:45:40 UTC

[jira] [Updated] (TAP5-1936) REOPEN TAP5-698 - PageTester's setupRequestFromURI incorrectly uses substring twice on same string

     [ https://issues.apache.org/jira/browse/TAP5-1936?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Denis Delangle updated TAP5-1936:
---------------------------------

        Fix Version/s:     (was: 5.3)
          Description: 
I get this error when clicking a link with parameters in PageTester
java.lang.StringIndexOutOfBoundsException: String index out of range: -1
       at java.lang.String.substring(String.java:1937)
       at java.lang.String.substring(String.java:1904)
       at org.apache.tapestry5.test.PageTester.setupRequestFromURI(PageTester.java:345)

That bug was corrected revision 896555 and put back revision 1185926.
The correction is trivial but have been done twice, so the bug is
still there.

I submit a patch to correct it

  was:
Incoming linkPath: /filelist.delete/c$002ftest2-2510777506787452085.txt?t:ac=c
int comma is set to 52
path: /filelist.delete/c$002ftest2-2510777506787452085.txt
then trying to do a path.substring(comma+1)

Fix: use linkPath instead of path at line 304:

if (comma > 0)
            decodeParametersIntoRequest(path.substring(comma + 1));

    Affects Version/s:     (was: 5.1.0.5)
                       5.3.3
    
> REOPEN TAP5-698 - PageTester's setupRequestFromURI incorrectly uses substring twice on same string
> --------------------------------------------------------------------------------------------------
>
>                 Key: TAP5-1936
>                 URL: https://issues.apache.org/jira/browse/TAP5-1936
>             Project: Tapestry 5
>          Issue Type: Bug
>          Components: tapestry-test
>    Affects Versions: 5.3.3
>            Reporter: Denis Delangle
>            Assignee: Howard M. Lewis Ship
>
> I get this error when clicking a link with parameters in PageTester
> java.lang.StringIndexOutOfBoundsException: String index out of range: -1
>        at java.lang.String.substring(String.java:1937)
>        at java.lang.String.substring(String.java:1904)
>        at org.apache.tapestry5.test.PageTester.setupRequestFromURI(PageTester.java:345)
> That bug was corrected revision 896555 and put back revision 1185926.
> The correction is trivial but have been done twice, so the bug is
> still there.
> I submit a patch to correct it

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira