Bug 34756

Summary: Cancelling a hold does not cancel the related transfer
Product: Koha Reporter: Julian Maurice <julian.maurice>
Component: CirculationAssignee: Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski>
Status: Needs Signoff --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: baptiste.wojtkowski, emily.lamancusa, gmcharlt, jake.deery, kyle.m.hall, kyle, martin.renvoize
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 34756: Cancelling a hold does not cancel the related transfer
Bug 34756: Cancelling a hold does not cancel the related transfer
Bug 34756: Cancelling a hold does not cancel the related transfer

Description Julian Maurice 2023-09-11 14:11:42 UTC
When a hold is confirmed and its pickup location is different from the item's location, a transfer is initiated.
If the hold is cancelled, the item remains in transfer whereas it is no longer needed at the transfer destination.

Is this normal behaviour ? It seems that staff users should at least see a message about the transfer and maybe have the ability to cancel it.

Steps to reproduce:
1. Create an item at library A
2. Place a hold on this item with a pickup location at library B
3. Check in the item at library A
4. Confirm the hold. The item should now be in transit to library B
5. Cancel the hold. There is no message about the probably useless transfer.
Comment 1 Baptiste Wojtkowski (bwoj) 2024-06-27 14:27:19 UTC
Created attachment 168198 [details] [review]
Bug 34756: Cancelling a hold does not cancel the related transfer

When a hold is confirmed and its pickup location is different from the item's location, a transfer is initiated.
If the hold is cancelled, the item remains in transfer whereas it is no longer needed at the transfer destination.

Is this normal behaviour ? It seems that staff users should at least see a message about the transfer and maybe have the ability to cancel it.

Steps to reproduce:
1. Create an item at library A
2. Place a hold on this item with a pickup location at library B
3. Check in the item at library A
4. Confirm the hold. The item should now be in transit to library B
5. Cancel the hold. There is no message about the probably useless transfer.

NEW TEST PLAN
1. Create an item at library A
2. Place a hold on this item with a pickup location at library B
3. Check in the item at library A
3-bis. Try to cancel the hold: notice there is no message
4. Confirm the hold. The item should now be in transit to library B
5. Cancel the hold. There is now a message to propose you to cancel the
   related transfer
6. Do not check the checkbox. There is no hold but still a transfer
7. Repeat the whole process checking the checkbox. Thers is no hold and
   the transfer has been cancelled.
Comment 2 Jake Deery 2024-07-01 11:21:00 UTC
Created attachment 168313 [details] [review]
Bug 34756: Cancelling a hold does not cancel the related transfer

When a hold is confirmed and its pickup location is different from the item's location, a transfer is initiated.
If the hold is cancelled, the item remains in transfer whereas it is no longer needed at the transfer destination.

Is this normal behaviour ? It seems that staff users should at least see a message about the transfer and maybe have the ability to cancel it.

Steps to reproduce:
1. Create an item at library A
2. Place a hold on this item with a pickup location at library B
3. Check in the item at library A
4. Confirm the hold. The item should now be in transit to library B
5. Cancel the hold. There is no message about the probably useless transfer.

NEW TEST PLAN
1. Create an item at library A
2. Place a hold on this item with a pickup location at library B
3. Check in the item at library A
3-bis. Try to cancel the hold: notice there is no message
4. Confirm the hold. The item should now be in transit to library B
5. Cancel the hold. There is now a message to propose you to cancel the
   related transfer
6. Do not check the checkbox. There is no hold but still a transfer
7. Repeat the whole process checking the checkbox. Thers is no hold and
   the transfer has been cancelled.

Signed-off-by: Jake Deery <jake.deery@ptfs-europe.com>
Comment 3 Jake Deery 2024-07-01 11:27:06 UTC
Ran through the test plan and looks all good to me. It would be nice if, when checking in at Library B, the message said something like "return cancelled hold" instead of "return this item to library a"

I'm nitpicking, though, and this is probably not within the scope of this bug.

Signed off with thanks
Comment 4 Baptiste Wojtkowski (bwoj) 2024-07-10 07:43:05 UTC
Hi ! Thank you so much for signing off and commenting :)

I created a bug for your proposal, https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37299
I'll work on it once the QA validates the approach chosen in this one.
Comment 5 Baptiste Wojtkowski (bwoj) 2024-10-22 13:02:30 UTC
Created attachment 173138 [details] [review]
Bug 34756: Cancelling a hold does not cancel the related transfer

When a hold is confirmed and its pickup location is different from the item's location, a transfer is initiated.
If the hold is cancelled, the item remains in transfer whereas it is no longer needed at the transfer destination.

Is this normal behaviour ? It seems that staff users should at least see a message about the transfer and maybe have the ability to cancel it.

Steps to reproduce:
1. Create an item at library A
2. Place a hold on this item with a pickup location at library B
3. Check in the item at library A
4. Confirm the hold. The item should now be in transit to library B
5. Cancel the hold. There is no message about the probably useless transfer.

NEW TEST PLAN
1. Create an item at library A
2. Place a hold on this item with a pickup location at library B
3. Check in the item at library A
3-bis. Try to cancel the hold: notice there is no message
4. Confirm the hold. The item should now be in transit to library B
5. Cancel the hold. There is now a message to propose you to cancel the
   related transfer
6. Do not check the checkbox. There is no hold but still a transfer
7. Repeat the whole process checking the checkbox. Thers is no hold and
   the transfer has been cancelled.

Signed-off-by: Jake Deery <jake.deery@ptfs-europe.com>