DO NOT REPLY [Bug 17624] New: - Default convertion patterns in LocaleConverUtils do not work

[bu...@apache.org]

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

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=17624

Default convertion patterns in LocaleConverUtils do not work

           Summary: Default convertion patterns in LocaleConverUtils do not
                    work
           Product: Commons
           Version: 1.6 Final
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Bean Utilities
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: Michael@ITSolv.ca


I have found that although individual converters usually work well,
LocaleConvertUtils has probably never worked (there is an error during
static initialization of static ). 

If possible following changes should be adressed by the fix (patch is available 
and I'll do my best to attach it to the bug report):
 
- Converters should handle the null patterns more gracefully. As converters
default pattern to null in  constructors, it makes sense to handle it,
rather then throw NullPntrExc.

- Create LocaleConvertUtilsTestCase should be created class trying to make it 
as close as possible to original ConvertUtilsTestCase

- Improve parse method for Number related converters so they return
instance of the proper type (Integer, Byte, etc.). this was mainly to pass
the test case.

- Made default format patterns of java.sql.Date, Time, and Timestamp classes
consistent with default behaviour of the valueOf / toString methods.

- Change the build file by removing converter specific tests and
adding LocaleConvertUtils test, as it seems to be the case for the
traditional converters. Maybe we should keep all test cases though?

- Added a little logging where code was changed.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org