Attachment 140000 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

0001-Bug-31447-Use-holds-pick-up-branch-in-Please-confirm.patch (text/plain), 2.93 KB, created by Emmi Takkinen on 2022-08-31 11:45:35 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Emmi Takkinen

Created: 2022-08-31 11:45:35 UTC

Size: 2.93 KB

patch

obsolete

>From 189dac56f6324ffa459d6c251af8ed2722326322 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
>---
> C4/Circulation.pm | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 87f892ab4b..bb05fc8983 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -1187,7 +1187,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};
>                 }
>@@ -1198,7 +1198,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};
>                 }
>@@ -1209,7 +1209,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.34.1
>

Actions: View | Diff | Splinter Review

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