From dc39e4b410c173d497ce41651dbf39d6a72dd461 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 29 Oct 2015 07:48:02 -0400 Subject: [PATCH] Bug 15079: Batch Delete Records Error MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit When using the batch deletion tool, if the script attempts to delete an undeletable item, the script will die with the error: Label not found for "next BIBLIONUMBER" at /usr/share/koha/intranet/cgi-bin/tools/batch_delete_records.pl line 172. Test Plan: 1) Attempt to batch delete a record with an item that cannot be deleted 2) Note error 3) Apply this patch 4) Repeat step 1 5) Note the error no longer occurs Signed-off-by: Hector Castro Works as advertised. Follow comment 5 to reproduce the issue Signed-off-by: Jonathan Druart I have not tested this patch, but looking at the code it is obviously a mistake. If an error occured, we rollback and skip this record. Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 706a31b50d52ef1ba48c568d7e60c376cfe87983) Signed-off-by: Frédéric Demians --- tools/batch_delete_records.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/batch_delete_records.pl b/tools/batch_delete_records.pl index b0523d536c..65cbf21e3c 100755 --- a/tools/batch_delete_records.pl +++ b/tools/batch_delete_records.pl @@ -169,7 +169,7 @@ if ( $op eq 'form' ) { error => ($@ ? $@ : $error), }; $dbh->rollback; - next BIBLIONUMBER; + next RECORD_IDS; } } -- 2.39.5