[ 
https://issues.apache.org/jira/browse/OFBIZ-3026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12844098#action_12844098
 ] 

BJ Freeman commented on OFBIZ-3026:
-----------------------------------

from a programmers point of view (I coded it right your data is wrong) is 
valid, but not acceptable in a environment where human err can injected bad 
data, from an external source.
I am not sure if it is called the same but the Term "bullet proofing" code 
means the programmer has taken into account the possible errors that could be 
induced and develop helpful messages or some compensation for the error to 
allow the correct operation to be performed.

IMHO bullet proofing for this is lacking.


> Datafile does not catch lack of delimiter if at end of line.
> ------------------------------------------------------------
>
>                 Key: OFBIZ-3026
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3026
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: Release Branch 9.04, SVN trunk
>         Environment: 9.04 revision 823664
>            Reporter: BJ Freeman
>            Assignee: Jacques Le Roux
>             Fix For: Release Branch 9.04
>
>
> datafile that does not have the last delimiter before the CRLF.
> 2009-10-10 20:33:19,265 (http-0.0.0.0-8443-2) 
> [ModelDataFileReader.java:135:INFO ] -- getModelDataFile: #1 Loaded dataFile: 
> Product
> 2009-10-10 20:33:19,265 (http-0.0.0.0-8443-2) [ModelDataFileReader.java:74 
> :INFO ] [ModelDataFileReader.getModelDataFileReader] : returning reader.
> 2009-10-10 20:33:19,265 (http-0.0.0.0-8443-2) [ModelDataFileReader.java:74 
> :INFO ] [ModelDataFileReader.getModelDataFileReader] : returning reader.
> 2009-10-10 20:33:19,265 (http-0.0.0.0-8443-2) [                       ?:?  
> :INFO ] 
> ---- exception report 
> ----------------------------------------------------------
> Exception: org.ofbiz.datafile.DataFileException
> Message: Field map could not be read from a line (1) with only 234 chars. 
> (null)
> ---- cause 
> ---------------------------------------------------------------------
> Exception: java.util.NoSuchElementException
> Message: null
> ---- stack trace 
> ---------------------------------------------------------------
> java.util.NoSuchElementException
> java.util.StringTokenizer.nextToken(Unknown Source)
> org.ofbiz.datafile.Record.createDelimitedRecord(Record.java:555)
> org.ofbiz.datafile.RecordIterator.getNextLine(RecordIterator.java:123)
> org.ofbiz.datafile.RecordIterator.setupStream(RecordIterator.java:79)
> org.ofbiz.datafile.RecordIterator.<init>(RecordIterator.java:61)
> org.ofbiz.datafile.DataFile.makeRecordIterator(DataFile.java:165)
> org.ofbiz.datafile.DataFile.readDataFile(DataFile.java:124)
> org.ofbiz.datafile.DataFile.readFile(DataFile.java:63)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to