From b57fb0a9b20333ce52536b319342c2dd5021e3da Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Wed, 24 Aug 2016 11:44:55 +0200 Subject: [PATCH] Bug 16935 - launch export_records.pl with deleted_barcodes param fails MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit When launching misc/export_records.pl with this command line : misc/export_records.pl --date=`date +%d/%m/%Y` --deleted_barcodes --filename=/tmp/koha.mrc You get this error message : DBD::mysql::db selectall_arrayref failed: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 4 [for Statement " ( SELECT DISTINCT barcode FROM deleteditems WHERE deleteditems.biblionumber = ? "] at misc/export_records.pl line 189. This is because of a '(' after 'q|', looks like a typo. Also, this patch removes useless var $q. Test plan : - Delete an item with barcode - Without patch, run : misc/export_records.pl --date=`date +%d/%m/%Y` --deleted_barcodes --filename=/tmp/koha.mrc => You get dirty MySQL - Without patch, run the same command => No error and the file is generated Signed-off-by: Nicolas Legrand Signed-off-by: Jonathan Druart Signed-off-by: Kyle M Hall (cherry picked from commit dbab260afccc04fac8d088839d6017242c818917) Signed-off-by: Frédéric Demians --- misc/export_records.pl | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/misc/export_records.pl b/misc/export_records.pl index b9a2d3a3b5..ccbade235a 100755 --- a/misc/export_records.pl +++ b/misc/export_records.pl @@ -184,14 +184,12 @@ if ( @record_ids and $id_list_file ) { if ($deleted_barcodes) { for my $record_id ( @record_ids ) { - my $q = q| - |; - my $barcode = $dbh->selectall_arrayref(q| ( + my $barcode = $dbh->selectall_arrayref(q| SELECT DISTINCT barcode FROM deleteditems WHERE deleteditems.biblionumber = ? |, { Slice => {} }, $record_id ); - say $_->{barcode} for @$barcode + say $_->{barcode} for @$barcode; } } else { -- 2.20.1