embedded comma in double quoted string causes failures

jokedst / CsvQuery

Plugin for Notepad++ that treats CSV files as (read only) SQL tables

153 stars 28 forks source link

embedded comma in double quoted string causes failures #10

Closed VintageGeek closed 6 years ago

VintageGeek commented 6 years ago

Using this example: "ControlID","Status","FeatureName","ResourceGroupName","ResourceName","ChildResourceName","ControlSeverity","IsBaselineControl","SupportsAutoFix","Description","Recommendation","DetailedLogFile" "test123","test123","test123","test123","test123","test123","test123","test123","test123","test123","comma goes here, ","test123"

I get the error above. It doesn't prompt to manually parse, but if i manually parse with default settings, file is read correctly.

v1.2.5

Thanks!

jokedst commented 6 years ago

Thank's for the bug report. It seems the analyzer decides that the best separator is double quotes, not comma... (all rows has the same number of quotes, but not two rows has the same number of commas).

I could make a quick fix to avoid quotes (which would make the manual settings pop up instead). Long term quoted string analysis will be added

VintageGeek commented 6 years ago

Ah, understood. I'll make the change on my local repo and wait for your longer term fix