[Bug 29790] New: Deleting serial items fail without warning
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Bug ID: 29790 Summary: Deleting serial items fail without warning Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: Serials Assignee: jonathan.druart+koha@gmail.com Reporter: jonathan.druart+koha@gmail.com QA Contact: testopia@bugs.koha-community.org If the deletion of a serial item failed, the UI does not provide a warning/error message. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |29788 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29788 [Bug 29788] Make Koha::Item->safe_to_delete return a Koha::Result::Boolean object -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 --- Comment #1 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Created attachment 128992 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128992&action=edit 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." -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off Patch complexity|--- |Trivial patch CC| |tomascohen@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #128992|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 --- Comment #2 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 129006 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129006&action=edit 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 <tomascohen@theke.io> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #129006|0 |1 is obsolete| | --- Comment #3 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 129168 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129168&action=edit 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 <tomascohen@theke.io> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |22.05.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 --- Comment #4 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to master for 22.05, thanks to everybody involved 🦄 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Version(s)|22.05.00 |22.05.00,21.11.02 released in| | Status|Pushed to master |Pushed to stable --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to 21.11.x for 21.11.02 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29790 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@bywatersolutions.com --- Comment #6 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- With this patch applied to 21.05.x, following the test plan, the issue is deleted but the item is not and I receive no error message. I see no change in behavior with or without the patch. Jonathan, can you take a look? I'm happy to backport, but this doesn't seem to do the trick. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org