Bug 37076

Summary: Incorrect needsconfirmation code RESERVED_WAITING
Product: Koha Reporter: Matt Blenkinsop <matt.blenkinsop>
Component: CirculationAssignee: Matt Blenkinsop <matt.blenkinsop>
Status: Pushed to oldstable --- QA Contact: Emily Lamancusa (emlam) <emily.lamancusa>
Severity: trivial    
Priority: P5 - low CC: david, emily.lamancusa, fridolin.somers, gmcharlt, kyle.m.hall, lchirlias, lucas
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 an incorrect value used in the OPAC self-checkout code (this new feature was added to Koha 23.11 by Bug 30979 - Add ability for OPAC users to checkout to themselves). Because of the incorrect value in the code (RESERVED_WAITING instead of RESERVE_WAITING - the 'D' is incorrect), patrons could self check out an item on hold for another patron, instead of getting an error message "This item appears to be on hold for another patron, please return it to the desk".
Version(s) released in:
24.11.00.24.05.06,23.11.11
Circulation function:
Bug Depends on: 30979    
Bug Blocks:    
Attachments: Bug 37076: Fix incorrect needsconfirmation code
Bug 37076: Fix incorrect needsconfirmation code
Bug 37076: Fix incorrect needsconfirmation code

Description Matt Blenkinsop 2024-06-12 13:36:09 UTC
There are two instances where RESERVED_WAITING is used as a needsconfirmation code when we should be using RESERVE_WAITING
Comment 1 Matt Blenkinsop 2024-06-12 13:40:09 UTC
Created attachment 167658 [details] [review]
Bug 37076: Fix incorrect needsconfirmation code

This patch corrects two instances of RESERVED_WAITING to RESERVE_WAITING

Test plan:
1) Navigate to system preferences and search for AllowItemsOnHoldCheckoutSCO
2) Observe from the description that the correct status code is RESERVE_WAITING
3) Search the repository for RESERVED_WAITING - you should find two instances, one in Checkouts.pm and one in checkout.js
4) Apply patch
5) Repeat step 3 - there should be 0 instances
Comment 2 Biblibre Sandboxes 2024-06-19 09:23:38 UTC
Created attachment 167883 [details] [review]
Bug 37076: Fix incorrect needsconfirmation code

This patch corrects two instances of RESERVED_WAITING to RESERVE_WAITING

Test plan:
1) Navigate to system preferences and search for AllowItemsOnHoldCheckoutSCO
2) Observe from the description that the correct status code is RESERVE_WAITING
3) Search the repository for RESERVED_WAITING - you should find two instances, one in Checkouts.pm and one in checkout.js
4) Apply patch
5) Repeat step 3 - there should be 0 instances

Signed-off-by: Lucas <lchirlias@ville-roubaix.fr>
Comment 3 Lucas 2024-06-19 09:28:05 UTC
> 3) Search the repository for RESERVED_WAITING - you should find two instances, one in Checkouts.pm and one in checkout.js

I checked in git.koha-community.org 

> 5) Repeat step 3 - there should be 0 instances

I looked at the patch and it applied
Comment 4 Emily Lamancusa (emlam) 2024-08-23 18:36:44 UTC
Created attachment 170665 [details] [review]
Bug 37076: Fix incorrect needsconfirmation code

This patch corrects two instances of RESERVED_WAITING to RESERVE_WAITING

Test plan:
1) Navigate to system preferences and search for AllowItemsOnHoldCheckoutSCO
2) Observe from the description that the correct status code is RESERVE_WAITING
3) Search the repository for RESERVED_WAITING - you should find two instances, one in Checkouts.pm and one in checkout.js
4) Apply patch
5) Repeat step 3 - there should be 0 instances

Signed-off-by: Lucas <lchirlias@ville-roubaix.fr>
Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Comment 5 Emily Lamancusa (emlam) 2024-08-23 18:46:41 UTC
Good catch! Passing QA and upgrading this to a bug, since it fixes a wrong error message in OPAC self-checkout (not the SCO module).
Comment 6 Katrin Fischer 2024-08-26 15:43:42 UTC
Pushed for 24.11!

Well done everyone, thank you!
Comment 7 Lucas Gass (lukeg) 2024-11-15 17:41:57 UTC
Backported to 24.05.x for upcoming 24.05.06
Comment 8 Fridolin Somers 2024-12-02 08:57:59 UTC
Pushed to 23.11.x for 23.11.11