gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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 <oleonard@myacpl.org>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit fd425e815c)
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
This commit is contained in:
Bernard Scaife 2025-07-08 15:57:06 +00:00 committed by Paul Derscheid
parent a00b2503b4
commit 65caca0ba8
No known key found for this signature in database
GPG Key ID: C25F9FD44CED595B
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt
View File

@ -388,6 +388,7 @@
[% INCLUDE 'csrf-token.inc' %]
<input type="hidden" name="import_batch_id" value="[% batch_lis.import_batch_id | html %]" />
<input type="hidden" name="op" value="cud-clean-batch" />
<input type="hidden" id="import_status" value="[% batch_lis.import_status | html %]" />
<button type="submit" class="btn btn-default btn-xs"><i class="fa fa-eraser"></i> Clean</button>
</form>
[% 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(