Attachment 158140 Details for Bug 34529 – Bug 34529: Offline circulation should be able to accept userid as well as cardnumber

[patch] Bug 34529: Offline circulation should be able to accept userid as well as cardnumber

Bug-34529-Offline-circulation-should-be-able-to-ac.patch (text/plain), 2.58 KB, created by Katrin Fischer on 2023-10-31 15:48:47 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2023-10-31 15:48:47 UTC

Size: 2.58 KB

patch

obsolete

>From a591b42fe1e38c7ab80e4f4ab014ebafeb264ade Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Mon, 14 Aug 2023 10:55:09 -0400
>Subject: [PATCH] Bug 34529: Offline circulation should be able to accept
> userid as well as cardnumber
>
>Like everywhere else in Koha, offline circulation should be able to use userid as well as cardnumber to look up the patron for a transaction ( checkouts and fine payments ).
>
>Test Plan:
>1) Apply this patch
>2) Restart all the things!
>3) Download the test.koc file attached to this bug
>4) Verify you have an item with the barcode 39999000001396
>   create it if needed
>5) Set a patron's username to "hacevedo"
>6) Upload and import the test.koc file into offline circ
>7) Verify the item was checked out and the payment was made
>
>Signed-off-by: Philip Orr <philip.orr@lmscloud.de>
>---
> C4/Circulation.pm    | 4 +++-
> offline_circ/list.pl | 8 +++++++-
> 2 files changed, 10 insertions(+), 2 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index baab76fef6..50123d37a1 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -4199,6 +4199,7 @@ sub ProcessOfflineIssue {
>     my $operation = shift;
> 
>     my $patron = Koha::Patrons->find( { cardnumber => $operation->{cardnumber} } );
>+    $patron ||= Koha::Patrons->find( { userid => $operation->{cardnumber} } );
> 
>     if ( $patron ) {
>         my $item = Koha::Items->find({ barcode => $operation->{barcode} });
>@@ -4232,7 +4233,8 @@ sub ProcessOfflineIssue {
> sub ProcessOfflinePayment {
>     my $operation = shift;
> 
>-    my $patron = Koha::Patrons->find({ cardnumber => $operation->{cardnumber} });
>+    my $patron = Koha::Patrons->find( { cardnumber => $operation->{cardnumber} } );
>+    $patron ||= Koha::Patrons->find( { userid => $operation->{cardnumber} } );
> 
>     $patron->account->pay(
>         {
>diff --git a/offline_circ/list.pl b/offline_circ/list.pl
>index 252dc8594b..2838a18530 100755
>--- a/offline_circ/list.pl
>+++ b/offline_circ/list.pl
>@@ -48,7 +48,13 @@ for (@$operations) {
>         $_->{'bibliotitle'}    = $biblio->title;
>         $_->{'biblionumber'}   = $biblio->biblionumber;
>     }
>-    my $patron             = $_->{cardnumber} ? Koha::Patrons->find( { cardnumber => $_->{cardnumber} } ) : undef;
>+
>+    my $patron =
>+        $_->{cardnumber}
>+        ? Koha::Patrons->find( { cardnumber => $_->{cardnumber} } )
>+        || Koha::Patrons->find( { userid => $_->{cardnumber} } )
>+        : undef;
>+
>     if ($patron) {
>         $_->{'borrowernumber'} = $patron->borrowernumber;
>         $_->{'borrower'}       = ($patron->firstname ? $patron->firstname:'').' '.$patron->surname;
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34529: 154403 | 154411 | 155969 | 155970 | 158140 | 158141 | 158321 | 158322 | 158323 | 158324 | 158380 | 158381 | 158382 | 158383