To reproduce: 1. Place a hold on item A for patron A 2. Check out item A to patron B 3. See the prompt to confirm the checkout because the hold for patron A exists. Check the 'cancel hold' checkbox and click the 'No, don't check out' button 4. Notice the hold for patron A is NOT cancelled <-- bug 5. Attempt to check out item to patron B again 6. This time, check the 'cancel hold' checkbox but click the 'Yes, check out' button 7. Notice that item A is successfully checked out to patron B and the hold for patron A has been cancelled. Therefore, the 'cancel hold' checkbox does not work if the checkout is not completed.
Looks like a long standing issue and I did not find a duplicated bug report, that's weird. At first glance I would say the checkbox is only taken into consideration for "Yes, check out" https://snipboard.io/jEXNBP.jpg Not sure how we could make it clear on the interface.
Yes it is a weird one. I think given that the option is a checkbox, it does imply that it's a separate action to the button, and you should be able to cancel a hold while clicking any button. In this case I think we just need to ensure that the cancel hold checkbox is considered if any of the buttons is clicked. Otherwise, the checkbox should be removed and the cancel hold action should be a second screen confirmation, after choosing "Yes, check out".
Something like that? https://snipboard.io/UXsqMB.jpg
(In reply to Jonathan Druart from comment #3) > Something like that? https://snipboard.io/UXsqMB.jpg Actually it's more than 2 options as there is also another checkbox for "Revert waiting status", if the hold is waiting.
(In reply to Jonathan Druart from comment #4) > (In reply to Jonathan Druart from comment #3) > > Something like that? https://snipboard.io/UXsqMB.jpg > > Actually it's more than 2 options as there is also another checkbox for > "Revert waiting status", if the hold is waiting. That could work! Would just also need the "Yes, check out and revert waiting status" option too. And to standardise the style of the other buttons.