BdR76 / CSVLint

CSV Lint plug-in for Notepad++ for syntax highlighting, csv validation, automatic column and datatype detecting, fixed width datasets, change datetime format, decimal separator, sort data, count unique values, convert to xml, json, sql etc. A plugin for data cleaning and working with messy data files.
GNU General Public License v3.0
167 stars 10 forks source link

Number of columns with quotes not valid #2

Closed becabbage closed 3 years ago

becabbage commented 3 years ago

First of all, thanks and congratulations for the helpful and great plugin.

I found an issue when validating columns that contain quoted delimiter-characters,

Quote character is: " (double-quotes) Delimiter character is: , (comma)

This is my validation config:

Format=CSVDelimited
ColNameHeader=True
Col1=a Text Width 10
Col2=b Text Width 10
Col3=c Text Width 10

This is my test data:

a,b,c
a1,a2,a3
"a1","a,2","a3"
"a1","a2","a3"

The 3rd line throws an validation error, although the 2nd field "a,2" is quoted validly.

This is the error-message:

** error line 3: Too many columns
Inspected 4 lines, 1 data errors found, time elapsed 00:00:00.000

Would be great if you could look into this.

Thanks & best regards, Bernhard

BdR76 commented 3 years ago

Thanks for your feedback, the support for quoted strings is on the list of missing features. I will continue working on this plugin again soon, and this feature will be at the top of the priority list.

BdR76 commented 3 years ago

FYI quoted strings are now supported, plus it's now easier to spot any missing quotes in combination with the new syntax highlighting feature

becabbage commented 3 years ago

Yeah, that's amazing. Thanks!

On Wed 25. Aug 2021 at 15:01, BdR76 @.***> wrote:

FYI quoted strings are now supported, plus it's now easier to spot any missing quotes in combination with the new syntax highlighting feature

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/BdR76/CSVLint/issues/2#issuecomment-905480669, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALPHLESEGJ3QP3IMWGLHTWTT6TSSBANCNFSM44O564AA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .