Currently an item can be marked withdrawn if it is checked out to a patron or in transit from one branch to another. This causes confusion for staff checking an item back in or receiving the transfer. If an item is checked out or in transit it should not be able to be marked withdrawn.
Conversely, it would be nice to have the option to have items check-in automatically (be removed from a customer's account) when a staff member marks them as withdrawn, so that they don't linger forever in both statuses.
+! agreed - If an item is checked out or in transit, it should not be able to be marked withdrawn.
Also agree! This is a major cause for confusion with our staff as well. Preventing items that aren't checked in from being withdrawn would be a huge help.
Automatic return might not be very forgiving if you ended up picking a wrong item. Maybe easier to warn/deactivate and have the staff person return it if it's required.
I would like a message to pop up when you try to check in a withdrawn item that notifies staff that the item is withdrawn. Presently, it returns the message that the item was not checked out even on items that are both checked out and withdrawn
Created attachment 176241 [details] [review] Bug 23010: Add system preference PreventWithdrawingItemsStatus
(In reply to Felicity Brown from comment #1) > Conversely, it would be nice to have the option to have items check-in > automatically (be removed from a customer's account) when a staff member > marks them as withdrawn, so that they don't linger forever in both statuses. This is a cool idea and I encourage you to file it as a separate bug!
Created attachment 177299 [details] [review] Bug 23010: Add ability to block withdrawning of items with certain statuses
Created attachment 177402 [details] [review] Bug 23010: Handle error in updateitem.pl and template
Created attachment 177413 [details] [review] Bug 23010: Add unit tests
Created attachment 177434 [details] [review] Bug 23010: Add system preference PreventWithdrawingItemsStatus To test: 1. APPLY PATCH, updatedatabase, and restart_all 2. Search for the system preference, PreventWithDrawingItemsStatus 3. Select both in transit and checked out, then save 4. Find an item and put it in transit. 5. Now from the item detail page, go to manage items > edit item 6. Using the item that was put in transit, attempt to change the status to withdrawn. 7. You should see an error at the top of the page 'In transit item cannot be withdrawn.' Make sure the item did not get marked as withdrawn. 8. Using a different item, do a checkout 9. Now from the item detail page, go to manage items > edit item 10. Using the checked out item, attempt to change the status to withdrawn. 11. vYou should see an error at the top of the page 'Onloan item cannot be withdrawn.' Make sure the item did not get marked as withdrawn. 12. Now, using the in transit item, go to the items tab. ( catalogue/moredetail.pl?biblionumber=x ) 13. Under Statuses, attempt to withdraw the item. 14. You should see an error near the top of the page "Cannot withdraw item in transit". 15. Now, using the checked out item, go to the items tab. ( catalogue/moredetail.pl?biblionumber=144 ) 16. Under Statuses, attempt to withdraw the item. 17. You should see an error near the top of the page "Cannot withdraw checked out item.". 18. Using both the in-transit and checked out item, attempt to modify the status through batchMod. 19. Ensure that the items are not being updated to withdrawn. 20. Turn off PreventWithDrawingItemsStatus system preferece. 21. Going through each of the steps again, except this time your items should successfully withdraw. prove t/db_dependent/Koha/Item.t
Created attachment 177435 [details] [review] Bug 23010: Add ability to block withdrawning of items with certain statuses
Created attachment 177436 [details] [review] Bug 23010: Handle error in updateitem.pl and template
Created attachment 177437 [details] [review] Bug 23010: Add unit tests
Created attachment 177451 [details] [review] Bug 23010: Add system preference PreventWithdrawingItemsStatus To test: 1. APPLY PATCH, updatedatabase, and restart_all 2. Search for the system preference, PreventWithDrawingItemsStatus 3. Select both in transit and checked out, then save 4. Find an item and put it in transit. 5. Now from the item detail page, go to manage items > edit item 6. Using the item that was put in transit, attempt to change the status to withdrawn. 7. You should see an error at the top of the page 'In transit item cannot be withdrawn.' Make sure the item did not get marked as withdrawn. 8. Using a different item, do a checkout 9. Now from the item detail page, go to manage items > edit item 10. Using the checked out item, attempt to change the status to withdrawn. 11. vYou should see an error at the top of the page 'Onloan item cannot be withdrawn.' Make sure the item did not get marked as withdrawn. 12. Now, using the in transit item, go to the items tab. ( catalogue/moredetail.pl?biblionumber=x ) 13. Under Statuses, attempt to withdraw the item. 14. You should see an error near the top of the page "Cannot withdraw item in transit". 15. Now, using the checked out item, go to the items tab. ( catalogue/moredetail.pl?biblionumber=144 ) 16. Under Statuses, attempt to withdraw the item. 17. You should see an error near the top of the page "Cannot withdraw checked out item.". 18. Using both the in-transit and checked out item, attempt to modify the status through batchMod. 19. Ensure that the items are not being updated to withdrawn. 20. Turn off PreventWithDrawingItemsStatus system preferece. 21. Going through each of the steps again, except this time your items should successfully withdraw. prove t/db_dependent/Koha/Item.t Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Created attachment 177452 [details] [review] Bug 23010: Add ability to block withdrawning of items with certain statuses Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Created attachment 177453 [details] [review] Bug 23010: Handle error in updateitem.pl and template Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Created attachment 177454 [details] [review] Bug 23010: Add unit tests Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
SO, it works as described, no possibility to set withdraw when PreventWithDrawingItemsStatus is active (Items tab, edit item, batchMod). What about clearing withdraw status when PreventWithDrawingItemsStatus is active -- still possible (Items tab, edit item), not possible (batchMod)?