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

Andreas Andreou resolved TAPESTRY-1974.
---------------------------------------

       Resolution: Fixed
    Fix Version/s: 4.1.7
         Assignee: Andreas Andreou

> DataSqueezerImpl: DataSqueezer.adaptor-prefix-taken: wrong argument to error 
> message -> conflicting prefix not shown
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-1974
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1974
>             Project: Tapestry
>          Issue Type: Bug
>          Components: Framework
>    Affects Versions: 4.1.3
>            Reporter: Oliver Spier
>            Assignee: Andreas Andreou
>            Priority: Minor
>             Fix For: 4.1.7
>
>
> When trying to register a SqueezeAdaptor with a prefix that is already used 
> by another registered SqeezeAdaptor, the error message says:
> "An adaptor for prefix '' is already registered."
> i.e. it does not specify _which_ prefix is already registered. The Exception 
> in DataSqueezerImpl.register(SqeezeAdaptor) is thrown as follows:
>             if (_adaptorByPrefix[offset] != null)
>                 throw new IllegalArgumentException(Tapestry.format(
>                         "DataSqueezer.adaptor-prefix-taken",
>                         Character.valueOf( prefix.charAt(i) ) ));
>                         prefix.substring(i, i)));
> Fix suggestion: The call to prefix.substring(i, i) should be replaced with 
> prefix.substring(i, i+1) 
> (see 
> http://java.sun.com/javase/6/docs/api/java/lang/String.html#substring(int,%20int)
>  )

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

Reply via email to