When one patron has a waiting hold, Koha pops up a modal when checking out to any patrons. To recreate: - enable WaitingNotifyAtCheckout - have or create a waiting hold for a patron - check out an item to a different patron who does not have a waiting hold - Koha says "This patron has waiting holds that are available for checkout"
Created attachment 167758 [details] [review] Bug 37055: Check count of holds, not if defined To test: - enable WaitingNotifyAtCheckout - have or create a waiting hold for a patron - check out an item to a different patron who does not have a waiting hold - Koha says "This patron has waiting holds that are available for checkout" - Apply patch - Reload patron (click 'Checkout' tab) - Check out an item - no warning - Find the patron with waiting hold - Check out an item - warning - Checkout the waiting item - no warning
Everything works as per the test plan. However, WaitingNotifyAtCheckout no longer seems to work for me as it is supposed to for a patron that actually has holds: 1. Place a hold for a patron (I changed Mary to the Centerville library, and paced a hold for record with an item in Centerville) 2. For the same patron, check out another item to them (I checked out an item based at Centerville). 3. The message saying waiting holds available DOESN'T appear.
I retested and everything seems to work as expected David. I did find one time where the warning didn't appear, but that was because I had pulled up the patron's account before marking the hold waiting. I checked in the item and confirmed the hold on another tab, and had not reloaded the patron. The way the preference works, it sends the waiting holds to the page, and then at checkout time, before the new barcode is submitted, we check to see if there are waiting holds that don't match the current barcode. This means if the item is marked waiting after loading the patron's checkout page they won't be warned. Can you retest, making sure to refresh the checkout page before checking out an item?
Created attachment 167980 [details] [review] Bug 37055: Check count of holds, not if defined To test: - enable WaitingNotifyAtCheckout - have or create a waiting hold for a patron - check out an item to a different patron who does not have a waiting hold - Koha says "This patron has waiting holds that are available for checkout" - Apply patch - Reload patron (click 'Checkout' tab) - Check out an item - no warning - Find the patron with waiting hold - Check out an item - warning - Checkout the waiting item - no warning Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
*** Bug 37199 has been marked as a duplicate of this bug. ***
I can't get the test plan to pass. I'm creating the waiting hold by placing a hold on an item and manually setting "found" to be "W" in the database but the final stage of the test plan is failing when I try to check out the waiting item - I still get a warning. This could just be me not understanding the test plan properly, anything I might be missing?
Hm, why manually updating the found status? I wonder if you are missing some data then, like the waiting date etc. Maybe try to check it in for setting the correct status?
Created attachment 168364 [details] [review] Bug 37055: Check count of holds, not if defined To test: - enable WaitingNotifyAtCheckout - have or create a waiting hold for a patron - check out an item to a different patron who does not have a waiting hold - Koha says "This patron has waiting holds that are available for checkout" - Apply patch - Reload patron (click 'Checkout' tab) - Check out an item - no warning - Find the patron with waiting hold - Check out an item - warning - Checkout the waiting item - no warning Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org> Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Turns out I just don't know how Koha works! Test plan now succeeds and QA tests are all good
Pushed for 24.11! Well done everyone, thank you!
*** Bug 37239 has been marked as a duplicate of this bug. ***
Any chance of a backport to 24.05?
Backported to 24.05.x for upcoming 24.05.05
Depends on Bug 34668 not in 23.11.x
This fixes something not working - no interface changes, no updates to the manual required.