I select "quote values" from the CSV display plugin's options and views_export quotes all values in rows but the header row.

Some of the header values contain the separator character (,) and as a result my CSV files break. Will follow up with patch asap.

CommentFileSizeAuthor
#1 512630-1_quote_headers.patch955 bytesalex_b

Comments

alex_b’s picture

alex_b commented
Assigned: alex_b » Unassigned
Status: Active » Needs review
StatusFileSize
new 512630-1_quote_headers.patch955 bytes

Attached patch formats headers just as themed_rows are being formatted. Works fine on my web site, will push this to production now. Looking forward to reviews.

alex_b’s picture

alex_b commented

ping?

andreiashu’s picture

andreiashu commented
Status: Needs review » Reviewed & tested by the community

Hi,

I just looked over your patch and it looks good. It shouldn't do anything wrong.

RTBC?

neclimdul’s picture

neclimdul
He/Him
commented
Status: Reviewed & tested by the community » Fixed

It didn't actually apply but I pulled the code out and committed it.

alex_b’s picture

alex_b commented

thank you.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.