gradle-license-plugin icon indicating copy to clipboard operation
gradle-license-plugin copied to clipboard

CSV writer doesn't escape / remove commas & newlines

Open agonzalez-plume opened this issue 2 years ago • 0 comments

The csv output formatter gets confused when commas exist in the text being outputted. The commas could simply be removed is probably the easiest.

As well newline characters seem to confused it too for instance '\n' in the description causes problems.

fixing could be as simple as doing: csvreport.kt:82:

element.replace(",", " ").replace("\n"," ")

I haven't done a plugin change before, so not entirely sure how to test out the change locally before committing.

I have good results with the json output, so I'm using dasel to convert json to csv and it's close to what I want, but the csvwriter output would be better.

I see a similar problem with HTML output being mixed up, but I think some others were seeing similar issues.

agonzalez-plume avatar Aug 10 '22 01:08 agonzalez-plume