[
https://issues.apache.org/jira/browse/CAMEL-12260?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16367433#comment-16367433
]
ASF GitHub Bot commented on CAMEL-12260:
----------------------------------------
GitHub user dmvolod opened a pull request:
https://github.com/apache/camel/pull/2228
CAMEL-12260: Default value for String field results is null for CSV/Bindy
The DataField.defaultValue always "" for String by default.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dmvolod/camel CAMEL-12260
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/camel/pull/2228.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2228
----
commit e16251daa7ed03c9fd4ffef54a03c1947bad5e03
Author: Dmitry Volodin <dmvolod@...>
Date: 2018-02-16T14:55:00Z
CAMEL-12260: Default value for String field results is null for
CSV/Bindy
----
> 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.20.3, 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)
