Summary: | Add ability to remove 'checked out' items from course reserves | ||
---|---|---|---|
Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
Component: | Course reserves | Assignee: | Rogan Hamby <rogan.hamby> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | 1joynelson, david, holsen, kelly, martin.renvoize, rogan.hamby, tmisilo |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: |
This enhancement allows the removal of items that are listed on a course reserve and are checked out. Previously, checked out items could not be removed.
|
|
Version(s) released in: |
20.05.00
|
Circulation function: | |
Attachments: |
small patch to allow removal on course reserve items
Bug 15377: Remove 'checked out' items from course reserves Bug 15377: Remove 'checked out' items from course reserves Bug 15377: Allow removal of 'checked out' items from course reserves Bug 15377: (QA follow-up) Fix capitalization and column heading |
Description
Nick Clemens (kidclamp)
2015-12-15 14:14:56 UTC
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? 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. *** |