From 2beb49666e0cc52b9b7975d42930fe180bce50f9 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 4 Jul 2018 15:05:25 -0300 Subject: [PATCH] Bug 21035: Handle new lines when running reports See https://metacpan.org/pod/Text::CSV_XS#Embedded-newlines Test plan: 1) Choose two items, say barcode '123' and '456' 2) Change the public note on 123 to read Line1 Line2 (I.e. type 'Line1', then press Enter, type 'Line2' and click update). 3) Change the public note on 456 to read Public note has one and only one line. Click update. 4) Create a report with the followng query: select barcode, itemnotes from items where barcode in ( '123', '456' ) Let's say that this is report number 10. 5) run ./misc/cronjobs/runreport.pl --format=csv REPORT_ID: => You should see both lines --- misc/cronjobs/runreport.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/misc/cronjobs/runreport.pl b/misc/cronjobs/runreport.pl index cb10125938..65b0e97e26 100755 --- a/misc/cronjobs/runreport.pl +++ b/misc/cronjobs/runreport.pl @@ -273,6 +273,7 @@ foreach my $report_id (@ARGV) { $message = $cgi->table(join "", @rows); } elsif ($format eq 'csv') { my $csv = Text::CSV_XS->new({ + binary => 1, quote_char => $quote, sep_char => $separator, }); -- 2.11.0