From 1ae360a87ff2d542c7c1b97d0df95c5044168b75 Mon Sep 17 00:00:00 2001 From: Bernard Scaife Date: Tue, 8 Jul 2025 15:57:06 +0000 Subject: [PATCH] Bug 38661: Add warning when deleting import batch When a batch is cleaned from the staged marc records screen, the user may be unaware of records that are imported within the batch which they may want to revert in future. Cleaning the batch unlinks these and makes reversion / cleanup difficult. This patch gives an extra warning when cleaning a batch where there are still linked records attached. Test plan: 1. Stage a marc record from a mrc file 2. Import the batch 3. Return to the managed staged records screen. 4. Click "clean" 5. Note message is "Clear all reservoir records staged in this batch? This cannot be undone." 6. Apply patch and restart all. 7. Repeat steps 1-4 with a new batch 8. Note message has changed to "This batch contains imported records. Are you sure you wish to clear all the reservoir records? This cannot be undone." Signed-off-by: Owen Leonard --- .../prog/en/modules/tools/manage-marc-import.tt | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt index 5ba37a094e3..e1c25ae544c 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt @@ -388,6 +388,7 @@ [% INCLUDE 'csrf-token.inc' %] + [% ELSIF ( batch_lis.import_status == 'cleaned' ) %] @@ -634,8 +635,12 @@ if ($(this).hasClass("batch_delete")) { return confirm(_("Are you sure you want to permanently delete this batch?")); } else { + if ($(this).closest('form').find('input#import_status').val() === "imported" ) { + return confirm(_("This batch contains imported records. Are you sure you wish to clear all the reservoir records? This cannot be undone.")); + } else { return confirm(_("Clear all reservoir records staged in this batch? This cannot be undone.")); } + } }); [% IF job_enqueued %] setTimeout( -- 2.39.5