Summary: | Batch checkout silently fails if item contains materials specified (952$3) | ||
---|---|---|---|
Product: | Koha | Reporter: | Catrina Berka <catrina> |
Component: | Circulation | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | RESOLVED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | normal | ||
Priority: | P5 - low | CC: | david, fridolin.somers, gmcharlt, kyle.m.hall, lucas, nick |
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 fixes the batch checkout when an item has a 952$3 value (Materials specified (bound volume or other part)). The text for the 952$3 was not displayed on the batch check out screen, it would silently fail, and then not check out the item to the patron (with no warning given). (This requires setting these system preferences CircConfirmItemParts, BatchCheckouts, BatchCheckoutsValidCategories (all).)
|
|
Version(s) released in: |
24.11.00,24.05.06
|
Circulation function: | |
Bug Depends on: | 37396 | ||
Bug Blocks: | |||
Attachments: |
Screenshot where the 952$3 should display
Bug 37424: Display additional materials note in batch checkout Bug 37424: Display additional materials note in batch checkout Bug 37424: Display additional materials note in batch checkout |
Created attachment 169438 [details] [review] Bug 37424: Display additional materials note in batch checkout This patch adds a missing conditional for ADDITIONAL_MATERIALS to the batch checkout table Test plan: 1. Set a staff member with circ permissions, including FORCE_CHECKOUT 2. Turn on: CircConfirmItemParts, BatchCheckouts, BatchCheckoutsValidCategories (all) 3. Log in as staff member in step 1 4. Attempt to checkout an item with a 952$3 from the batch checkout tab 5. The item with a 952$3 displays, but the copy/text of the materials specified note does not. 6. Do not confirm checkout 7. Apply patch 8. Try again, the materials note should show this time 9. Confirm checkout 10. Success! Item is checked out Created attachment 169447 [details] [review] Bug 37424: Display additional materials note in batch checkout This patch adds a missing conditional for ADDITIONAL_MATERIALS to the batch checkout table Test plan: 1. Set a staff member with circ permissions, including FORCE_CHECKOUT 2. Turn on: CircConfirmItemParts, BatchCheckouts, BatchCheckoutsValidCategories (all) 3. Log in as staff member in step 1 4. Attempt to checkout an item with a 952$3 from the batch checkout tab 5. The item with a 952$3 displays, but the copy/text of the materials specified note does not. 6. Do not confirm checkout 7. Apply patch 8. Try again, the materials note should show this time 9. Confirm checkout 10. Success! Item is checked out Signed-off-by: Catrina Berka <catrina@bywatersolutions.com> Hi Catrina. It looks like you've signed this off. So that you get the credit on the dashboard (https://dashboard.koha-community.org/), could you update the status to "Signed off"? (It will also move the bug into the QA queue.) David Created attachment 174912 [details] [review] Bug 37424: Display additional materials note in batch checkout This patch adds a missing conditional for ADDITIONAL_MATERIALS to the batch checkout table Test plan: 1. Set a staff member with circ permissions, including FORCE_CHECKOUT 2. Turn on: CircConfirmItemParts, BatchCheckouts, BatchCheckoutsValidCategories (all) 3. Log in as staff member in step 1 4. Attempt to checkout an item with a 952$3 from the batch checkout tab 5. The item with a 952$3 displays, but the copy/text of the materials specified note does not. 6. Do not confirm checkout 7. Apply patch 8. Try again, the materials note should show this time 9. Confirm checkout 10. Success! Item is checked out Signed-off-by: Catrina Berka <catrina@bywatersolutions.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Very small template change, will pick for 24.11. Pushed for 24.11! Well done everyone, thank you! Backported to 24.05.x for upcoming 24.05.06 This is a bug fix and no interface changes, no updates to the manual required. |
Created attachment 169379 [details] Screenshot where the 952$3 should display As noted in 15718 and 23780, when using the batch checkout feature, there’s no preview of the 952$3 in the confirm checkout screen to let you know why the checkout needs confirmation, but also the checkout silently fails even when selecting "Checkout or renew". It sounded most similar to 30995, but my user has the force_checkout permission, so that doesn’t apply. Test plan: 1. Set a staff member with circ permissions, including FORCE_CHECKOUT 2. Turn on: CircConfirmItemParts, BatchCheckouts, BatchCheckoutsValidCategories (all) 3. Log in as staff member in step 1 4. Attempt to checkout an item with a 952$3 from the batch checkout tab 5. The item with a 952$3 displays, but the copy/text of the materials specified note does not. 6. When you select “Checkout or renew,” you are taken back to the batch check out screen, but the item with the 952$3 is NOT checked out to the patron, nor are you given any warning that an error occurred.