[jira] [Commented] (OFBIZ-4837) Separator Error in data file tools
[ https://issues.apache.org/jira/browse/OFBIZ-4837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14501147#comment-14501147 ] Deepak Dixit commented on OFBIZ-4837: - Thanks Gil Portenseigne and Aditya. Your patch has been committed at following revision: Trunk at r#1674447 14.12 at r#1674448 13.07 at r#1674449 12.04 at r#1674450 Separator Error in data file tools -- Key: OFBIZ-4837 URL: https://issues.apache.org/jira/browse/OFBIZ-4837 Project: OFBiz Issue Type: Bug Components: framework Affects Versions: Trunk Reporter: Gil Portenseigne Assignee: Deepak Dixit Priority: Minor Labels: data, parsing Fix For: 14.12.01, 12.04.06, 13.07.02, Upcoming Branch Attachments: OFBIZ-4837.patch, OFBIZ-4837.patch, OFBIZ-4837_2.patch, dataDefinition.xml, dataSample.csv, result.xml, resultBefore.xml Original Estimate: 0.5h Remaining Estimate: 0.5h In https://demo-trunk.ofbiz.apache.org/webtools/control/viewdatafile There is a bug when defining simple separator (for instance ,) in definition file, and when in data file a string data contains the separator. This one sould not be interpreted in data parsing. To illustrate a bit more, i add small sample files to reproduce the problem. And the patch which fix the bug. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-4837) Separator Error in data file tools
[ https://issues.apache.org/jira/browse/OFBIZ-4837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13259665#comment-13259665 ] Gil Portenseigne commented on OFBIZ-4837: - I didn't comment the code : I add strVal != null, to avoid NPE The while loop is here to concatene the string value until the ending string separator is found. Separator Error in data file tools -- Key: OFBIZ-4837 URL: https://issues.apache.org/jira/browse/OFBIZ-4837 Project: OFBiz Issue Type: Bug Components: framework Affects Versions: SVN trunk Reporter: Gil Portenseigne Priority: Minor Labels: data, parsing Attachments: OFBIZ-4837.patch, dataDefinition.xml, dataSample.csv, result.xml, resultBefore.xml Original Estimate: 0.5h Remaining Estimate: 0.5h In https://demo-trunk.ofbiz.apache.org/webtools/control/viewdatafile There is a bug when defining simple separator (for instance ,) in definition file, and when in data file a string data contains the separator. This one sould not be interpreted in data parsing. To illustrate a bit more, i add small sample files to reproduce the problem. And the patch which fix the bug. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (OFBIZ-4837) Separator Error in data file tools
[ https://issues.apache.org/jira/browse/OFBIZ-4837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13259955#comment-13259955 ] Erwan de FERRIERES commented on OFBIZ-4837: --- Hi Gil, there is no need to rename your patches, Jira is directly deactivating the oldest one. Also, maybe the best test to explain your work is to add unit tests. thanks Separator Error in data file tools -- Key: OFBIZ-4837 URL: https://issues.apache.org/jira/browse/OFBIZ-4837 Project: OFBiz Issue Type: Bug Components: framework Affects Versions: SVN trunk Reporter: Gil Portenseigne Priority: Minor Labels: data, parsing Attachments: OFBIZ-4837.patch, OFBIZ-4837_2.patch, dataDefinition.xml, dataSample.csv, result.xml, resultBefore.xml Original Estimate: 0.5h Remaining Estimate: 0.5h In https://demo-trunk.ofbiz.apache.org/webtools/control/viewdatafile There is a bug when defining simple separator (for instance ,) in definition file, and when in data file a string data contains the separator. This one sould not be interpreted in data parsing. To illustrate a bit more, i add small sample files to reproduce the problem. And the patch which fix the bug. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira