Summary: | Add warning when deleting import batch | ||
---|---|---|---|
Product: | Koha | Reporter: | Bernard <bernard.scaife> |
Component: | MARC Bibliographic record staging/import | Assignee: | Bernard <bernard.scaife> |
Status: | Pushed to stable --- | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | lucas |
Version: | unspecified | Keywords: | release-notes-needed |
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
25.11.00,25.05.03
|
Circulation function: | |
Attachments: |
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
Bug 38661: Add warning when deleting import batch Bug 38661: (follow-up) Eliminate duplicate ids |
Description
Bernard
2024-12-10 10:39:38 UTC
Created attachment 183872 [details] [review] 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 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." Created attachment 183888 [details] [review] 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> Created attachment 183889 [details] [review] Bug 38661: (follow-up) Eliminate duplicate ids The id attribute should be unique on the page, so I'm changing it to a name attribute and updating the corresonding selector. Also, we can eliminate find() here: $(this).closest('form').find('input... ...because $(this) IS the form, so we can optimize a tiny bit by removing the find(). Signed-off-by: Owen Leonard <oleonard@myacpl.org> Should this be 'Passed QA'? The changes here are minimal and everything looks good, keeping this as PQA even though it seems to lacking a proper QA. I'll consider it QA'ed by me. :) Please add release notes! Nice work everyone! Pushed to main for 25.11 Nice work everyone! Pushed to 25.05.x |