View Issue Details

IDProjectCategoryView StatusLast Update
0006053JEDI VCL00 JVCL Componentspublic2013-12-16 17:05
ReporterleusAssigned Toobones 
PrioritynormalSeveritymajorReproducibilityalways
Status resolvedResolutionsuspended 
Product Version3.47 
Target VersionFixed in Version 
Summary0006053: Exception: "CSV string is too long" with non-escaped lines containing internal quotes
DescriptionTrying to import a simple CSV file (separated by semicolon) and get an exception on a line with a double quote inside a column:

CSV string is too long: 78494010-1;SOC COMERCIAL O"HIGGINS LIMIT.

I've attached a project that reproduces the issue. In the Delphi IDE try to change the dataset Active property to true and you should get the exception.

I've added MadExcept's information.
Additional InformationRemoved by reporter's request
TagsNo tags attached.

Relationships

has duplicate 0006149 closed TJvCSVDataSet : Error String too long. 

Activities

2012-12-26 02:35

 

test_csv.zip (490,458 bytes)

leus

2012-12-26 02:36

reporter   ~0020321

Hm, sorry for the spam - didn't notice how verbose MadExcept is.

obones

2013-01-15 16:09

administrator   ~0020382

Double quotes should be escaped, having them like that is a sure way to wreak havoc

leus

2013-01-16 05:30

reporter   ~0020394

A double quote located anywhere but right at the beginning of a new column shouldn't have any special meaning. If it does, it is a bug.

leus

2013-01-16 05:31

reporter   ~0020395

By the way, is it possible to remove the "Additional information" field? It is shameful (and full of private information!)

Arioch

2013-08-31 03:52

developer   ~0020608

Last edited: 2013-08-31 03:56

Can you just press "Edit" button to the left down from the list of attached files ?

----

http://tools.ietf.org/html/rfc4180

 If fields are not enclosed with double quotes, then
        double quotes may not appear inside the fields.

leus

2013-09-02 17:17

reporter   ~0020629

Ok, not a bug. Is it possible to make it support it anyways? Excel consumes and creates this file without problems. I'll take a look but I remember getting lost in the source last time.

Arioch

2013-09-03 08:17

developer   ~0020632

Good luck. TDataSet internals are a very complicated thing, i remember it from my TDbf days.

Now that Joanna Carter removed Delphi articles from her site, including DS internal mechanics review, that would be quite a task for you.

obones

2013-12-13 11:46

administrator   ~0020801

So, what do we do here? Simply resolve as "not fixable" ?

Arioch

2013-12-13 12:13

developer   ~0020807

I'd close as "postponed" or what that is in English. If someone would implement - he is welcome to add patch here. If no one would - then be it.

It is "fixable" - it is that none of us has resources/desire to design and implement the fix.

obones

2013-12-16 17:05

administrator   ~0020868

Suspending

Issue History

Date Modified Username Field Change
2012-12-26 02:35 leus New Issue
2012-12-26 02:35 leus File Added: test_csv.zip
2012-12-26 02:37 leus Note Added: 0020321
2013-01-15 16:09 obones Note Added: 0020382
2013-01-15 16:09 obones Status new => feedback
2013-01-16 05:30 leus Note Added: 0020394
2013-01-16 05:31 leus Note Added: 0020395
2013-08-31 03:52 Arioch Note Added: 0020608
2013-08-31 03:53 Arioch Relationship added has duplicate 0006149
2013-08-31 03:56 Arioch Note Edited: 0020608
2013-08-31 03:57 Arioch Additional Information Updated
2013-09-02 17:17 leus Note Added: 0020629
2013-09-03 08:17 Arioch Note Added: 0020632
2013-09-03 08:19 Arioch Summary Exception: "CSV string is too long" => Exception: "CSV string is too long" with non-escaped lines containing internal quotes
2013-12-13 11:46 obones Note Added: 0020801
2013-12-13 12:13 Arioch Note Added: 0020807
2013-12-16 17:05 obones Note Added: 0020868
2013-12-16 17:05 obones Status feedback => resolved
2013-12-16 17:05 obones Resolution open => suspended
2013-12-16 17:05 obones Assigned To => obones