From de46f4c0692c0d56370359a65016759870b73717 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 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 --- 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 83d8a7b..34b4da3 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.1.4