From 381898d8ca2a78319dfc9e6d8e44564754172766 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 4 Jan 2022 15:59:15 +0100 Subject: [PATCH] Bug 29790: Restore warning if deletion of serial item fails If the deletion of a serial item failed, the UI did not provide a warning/error message. Test plan: 0. Create a new subscription with "Create an item record when receiving this serial" 1. Receive a new item, set a barcode 2. Check it out 3. Select the item you have received in the serial item list (page "Serial collection information") and click "Delete selected issues" 4. Tick "Delete the associated items" and confirm the deletion => Without this patch the deletion fail but the UI does not warn it => With this patch applied you see a warning "one or more associated items could not be deleted at this time." Signed-off-by: Tomas Cohen Arazi Signed-off-by: Kyle M Hall Signed-off-by: Fridolin Somers --- serials/serials-collection.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/serials/serials-collection.pl b/serials/serials-collection.pl index 963eeeb577..2179b83069 100755 --- a/serials/serials-collection.pl +++ b/serials/serials-collection.pl @@ -123,9 +123,9 @@ if($op eq 'delete_confirm'){ } my $items = Koha::Items->search({ itemnumber => \@itemnumbers }); while ( my $item = $items->next ) { - my $error = $item->safe_delete; + my $deleted = $item->safe_delete; $template->param(error_delitem => 1) - if $error eq '1'; + unless $deleted; } } for my $serialid (@serialsid){ -- 2.39.2