Currently if an item is checked out it cannot be removed from course reserves. Item types can change while on reserve, so this would have to be taken into account in allow their removal. I would say the current checkout should be unaffected as regards due date, but renewals/overdues should obey the original item type after removal from course
We are using the Course Reserves module and finding the inability to remove checked-out items to be a big handicap.
Removing the check for a circulation preventing the removal is fairly straightforward. Should we add a confirmation dialogue or some visual indicator that the item is currently checked out?
Yes, I think some sort of indicator of checked out status would be needed. Right now the status shows by default in the course reserve view, and it is italicized when the item is checked out. I think that is tied to the "can't do anything" part, but something similar could still be used to draw attention to it?
Created attachment 97369 [details] [review] small patch to allow removal on course reserve items I considered a confirmation dialogue that alerted the user to the fact that there was a circulation but realized the original intent was to put that information in front of the user immediately. Thus this patch simply moves the old criteria for restricting removal of a reserve item to information in the grid. So, the patch does two things 1) removes the restriction of not allowing a reserve to be removed if it is onloan 2) adds a column to the display that says with the item is checked out or available based on the onloan flag
Created attachment 99065 [details] [review] Bug 15377: Remove 'checked out' items from course reserves 1) removes the restriction of not allowing a reserve to be removed if it is onloan 2) adds a column to the display that says with the item is checked out or available based on the onloan flag Test plan: 1) Create a new course reserves list (Home > Course reserves) 2) Add some items to the course 3) Check out an item 4) Note that the Remove button on the course list page is greyed out for the item checked out 5) Apply the patch 6) Refresh the page 7) Note that the Remove button is no longer greyed out 8) Click the Remove button 9) A message pops up 'Are you sure you want to remove this item from the course?' - click OK 10) The item is no longer shown on the course list page Signed-off-by: David Nind <david@davidnind.com>
Comment on attachment 97369 [details] [review] small patch to allow removal on course reserve items Not sure what happened here - have obsoleted the old patch
Asked Kyle to take a look here and he wanted to get some libraries to take a look.
Created attachment 99321 [details] [review] Bug 15377: Remove 'checked out' items from course reserves 1) removes the restriction of not allowing a reserve to be removed if it is onloan 2) adds a column to the display that says with the item is checked out or available based on the onloan flag testplan: add an item to course reserves, check the item out, check that the Ciruclation column changes to Checked Out and that you can remove it Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Hannah Olsen <holsen@duncanville.com>
Created attachment 99845 [details] [review] Bug 15377: Allow removal of 'checked out' items from course reserves 1) removes the restriction of not allowing a reserve to be removed if it is onloan 2) adds a column to the display that says with the item is checked out or available based on the onloan flag testplan: add an item to course reserves, check the item out, check that the Ciruclation column changes to Checked Out and that you can remove it Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Hannah Olsen <holsen@duncanville.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 99846 [details] [review] Bug 15377: (QA follow-up) Fix capitalization and column heading Checked Out -> Checked out Circulation -> Status (matching other pages with this information) Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Welcome aboard Rogan Hamby, great to have a new name to add to the contributors' list :)
Nice work everyone! Pushed to master for 20.05
enhancement not backported to 19.11.x
*** Bug 22599 has been marked as a duplicate of this bug. ***