Summary: | "Delete all items" doesn't check issues | ||
---|---|---|---|
Product: | Koha | Reporter: | Chris Cormack <chris> |
Component: | Cataloging | Assignee: | Galen Charlton <gmcharlt> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | P1 - high | CC: | jwagner, mathsabypro, wizzyrea |
Version: | Main | ||
Hardware: | PC | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: | This patch replaces DellItem by DellItemCheck in "delete all items". |
Description
Chris Cormack
2010-05-21 01:12:41 UTC
I just checked this feature in the 3.2 release candidate, and Delete All Items still did not block deletion of an item that was checked out. The item record was removed, leaving an orphan record in the issues table and attached to the patron's checkout list. I'm resetting the priority on this to blocker because I think this is a "don't use until it's fixed" problem. As a further update, I tested this with a triggered hold ("item is waiting") and an untriggered hold. Deleting all items removed the triggered hold from that patron's record, but did not remove the untriggered hold, presumably on the theory that items could be added back to the title. If the title is then deleted, the untriggered hold is also deleted. This feature should have a warning at the delete all items stage for checkouts and holds. Pushed patches so that items on loan, waiting on the hold shelf, or in transit for a hold request are no longer deleted. Downgrading bug, but leaving open for the cosmetic issue of giving the operator an explicit indication that some items were not deleted. Waiting and transit statuses are still ignored in master as of 4-1-11. http://screencast.com/t/Lh5vwCxetec Also, it does tell you that some items weren't deleted, so that part's fixed. :) I believe this is fixed - you have to cancel all holds now before you can delete all items. My test record had a waiting and a reserved (not yet pulled) item. |