Attachment 66356 Details for Bug 12598 – Bug 12598: Fix conflict with 17829 - GetMember

[patch] Bug 12598: Fix conflict with 17829 - GetMember

Bug-12598-Fix-conflict-with-17829---GetMember.patch (text/plain), 1.48 KB, created by Jonathan Druart on 2017-08-22 16:35:49 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2017-08-22 16:35:49 UTC

Size: 1.48 KB

patch

obsolete

>From 6d6e4ffb0df8d9dba9b43068cd7b8ae06ea6d2f4 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Tue, 25 Jul 2017 10:10:54 -0300
>Subject: [PATCH] Bug 12598: Fix conflict with 17829 - GetMember
>
>Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>
>---
> Koha/Patrons/Import.pm | 12 ++++++++----
> 1 file changed, 8 insertions(+), 4 deletions(-)
>
>diff --git a/Koha/Patrons/Import.pm b/Koha/Patrons/Import.pm
>index 10173bb755..965db661c9 100644
>--- a/Koha/Patrons/Import.pm
>+++ b/Koha/Patrons/Import.pm
>@@ -163,10 +163,7 @@ sub import_patrons {
>         my $borrowernumber;
>         my $member;
>         if ( defined($matchpoint) && ( $matchpoint eq 'cardnumber' ) && ( $borrower{'cardnumber'} ) ) {
>-            $member = GetMember( 'cardnumber' => $borrower{'cardnumber'} );
>-            if ($member) {
>-                $borrowernumber = $member->{'borrowernumber'};
>-            }
>+            $member = Koha::Patrons->find( { cardnumber => $borrower{'cardnumber'} } );
>         }
>         elsif ($extended) {
>             if ( defined($matchpoint_attr_type) ) {
>@@ -180,6 +177,13 @@ sub import_patrons {
>             }
>         }
> 
>+        if ($member) {
>+            $member = $member->unblessed;
>+            $borrowernumber = $member->{'borrowernumber'};
>+        } else {
>+            $member = {};
>+        }
>+
>         if ( C4::Members::checkcardnumber( $borrower{cardnumber}, $borrowernumber ) ) {
>             push @errors,
>               {
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 12598: 29807 | 29808 | 29809 | 29939 | 30755 | 30780 | 35557 | 35558 | 35559 | 35560 | 35848 | 35849 | 35850 | 35851 | 35964 | 36684 | 36685 | 36687 | 36688 | 36690 | 36692 | 36693 | 36694 | 36695 | 36696 | 39124 | 39125 | 39126 | 39127 | 39128 | 41790 | 41791 | 41792 | 41793 | 41794 | 44065 | 44066 | 44067 | 44068 | 44069 | 48753 | 48754 | 48755 | 48756 | 48757 | 50693 | 50694 | 50695 | 50696 | 50697 | 50698 | 50699 | 50700 | 50704 | 51122 | 51357 | 51358 | 51359 | 51360 | 51361 | 51362 | 51363 | 51364 | 51365 | 51366 | 51367 | 51368 | 51369 | 51370 | 51371 | 51372 | 52323 | 52324 | 52325 | 52326 | 52327 | 52328 | 52329 | 52330 | 52331 | 52332 | 52333 | 52540 | 64437 | 64438 | 64855 | 65214 | 65215 | 65216 | 65217 | 65218 | 65819 | 65820 | 65821 | 65822 | 65823 | 65824 | 66353 | 66354 | 66355 | 66356 | 66357 | 66358 | 66359 | 66575 | 66576 | 66577 | 66578 | 66579 | 66580 | 66581 | 66582 | 67276 | 67277 | 70265 | 70266 | 71755 | 71756 | 71757 | 71758 | 71759 | 71760 | 71761 | 71762 | 71763 | 71764 | 71765 | 71766