[ 
https://issues.apache.org/jira/browse/TAPESTRY-1974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12779828#action_12779828
 ] 

Hudson commented on TAPESTRY-1974:
----------------------------------

Integrated in tapestry-4.1-trunk #11 (See 
[http://hudson.zones.apache.org/hudson/job/tapestry-4.1-trunk/11/])
    

> 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