Bug 14518: Allow deleting records with holds (and no item) after confirmation
authorRemi Mayrand-Provencher <remi.mayrand-provencher@inLibro.com>
Wed, 20 May 2015 17:26:23 +0000 (13:26 -0400)
committerTomas Cohen Arazi <tomascohen@theke.io>
Mon, 5 Oct 2015 15:33:31 +0000 (12:33 -0300)
commit69400a635c98f2da7ddfc3d4902a087cff12dce3
treeb2f73a2c26cb1d824f1bb8f8d2df0ab8884449e8
parent26383113d0cc7edf58458795349b252dfbee25c6
Bug 14518: Allow deleting records with holds (and no item) after confirmation

Cleaning up records with no item but holds is impossible since
the code does not consider the result from the confirmation popup.

This situation happens to our client in their (annual) cleanup.

I. Setup
Create the following if needed:
1) A patron for placing holds and checkouts
2) A test bibliographic record
3) An item for the record. Set the barcode to "DELETEME"

II. Corrupting the Database
1) Checkout the item "DELETEME"
2) Place a record level hold on the test record (next available item)
3) Checkin the item "DELETEME"
4) MySQL: DELETE FROM items WHERE barcode="DELETEME";

III. Deleting the record.
1) On the test record's details page: Edit -> Delete record
2) Click 'yes' in the prompt
3) Nothing happens.

IV. Fixing the bug
1) Apply the patch
2) Repeat step III to delete the record
3) Record is deleted

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Note: It's not possible in Koha to place a hold on a
record with items, so the effect described here is more
likely to be the result of a migration issue or another
bug. But doesn't hurt to provide a way out :)
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc