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
            Priority: Minor


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.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to