DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGˇ
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=33839>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDˇ
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33839

           Summary: DecimalLocaleConverter und subClasses:
                    ConversionException not thrown
           Product: Commons
           Version: unspecified
          Platform: Macintosh
        OS/Version: Mac OS X 10.3
            Status: NEW
          Severity: major
          Priority: P2
         Component: Bean Utilities
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: [EMAIL PROTECTED]


To simulate the error run the TestCase 
org.apache.commons.beanutils.locale.LocalConvertUtilsTestCase.fixmetestNegativeScalar.

The DecimalLocaleConverter has an error in constructor chaining. 
The following constructor
>>>
    public DecimalLocaleConverter(Locale locale, String pattern, boolean 
locPattern) {

        this(null, locale, pattern, locPattern);
    }
<<<
calls the constructor
>>>
        public DecimalLocaleConverter(Object defaultValue, Locale locale, 
String pattern, boolean 
locPattern) {

        super(defaultValue, locale, pattern, locPattern);
        
    }
<<<
which always returns a default value.
The constructor has to call anlogous to DateLocalConverter 
>>>
        public DecimalLocaleConverter(Locale locale, String pattern, boolean 
locPattern) {
        super(locale, pattern, locPattern);
    }
<<<

By the way: I missing a CalendarLocalConverter as a subclass of 
DateLocaleConverter!

Thanks,

Stefan Lötscher, Inventage

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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

Reply via email to