[ 
https://issues.apache.org/jira/browse/CAMEL-12260?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16367438#comment-16367438
 ] 

ASF GitHub Bot commented on CAMEL-12260:
----------------------------------------

oscerd commented on issue #2228: CAMEL-12260: Default value for String field 
results is null for CSV/Bindy
URL: https://github.com/apache/camel/pull/2228#issuecomment-366259140
 
 
   LGTM, since this is a change in the current behavior of 2.20.x I'll merge 
this one only on master.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> Default value for String field results in null for CSV / Bindy
> --------------------------------------------------------------
>
>                 Key: CAMEL-12260
>                 URL: https://issues.apache.org/jira/browse/CAMEL-12260
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-bindy
>    Affects Versions: 2.20.2
>            Reporter: Pauli Borodulin
>            Assignee: Dmitry Volodin
>            Priority: Minor
>             Fix For: 2.21.0
>
>
> **CAMEL-5018 added "support the set the default value on the field of bindy 
> CVS", which is nice, but the default value is useless for Strings, because 
> BindyCSVFactory turns the empty value to null even when defaultValue = "", see
> [https://github.com/apache/camel/blob/camel-2.20.x/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java#L230]
> and
> [https://github.com/apache/camel/blob/camel-2.20.x/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyAbstractFactory.java#L234]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to