search

wadetb / Sublime-Text-Advanced-CSV

Efficiently format, edit, arrange, and evaluate cells in CSV files
169 stars 14 forks source link

Making auto_quote system false by default #48

Open VictorShima opened 6 years ago

VictorShima commented 6 years ago

First of, Thank you very much for working on this plugin!

When using auto_quote: true the following piece of CSV will have its quotes removed when a CSV > Compact columns is used:

oranges,"tasty
fruits"
apples,"delicious treats"

Becomes:

oranges,tasty
fruits
apples,delicious treats

It seems like the command is not respecting line breaks inside quotes, turning a valid CSV with 2 rows into an invalid CSV with 3 rows. The Section 2.6 of a CSV Standard describes the usage of line breaks inside quotes.

Removing the auto_quote configuration makes it work wonderfully out-of-the-box. Other issues also seem to suggest that the auto_quote is having couple of edge cases and past problems.

Would you consider making auto_quote disabled on default?

wadetb commented 6 years ago

Hi @VictoryShima - thanks for the report! I'll investigate the bugs with auto quote and will consider disabling it.