[ 
https://issues.apache.org/jira/browse/BETWIXT-58?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12509268
 ] 

Martin van den Bemt commented on BETWIXT-58:
--------------------------------------------

Oh forgot to add : since value can be null, I decided to just return "" instead 
of value (since the javadoc says it will not return null)

> Empty string bug
> ----------------
>
>                 Key: BETWIXT-58
>                 URL: https://issues.apache.org/jira/browse/BETWIXT-58
>             Project: Commons Betwixt
>          Issue Type: Bug
>    Affects Versions: Nightly Builds
>         Environment: N/A
>            Reporter: Kevin Waugh
>            Priority: Minor
>             Fix For: Nightly Builds
>
>
> If you create a bean with a string property and set the string to be the 
> empty string then write the bean to xml using Betwixt upon reading the bean 
> the string property will be set to null.
> The stringToObject method in ConvertUtilsObjectStringConverter does not obey 
> it's javadoc description.  The following patch still violates the 
> description, but fixes the bug stated.
> Index: 
> src/java/org/apache/commons/betwixt/strategy/ConvertUtilsObjectStringConverter.java
> ===================================================================
> --- 
> src/java/org/apache/commons/betwixt/strategy/ConvertUtilsObjectStringConverter.java
>  (revision 534335)
> +++ 
> src/java/org/apache/commons/betwixt/strategy/ConvertUtilsObjectStringConverter.java
>  (working copy)
> @@ -61,7 +61,7 @@
>      public Object stringToObject(String value, Class type, String flavour, 
> Context context) {
>          if (value == null || "".equals(value))
>          {
> -            return null;
> +            return value;
>          }
>          return ConvertUtils.convert( value, type );

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