Attachment 140338 Details for Bug 31447 – Bug 31447: Use holds pick up branch in "Please confirm checkout" message

[patch] Bug 31447: Use holds pick up branch in "Please confirm checkout" message

Bug-31447-Use-holds-pick-up-branch-in-Please-confi.patch (text/plain), 2.99 KB, created by ByWater Sandboxes on 2022-09-08 12:56:35 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: ByWater Sandboxes

Created: 2022-09-08 12:56:35 UTC

Size: 2.99 KB

patch

obsolete

>From 4e151d53ae0b0d79ddaa766fa1bb528ac5758131 Mon Sep 17 00:00:00 2001
>From: Emmi Takkinen <emmi.takkinen@koha-suomi.fi>
>Date: Tue, 23 Aug 2022 13:48:39 +0300
>Subject: [PATCH] Bug 31447: Use holds pick up branch in "Please confirm
> checkout" message
>
>When one tries to check out item which has hold in it,
>"Please confirm checkout" message uses patrons home library
>instead of holds pick up library. It would be more logical
>to use latter here.
>
>To test:
>1. Find record with holds.
>2. For first priority hold, change it's pick up library to differ from patrons homebranch if needed.
>3. Check out records item for a different patron.
>=> Note that notice reads: "Item ... has been on hold for ... at [patrons homebranch] since ...".
>4. Apply this patch.
>5. Repeat steps 2 and 3.
>=> Notice should now read: "Item ... has been on hold for ... at [holds pick up branch] since ...".
>
>Sponsored-by: Koha-Suomi Oy
>
>Signed-off-by: Axelle Clarisse <axelle.clarisse@univ-amu.fr>
>---
> C4/Circulation.pm | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 58ceeae3d1..7fa93fe227 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -1186,7 +1186,7 @@ sub CanBookBeIssued {
>                     $needsconfirmation{'ressurname'} = $patron->surname;
>                     $needsconfirmation{'rescardnumber'} = $patron->cardnumber;
>                     $needsconfirmation{'resborrowernumber'} = $patron->borrowernumber;
>-                    $needsconfirmation{'resbranchcode'} = $patron->branchcode;
>+                    $needsconfirmation{'resbranchcode'} = $res->{branchcode};
>                     $needsconfirmation{'resreservedate'} = $res->{reservedate};
>                     $needsconfirmation{'reserve_id'} = $res->{reserve_id};
>                 }
>@@ -1197,7 +1197,7 @@ sub CanBookBeIssued {
>                     $needsconfirmation{'ressurname'} = $patron->surname;
>                     $needsconfirmation{'rescardnumber'} = $patron->cardnumber;
>                     $needsconfirmation{'resborrowernumber'} = $patron->borrowernumber;
>-                    $needsconfirmation{'resbranchcode'} = $patron->branchcode;
>+                    $needsconfirmation{'resbranchcode'} = $res->{branchcode};
>                     $needsconfirmation{'resreservedate'} = $res->{reservedate};
>                     $needsconfirmation{'reserve_id'} = $res->{reserve_id};
>                 }
>@@ -1208,7 +1208,7 @@ sub CanBookBeIssued {
>                     $needsconfirmation{'ressurname'} = $patron->surname;
>                     $needsconfirmation{'rescardnumber'} = $patron->cardnumber;
>                     $needsconfirmation{'resborrowernumber'} = $patron->borrowernumber;
>-                    $needsconfirmation{'resbranchcode'} = $patron->branchcode;
>+                    $needsconfirmation{'resbranchcode'} = $res->{branchcode};
>                     $needsconfirmation{'resreservedate'} = $res->{reservedate};
>                     $needsconfirmation{'reserve_id'} = $res->{reserve_id};
>                 }
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 31447: 140000 | 140338 | 141292 | 143112 | 143349 | 143386 | 143776 | 143777 | 143813 | 143814 | 144241