[ 
https://issues.apache.org/jira/browse/BEANUTILS-283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12503448
 ] 

Niall Pemberton commented on BEANUTILS-283:
-------------------------------------------

Sorry - DateConverter is new - not refactored!
(see BEANUTILS-255)

> ConvertUtilsBean doesn't handl conversion to String correctly
> -------------------------------------------------------------
>
>                 Key: BEANUTILS-283
>                 URL: https://issues.apache.org/jira/browse/BEANUTILS-283
>             Project: Commons BeanUtils
>          Issue Type: Bug
>          Components: ConvertUtils & Converters
>    Affects Versions: Nightly Builds
>         Environment: commons-beanutils-20070610-src.zip
>            Reporter: Josef Cacek
>            Assignee: Niall Pemberton
>             Fix For: 1.8.0
>
>         Attachments: Test.java
>
>
> Either lookup(Class, Class) or convert(Object, Class) method doesn't work 
> correctly in class ConvertUtilsBean.
> Problematic is the case when target class is String.
> The code from convert(..) which doesn't work:
>         Object converted = value;
>         Converter converter = lookup(sourceType, targetType);
>         if (converter != null) {
>             if (log.isTraceEnabled()) {
>                 log.trace("  Using converter " + converter);
>             }
>             converted = converter.convert(targetType, value);
>         }
> // here we already have converted value (String), but we will rewrite it by 
> default .toString() - it's a bug
>         if (targetType == String.class && value != null) {
>             converted = value.toString();
>         }

-- 
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